MVC 视图中过多的 Null 检查

Question

这是一个关于风格和设计而不是语法的问题。

我的域模型具有多个（由于缺乏更好的术语）导航属性。因此，在具有 Bar 类型属性的 Foo 强类型详细信息视图中，我可以执行以下操作：

<%: Foo.Bar.Name %>

但是，有时 Bar 为 Null，所以我最终会得到类似的结果：

<%: Foo.Bar == null ? String.Empty : Foo.Bar.Name %>

在其他情况下，因为导航属性的便利性，我可以做更多的链接。然而，缺点是在我的视图中引入了更多的空检查。

作为替代方案，我可以在 ViewModel 中执行所有空检查，以便将一些“干净”的内容传递给视图。我正在寻找一些想法或常识指南，以避免在我的观点中进行过多的空检查。

PS 我正在使用 ASP.NET MVC，但我认为这个问题可能与其他 MVC 框架相关。

Answer 1

你自己已经回答了：

作为替代方案，我可以在 ViewModel 中执行所有 null 检查，以便将一些“干净”的内容传递给 View。

就是这样，使用 ViewModel。

Answer 2

不确定它在您的上下文中是否真的有意义，但是当您被过多的空检查困扰时，您应该考虑 空对象 模式。

Answer 3

创建一个帮助器来检查 null 并返回适当的默认值。

例如，如果您有一个电话号码字段，则可以向其传递该字段和一个常量 PHONE。帮助器检查是否为空，如果为空则显示“没有可用的电话号码”。

这样您就可以将所有空检查保留在一个地方。

Answer 4

一种选择是使用助手，如此处所述。

Answer 5

您可以为模型中的导航属性提供延迟初始化。这样您就可以最大限度地减少遇到空值的风险。

public class Foo
{
    private Bar _bar;

    public Bar Bar
    {
        get
        {
            if (_bar == null)
            {
                _bar = new Bar();
            }
            return _bar;
        }
    }
}

public class Bar
{
    public string Name { get; set; }
}

Answer 6

首先考虑在您的模型中，如果 Bar 为 null，则 Foo 的实例是否有效。如果不是，那么您应该在 Bar 属性的构造函数和/或设置器中放置一个保护子句。

public class Foo{

        public BarClass _bar;

        public BarClass Bar{
            get{return _bar; }
            set{
                if(value == null)
                   throw new ArgumentNullException();
                _bar = value; } 
            } 

           public Foo(BarClass bar){
              if(bar == null) throw new ArgumentNullException();
              _bar = bar;
          } 
        }

其次，过多且重复的空检查会产生代码异味。您应该使用重构用多态性替换条件。您可以根据空对象模式创建空对象。然后确保 Bar 属性始终使用 NullBar 进行初始化。

public class Foo{

    public BarClass _bar;

    public BarClass Bar{
        get{return _bar ?? new NullBar(); }
        set{ _bar = value ?? new NullBar(); } 
    } 

       public Foo(){
          Bar = new NullBar(); // initialization
       } 
    }

那么 Bar 永远不会为空。