Комментарии 23
Так и должна быть оформлена, на JS, как и на Си, особо не разгуляешься.
На каком-нибудь С++, C# или Rust надо делать типы Point и Vector с переопределёнными арифметическими операторами, это уменьшит число строк вдвое с увеличением понятности.
Всё это полная чушь.
Линтеры, которые следят почти исключительно за оформлением кода — просто слабые. Их слабость может быть вызвана сложностью/гибкостью языка, который слишком сложно анализировать, но это ничего не меняет — существуют линтеры, по крайней мере в других языках, которые способны на очень многое помимо проверки оформления кода.
Если Вы способны отличить плохой код только по косвенному признаку (плохое оформление), то у меня для Вас печальная новость: делать ревью Вам ещё рановато. Да, требование запускать линтер приводит к тому, что и хороший и плохой код выглядит одинаково… но нет, это не мешает, а помогает при ревью, потому что легче сосредоточиться на том, что и как этот код делает, не отвлекаясь на то, как он оформлен.
И, как Вам уже ответили выше, если следование требованиям линтера приводит к ухудшению читабельности кода, то им не надо следовать — надо или менять настройки линтера, или отключать его (в идеале — только мешающее правило, а не весь линтер) локально для данной строки/блока/функции (обычно это делается комментарием в определённом формате).
Зависит от качества линтера. Например, в среднем проекте на Go (около 50к строк) у меня директивы для линтера встречаются один раз на 488 строк, в мелком проекте на Perl (около 2к строк) они встречаются один раз на 89 строк. В обоих случаях чтению кода они совершенно точно не мешают.
Ну конечно не мешает. Любителям линтеров даже лишний пробел мешает читать код. Что уж говорить про целые строки комментариев врезанные по среди логики. А если вам не мешает, так вам и линтер не нужен. Вы видимо обладаете навыком чтения кода игнорируя лишнее.
Вполне возможно, на переделку нужно отправить не код, а конфиг линтера.
А вы когда-нибудь пробовали продавить изменения в конфиг линтера не будучи при этом тимлидом?
Ну либо самого ревьюера.
А вы когда-нибудь пробовали изменить представление человека о хороших и плохих практиках не будучи при этом его прямым руководителем?
Отсутствие ошибок линтера — только одно из условий, хоть и обязательное
Именно. Это лишь необходимое условие. А автор подумал, что оно же и достаточное. :)
Макдональдс копнули глубже.
С красивыми девушками болтают больше – следующие покупатели ждут дольше.
С баб Манями болтают дольше – следующие покупатели ждут дольше.
С женщинами вообще…
Мак нанимал в официантки и поварихи только яценосителей!
Зарплаты были чуть пожирнее, но это было так наркотически выгодно, что, естественно, даже запретили законом.
О линтерах, качестве кода, качестве вообще и управлении качеством