Question

我需要多个类构造函数的帮助。我不想重复我的代码，但是如何？

    public EventModel(string name, DateTime startTime, DateTime endTime)
        : base(name)
    {
        StartTime = startTime;
        EndTime = endTime;
    }

    public EventModel(Guid id, string name, DateTime startTime, DateTime endTime)
        : base(id, name)
    {
        StartTime = startTime;
        EndTime = endTime;
    }

我看起来像这样：

    public EventModel(Guid id, string name, DateTime startTime, DateTime endTime)
        : this(name, startTime, endTime), base(id, name)
    {
    }

Answer 1

通常在这种情况下，我会将代码重构为常用方法，但缺点是您无法设置readonly个字段。

public EventModel(string name, DateTime startTime, DateTime endTime)
    : base(name)
{
    Initialize(startTime, endTime);
}

public EventModel(Guid id, string name, DateTime startTime, DateTime endTime)
    : base(id, name)
{
    Initialize(startTime, endTime);
}

private void Initialize(DateTime startTime, DateTime endTime)
{
    StartTime = startTime;
    EndTime = endTime;
}

Answer 2

如果您可以修改基类以接受可为空的Guid，则可以将id参数设为可选：

public EventModel(
   string name,
   DateTime startTime,
   DateTime endTime,
   Guid? id = null)
    : base(name, id)
{
   StartTime = startTime;
   EndTime = endTime;
}

然后在您的基类中，将null id视为仅包含name的构造函数。这样做可能会阻止您避免在基类中重复，但同时您可以减少基类中的重复。

此外，我强烈建议您拨打Guid类型参数guid。缩短到id几乎没有任何好处，并且更有可能混淆和减慢那些稍后查看代码的人。

Answer 3

为构造函数提供默认值。

    public EventModel(Guid id = 0, string name = "Default", DateTime startTime = new DateTime(0), DateTime endTime = new DateTime(0))
    : base(id, name)
{
    StartTime = startTime;
    EndTime = endTime;
}

Answer 4

让基类像这样实现。以下只是一个例子

基类

public Text(): this(0, 0, null) {}
public Text(int x, int y): this(x, y, null) {}
public Text(int x, int y, string s) {
  // Actual constructor implementation

使用示例

Text t1 = new Text();               // Same as Text(0, 0, null)
Text t2 = new Text(5, 10);            // Same as Text(5, 10, null)
Text t3 = new Text(5, 20, "Hello");

如何避免使用多个构造函数重复代码？

4 个答案: