我是MVC的新手,我正在试图弄清楚为什么没有显示验证摘要。这是代码:
视图:
<% using(var form = Html.BeginForm("Create", "User"))
{%>
<table>
<thead>
<th>Create User</th>
<th><%= Html.ValidationSummary(false) %></th>
</thead>
<tbody>
<tr>
<td><%= Html.LabelFor(model => model.Creating.Username) %>:</td>
<td><%= Html.TextBoxFor(model => model.Creating.Username) %></td>
</tr>
<tr>
<td><%= Html.LabelFor(model => model.Creating.Firstname) %>:</td>
<td><%= Html.TextBoxFor(model => model.Creating.Firstname)%></td>
</tr>
<tr>
<td><%= Html.LabelFor(model => model.Creating.Lastname) %></td>
<td><%= Html.TextBoxFor(model => model.Creating.Lastname) %></td>
</tr>
<tr>
<td colspan="2">
<input type="submit" value="Create" />
</td>
</tr>
</tbody>
</table>
<%}%>
相关控制器方法:
[HttpPost]
public ActionResult Create(User creating)
{
var response = _service.Save(creating);
if (response.Success)
return RedirectToAction("Index");
response.Errors.CopyToModelState(this.ModelState);
return RedirectToAction("Index");
}
业务逻辑方法:
public Response Save(User user)
{
//Place Validation logic here
//Check username is between 3-30 characters and make sure the username is unique
//return response if username fails business rules
bool isDataInvalid = false;
List<ValidationError> errorList = new List<ValidationError>();
if ((user.Username.Length < 3) || user.Username.Length > 30)
{
ValidationError invalidUsernameLengthError = new ValidationError();
invalidUsernameLengthError.Property = "Creating.Username";
invalidUsernameLengthError.ErrorMessage = "must be between 3 and 30 characters long";
errorList.Add(invalidUsernameLengthError);
isDataInvalid = true;
}
if (isDataInvalid)
{
return new Response()
{
Success = false,
Errors = errorList
};
}
_repository.Save(user);
return new Response()
{
Success = true
};
}
帮助方法:
public static void CopyToModelState(this List<ValidationError> errors, ModelStateDictionary modelState)
{
foreach (var error in errors)
{
modelState.AddModelError(error.Property, error.ErrorMessage);
}
}
逻辑按照预期行事,但没有任何显示。我已经检查了输出的HTML,并且验证没有被写入。我已经尝试将模型的属性分配给modelState并直接在相关字段上显示验证,但这也不起作用。有什么想法吗?
啊!执行RedirectToAction会导致新请求,因此错误数据将丢失。此外,我正在使用不同的控制器,所以我需要显式调用原始视图(Index.aspx)。我的index.aspx期望的模型实际上不是用户对象,它是一个不同的列表对象,所以我需要这样做:
var users = _service.FindAll();
return View("Index", new UserListModel() { Users = users });
而不是RedirectToAction。这显然是验证错误的标准模式 - 成功(没有错误)您使用RedirectToAction,但是对于错误,您需要返回正确的视图。
好的,谢谢mattytommo - 这真的很有用。我仍然遇到问题。我现在有这个用于控制器 - 类似于你的建议,但仍然没有显示错误消息。我尝试了数据注释但无法使其工作(我正在使用MVC2),并且一直在尝试我能想到的所有内容以修复现有代码。
[HttpPost]
public ActionResult Create(User creating)
{
var response = _service.Save(creating);
if (response.Success)
return RedirectToAction("Index");
foreach (var error in response.Errors)
{
ModelState.AddModelError(error.Property, error.ErrorMessage);
}
return RedirectToAction("Index");
}
还有什么想法吗?我很欣赏这个建议!
答案 0 :(得分:3)
我认为您按值传递了ModelState
变量,因此您在该函数中所做的更改实际上并未保存。尝试重现你的功能,但对于实际的ModelState
对象,就像这样:
[HttpPost]
public ActionResult Create(User creating)
{
var response = _service.Save(creating);
if (response.Success)
return RedirectToAction("Index");
foreach (var error in response.Errors)
{
ModelState.AddModelError(error.Property, error.ErrorMessage);
}
return View(creating);
}
您还应该查看MVC数据注释,可以通过将这两个属性放在Username
属性上来简单地替换您的验证:
[MinLength(3), MaxLength(30)]
public string UserName { get; set; }
或只使用StringLength
的一个属性(感谢@SimonWhitehead):
[StringLength(30, MinimumLength = 3)]
数据注释:Here