ML
Dmitry Russ (Aleksandrov)
Не лучше when is_number(A) andalso A > 0 использовать? Т.е. так понятнее, что именно проверяется.
Лениво
Size: a a a
2020 March 08
TH
У нас такое кодревью не пройдет. Все, что нужно угадывать отправляется автору назад. Вне зависимости, корявый нейминг или нечитабельная логика
MK
Лениво
И A - 0 действительно работает как надо внутри стража?
ML
И A - 0 действительно работает как надо внутри стража?
да, но надо не забывать, что это эксепшн, а значит в некоторых местах этим нельзя пользоваться хотя бы из-за этого
MK
А, так оно таки кинет исключение?
MK
Я подумал, молча пойдёт проверять следующую сигнатуру
ML
эксепшны в гардах ловятся и не крешат программу, но есть более неприятные случаи:
1> A = undefined.
undefined
2> A - 0 > 0 orelse true.
** exception error: an error occurred when evaluating an arithmetic expression
in operator -/2
called as undefined - 0
ML
3> [V || V <- [1,2,3,4], true].
[1,2,3,4]
4> [V || V <- [1,2,3,4], A - 0 > 0 orelse true].
[]
ML
2> [V || V <- [1,2,3,4], true orelse A - 0 > 0].
[1,2,3,4]
3> [V || V <- [1,2,3,4], true or A - 0 > 0].
[]
or не ленивый, orelse ленивый
СИ
возможно, такой угрюмый код в макрос обернуть лучше, типа expect(is_numeric(...)), чтобы понятно было что исключение будет
MK
У нас такое кодревью не пройдет. Все, что нужно угадывать отправляется автору назад. Вне зависимости, корявый нейминг или нечитабельная логика
Я вот помню как однажды написал
!!x на каком-то языке типа Perl, так это у всех почему-то мозг взорвало.MK
или C
MK
Хотя например в ядре Linux двойное отрицание используется повсюду
MK
Так что угадывание зависит не только от автора ребуса, но и от способностей читателя.
TH
Сложность софтра растёт и сбивает с мысли какой-нибудь смысловой кульбит в коде, когда пытаешься почитать чей-то код. Или даже свой, спустя какое-то время.
MK
А еще хороший пример на мой взгляд - это код nginx
MK
Там у авторов есть неформальный "словарик", и допустим переменная с запросом во всем коде обозначается буквой
r.MK
И если ты этот словарик не знаешь, то будешь долго вдуплять что тут происходит, потому-что 90% названий одно, двух буквенные