Как я могу поделиться кодом конструктора с членами класса readonly?

Question

У меня есть класс, FooBarSet с одним «ядром» блока логики инициализации.

A FooBar изготовлен из Foo и Bar. Класс FooBarSet инициализируется списком FooBar. FooBarSet может также быть инициализирован отдельными параллельными Foo и Bar списками.

В идеале, я мог бы работать так:

public class FooBarSet 
{ 
    private readonly List<FooBar> _list; 

    // Primary constructor. 
    public FooBarSet(List<FooBar> foobarList) 
    { 
     // Contracts and initialization... 
     _list = foobarList; 
    } 

    // Secondary constructor. 
    public FooBarSet(List<Foo> fooList, List<Bar> barList) 
    { 
     // Zip a new list of new FooBars 
     var zipped = fooList.Zip(barList, 
           (foo, bar) => new FooBar(foo, bar)); 

     // Call primary constructor with zipped list. 
     this(zipped); 
    } 
} 

Это C#, а не Java, поэтому this(zipped) является незаконным. Общее решение, as in this answer, чтобы вытащить инициализации ядра в общий частный метод:

public class FooBarSet 
{ 
    private readonly List<FooBar> _list; 

    // Common "constructor" called from actual constructors. 
    private Init(List<FooBar> foobarList) 
    { 
     // Contracts and initialization... 
     _list = foobarList; 
    } 

    public FooBarSet(List<FooBar> foobarList) 
    { 
     Init(foobarList); 
    } 

    public FooBarSet(List<Foo> fooList, List<Bar> barList) 
    { 
     var zipped = fooList.Zip(barList, 
           (foo, bar) => new FooBar(foo, bar)); 

     Init(zipped); 
    } 

} 

Однако, это не работает, либо из-за readonly _list поля.

Предполагая, что _list должно быть readonly, как я могу заставить эти конструкторы делиться кодом инициализации?

Answer 1

Простой ответ, чтобы получить метод Init вернуть значение, которое будет установлено в конструкторе:

public class FooBarSet 
{ 
    private readonly List<FooBar> _list; 

    // Common "constructor" called from actual constructors. 
    private List<FooBar> Init(List<FooBar> foobarList) 
    { 
     // Contracts and initialization... 
     return whateverList; 
    } 

    public FooBarSet(List<FooBar> foobarList) 
    { 
     _list = Init(foobarList); 
    } 

    public FooBarSet(List<Foo> fooList, List<Bar> barList) 
    { 
     var zipped = fooList.Zip(barList, 
           (foo, bar) => new FooBar(foo, bar)); 

     _list = Init(zipped); 
    } 

} 

Answer 2

Вы можете переместить «молния» код статической функции и использовать это:

public class FooBarSet 
{ 
    private readonly List<FooBar> _list; 

    private static List<FooBar> Prepare(List<Foo> fooList, List<Bar> barList) 
    { 
     return fooList.Zip(barList, (foo, bar) => new FooBar(foo, bar)); 
    } 

    public FooBarSet(List<Foo> fooList, List<Bar> barList) : 
     this(Prepare(fooList, barList)) 
    { 
    } 

    public FooBarSet(List<FooBar> zippedList) 
    { 
     _list = zippedList; 
    } 
}