【发布时间】：2021-11-17 20:40:03
【问题描述】：
我正在为具有 5 个插槽的产品开发一个电子储物柜系统的概述。 我想要实现的是自动填充每个“储物柜插槽”的下拉菜单并在每个插槽中选择正确的产品。
也可以有多个储物柜，所以我不知道实现这一目标的最佳和最有效的方法。
索引.cshtml
@foreach (var locker in Model.Lockers)
{
<div class="lockBox slot1 unavailible">
<div>
<select asp-items="Model.OptionSlot1">
<option value="">– Empty slot –</option>
</select>
</div>
1
</div>
<div class="lockBox slot2 unavailible">
<div>
<select asp-items="Model.OptionSlot2">
<option value="">– Empty slot –</option>
</select>
</div>
2
</div>
<div class="lockBox slot3 unavailible">
<div>
<select asp-items="Model.OptionSlot3">
<option value="">– Empty slot –</option>
</select>
</div>
3
</div>
<div class="lockBox slot4 unavailible">
<div>
<select asp-items="Model.OptionSlot4">
<option value="">– Empty slot –</option>
</select>
</div>
4
</div>
<div class="lockBox slot5 unavailible">
<div>
<select asp-items="Model.OptionSlot5">
<option value="">– Empty slot –</option>
</select>
</div>
5
</div>
}
索引.cshtml.cs
var userId = User.FindFirstValue(ClaimTypes.NameIdentifier);
Lockers = await LockerManager.GetLockers(userId);
Products = await ProductManger.GetProducts(userId);
Box = await BoxManager.GetBoxes(userId);
var box1 = Box.Where(a => a.BoxNo == 1).ToList()[0];
var prod1 = Products.Where(p => p.Id == box1.ProductId).ToList()[0];
OptionsSlot1 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box1.ProductId == prod1.Id ? true : false
}).ToList();
var box1 = Box.Where(a => a.BoxNo == 1).ToList()[0];
var prod1 = Products.Where(p => p.Id == box1.ProductId).ToList()[0];
OptionsSlot1 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box1.ProductId == prod1.Id ? true : false
}).ToList();
var box2 = Box.Where(a => a.BoxNo == 2).ToList()[0];
var prod2 = Products.Where(p => p.Id == box2.ProductId).ToList()[0];
OptionsSlot2 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box2.ProductId == prod2.Id ? true : false
}).ToList();
var box3 = Box.Where(a => a.BoxNo == 3).ToList()[0];
var prod3 = Products.Where(p => p.Id == box3.ProductId).ToList()[0];
OptionsSlot3 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box3.ProductId == prod3.Id ? true : false
}).ToList();
var box4 = Box.Where(a => a.BoxNo == 4).ToList()[0];
var prod4 = Products.Where(p => p.Id == box4.ProductId).ToList()[0];
OptionsSlot4 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box4.ProductId == prod4.Id ? true : false
}).ToList();
var box5 = Box.Where(a => a.BoxNo == 5).ToList()[0];
var prod5 = Products.Where(p => p.Id == box5.ProductId).ToList()[0];
OptionsSlot5 = Products.Select(a => new SelectListItem
{
Value = a.Id.ToString(),
Text = a.Name,
Selected = box5.ProductId == prod5.Id ? true : false
}).ToList();
【问题讨论】：
-
您要求使其更小或更高效。您是否遇到此代码的性能问题？还是你觉得代码太多？
-
减少代码量的一件事是删除所有
? true : false。这就像说“如果值为真，则值为真，但如果值为假，则值为假”，而不仅仅是“值为真”或“值为假”。 -
@WouterdeKort，不，我没有遇到性能问题，但是一旦有 200 个插槽的储物柜，这种伤口是可持续的。
-
我希望看到一个循环然后为每个插槽编写代码。
-
您可以改用字典。并在 cshtml 中使用局部视图
标签： c# .net razor-pages