возвращение стиль значение вопрос

Question

Допустим, у меня есть следующий код

public static string GetXMLValue() 
{ 
    XDocument settingsFile = XDocument.Load("Settings.xml"); 
    return settingsFile.Element("Settings").Element("GenericValue").Value; 
} 

Он просто читает файл настройки XML и возвращает значение GenericValue. Это не может быть проще. Теперь мой вопрос заключается в том, будет ли он предоставлять любые benifit (читаемость, производительность, синтаксически, maintainablitiy и т. Д.), Чтобы сначала вернуть возвращаемое значение в строковой переменной, а затем вернуться? Или это лучше всего оставить так, как есть?

Answer 1

Чтобы быть честным, простота методов делает его доступным для чтения даже в «одной» линии:

public static string GetXMLValue() 
{ 
    return XDocument 
      .Load("Settings.xml") 
      .Element("Settings") 
      .Element("GenericValue") 
      .Value; 
} 

Answer 2

С точки зрения читаемости присвоение значений переменной и ее возврат определенно помогут.

Answer 3

Есть несколько ситуаций, в которых я вижу значение в создании вспомогательной переменной:

• Я хочу утверждать что-то о нем, как предпосылка (например, не пустая строка, минимальная/максимальная длина; и т. д.)
• У меня возникли проблемы, и я хочу отлаживать значение более легко.

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

Answer 4

бы это обеспечить любой benifit [...] в первую очередь возвращаемого значения в a строковая переменная, а затем возвращается? Или это лучше всего оставить так, как есть?

Функция настолько проста, что это просто не имеет значения, поэтому не теряйте сон об этом. Как только функция становится более сложной, вы всегда можете ее переосмыслить.

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

Как и в стороне:

Что я нахожу гораздо более сомнителен, что вы читаете внешний ресурс (файл) в методе получения. Вызов операций, которые могут иметь побочные эффекты (например, чтение файла) в геттере, - это плохой стиль IMHO. Таким образом, каждый вызывающий элемент getter должен обрабатывать IOExceptions при чтении файла.

Подумайте об этом, например, передав информацию через конструктор (либо прочитайте файл из конструктора, либо передайте объект, который заботится о предоставлении информации). Это отделит ваш дизайн и упростит, например, повторного использования и модульного тестирования.