Что в какой ситуации выбирать?
В частности:
Пока меня не было, вопрос отредактировали. Почему Task.Result - свойство, а не метод? Что следовало бы поставить и почему? У него есть только геттер и оно не кешируется (наверно).
Связанный вопрос: Для чего нужны свойства?
С точки зрения рантайма разницы, конечно, нет. Свойство есть не более чем пара из двух функций — геттера и сеттера, причём эти функции, формально говоря, вовсе не обязаны читать/устанавливать какое-либо поле объекта.
Разница есть в семантике. Когда мы говорим об объекте, мы подразумеваем абстракцию чего-нибудь из окружающего мира. Это звучит довольно тривиально, но из этого следуют нетривиальные выводы.
У объекта есть данные, которыми он обладает, и поведение. То, что является данными, моделируется свойствами. То, что является поведением, моделируется методами. При этом совершенно не важно, представлены ли данные объекта полем, несколькими полями, или вообще вычисляются на лету — это подробности имплементации, которые никому «снаружи» не интересны. Это является одной из причин того, почему «наружу» нужно выставлять свойства, а не поля.
Допустим, у вас есть собака. У неё есть цвет шерсти, кличка, и она умеет выполнять команды. Теперь давайте рассмотрим разницу.
Кличка — это есть свойство собаки: это нечто, что есть объект (строка), присущий самой собаке. Её можно установить (например, новый хозяин собаки может поменять кличку), и прочитать. Соответственно, её нужно моделировать свойством:
public string Name { get; set; }
Цвет шерсти собаки — тоже её свойство. Но его невозможно «установить» снаружи (отвлечёмся от возможности перекрасить собаку: цвет краски и цвет шерсти — не одно и то же). Поэтому его нужно моделировать свойством, которое доступно только на чтение:
private readonly Color color;
public Color Color { get { return color; } }
(или для вышедшего пару дней назад C# 6 можно просто
public Color Color { get; }
поскольку значение можно установить в конструкторе).
Теперь, выполнение команды. Это не свойство самой по себе собаки, это ей поведение. Таким образом, правильно будет моделировать выполнение команды так:
public void ExecuteCommand(DogCommand command) { ... }
Заметьте, что формальные признаки («я только получаю значение поля!») нерелевантны. Например, если у вас команда, которую в данный момент выполняет собака, записывается во внутреннее поле, и вы хотите иметь возможность узнать, какую же команду собака выполняет в данный момент, то неправильно будет оформлять команду так:
private DogCommand currentCommand = null;
public DogCommand CurrentCommand
{
get { return currentCommand; }
set
{
if (currentCommand != null)
StopExecuting();
currentCommand = value;
if (currentCommand != null)
StartExecuting();
}
}
потому, что команда — это не неотъемлемое свойство самой собаки, а лишь её поведение. Правильно будет, например, так:
bool isExecutingCommand = false;
DogCommand currentCommand = null;
public void ExecuteCommand(DogCommand command)
{
if (isExecutingCommand)
CancelCommand();
isExecutingCommand = true;
currentCommand = command;
StartCommand(command);
}
public DogCommand GetCurrentExecutingCommand()
{
return isExecutingCommand ? currentCommand : null;
}
Методы - это действия. Свойства - это данные.
Вопрос подробно разобран в MSDN: Property Usage Guidelines: Properties vs. Methods
Если то, что вы пишете выглядит как кусок данных объекта - делайте его свойством. Например Control.Name.
Если то, что вы пишете:
.ToString())то стоит использовать метод.
DateTime.Now :)
– Pavel Mayorov
Jul 31 '15 at 13:09
Свойства - это тоже методы. Свойства компилируются в методы - гетеры и сетеры, причём даже гетер и сетер могут быть с более чем одним параметром, потому что они - самые что ни есть обычные методы со всеми их прелестями, но на C# такую конструкцию не написать, а вот на IL или VB пожалуйста: https://msdn.microsoft.com/en-us/library/bc3dtbky.aspx
Итого, свойства - это более удобный способ написания методов GetSomeValue() и SetSomeValue(value). Соответственно, отвечая на вопрос автора: для удобства и лаконичности стоит использовать свойства вместо методов, если их семантика GetValue и SetValue(value).
На этот вопрос не существует безусловно правильного ответа, поэтому всё, что будет написано в качестве ответов — не более, чем мнения.
Итак, личное мнение: свойства позволяют упростить работу с объектом. По большому счёту, всегда, когда есть возможность, лучше использовать свойства вместо методов, за исключением нескольких случаев.
Например, возможность есть, если у вас метод без параметров возвращает какое-то значение, не меняя состояния объекта — это свойство только для чтения. Здесь неважно, возвращается ли обычное скрытое поле или результат вычислений.
Что касается исключений, то они таковы: иногда для соблюдения инкапсуляции Вы не должны позволять менять отдельные свойства объекта. Пара или тройка свойств всегда должны меняться вместе и согласовано, чтобы обеспечить инвариантность объекта. В этом случае Вы можете сделать эти свойства только для чтения и добавить метод с двумя-тремя параметрами, который проверяет правильность инварианта на входе.
Другое исключение касается времени выполнения вычислений. Если время может оказаться "достаточно" большим, правильнее написать метод, а не свойство. Насколько понимаю, дело здесь в "психологии": когда вы видите Tree.Depth, то конструкция воспринимается как обычное получение значения поля, хотя в действительности может быть рекурсивным методом с большим временем выполнения.
Поскольку сами свойства это не более чем "сахар", то это часто вопрос личных предпочтений, но в целом всё уже правильно расписал PashaPash. Свойства должны выглядеть так, будто это поля вашего объекта, свойства лишь позволяют удобно их скрывать. Конкретно по вашим вопросам:
