CA2227和ASP.NET模型绑定

问题描述 投票:5回答:2

最近,我一直在努力在我的项目中启用更多的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的情况下将模型中的项目绑定?还是在这里添加抑制消息是我最好的选择?

c# asp.net asp.net-mvc code-analysis fxcop
2个回答
3
投票

我相信您需要创建一个自定义ModelBinder来解决此问题,这不值得。在这种情况下,由于违反规则有合理的好处,因此请继续加以制止。


0
投票

这是应该取消此规则的情况,因为这些模型类还可以作为传输数据对象执行工作。

引用规则文档:

何时禁止显示警告

如果该属性是数据传输对象(DTO)类的一部分,则可以禁止显示警告。否则,请勿抑制此规则的警告。

https://docs.microsoft.com/pt-br/visualstudio/code-quality/ca2227?view=vs-2019

© www.soinside.com 2019 - 2024. All rights reserved.