MVC中，提交后界面Model值未能更新的原因及优化。

假设有如下代码：

View/Time/index.cshtml

   
   


    
    
@model  MvcApplication1.Models.timeModel 

    
    
<h2>Time</h2> 

    
    
@using (Html.BeginForm("Index", "Time")) 

    
    
{   

    
    
   <B>Get Current time:</B> @Html.TextBoxFor(m => m.currenttime)<br /> 

    
    
   <B>Current time:</B> @Model.currenttime<br /> 

    
    
   <p>   

    
    
   <input type="submit" value="search" />   

    
    
   </p>   

    
    
} 
   
   

TimeController.cs

   
   


    
    
        public ActionResult Index() 

    
    
        { 

    
    
            MvcApplication1.Models.timeModel model = new Models.timeModel(); 

    
    
            model.currenttime = DateTime.Now.TimeOfDay; 

    
    
            return View(model); 

    
    
        } 

    
    
 

    
    
        [HttpPost] 

    
    
        public ActionResult Index(MvcApplication1.Models.timeModel model) 

    
    
        { 

    
    
            model.currenttime = DateTime.Now.TimeOfDay; 

    
    
            return View(model); 

    
    
        } 
   
   

结果如下：

点击submit之后

点击后发现，textboxfor中的值并没有改变，但是model的值其实已经改变了。

这是因为controller的ModelState属性，是dictionary类型，它包含了所有的要提交的值和验证的错误信息。在返回View的时候，文本框之类的值由ModelState所持有，因此没有得到更新，无法在界面上显示。最简单的做法就是使用ModelState.Clear()，或者ModelState.Remove("currenttime"); 之类的方法。但是也存在一些风险，比如把验证的错误信息也清除了。

先看一下效果。

   
   


    
    
public ActionResult Index() 

    
    
{ 

    
    
    MvcApplication1.Models.timeModel model = new Models.timeModel(); 

    
    
    model.currenttime = DateTime.Now.TimeOfDay; 

    
    
    return View(model); 

    
    
} 

    
    
 

    
    
//每次post重新赋值时间给model 

    
    
[HttpPost] 

    
    
public ActionResult Index(MvcApplication1.Models.timeModel model) 

    
    
{ 

    
    
    ModelState.Remove("currenttime"); 

    
    
    model.currenttime = DateTime.Now.TimeOfDay; 

    
    
    return View(model); 

    
    
} 
   
   

但是这种方法并不安全，因为如果是clear方法可能会清空ModelState中验证信息。而且刷新页面的时候，也会有重复提交的风险。

改成如下方式，把post方式转换成get方式，这样就避免了刷新时候会弹出重复提交的按钮。用TempData或者Session作为临时保存数据。

   
   


    
    
public ActionResult Index() 

    
    
       { 

    
    
           Models.timeModel model = new Models.timeModel(); 

    
    
           if (TempData["timeModel"] != null) 

    
    
           { 

    
    
               model = TempData["timeModel"] as Models.timeModel; 

    
    
           } 

    
    
           else 

    
    
           { 

    
    
               model.currenttime = DateTime.Now.TimeOfDay; 

    
    
           } 

    
    
           return View(model); 

    
    
       } 

    
    
 

    
    
 

    
    
       //把post方式转换成get方式 

    
    
       [HttpPost] 

    
    
       public ActionResult Index(MvcApplication1.Models.timeModel model) 

    
    
       { 

    
    
           model.currenttime = DateTime.Now.TimeOfDay; 

    
    
           TempData["timeModel"] = model; 

    
    
           return RedirectToAction("Index"); 

    
    
       }