Чистота кода VS оптимизация

Цитата Serafim @ 07.04.18, 00:40

Очевидно, что второй вариант на порядок лучше

Он лучше, когда основной код занимает больше 1 строки, а здесь какой смысл в этом нагромождении?

Цитата Pavia @ 06.04.18, 17:04

Но это противоречит безопасности if без else.

Как же делать if, в котором не нужен else? Прописывать else; тоже не фонтан...

Цитата Jin X @ 08.04.18, 06:56

Цитата Serafim @ 07.04.18, 00:40

Очевидно, что второй вариант на порядок лучше

Он лучше, когда основной код занимает больше 1 строки, а здесь какой смысл в этом нагромождении?

Сейчас там 1 строка, завтра может быть 10. Без "нагромождения" нужно будет переписывать.
pull-request'ы будут больше и дольше будешь апрувов ждать

Сообщение отредактировано: D_KEY - 08.04.18, 08:21

Цитата Jin X @ 08.04.18, 06:56

Он лучше, когда основной код занимает больше 1 строки, а здесь какой смысл в этом нагромождении?

Ранний выход из функции проще читается. Я хз как сформулировать своими словами почему именно проще, ты меня в тупик поставил

D_KEY, намекаешь на то, что все будут пытаться исправить?

Serafim, я понимаю, когда там хотя бы 2-3 строки основного кода. Или если есть большая вероятность, что код будет дописан. Но если предпосылок к расширению кода пока особых нет, чем это лучше?

const fetchData = (id) => {
  if (!id) return;
  fetch('/data/' + id)
}

против

const fetchData = (id) => {
  if (id) fetch('/data/' + id)
}

Хоть со {скобками}, хоть без...

Цитата Jin X @ 06.04.18, 12:46

Цитата villain @ 06.04.18, 12:21

как это не углубляйтесь? возможно там вообще деление не нужно

Суть вообще не в этом. Будем считать, что алгоритм достаточно оптимален (это часть кода разложения на простые множители и манипуляции с ними), но есть 2 вот таких варианта реализации
Там вообще умножение должно быть, я заменил его на деление для пущего эффекта

это не 2 варианта реализации, это 2 разных алгоритма
первый лучше - нет внезапного выхода из цикла, 2 условия vs 4
а за

while 1:
   ...
   if n % i: break

отдельный котёл положен

Цитата villain @ 09.04.18, 05:29

это не 2 варианта реализации, это 2 разных алгоритма

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

Цитата villain @ 09.04.18, 05:29

а за
...
отдельный котёл положен

Это отсутствие do-while в Python. Можно поменять 2 последние строки местами, как вариант, а перед циклом написать k=1

Цитата Jin X @ 09.04.18, 11:12

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

порядок действий разный? значит разные.

Цитата Jin X @ 09.04.18, 11:12

Это отсутствие do-while в Python. Можно поменять 2 последние строки местами, как вариант, а перед циклом написать k=1

зачем, если есть вариант без этих костылей?

Цитата Jin X @ 08.04.18, 14:33

Serafim, я понимаю, когда там хотя бы 2-3 строки основного кода. Или если есть большая вероятность, что код будет дописан. Но если предпосылок к расширению кода пока особых нет, чем это лучше?

Как минимум тем, что в первом варианте явно видна сигнатура Promise|null, а во втором - нет.

Да и, повторюсь, за опускание фигурных скобок хочется убивать В культурном мире - это как код без отступов - признак плохих манер или неуча за клавиатурой. По крайней мере в JS и некоторых других языках, где уже давно есть codestyle стандарты.

Цитата Serafim @ 09.04.18, 13:13

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

Ох уж мне эти "культурные люди", все-то они возводят в абсолют

void doSmthng(){
if(smthngBad1)throw new Exception("Something bad 1");
if(smthngBad2)throw new Exception("Something bad 2");
if(smthngBad3)throw new Exception("Something bad 3");
 
do();
}

Появление фигурных скобок читаемость ухудшит в разы.

Сообщение отредактировано: Астарот - 09.04.18, 14:34

Цитата Астарот @ 09.04.18, 14:33

Появление фигурных скобок читаемость ухудшит в разы.

Ага, конечно

function doSmthng() {
    switch (true) {
        case smthngBad1:
            throw new Exception("Something bad 1");
        case smthngBad2:
            throw new Exception("Something bad 1");
        case smthngBad3:
            throw new Exception("Something bad 1");
    }
 
    do();
}

Добавлено 09.04.18, 14:52
И то, это потому, что в JS нет нормального паттерн матчинга

Цитата Serafim @ 09.04.18, 14:44

Ага, конечно

И зачем ты три легко читаемые строчки превратил в... это? Да и речь шла про

Цитата Serafim @ 09.04.18, 13:13

за опускание фигурных скобок хочется убивать

Ну, вот случай, когда опускать их прямо показано

Цитата Serafim @ 09.04.18, 14:44

Ага, конечно

function doSmthng() {
    switch (true) {
        case smthngBad1:
            throw new Exception("Something bad 1");
        case smthngBad2:
            throw new Exception("Something bad 1");
        case smthngBad3:
            throw new Exception("Something bad 1");
    }
 
    do();
}

Вот за такое и правда, хочется взять и у@#$ть. Мне понадобилось несколько секунд, чтобы понять что за муйня такая этот ваш switch(true).

Сообщение отредактировано: applegame - 09.04.18, 15:39

Цитата applegame @ 09.04.18, 15:38

Вот за такое и правда, хочется взять и у@#$ть.

Редкий случай, но я вот тут соглашусь

Цитата Serafim @ 09.04.18, 14:44

И то, это потому, что в JS нет нормального паттерн матчинга

И чем тебе в этом случае поможет "нормальный" паттерн-матчинг?

Сообщение отредактировано: applegame - 09.04.18, 15:43

Цитата applegame @ 09.04.18, 15:42

И чем тебе в этом случае поможет "нормальный" паттерн-матчинг?

Думаю речь тут идет в том числе о том, что в js в case можно методы вызывать, то есть за исключением многословности тот же if получается.