k
сразу видно, симфони вик не читаешь!
это паблики только так работают
это паблики только так работают
отличная причина начать делать все проперти паблик!
Size: a a a
2021 March 11
ИЛ
есть у них нужда видеть это приватное свойство или они в рид модели отдельной видят?
факт канцелед - публичный вне зависимости от того, что там есть в рид модели
завтра придётся писать автоматизацию основанную на этом факте, а в основной модели кто-то перестарался с приватами
завтра придётся писать автоматизацию основанную на этом факте, а в основной модели кто-то перестарался с приватами
ИЛ
отличная причина начать делать все проперти паблик!
хе хе, просто в пхп нет дружественных классов
поэтому отдельные паблики на самом деле не должны быть пабликами
публичные конструкторы в сервисах нужны только для контейнера и для тестов, но никого не смущает это
а вот если публичное свойство - всё, диверсия
поэтому отдельные паблики на самом деле не должны быть пабликами
публичные конструкторы в сервисах нужны только для контейнера и для тестов, но никого не смущает это
а вот если публичное свойство - всё, диверсия
KN
отличная причина начать делать все проперти паблик!
я и так делаю для DTO или если они иммутабельные =)
KN
и проблемы с полями в пыхе ровно две:
1) нет свойств
2) нет возможности декларации на уровне интерфейсов
1) нет свойств
2) нет возможности декларации на уровне интерфейсов
KN
вторая решается через декларацию докблока в интерфейсе, а первая либо через костыли, либо никак
VC
факт канцелед - публичный вне зависимости от того, что там есть в рид модели
завтра придётся писать автоматизацию основанную на этом факте, а в основной модели кто-то перестарался с приватами
завтра придётся писать автоматизацию основанную на этом факте, а в основной модели кто-то перестарался с приватами
ну сделать геттер делов-то, это не убрать, если он вдруг оказывается лишним
KN
так что если они иммутабельные (благо есть 3 разных варианта это обозначить), то причин не делать поля публичными ровно ноль
ИЛ
ну сделать геттер делов-то, это не убрать, если он вдруг оказывается лишним
дык, не надо бояться, если интерфейс вдруг полнее, чем необходимо
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
VC
хе хе, просто в пхп нет дружественных классов
поэтому отдельные паблики на самом деле не должны быть пабликами
публичные конструкторы в сервисах нужны только для контейнера и для тестов, но никого не смущает это
а вот если публичное свойство - всё, диверсия
поэтому отдельные паблики на самом деле не должны быть пабликами
публичные конструкторы в сервисах нужны только для контейнера и для тестов, но никого не смущает это
а вот если публичное свойство - всё, диверсия
вот о дружественных тестах думал как раз. но если похожим образом приспичиввает, то предпочту с помощью “магии” читать приватное свойство или интеграционный тест написать, чем вводить геттер/иззер в формально публичный API объекта, пускай даже с internal аннотацией
VC
дык, не надо бояться, если интерфейс вдруг полнее, чем необходимо
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
если он полнее чем необходимо, то рано или поздно на него кто-нибудь завяжется как на workaround для своей задачи и вот у нас уже не необходимый каплинг )
ЕР
вот о дружественных тестах думал как раз. но если похожим образом приспичиввает, то предпочту с помощью “магии” читать приватное свойство или интеграционный тест написать, чем вводить геттер/иззер в формально публичный API объекта, пускай даже с internal аннотацией
Рефлексия есть, чего не хватает то?
ЕР
дык, не надо бояться, если интерфейс вдруг полнее, чем необходимо
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
никто же не заставляет использовать всё и всегда
а то дай волю, из доктрины повыпиливают чтение, раз уж чтение теперь отдельно
Interface Segregation Principle? SOLID? Не, не слышали?
ИЛ
Interface Segregation Principle? SOLID? Не, не слышали?
это тут при чём?
ЕР
это тут при чём?
При твоих советах вредных
VC
Рефлексия есть, чего не хватает то?
я биндить кложуру предпочитаю ) но это же костыли всё равно по сути
ЕР
я биндить кложуру предпочитаю ) но это же костыли всё равно по сути
Почему костыли?
VC
Почему костыли?
обход явно установленных ограничений видимости
ИЛ
При твоих советах вредных
ты из доктрины уже выпилил чтение?
KN
я биндить кложуру предпочитаю ) но это же костыли всё равно по сути
кстати надо потестить что быстрее, бинд кложуры или получение через рефлексию)