class 方法返回“this”是不好的做法吗？

Question

public class Chain
{
    public string ChainString;

    public Chain() 
    {
        ChainString = "{}"; 
    }

    public Chain AddLink()
    {
        ChainString += "-{}";
        return this; // is this a bad idea?
    }
}

在上面的示例中， AddLink方法返回this 。 我想这样做的原因是为了更易读的实例化，如下所示。

// more verbose (here, `AddLink` returns void)
Chain myChain = new Chain();
myChain.AddLink();
myChain.AddLink();
myChain.AddLink();
// myChain.ChainString = "{}-{}-{}-{}"

// nicer
Chain myChain = new Chain()
    .AddLink()
    .AddLink()
    .AddLink();
// myChain.ChainString = "{}-{}-{}-{}"

有什么理由我不应该这样做吗？ 我想不出任何东西，但感觉有点老套，而且我还没有在其他地方看到过。

Answer 1

不，这是流畅界面的常见模式。

Answer 2

这可以。 我还可以建议：

public Chain AddLink(int links = 1)
{
    if (links <= 0) return this;

    string link = "-{}";
    var buf = new StringBuilder(ChainString, ChainString.Length + (links * link.Length));
    for(int i=0; i<links; i++)
    {
        buf.Append(link);
    }

    ChainString = buf.ToString();
    return this; 
}

Answer 3

如上所述，这是一种非常常见的设计模式，旨在使代码更具可读性。 您可能会发现用于描述此类代码的几个术语（流畅编码、流畅风格、流畅 api、流畅界面）。 另一方面，调试起来很困难，有时甚至是不可能的。 中间结果是无法实现的。