Size: a a a

2020 October 24

ЕР

Евгений Ромашкан... in PHP
Dmitriy Tkachenko
И там после исправления 500 ошибок псалма были только итерабл тип
+ также
источник

DT

Dmitriy Tkachenko in PHP
Который отключается в настройках пхпстан 2мя директивами и ошибок больше нет
источник

АГ

Алексей Гевондян... in PHP
Евгений Ромашкан
На все эти ошибки будет жаловаться и псалм
псалм другое проверяет. псалм больше по типам, пхпстан это не замечает например, а пхпстан больше про архитектуру (условно), именование (всякое разное), и прочее
источник

ЕР

Евгений Ромашкан... in PHP
Алексей Гевондян
псалм другое проверяет. псалм больше по типам, пхпстан это не замечает например, а пхпстан больше про архитектуру (условно), именование (всякое разное), и прочее
Тоже самое он проверяет
источник

АГ

Алексей Гевондян... in PHP
например стен у нас ругается на переменные длиннее 40 символов. ну сокращаем. вот в последней версии подвезли
источник

АГ

Алексей Гевондян... in PHP
Евгений Ромашкан
Тоже самое он проверяет
ну тебе виднее.
источник

ЕР

Евгений Ромашкан... in PHP
Алексей Гевондян
например стен у нас ругается на переменные длиннее 40 символов. ну сокращаем. вот в последней версии подвезли
лол
источник

ЕР

Евгений Ромашкан... in PHP
Архитектура
источник

ЕР

Евгений Ромашкан... in PHP
Ок
источник

DT

Dmitriy Tkachenko in PHP
Евгений Ромашкан
Тоже самое он проверяет
Не, он ещё вардампы там смотрит и подобную уж совсем дичь
источник

ЕР

Евгений Ромашкан... in PHP
Dmitriy Tkachenko
Не, он ещё вардампы там смотрит и подобную уж совсем дичь
Ну в смысле псалм проверяет всё то же что и пхпстан + экслюзивные фичи
источник

ЕР

Евгений Ромашкан... in PHP
Кроме той разницы с перечислениями, и, видимо, длины переменных :D
источник

АГ

Алексей Гевондян... in PHP
Евгений Ромашкан
Архитектура
ну а чему ты отнесешь рекомендации следующие:
- слишком много методов
- слишком сложный метод
- не используйте статические функции
- не используйте статические переменные
- не используйте трейты
- не используйте обращение динамическое к методам и полям, вместо этого сделайте явное.
- не используйте абстрактные методы, используйте интерфейсы
- не возвращайте ассоциативные массивы, используйте дто
- используйте отдельную фабрику для создания объектов, вместо возвращения через нью
- методы, возвращающие бул, должны называться на is/has
- абстрактный класс должен иметь префикс abstract
- класс со статическими методами должен иметь постфикс static
источник

АГ

Алексей Гевондян... in PHP
....
источник

АГ

Алексей Гевондян... in PHP
и многое другое, где-то половину сейчас на вскидку вспомнил
источник

АГ

Алексей Гевондян... in PHP
из того, что засапрешено
источник

АГ

Алексей Гевондян... in PHP
псалму, к слову - на все на это - абсолютно насрать
источник

ЕР

Евгений Ромашкан... in PHP
Некоторые сомнительные, конечно, но ок)
У меня на проекте покрытом псалмом никаких ишьюсов кроме iterable от пхпстана не было, может потому что я и так всякие трейты не использую)
источник

АГ

Алексей Гевондян... in PHP
и псалм и стен на максималках
источник

АГ

Алексей Гевондян... in PHP
ну а у нас ларавел) так что мы страдаем)
источник