Если ошибка;сделать что-то / если сделать что-то еще ошибка - PullRequest
0 голосов
/ 31 мая 2018

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

if (a) { code } else { different code }

вместо

if (! a) { different code } else { code }

, но в случае проверки ввода я думаю, чтоСледующий пример не легко читается:

первый пример

if (something) {
    if (somethingElse) {
        doSomething
    } else {
        error message: somethingElse is wrong (different message)
        exit
    }
} else {
    error message: something is wrong
    exit
}

Я думаю, что этот пример лучше (более читабелен), чем построение пирамиды условий:

второй пример

if (! something) {
    error message: something is wrong
    exit
}
if (! somethingElse) {
    error message: somethingElse is wrong (different message)
    exit
}

doSomething

exit

Вы бы выбрали первый или второй пример?

...