Различия между вызывающим методом внутри или снаружи конструктора

Если ваш метод GetValue является статическим методом в вашем классе, вы можете использовать его в Field Initialization. Если это метод экземпляра, вы получите сообщение об ошибке.

Ваш первый код использует Field Initialization, тогда как во втором коде вы инициализируете поле в конструкторе.

Поля C#

Поля инициализируются непосредственно перед конструктором для   экземпляр объекта, поэтому, если конструктор присваивает значение   поле, оно перезапишет любое значение, указанное во время объявления поля.

Насколько лучше, , я бы сказал, это зависит от вашего требования . Обычно, если вы передаете некоторый параметр конструктору, который вы хотите назначить для некоторого поля, вы можете сделать это только в конструкторе. Но если вы хотите иметь какое-то значение по умолчанию для полей, перед выполнением конструктора тогда полезно иметь полевой инициализатор.

Один случай, когда инициализация поля может быть осуществимой, а читаемость читаемости - в следующем случае

class Test
{
    string myVar;
    string anotherVar;

    public Test()
    {
        myVar = "one default";
        anotherVar = "another default";
    }
    public Test(string s) //: this()
    {
        myVar = s;
        anotherVar = "another default";
    }
}

Вероятно, вы не хотите вызывать конструктор по умолчанию, потому что он может быть ошибочным в некоторых случаях для инициализации некоторой переменной два раза (в моем случае это окончательно нормально); Поэтому в приведенном выше примере я переместил anotherVar в инициализацию поля.

Note: speaking about performance, which I don't on such nano-optimization levels - when you use field initialize your field receive it first value as desired. When you initialize it from constructor, it already have its default value, so technically speaking you assign a field twice. But this penalty is so small, that I can't imagine scenarios, in which it is plausible.

Код, который вы опубликовали, не компилируется, поэтому я пошел вперед и сделал несколько возможных вариантов.

1. Статическое поле и статический метод

public class Test
{
    static string myVar = GetValue();

    public Test()
    {
    }

    static string GetValue()
    {
        return String.Empty;
    }
}

В этом случае не имеет значения, где вы устанавливаете статическое поле: в конструкторе static (не представленном в коде) или в качестве инициализации статического поля. CLR будет инициализироваться как при загрузке типа в AppDomain. (Существует последовательность, такая как CLR вызывает статический конструктор вначале, а затем устанавливает все статические поля или наоборот - эта последовательность вышла из вашего контроля, хотя).

2. Настройка свойства в конструкторе

public class Test1
{
    string MyVar{get;set;}

    public Test1()
    {
        MyVar = GetValue();
    }

    string GetValue()
    {
        return String.Empty;
    }
}

Это зависит от типа и использования. В этом конкретном случае я, вероятно, избегал бы этого делать, поскольку конструктор должен был быть легким. Если вы выполняете какую-либо тяжелую обработку, лучше переместить это в метод Initialize с правильной обработкой ошибок и блоками try-catch . Если вы не выполняете тяжелую работу, чтобы настроить свойство, просто поместите значение по умолчанию в конструктор, например MyVar = "EMPTY" .

3. Установить свойство в другом классе.

public class Test
{
    public string MyVar{get;set;}

    public Test(){ }

    public string GetValue()
    {
        return String.Empty;
    }
}

//somewere else 
Test t = new Test();
t.MyVar = t.GetValue();

Это выглядит немного странно, поскольку класс предоставляет состояние (свойство) и означает получение текущего состояния (метод GetValue). Два члена (свойство и метод) в этом случае, вероятно, должны быть объединены в один getter .