ASP.NET MVC 不良做法：具有必需属性的可选子模型

Question

Google 上有大量资源，但我无法完全理解在我的场景中需要做什么：

我有这门课：

public class CompanyLanguage : EntityBase
{
    public int CompanyId { get; set; }
    public int LanguageId { get; set; }
    public bool IsDefault { get; set; }


    public virtual Company Company { get; set; }
    public virtual Language Language { get; set; }
}

Language 定义为：

public class Language:EntityBase
{
    [Required]
    [DisplayName("Language Code")]
    public string LanguageCode { get; set; }

    [Required]
    [MaxLength(2, ErrorMessage ="2 characters maximum")]
    [DisplayName("2 Char Language Code")]
    public string LanguageCode2Char { get; set; }

    [Required]
    [DisplayName("Language Name")]
    public string LanguageName { get; set; }

    public virtual List<LabelLanguage> LabelLanguages { get; set; }
}

运行Fortify Scan 会将以下问题作为高优先级返回：

（ASP.NET MVC 不良做法：具有必需属性的可选子模型）

我们无法运行强化扫描 - 它正在由其他人运行，因此我需要正确进行更改，以免它直接返回。

我查看过的所有资源都表明可以进行低发攻击 - 即空 Language，即使 Language 具有一些必需的属性。

对我来说，这是一个有效的场景——Language 的必需属性仅在 Language 不为空时才需要。

那么我应该怎么做才能解决这个问题？我是需要public int LanguageId { get; set; }，还是public virtual Language Language { get; set; }，还是两者都需要？

或者我完全错了，我必须做其他事情？正如我所说，我无法测试这些，因为必须将软件发送出去进行测试，否则我会尝试各种方式。

Answer 1

总结我们从 cmets 的讨论。

1. 创建一个仅对满足相应视图所需的信息进行建模的视图模型。
2. 从域 ef 模型中填充控制器操作中的视图模型
3. 使用 linq 查询或 Automapper 直接将项目投影到视图模型中。

您的问题的示例视图模型

public class CompanyLanguageEditViewModel
{
    [DisplayName("Company")]
    [Required]
    public int CompanyId { get; set; }

    [DisplayName("Language")]
    [Required]
    public int LanguageId { get; set; }
    public bool IsDefault { get; set; }

    public IEnumerable<SelectListItem> Companies{ get; set; }
    public IEnumerable<SelectListItem> Languages { get; set; }
}

在你看来，你可以使用

@Html.DropDownListFor(x => x.CompanyId, Model.Companies);

您的标签将是 Country 并且您只会回复您需要的内容