Question

我正在制作一个MVC在线网络购物应用，所以在购物车页面之后我有ProcessStepOne动作，用户应在其中填充他的数据。

[Authentication] 
public ActionResult ProcessStepOne()
{

    ProcessOrderViewModel model = GetOrderData();            
    return View("ProcessOrderStepOne", model);

}


private ProcessOrderViewModel GetOrderData()
{
    ProcessOrderViewModel model = (ProcessOrderViewModel)Session["Process"];
    if(model==null)
    {
        model = new ProcessOrderViewModel();
    }
    return model;
}

我的ProcessOrderViewModel模型是：

public class ProcessOrderViewModel 
{
    public ProcessOrderViewModel()
    {
        this.PrivateIndividualData = new PrivateIndividualModel();
        this.OrderDiscoutPrice = new OrderDiscountPriceModel();
    }
    public PrivateIndividualModel PrivateIndividualData { get; set; }
    public OrderDiscountPriceModel OrderDiscoutPrice { get; set; }
}

我的观点ProcessOrderStepOne是：

@model  ProcessOrderViewModel

<form id="formOrderData" method="post" action="@Url.Action("ProcessStepTwo")">           
    <div class="row">
        <div class="col-xs-12 col-sm-6">
            <div class="form-group">
                <label> First Name </label>
                @Html.TextBoxFor(x => x.PrivateIndividualData.FirstName, new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.PrivateIndividualData.FirstName)
            </div>
        </div>

        <div class="col-xs-12 col-sm-6">
            <div class="form-group">
                <label> Last Name </label>
                @Html.TextBoxFor(x => x.PrivateIndividualData.Family, new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.PrivateIndividualData.Family)
            </div>
        </div>

        <div class="col-xs-12 col-sm-6">
            <div class="form-group">
                <label>Email</label>
                @Html.TextBoxFor(x => x.PrivateIndividualData.Email, new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.PrivateIndividualData.Email)
            </div>
        </div>

        <div class="col-xs-12 col-sm-6">
            <div class="form-group">
                <label for="tel">@StringResources.GetResourceString("UserManagement", "Register_PhoneLabel")</label>
                @Html.TextBoxFor(x => x.PrivateIndividualData.Telephone, new { @class = "form-control" })
                @Html.ValidationMessageFor(x => x.PrivateIndividualData.Telephone)
            </div>
        </div>
    </div>
</form>

所以，我的第二步是检查用户输入的值，然后显示他进行验证。如果一切正常，我会将数据保存在会话中，以便用户可以返回上一个ProcessStepOne操作或继续执行MakeOrder操作。

public ActionResult ProcessStepTwo(ProcessOrderViewModel model)
{
    if (ModelState.IsValid)
    {
        Session["Process"] = model;
        return View("ProcessOrderStepTwo", model);
    }
    return View("ProcessOrderStepOne", model);
 }

观点是：

@model ProcessOrderViewModel

<section class="form-section">                    
    <p>
        <a href='@Url.Action("ProcessStepOne")'>CHANGE</a>
        <span class="text-semibold">Name:</span> @Model.PrivateIndividualData.FirstName <br>
        <span class="text-semibold">Last Name:</span> @Model.PrivateIndividualData.Family <br>
        <span class="text-semibold">E-mail:</span>  @Model.PrivateIndividualData.Email<br>
        <span class="text-semibold">Телефон:</span>@Model.PrivateIndividualData.Telephone <br>                           
    </p>
</section>
<a href='@Url.Action("MakeOrder")'>PROCEED TO MAKE ORDER</a>

而且，这是我从会话中获取数据的最后一个操作：

public ActionResult MakeOrder()
{
    var data = (ProcessOrderViewModel)System.Web.HttpContext.Current.Session["Process"];   
    // make order with this data
    return View("Thank You");
}

所以我的问题是：在MakeOrder操作中，我只是从会话中获取模型而不进行任何验证。如何再次验证它，并且正确地将数据存储在会话中，以便我可以返回进行修改或继续进行订购。这是正确的做法，因为我无法想到另一种方式。

Answer 1

在MakeOrder操作中，我只是从会话中获取模型而不进行任何验证。我该如何再次验证它？

您可以在控制器中使用TryValidateModel，在从Session中检索模型后重新验证模型。但是，从我的测试来看，这个方法似乎只验证了模型中的一个深度。考虑到这一点，您的代码将类似于以下内容：

public ActionResult MakeOrder()
{
    var data = (ProcessOrderViewModel)System.Web.HttpContext.Current.Session["Process"]; 

    bool individualDataValid = TryValidateModel(data.PrivateIndividualData);
    bool discountDataValid = TryValidateModel(data.OrderDiscoutPrice);

    if (individualDataValid && discountDataValid)
    {
        // make order with this data
        return View("Thank You");
    }

    //Error condition
    return View("Error");
}

将数据存储在会话中是否正确，以便我可以返回进行修改或继续进行订购。这是正确的做法，因为我想不出另一种方式。

就个人而言，如果可以，我会避免使用Session。但是，我认为像你这样的“购物篮”要求是完全合理的。我要说的是，从维护和测试的角度来看，如果为Session访问创建包装器对象可能会更好。请参阅Session variables in ASP.NET MVC。

这有以下好处：

您的所有会话访问都在一个地方。不是分散在控制器之外。
您的解决方案中没有使用“Magic Strings”进行会话访问。
作为一个额外的好处，因为您已将会话访问抽象为对象，您的控制器可以更“可测试”（假设它不是静态包装器）。

验证从会话中提取的MVC模型

1 个答案: