Ну не бывает такого
10.04.2019, 18:50
типичная ситуация с код-ревью:
5 строчек кода -- 10 чендж-реквкстов
500 строчек кода -- "вроде ок"
5 строчек кода -- 10 чендж-реквкстов
500 строчек кода -- "вроде ок"
Вот тут лучше сделать стрелочной функцией, смотри как коротко стало, одной строкой. Тогда уж и переменную назвём n, зачем ей длинное название, если она нигде дальше не используется.
Эм... а тут что? Стрелочная функция? А что это за w ей передаётся? Не, так не пойдёт, вообще нифига не понятно, анонимную функцию давай напиши, по-старинке.
Эм... а тут что? Стрелочная функция? А что это за w ей передаётся? Не, так не пойдёт, вообще нифига не понятно, анонимную функцию давай напиши, по-старинке.
Решается зафиксированным кодстайлом. Причем зачастую кодируется на уровне проекта и проходит автоформаттером.
А вообще доебываться на ревью последнее дело. Первое правило, которое человек на ревью должен понять - анализировать нужно качество решения, а не "а я бы по-другому сделал".
А вообще доебываться на ревью последнее дело. Первое правило, которое человек на ревью должен понять - анализировать нужно качество решения, а не "а я бы по-другому сделал".
Чтобы написать коммент, необходимо залогиниться
