Блог

 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 = "{}-{}-{}-{}"
 

Есть ли какая-либо причина, по которой я не должен этого делать? Я не могу придумать ни одного, но это кажется немного хакерским, и я не видел, чтобы это делалось где-либо еще.

Нет. Это обычный шаблон для fluent интерфейсов.

Это нормально. Могу ли я также предложить:

 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; 
}
 

Как было опубликовано выше, это очень распространенный шаблон проектирования, который направлен на то, чтобы сделать код более читаемым. Вы можете найти несколько терминов, используемых для описания такого кода (плавное кодирование, свободный стиль, свободный api, свободный интерфейс). С другой стороны, отлаживать сложно, а иногда даже невозможно. Промежуточные результаты недостижимы.