Вверх ↑
Разработчик
Ответов: 4668
Рейтинг: 420
#1: 2018-01-17 23:57:08 ЛС | профиль | цитата
AlexKir писал(а):
Но самое забавное вы ХВАСТАЕТЕСЬ ровно тем, что я ругаю, читабельность кода подобные пассажи резко снижают .

Возьмем строчку 206 из реально используемого мной кода:
let logins = room.getMembers().map(x => RegExp.escape(x.name)).sort().reverse().join('|');
Когда таких операций становится много, их, конечно же, стоит писать вот так (особенно в продакшен коде):
let logins = room.getMembers()
.map(x => RegExp.escape(x.name))
.sort()
.reverse()
.join('|');
Без лямда функции это выглядело бы минимум так:
let logins = room.getMembers();
for (let key in logins){
logins[key] = RegExp.escape(logins[key].name);
}
logins = logins.sort().reverse().join('|');
А если бы я еще и сортировал по своему порядку (+1 лямда функция) и еще какие-то пост-обработки добавлял, код бы растолстел еще и еще.

Сравнивайте сами, что проще и легче читается
карма: 10
0
Редактировалось 2 раз(а), последний 2018-01-17 23:58:48