NullReferenceException 包含 LINQ 表达式 MVC3

Question

Details.cshtml 页面中的以下代码：

 @{
            @Html.Raw(" | ");
            for (int i = 0; i < Model.Payments.Count; i++)
            {
                @Html.Raw("<a href=../../Payment/Details/" + Model.ID + "/" + Model.Payments.ElementAt(i).accountID + ">" + Model.Payments.ElementAt(i).Account.landIOC + "</a>");           
                @Html.Raw(" | ");
            }
        }

Model.Payments.ElementAt(i).Account 由于某种原因始终为 NULL，尽管一笔付款始终只有一个帐户并且我们在 LINQ 表达式 (PaymentController) 中包含 Account：

_erasDb.Payments.Include("Account").Include("Event").Where(...)

我们不知道为什么帐户为空。完整代码请参见：

http://pastebin.com/Yf1JDsMF

Answer 1

多次调用 ElementAt 效率非常低。只需切换到 foreach - 它应该修复 ElementAt 返回 null 的问题，更高效、更简单。

另外，为什么不只是：

<text> | </text>
foreach(var payment in Model.Payments)
{
    <a href=../../Payment/Details/@Model.ID/@payment.accountID">@payment.Account.landIOC</a>          
    <text> | </text>
}

如果您使用大量 @Html.Raw(...)，您可能错过了一个技巧...特别是，您很有可能打开 xss如果使用太多 .Raw，就会出现漏洞。

Answer 2

问题：您首先使用数据库、模型还是代码的 EF 方案？
在您看来，您可以使用 ; | 替换 Html.Raw(" | ");
同样，您的链接生成策略有点固定。我想这些链接有相应的操作，因此必须有路由，您可以使用 Html.ActionLink。
使用 foreach(var m in Model.Payments) 进行迭代可以使您的代码更加漂亮：

@{
 <text> |
  @foreach(var m in Model.Payments){
    <text>
    @Html.ActionLink(m.Account.landIOC,"Details","Account",new{Id = m.ID, AccountId= m.AccountID}) |
    </text>
  }
</text>
}

现在这可能是一个起点。