Элегантная проверка на ноль и выход в C #

Question

Что такое элегантный способ написать это?

if (lastSelection != null)
{
    lastSelection.changeColor();
}
else
{
    MessageBox.Show("No Selection Made");
    return;
}

changeColor() - это функция void, а функция, выполняющая вышеуказанный код, также является функцией void.

Answer 1

Вы можете уменьшить беспорядок, изменив условие:

if (lastSelection == null)
{
    MessageBox.Show("No Selection Made");
    return;
}

lastSelection.changeColor();

Answer 2

Хотя мне лично очень не нравится иметь несколько возвращаемых операторов в функции, в соответствии с защитным программированием, которое я видел, вы перехватываете состояние ошибки и выходите и пропускаете что-либо еще:

if(lastSelection == null)
{
    MessageBox.Show("No Selection Made");
    return;
}

lastSelection.changeColor();

Трудно сказать, как бы я делал это в своей работе, не видя всей функции, к которой он относится.

Answer 3

Я хотел бы убедиться, что lastSelection никогда не будет нулевым, и иметь указатель на «пустой» объект в тех случаях, когда он будет нулевым в текущем проекте. Тогда нет необходимости в нулевой проверке. Разработка возможных ошибок вне системы - это всегда хорошая практика.

IOW, я бы спросил, является ли неравенство в коде тем, как вы делаете нулевую проверку, или это тот факт, что нулевая проверка требуется вообще .

Answer 4

Это еще один способ реализации этого.

        Action _action = (lastSelection != null
                              ?
                                  new Action(lastSelection.changeColor)
                              :
                                  () => Console.WriteLine("No Selection Made"));
        _action.Invoke();

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

Answer 5

Видя, что все получили хорошие ответы, НЕ делайте этого :

if (lastSelection == null) goto ERR;
lastSelection.changeColor();
//... possibly more stuff...
return;
ERR:
MessageBox.Show("No Selection Made");

Answer 6

Практика быстрого разгона - сначала написать ноль, как показано ниже:

if (null == lastSelection)
{
    MessageBox.Show("No Selection Made");
    return;
}

lastSelection.changeColor();

Answer 7

Я думаю, что ваш код хорош: сначала вы проверили нормальный регистр (lastSelection != null), а затем ненормальный.

Я могу просто добавить одну модификацию:

if (lastSelection != null)
{
    lastSelection.changeColor();
    //your other normal code
}
else
{
    MessageBox.Show("No Selection Made");
    //return; //no need for this now
}