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

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

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

10 ответов

13 просмотров

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

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

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

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

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

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

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

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

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

я бы назвал resident

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

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

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

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

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

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
~ 2m21s  nix shell github:nixos/nixpkgs#stack ~  stack ghc -- --version error: … while calling the 'derivationStrict' builtin at /builtin/derivation.nix:...
Rebuild your mind.
6
Карта сайта