Использование оператора else против неиспользованного присваивания

Question

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

У меня есть оператор if в контроллере mvc, который выглядит следующим образом:

if (submitButton != "Search")
{
ModelState.Clear();
}
TempData["searchParameter"] = searchParameter;

однакоесли условие ложно, TempData ["searchParameter"] никогда не используется.Лучше оставить код, как указано выше, или поместить назначение TempData ["searchParameter"] в оператор else?

Answer 1

Поместите его в оператор else.

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

Answer 2

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

В целом, я всегда использую оператор else, даже если поток кода был бы идентичен без него. Например:

if(null == foo) {
  return false;
} else {
  foo.Bar();
  return true;
}

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

Answer 3

В вашем примере TempData["searchParameter"] назначается независимо от того, submitButton != "Search". Это поведение, которое вы ожидаете?

Если нет, оператор else решит эту проблему за вас.

Answer 4

Я бы использовал инструкцию else.Делает это яснее и безопаснее, если в будущем будут сделаны какие-либо изменения.

Я думаю, это также хорошая практика.