Size: a a a

2021 July 30

AS

Anton Samofal in Laravel Pro
Ну это я на ходу выдумал рандомное... вот конкретно сейчас раздумываю как мне лучше $usedGiftCards или $giftCardsUsed
источник

S

Sergey in Laravel Pro
это же пыхапэ чатик. и заходить с него вот так с размаху с кэмэлкейсом - это опасно достаоочно! снейк кейс и только!
источник

AS

Anton Samofal in Laravel Pro
та сейчас речь даже не об этом) К слову, у меня здесь легаси, тут везде снейк кейс, но сам по себе кемел кейс мне больше нравится, дабы писать в единном стиле вместе с js
источник

?

? in Laravel Pro
Второй вариант более читабельнее как по мне
источник

AS

Anton Samofal in Laravel Pro
о, а вот и сторонник второго варианта есть)
источник

AS

Anton Samofal in Laravel Pro
но судя по опросу, первый вариант пока лидирует
источник

?

? in Laravel Pro
Я в опросн первый вариант выбрал
источник

AN

Alexander N in Laravel Pro
Я больше склоняюсь к первому варианту конечно хоть и суть что хранится в переменной на 2 месте
источник

MG

Max Grom in Laravel Pro
Если вы ищете совета, то лучше всего если вы определите нужный формат в команде/проекте. Здесь можно собрать мнения, но работать с кодом вам и вашим коллегам. Попытайтесь формализировать это и дальше проект будет иметь целостный вид и все последубщие кто с ним будет работать может и будут удивляться принятому решению, но точно будут рады, что это не выглядит как зоопарк
источник

AS

Anton Samofal in Laravel Pro
Да, это понятно, мне просто стало интересно мнение людей здесь. Какое мнение популярней в обществе
источник

MG

Max Grom in Laravel Pro
Понял. Ну я вот удивлён что голосуют за второй вариант. Субъективно считаю, чем больше это похоже на нормальную английскую речь - тем лучше. Это как например использовать для методов которые возвращают bool не только префикс is - isActive(User $user) но и areActive(Collection $users). То есть чем более это читабельно - тем лучше
источник

MG

Max Grom in Laravel Pro
Ну а наличие одинаковых переменных в одном контексте как и прежде говорил - скорее свидетельствует о проблемах в этом контексте
источник

AS

Anton Samofal in Laravel Pro
Та я и сам в голосовалке все же отдал предпочтение первому варианту, хотя и второй для меня не так очевидно плох
источник

MG

Max Grom in Laravel Pro
А в таком варианте?
$postFactory->makeFrom($draftNew, $parametersNew);
$postFactory->makeFrom($newDraft, $newParameters);
источник

AS

Anton Samofal in Laravel Pro
Второй вариант лично мне нравится больше :) Но я поймал себя на мысли, что здесь роль может играть длина самой переменной. Когда это что-то короткое, например, $newOrder, $newTicket, $approvedInvoice то все прям очевидно. Глазам легко сразу увидеть, что имено хранит переменная. Но если имя пременной растет, то вот здесь я уже не всегда радуюсь ее описание в начале. Когде тебе нужно найти конкретную переменную, а приходится перечитывать описание кучи других, которые тебе сейчас не нужны...
источник

AS

Anton Samofal in Laravel Pro
Но я так же и согласен с мнение про контекст, не всегда нужно максимально описывать переменную в ее имени... вот только когда работаешь с легаси, не всегда скоуп узкий. Я сейчас рефакторю один метод создания ордера, его длина 848 строк, мать его
источник

AS

Anton Samofal in Laravel Pro
Например, можно встретить и вот такую переменную $custom_payment_methods_credit_note_amount
источник

S

Sergey in Laravel Pro
сначла глагол, затем сущность. fetchProducts, restoreCommit. бизнес логика по сущностям разносится по котнороллерам/сервисам. дидиди же.
источник

AS

Anton Samofal in Laravel Pro
Глагол в переменной?)
источник

S

Sergey in Laravel Pro
конкретно по озвученной проблеме, следует руководствоваться правилами языка. правильно new ticket. но да, глагол в переменных: fetchedProducts, restoredCommit
источник