Size: a a a

2021 March 11

SZ

Sergey Zolotov in PHP
Artem Molotov
Когда с одной стороны советуют следовать рекомендациям и практикам Гугла, а с другой стороны сотрудники из него уходят и жалуются на игнор инициатив по важным вещам, как при перфоманс ревью, к примеру...
ну слепо следовать и играть в крупную корпорацию никто не советует) а вот внутренние процессы довольно интересные
источник

AM

Artem Molotov in PHP
Sergey Zolotov
ну слепо следовать и играть в крупную корпорацию никто не советует) а вот внутренние процессы довольно интересные
эт я не придераюсь, просто иногда удивляют жизненные ситуации
источник

AM

Artem Molotov in PHP
в целом то оно понятно
источник

k

knopkod4v in PHP
Иван Лещёв
я вот вынужден от 3 до 6 лишних файлов править просто потому что нет автовайринга и  в сервисе часть вызовов не проксируется, а надо каждый новый прописывать
как #[Required] помогает прокисровать?
источник

ИЛ

Иван Лещёв in PHP
knopkod4v
как #[Required] помогает прокисровать?
это к автовайрингу
источник

SZ

Sergey Zolotov in PHP
Иван Лещёв
это к автовайрингу
лучше все же конструкторы юзать)
источник

k

knopkod4v in PHP
Sergey Zolotov
многие пилились как пилотные проекты, которые по какой-то причине не взлетели или не зашли юзерам. а обратную задачу на спилить никто не давал сразу
задача на выпил функциональности обычно не рассматривается менеджментом как важная, они обычно не особо задумываются, что мёртвый код стоит денег
а если там ещё и жопа с кодом, то его может быть и выпиливать тяжело (опасно), поэтому дев-ы тоже на это смотрят так
"задача вроде не обязательная, менеджмент не пушит, делать неохота, сложна, будем надеяться про это все забудут"
источник

SZ

Sergey Zolotov in PHP
knopkod4v
задача на выпил функциональности обычно не рассматривается менеджментом как важная, они обычно не особо задумываются, что мёртвый код стоит денег
а если там ещё и жопа с кодом, то его может быть и выпиливать тяжело (опасно), поэтому дев-ы тоже на это смотрят так
"задача вроде не обязательная, менеджмент не пушит, делать неохота, сложна, будем надеяться про это все забудут"
ну вот из-за этого мертвого кода и получаются сложности миграции на новую версию фреймворка/либы/языка. так и живут с 5.3 пхп
источник

SZ

Sergey Zolotov in PHP
а потом героически переписывают на го
источник

MM

Maksim Masiukevich in PHP
Всяко лучше, чем терпеть респонсы по 500 мс, че)
источник

MM

Maksim Masiukevich in PHP
Только вот где гарантия, что у этих же ребят го не будет отвечать по 500 мс, история умалчивает)
источник

ИЛ

Иван Лещёв in PHP
Sergey Zolotov
лучше все же конструкторы юзать)
дискуссионно
разве что если будешь переносить код целиком во фреймворк, который не умеет в эту аннотацию/атрибут
а если нет, то не вижу преимуществ перед недостатками
источник

SZ

Sergey Zolotov in PHP
Иван Лещёв
дискуссионно
разве что если будешь переносить код целиком во фреймворк, который не умеет в эту аннотацию/атрибут
а если нет, то не вижу преимуществ перед недостатками
ну тут несколько факторов
1. ты с required все равно не зафорсишь эти зависимости, например в тестах
2. кол-во зависимостей в пропертях может быстро выйти из под контроля
3. конструкторы красивше особенно с 8м пхп)
источник

ВУ

Валентин Удальцов... in PHP
сразу видно, кто тесты не пишет)))
потому что в тестах сделать new Service($a, $b) проще, чем new Service(); $service->a = $a; $service->b = $b;
источник

ВУ

Валентин Удальцов... in PHP
ну и потом psalm возругается, если у тебя свойство не сеттится при создании инстантса явно. и будет прав
источник

ИЛ

Иван Лещёв in PHP
Валентин Удальцов
ну и потом psalm возругается, если у тебя свойство не сеттится при создании инстантса явно. и будет прав
я уже пулреквест сделал
источник

ВУ

Валентин Удальцов... in PHP
Иван Лещёв
я уже пулреквест сделал
в Psalm? с точки зрения типов ты не прав)
источник

SZ

Sergey Zolotov in PHP
поле фактически может быть нулом
источник

ИЛ

Иван Лещёв in PHP
Валентин Удальцов
в Psalm? с точки зрения типов ты не прав)
в симфони плагин
источник

ВУ

Валентин Удальцов... in PHP
Sergey Zolotov
поле фактически может быть нулом
это ты про uninitialized? или про явный ?Service ?
источник