не видел

Привет.
Кто как относится к обьявлениям переменных внутри if в плане читабельности?

плохо читается, а дебажиться ещё хуже

ясненько, спасибо

Мрак)) Здається є інспекція в phpInspections плагіні. Називається assignment in condition.

Не вижу в этом проблем, особенно если во всем проекте все будут использовать так, то читаться будет нормально.

Лично мне такое край не нравится.
Но соглашусь, что если есть project's codestyle — проблем не будет.
Лишь бы придерживались все в команде и была автопроверка.

Просто буде в всьому проекті погано читатись )

Можно null object сделать там и все будет хорошо

В целом плохо. Читабельность еще полбеды, можно привыкнуть, а вот дебаг...

Нет ничего плохого в обьявлении переменной внутри IF, если соблюдены все условия:
- переменная будет использоваться только внутри иф блока
- вы точно уверены что возвращает выражение в переменную (тип), например bool или массив (пустой=false) или ?object, и т.д. к примеру, int, float - я бы поостерегся получать внутри if'a.
Но пример выше все равно деструктивен, по другой причине, два раза вычисляется метод, вдруг в getExtensionAttributes 10ти этажный SQL запрос? Ещё ладно если бы это был this->something - то есть метод под нашим явным контролем, а тут вообще тупо интерфейс входной параметр - то есть как реализовано мы заведомо не знаем.
Итого лучше получить сразу переменную, если она нулл - можно сразу выходить из метода, добавить еще один if, тогда обход айтемов будет без условных оператором, а по умолчанию метод будет возвращать true.
Ну правда защитники одного return'a в методе все равно могут прикопатся, но множественный возврат и в исходном коде есть))

і в результаті дивлячись на ці всі правила, умови і т.д. виникає питання, чому б просто не об’являти змінну завжди поза межами if і не заморочуватися?))

Конечно, так всегда лучше (не только в php), но когда ты пишешь уже 5 лет код, хочется его сделать как можно больше выразительнее. Главное не пытаться продавливать то или иное мнение на всю команду, особенно если там большой разброс по опыту.

все залежить від того хто ви, для кого та який обем повинні написати за наступну ніч
і якщо у вас термін на тиждень назад, причому, вам тільки годину назад, як сказали що все має працювати good
то там взагалі можна побачити що весь код взагалі генериться скриптами php))

звичайно коли мало часу, то не зважди виходить зробити так, як хотілося б - але це, на мою думку, стосується біль архітектури ніж самого кодстайлу) А щодо присвоєння змінної в if чи поза межами - на час практично не впливає) Навіть якщо піджимає час - то просто спочатку присвоїти змінній занчення, а тоді заюзати if якби сильно нікого не уповільнить, а читаємість і дебаг буде приємніший)

Привет)
Мы случайно заметили, что осталось всего 2 билета по цене Regular и цена переключится.
Предупредили вас, если что 😉
http://bit.ly/2VCU5Cs

Здравствуйте, подскажите пожалуйста - не могу прислать вам письмо на почту для получения промокода

Здравствуйте, напишите, пожалуйста, на эту почту: yana.borysova@fwdays.com

Спасибо, отправил