C# 构造函数 - 设置第二个属性

Question

我有这个类：

public class CalendarData_Day
{
    public DateTime Date { get; set; }
    public DayType TypeOfDay { get; set; }
    public bool Choose { get; set; }

    public CalendarData_Day(DateTime datum) : this(datum, DayType.Normal, true)
    {
    }

    public CalendarData_Day(DateTime datum, DayType typDne) : this(datum, typDne, true)
    {
    }

    public CalendarData_Day(DateTime datum, DayType typDne, bool vybran)
    {
        this.Date = datum;
        this.TypeOfDay = typDne;
        this.Choose = vybran;
    }
}

我想在第二个构造函数中检查 DayType 是否为 Weekend，如果不是则不发送到 Choose true but false。有人知道我该怎么做吗？ 我知道我可以添加到最后一个构造函数 if 并检查，但它似乎不适合我。我认为有更好的方法，我认为我应该以其他方式来做，或者这在最后一个构造函数中可以吗：

if (TypeOfDay == DayType.Weekend)
    this.Choose = false;

我知道它有效，但我不知道这是正确的方法。

编辑： 很抱歉我没有解释一切。有超过 2 个 DayType，假设有 Holiday、Work，...我希望用户可以仅使用第二个构造函数来调用类，如果 DayType 是 Weekend 或 Holiday，则 Choose 必须为 false，但如果是 Normal 或Work 应该为 true，否则用户必须使用最后一个构造函数并将 DayType 设置为 Work，并将 Choose 设置为 false。这很复杂，很抱歉我应该第一次写这个。

Answer 1

最好根据参数传递链式构造函数参数：

public CalendarData_Day(DateTime datum, DayType typDne)
    : this(datum, typDne, typeDne != DayType.Weekend)
{        
}

这样您就不需要将属性设置两次 - 一次设置为某种默认值，然后根据您已知的信息修复它。

我个人会将参数名称从 typDne 更改为 dayType 或类似名称。

编辑：我刚刚看到您正在考虑将测试放入最后构造函数而不是第二构造函数中。我希望按原样接受调用者为 vybran 给出的值，而不是有条件地忽略。您仅描述希望第二个构造函数检查DayType == Weekend - 而不是最后一个构造函数 - 因此只有第二个构造函数应该更改。

编辑：如果周末或假期的Choose必须为假，那么我会在最后一个构造函数中强制执行，但选择值 em> 在第二个构造函数中：

public CalendarData_Day(DateTime datum, DayType typDne)
    : this(datum, typDne,
           typeDne != DayType.Weekend && typeDne != DayType.Holiday)
{        
}

public CalendarData_Day(DateTime datum, DayType typDne, bool vybran)
{
    if (vybran && (typeDne == DayType.Weekend || typeDne == DayType.Holiday))
    {
        throw new ArgumentException(
           "vybran cannot be true for holiday or weekend dates", "vybran");
    }
    this.Date = datum;
    this.TypeOfDay = typDne;
    this.Choose = vybran;
}

Answer 2

验证构造函数中的参数是完全可以的。

不过，您应该确保如果条件为 false，您的代码仍然按预期工作：

if (TypeOfDay == DayType.Weekend)
{
   this.Choose = false;
}
else
{
   this.Choose = vybran;
}

如果传入的参数完全错误，您可以考虑抛出 ArgumentException。

Answer 3

如果您的构造函数之一具有非常广泛的逻辑，您可能会受益于调用私有构造函数的静态构造函数。

class MyClass
{
  private MyClass(...)
  {

  }
  public static MyClass CreateMyClassWithValidation(...)
  {
    if(....)
      return new MyClass(...);
  } 
}