А ты не думаешь, что такого рода проверки вообще не нужны? Точнее должны удаляться в рабочей версии, по подобию assert'ов. Соответственно размер не имеет особого значения.
Alexander Ulizko
· 10 months ago
Если бы можно было гарантировать, что весь код напишу я и буду поддерживать только я, то такие проверки можно и удалять. А для библиотечных функций, тем более, создаваемых с прицелом на то, чтобы отпустить их в "большой мир", такие проверки, имхо, must have.
Степан
· 8 months ago
Уж лучше эксепшн кинуть, чем возвращать не пойми что:
if (!when || !byWhom) { throw new Error("destroyTheWorld: required parameters 'when' and 'byWhom' are missing or undefined."); }
Так сразу будет понятно где и почему возникает проблема.
Alexander Ulizko
· 8 months ago
В данном случае пост немножко не о том, как должен сообщать метод о недостаточном количестве параметров. Но если затрагивать такую тему, как общую культуру программирования, то вы, безусловно, правы.
All Comments
if (!when || !byWhom) {
throw new Error("destroyTheWorld: required parameters 'when' and 'byWhom' are missing or undefined.");
}
Так сразу будет понятно где и почему возникает проблема.
недостаточном количестве параметров.
Но если затрагивать такую тему, как общую культуру программирования, то вы,
безусловно, правы.