我创建了一个在线商店,尝试实现“详细视图”功能。该产品已成功显示并具有必要的属性,但是在查看其他产品时,不会删除前一个产品并累积在存储中。因此,详细的产品评论表单包含我使用该请求添加的所有产品。
这是课程Detail
namespace App.Domain.Entities
{
public class Detail
{
private List<DetailLine> lineCollection = new List<DetailLine>();
public void AddItem(Product product)
{
DetailLine line = lineCollection
.Where(g => g.Product.ProductId == product.ProductId)
.FirstOrDefault();
if (line == null)
{
lineCollection.Add(new DetailLine
{
Product = product
});
}
}
public IEnumerable<DetailLine> Lines
{
get { return lineCollection; }
}
}
public class DetailLine
{
public Product Product { get; set; }
}
}
这是DetailController
namespace App.Web.Controllers
{
public class DetailController : Controller
{
public ViewResult Index(string returnUrl)
{
return View(new DetailIndexViewModel
{
Detail = GetDetail(),
ReturnUrl = returnUrl
});
}
private IProductRepository repository;
public DetailController(IProductRepository repo)
{
repository = repo;
}
public RedirectToRouteResult AddToDetail(int productId, string returnUrl)
{
Product product = repository.Products
.FirstOrDefault(g => g.ProductId == productId);
if (product != null )
{
GetDetail().AddItem(product);
}
return RedirectToAction("Index", new { returnUrl });
}
public Detail GetDetail()
{
Detail detail = (Detail)Session["Detail"];
if (detail == null)
{
detail = new Detail();
Session["Detail"] = detail;
}
return detail;
}
}
}
这是期望的结果:
这是麻烦的结果:
此索引视图
@model App.Web.Models.DetailIndexViewModel
@{
ViewBag.Title = "Подробное описание товара";
}
<table class="table">
<thead>
<tr>
<th>Название</th>
<th class="text-left">Цена</th>
<th class="text-left">Вес</th>
<th class="text-left">Вкус</th>
</tr>
</thead>
<tbody>
@foreach (var line in Model.Detail.Lines)
{
<tr>
<td class="text-left">@line.Product.Name</td>
<td class="text-left">@line.Product.Price.ToString("# руб")</td>
<td class="text-left">@line.Product.Mass</td>
<td class="text-left">@line.Product.Taste</td>
</tr>
}
</tbody>
</table>
<tr>
<th class="text-center" >Описание</th>
</tr>
@foreach (var line in Model.Detail.Lines)
{
<div class="container">
@line.Product.Description
</div>
}
答案 0 :(得分:1)
在ASP.NET MVC中,通常首选使用Session
以外的方法来存储数据。 Session
最适合multi-screen wizards使用,其中上一个视图/操作的结果适用于下一个视图/操作。
我会给你“快速”的答案;然后稍后将编辑我的答案以提供“更好”的方法。
您似乎在向Detail
类添加了多个产品线,而没有删除以前的产品线。
如果这是一个列表,但您不想显示多个,为什么要在List
中包含它?
现在,您AddItem
并未从Session
中删除项目。
要从Session
中删除内容;您需要将其添加到适当的位置:
var detail = (Detail)Session["Detail"];
detail.RemoveOlderItems(product);
Session["Detail"] = detail;
使用RemoveOlderItems
,如下所示:
public void RemoveOlderItems(Product product)
{
List<DetailLine> lines = lineCollection
.RemoveAll(g => g.Product.ProductId != product.ProductId);
lineCollection = lines;
}
很难说出您要显示的内容(我认为部分原因是我很难跟踪视图的屏幕快照,因为它不是使用本国语言显示的;但是我正在关注尽我所能)。但我还有一些澄清的意见和问题:
视图应处于什么抽象水平? Detail
充满了ProductLines
吗?您要显示多个产品线吗?根据显示的屏幕截图,您实际上只想显示一个。如果您只需要一个List
的想法,那么我将彻底删除ProductLine
的想法。
如果您可以进一步说明最终结果是什么,以及为什么List
似乎只希望显示一个结果,那将会有所帮助。
答案 1 :(得分:0)
我解决了添加到DetailController放弃会话的解决方案:
public Detail GetDetail()
{
Detail detail = (Detail)Session["Detail"];
if (detail == null)
{
detail = new Detail();
Session["Detail"] = detail;
}
else {
Session.Abandon();
}
return detail;
}
}
}
但是,最好避免这样的决定来解决这样的决定?