如何避免数据重复请求MVC

时间:2019-04-02 17:14:15

标签: c# asp.net-mvc

我创建了一个在线商店,尝试实现“详细视图”功能。该产品已成功显示并具有必要的属性,但是在查看其他产品时,不会删除前一个产品并累积在存储中。因此,详细的产品评论表单包含我使用该请求添加的所有产品。

这是课程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;
        }
    }
}

这是期望的结果: this is the desired result

这是麻烦的结果: this trouble result

此索引视图

@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>
}

2 个答案:

答案 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;
        }
    }
}

但是,最好避免这样的决定来解决这样的决定?