169 похожих чатов

Есть массив объектов residentsDataList, каждый объект хранит в себе некоторые

данные о пользователе, нормально ли в фильтре этого массива назвать переменную residentDataItem?

10 ответов

4 просмотра

item вполне достаточно, видно же по названию массива чего там лежит

Иван Степанов
Ревьювер не принял

А на скобки случайно не ругался?)) Можно и без них когда 1 аргумент) фиг знает

Иван Степанов
Ревьювер не принял

Почему не принял? Человек выше написал хорошую мысль, по названию массива видно что это список, item - пункт/еденица списка, + чем меньше скоуп переменной, тем короче ее можно называть (лично мое мнение). То есть очевидно, что глобальную константу называть item никто не будет, но тут видно по названию массива + скоуп маленький

Иван Степанов
Хз, главное, что сейчас приняли

Как по мне это тоже самое что писать вот так вот: user.userName <- можно просто name, потому что и так видно, что свойство относиться к юзеру

Иван Степанов
Хз, главное, что сейчас приняли

Если договорились с командой - то делайте так дальше, просто прикольно будет, если кто то из ревьюверов в следующем мр так же коротко назовет переменную)

я бы назвал resident

ну или вообще const ownersCount = residentsDataList.filter(({ isOwner }) => isOwner).length

Anton Permyakov
я бы назвал resident

Вот это лучший вариант, согласен

Похожие вопросы

Обсуждают сегодня

Кстати, а я вот тут подумал. Допустим, у нас имеется цикл который выполняет огромное количество итераций, но мы хотим, чтобы какие-то действия исполнилось только один раз. В Я...
The Bird of Hermes
23
Доброй ночи. Вопрос знатокам. Имеется некая таблица, результат которой выведен в DBGrid на форме. И есть форма, с помощью которой можно как добавить запись, так и отредактиров...
Евгений
28
а всё почему? потому что ассемблер в отличии от яву порождает множество пагубных привычек, среди которых например можно отметить использование глобальных переменных для всего ...
Mixail Frolov
35
анрей С какой целью зашёл?
КТ315
42
я часов 15 назад начал пытаться написать хоть что-то напоминающее ос и у меня есть проблема, которую я пытаюсь решить последние часов 5: есть крч 2 исходника с минимальным код...
Al1to
17
Кто нибудь из участников чата пишет на ассемблере под GNU/Linux?
Spiker01
24
Всем привет ребят! Мб кто-то сталкивался? adminСomponents .get("searchСomponents", use: searchComponents) .openAPI( summary: ...
Admin Adminov
2
Насколько моя реализация Exercise 1 плохая? data MessageType = Info | Warning | Error Int deriving (Show, Eq) type TimeStamp = Int data LogMessage = LogMessage ...
= ?
10
Кстати вопрос. Проанализировав свои проекты я понял, что в наиболее ценных из них Асм использовался для организации real-time процессов (внутри DOS). А какие есть способы сего...
Alexander Morozov
18
Я не очень много работал с потоками, тут возник вопрос - если будет одновременный доступ к памяти из двух разных потоков, это само по себе приведет к ошибке?
The Bird of Hermes
25
Карта сайта