三元最佳实践

Question

我有一些代码，尽管做起来很简单，但也许可能并不立刻明白。

I found @(Model.Count() == 0 ? "no" : Model.Count().ToString()) @(Model.Count() == 1 ? "person" : "people")
@foreach (var item in Model) {
   <div>@item.Name at @item.Email</div>
}

在编写大量这样的代码之前，我想知道这是否是实现此目的的好方法。

因此，问题是，在.NET中是否有更好的框架方法可以执行此操作，或者三元方法是否还可以

前提显然是

• 0条记录=我没有人
• 1条记录=我找到1个人
• 2条以上记录=我发现2个人

Answer 1

在我看来，将Ternary条件运算符用于这种条件绝对是很好的。

有经验的开发人员无需考虑即可理解它，但是如果您想让初学者更容易理解它，则也可以使用if和else构造。

但是我将使用Any()作为注释中提到的@ I4V。

I found @(Model.Any() ? Model.Count().ToString() : "no") @(Model.Count() == 1 ? "person" : "people")


@foreach (var item in Model) {
   <div>@item.Name at @item.Email</div>
}

Answer 2

如果您在几个地方这样做，扩展方法将同时解决您的问题（可读性和简化代码）

public static string PersonCountString(this IEnumerable<Person> personList)
{
    var count = personList.Count();
    return String.Format("{0} {1}", count > 0 ? count : "no",
                                    count == 1 ? "person" : "people");
}
...
I found (@Model.PersonCountString())

Answer 3

要回答您的问题：不，我发现oneliner不可读，对我来说就像@(() 0 ? "" : .().()) @(.() == 1 ? "" : "") ，更不用说对.Count()的多次调用了。

您可以创建一个（共享的）辅助方法，如下所示：

string GetCountWithDescription(int count, 
                               string singleItemDescription, 
                               string multipleItemsDescription)
{
    switch (count)
    {
        case 0:
            return "no " + multipleItemsDescription;
        case 1:
            return "1 " + singleItemDescription;
        default:            
            return count + " " + multipleItemsDescription;
    }
}

也可以重复使用，因此您可以将其粘贴到一个单独的文件中，以免使代码混乱，并且可以从每个视图中简单地调用它，如下所示：

@GetCountWithDescription(Model.Count(), "person", "people")

Answer 4

您想达到什么目的？ 更好的可读性或更快的代码（开发）？ 如果目标是提高可读性，那么我建议将三元运算保留在字符串中，例如：

string modelCountStr = Model.Count() == 0 ? "no" : Model.Count().ToString(); string modelPluralStr = Model.Count() == 1 ? "person" : "people";

Answer 5

考虑到如果存在任何用户，则必须使用Count() ：

@{ 
    var count = Model.Count();
}

I found @(count == 0 ? "no" : count) @(count == 1 ? " person" : " people")

Answer 6

使其更具可读性的另一种方法是选择一种解决方案，其中尽可能少地或理想地涉及零条件。

这是我的看法：

@{ 
    var count = Model.Count(); 
}
I found @string.Format(new[] { "no people", "{0} person", "{0} people"} [Math.Min(count, 2)], count)

可以说Math.Min负责某种分支，但是我认为这更容易理解。