данные о пользователе, нормально ли в фильтре этого массива назвать переменную residentDataItem?
item вполне достаточно, видно же по названию массива чего там лежит
Ревьювер не принял
А на скобки случайно не ругался?)) Можно и без них когда 1 аргумент) фиг знает
Почему не принял? Человек выше написал хорошую мысль, по названию массива видно что это список, item - пункт/еденица списка, + чем меньше скоуп переменной, тем короче ее можно называть (лично мое мнение). То есть очевидно, что глобальную константу называть item никто не будет, но тут видно по названию массива + скоуп маленький
Хз, главное, что сейчас приняли
Как по мне это тоже самое что писать вот так вот: user.userName <- можно просто name, потому что и так видно, что свойство относиться к юзеру
Если договорились с командой - то делайте так дальше, просто прикольно будет, если кто то из ревьюверов в следующем мр так же коротко назовет переменную)
я бы назвал resident
ну или вообще const ownersCount = residentsDataList.filter(({ isOwner }) => isOwner).length
Вот это лучший вариант, согласен
Обсуждают сегодня