最近,我一直在努力在我的项目中启用更多的Visual Studio代码分析规则。但是,我一直碰到规则CA2227:“集合属性应为只读”。
说我有一个像这样的模型类:
public class Foo
{
public string Name { get; set; }
public List<string> Values { get; set; }
}
而且我有一个强烈映射的视图页面:
@using (Html.BeginForm())
{
@Html.LabelFor(m => m.Name)
@Html.EditorFor(m => m.Name)
for (int i = 0; i < Model.Values.Count; i++)
{
<br />
@Html.LabelFor(m => Model.Values[i]);
@Html.EditorFor(m => Model.Values[i]);
}
<button type="submit">Submit</button>
}
使用ASP.NET MVC,我可以在控制器中编写一个动作,该动作将自动将此输入绑定到Foo
类型的类:
[HttpPost]
public ActionResult ProcessForm(Foo model)
{
return View(model);
}
此方法的问题在于,我的List<string>
自动属性违反了规则CA2227。如果我不进行模型绑定,则可以使该属性为只读,并在其他位置填充该集合。但是,该方法不适用于默认的模型绑定器。现在,我只是在视图模型中添加抑制消息。
是否有一种方法可以在不违反CA2227的情况下将模型中的项目绑定?还是在这里添加抑制消息是我最好的选择?
我相信您需要创建一个自定义ModelBinder
来解决此问题,这不值得。在这种情况下,由于违反规则有合理的好处,因此请继续加以制止。
这是应该取消此规则的情况,因为这些模型类还可以作为传输数据对象执行工作。
引用规则文档:
何时禁止显示警告
如果该属性是数据传输对象(DTO)类的一部分,则可以禁止显示警告。否则,请勿抑制此规则的警告。
https://docs.microsoft.com/pt-br/visualstudio/code-quality/ca2227?view=vs-2019