Какой самый чистый способ написать это, если .. тогда логика?

Question

Они оба делают одно и то же. Один способ лучше? Очевидно, что если я напишу код, я буду знать, что я сделал, но как насчет того, кто его читает?

if (!String.IsNullOrEmpty(returnUrl))
{
    return Redirect(returnUrl);
}
return RedirectToAction("Open", "ServiceCall");

ИЛИ

if (!String.IsNullOrEmpty(returnUrl))
{
   return Redirect(returnUrl);
}
else
{
    return RedirectToAction("Open", "ServiceCall");
}

Answer 1

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

if(String.IsNullOrEmpty(returnUrl)) { 
  return RedirectToAction("Open", "ServiceCall"); 
}
else{ return Redirect(returnUrl); }

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

Answer 2

С точки зрения обслуживания я предпочитаю первую версию. Я видел меньше ошибок с последовательным стилем «уйди раньше».

Тернарный оператор (? :) в порядке, но только если маловероятно, что новый код будет вставлен перед вторым оператором возврата. В противном случае, когда придет время добавить что-то новое во 2-й путь кода, троичный оператор должен в любом случае вернуться к блоку if / else.

Answer 3

Как здесь , но более читабельно:

return
  string.isNullorEmpty(returnUrl) ? 
     RedirectToAction("Open", "ServiceCall") :
     Redirect(returnUrl);

Этот способ работает, доступен для чтения и не является избыточным.

Answer 4

Первый. Вам больше ничего не нужно, если вы возвращаете значение внутри оператора if. Итак:

if (!String.IsNullOrEmpty(returnUrl))
    return Redirect(returnUrl);
return RedirectToAction("Open", "ServiceCall");

Answer 5

Мне нравится первый лучше.

if ( ) 
{
    return ... 
}
return 

Для меня это звучит как «по умолчанию», вы можете связать больше условий, но в конце есть значение по умолчанию.

Конечно, это вопрос стиля.

Дополнительный вопрос.

Это стиль C # - заключать квадратные скобки в одну строку?

if ( ) 
{
}
else
{
}

Я видел, как это проникло в примеры кода Java в SO, и мне интересно, что это коренная причина.

EDIT

@ Оуэн. Я имею в виду, это стиль C # с использованием этой формы?

if ()  
{
    code ... 
}
else
{
    code...
}

Вместо этого (это будет Java предпочтительнее )

if ( ) { 
    code ... 
} else { 
   code ...
}

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

Answer 6

Как насчет:

if (!String.IsNullOrEmpty(returnUrl))
    return Redirect(returnUrl);
else
    return RedirectToAction("Open", "ServiceCall");

Answer 7

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