我应该让我的对象属性为空还是为每种类型使用 CLR 默认值？

Question

我一直在尝试找出处理默认值的最佳方法。将 ID 值设置为 0 是有意义的，但如果它是货币值或其他最初未设置的值，当您稍后在代码中遇到它时，无法判断它是否已设置或设置为 0。如果货币值设置为空，那么你知道它没有被设置。此外，在处理数据库时，很容易知道是否需要将空值写入字段，而不是试图弄清楚它是否应该为空。

处理此问题的公认方法是什么？

Class MyModel
{
    public int Id {get;set;}
    public string Title {get;set;}
    public DateTime CreatedDate {get;set;}
    public bool IsActive {get;set;}

    //CLR will automatically set these values
    Public MyModel()
    {
        Id = 0; 
        Title = String.Empty;
        CreatedDate = "1/1/0001";
        IsActive = false;
    }
}

对

Class MyModel
{
    public int? Id {get;set;}
    public string Title {get;set;}
    public DateTime? CreatedDate {get;set;}
    public bool? IsActive {get;set;}

    //CLR will automatically set these values
    Public MyModel()
    {
        Id = null; 
        Title = null;
        CreatedDate = null;
        IsActive = null;
    }
}

Answer 1

您可以随时混合使用方法，具体取决于您的领域。

static class ID
{
    public const int Unassigned = -1;
}

class MyModel
{
    public int Id { get; private set; }
    public string Title { get; set; }
    public DateTime CreatedDate { get; set; }
    public bool IsActive { get; set; }
    public bool? IsAwesome { get; set; }

    Model () 
    {
        // you may use "default" constants...
        Id = ID.Unassigned;
    }

    // you may use optional parameters or overloads
    public MyModel (string title,
        DateTime created = DateTime.Now, // default values may be concrete 
        bool? isAwesome = null)          // or nullable as well
        : this ()                        // and you can chain-call constructors!
    {
        Title = title ?? "<no title>";   // you don't always want null to come through
        CreatedDate = created;     
        IsAwesome = isAwesome;    
    }
}

// Possible usages:
var model = new MyModel ("Hello", new DateTime (2001, 1, 1));
var model = new MyModel ("world", isAwesome: true);
var model = new MyModel (null) {
    IsActive = true
};

某些属性具有null 值可能是有意义的，例如“未设置”。

在您的示例中，模型在持久化到数据库之前可能真的没有Id。如果是这种情况，并且无 id 模型在业务逻辑方面有意义，则可以为空的 Id 比 Id = 0 更好。但是，如果您的应用程序从不使用无 id 模型，并且通常期望 Id 等于某个值，那么编写它会很疯狂

if (model.Id != null)

每次你想用它做点什么。

在这种情况下，您可能应该默认使用Id = 0。
你也可以引入一个常量（就像我在上面所做的那样），尽管我不推荐它用于除了 id 之外的任何东西，并且只有在它们被其他地方的代码大量使用的情况下。

同样，一切都取决于域。
您的工作是确保不能轻易创建违反业务规则的对象。

Answer 2

公认的处理方法是什么？

接受了吗？这取决于域。

如果将货币值设置为 null，则您知道它尚未设置。

不一定。想象一下，我正在使用一个银行应用程序，我想搜索一个特定的交易，我得到一个如下所示的对话框：

Enter the fields you know:
Transaction date: 6/26/2011
Payee: Apple
Amount:

现在TransactionSearchParameters.Amount 应该设置为空。您无法将其与未设置区分开来。

此外，在处理数据库时，很容易知道是否需要将空值写入字段而不是试图弄清楚它是否应该为空。

您应该花更多时间正确地为您的域建模，然后让 ORM 弄清楚如何将这些东西正确地放入数据库。

Answer 3

只是在这里添加一个想法，在适合使用时，总是有复杂类型的null object pattern。

Answer 4

这取决于您将如何使用您的代码。如果属性是简单类型，如整数和字符串，默认值更好，如果属性本身是对象，我使用空值，因为在 C#/Java/PHP 中，对象引用实际上是对象指针，更好用。

但是，如果您的属性是集合，例如 lists 或 maps ，那么创建集合并将其留空而不是 null 是一种“更好的做法”。

干杯。

Answer 5

嗯......“最好”的答案（在那里非常固执，但我有权）是当你不需要区分两者时，因为只有一种可能的状态，即有效：

class MyModel
{
    public int Id {get; private set;}
    public string Title {get; private set;}
    public DateTime CreatedDate {get; private set;}
    public bool IsActive {get; private set;}

    Public MyModel(int Id, string Title, DateTime CreatedDate, bool IsActive)
    {
        this.Id = Id; 
        this.Title = Title;
        this.CreatedDate = CreatedDate;
        this.IsActive = IsActive;
    }
}

我知道这并不总是可行的，例如 Query by Example。

通常会避免使用“将 ID 值设置为 0 有意义...”之类的“幻数”。一旦你现在这样做了，你编写的每一段代码都必须编码，以了解这些幻数是什么以及它们的含义。这是很少实现的，并且充满了令人讨厌的和容易出错的代码。

如果您只是必须区分具有值和不具有值的字段，那么您后面的示例会好一些。至少在这里你的值类型是明确有效或无效的。但是，使用 Nullable 意味着您必须使用不同的方法来确定类、字符串或其他引用类型是否无效。

IMO，您最好使用以下内容，尽管您将看到它变得非常冗长。

class MyModel
{
    private int _id;
    public bool HasId { get; set; }
    public int Id
    {
        get
        {
            if (!HasId) throw new System.InvalidOperationException();
            return _id;
        }
        set
        {
            HasId = true;
            _id = value;
        }
    }

    private string _title;
    public bool HasTitle { get; set; }
    public string Title
    {
        get
        {
            if (!HasTitle) throw new System.InvalidOperationException();
            return _title;
        }
        set
        {
            if (value == null) throw new System.ArgumentNullException("Title");
            HasTitle = true;
            _title = value;
        }
    }

    private DateTime _createdDate;
    public bool HasCreatedDate { get; set; }
    public DateTime CreatedDate
    {
        get
        {
            if (!HasCreatedDate) throw new System.InvalidOperationException();
            return _createdDate;
        }
        set
        {
            HasCreatedDate = true;
            _createdDate = value;
        }
    }

    private bool _isActive;
    public bool HasIsActive { get; set; }
    public bool IsActive
    {
        get
        {
            if (!HasIsActive) throw new System.InvalidOperationException();
            return _isActive;
        }
        set
        {
            HasIsActive = true;
            _isActive = value;
        }
    }
}

最后，如果你要走这条路，代码生成器会很好地为你服务。