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

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

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

10 ответов

11 просмотров

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

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

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

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

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

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

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

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

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

я бы назвал resident

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

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

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

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

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

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Мужики. привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных элементов...
Kraszx
2
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Товарищи, кто работа с iphelper? Или может я в самой логике ошибки фигачу, не пойму.... var ifTable : PMIB_IFTABLE; size, corSize: DWORD; Buffer ...
Warfarellen
4
я так понимаю, я так подозреваю, что создание такого плагина для человека, кто умеет писать плагины для делфи потребует минут 5-10 времени. но это мое подозрение. хотелось бы ...
Kraszx
7
Всем привет! Кто пользуется DevExpress, подскажите пожалуйста, реализован ли в TcxGrid в новых версиях поиск по датам как в Экселе (ну т.е. не просто список чекбоксов со значе...
A Z
4
Карта сайта