СП
На основании каких критериев определяется делать класс с единственной реализацией или же использовать интерфейс?
Вопрос возник на примере этого PR для
Вопрос возник на примере этого PR для
CsrfViewInjection.Size: a a a
2020 August 21
СП
а issue есть?
нет, но в чатике нашёл, что вроде yii-debug надо подключить и будет выводится... попробую вечером.
Д
На основании каких критериев определяется делать класс с единственной реализацией или же использовать интерфейс?
Вопрос возник на примере этого PR для
Вопрос возник на примере этого PR для
CsrfViewInjection.В интернете есть много инфы на эту тему
СП
Что гуглить? Мне что-то не то все выдает
DS
Так и не вынесли валидацию, сейчас там есть нерабочие кейсы
https://github.com/yiisoft/yii-event/blob/master/src/EventDispatcherProvider.php#L87
https://github.com/yiisoft/yii-event/blob/master/src/EventDispatcherProvider.php#L87
DS
На основании каких критериев определяется делать класс с единственной реализацией или же использовать интерфейс?
Вопрос возник на примере этого PR для
Вопрос возник на примере этого PR для
CsrfViewInjection.Конкретно в этом пр не нужен, поищи для чего я предлагал его ввести и какую проблему это должно было решить. Смысл только если вынести реализацию из этого пакета
СП
Конкретно в этом пр не нужен, поищи для чего я предлагал его ввести и какую проблему это должно было решить. Смысл только если вынести реализацию из этого пакета
Да, если выносить из пакета, то точно нужна.
Я не понимаю, как понять, что для класса не нужны будут альтернативные реализации. В данном случае почему у пользователя НИКОГДА не возникнет задачи подменить реализацию?
Хотя пока это писал... захочет подменить - подменит класс. Так?
Я не понимаю, как понять, что для класса не нужны будут альтернативные реализации. В данном случае почему у пользователя НИКОГДА не возникнет задачи подменить реализацию?
Хотя пока это писал... захочет подменить - подменит класс. Так?
СП
То есть если реализация одна и она в одном и том же пакете, то интерфейс не нужен. Правильно?
DS
То есть если реализация одна и она в одном и том же пакете, то интерфейс не нужен. Правильно?
Нет, нет так. Я еще не учел один момент, когда с yii приложением будет использоваться не yiisoft/router, так что в этом случае ты прав, возможна подмена реализации, тогда интерфейс нужен
DS
А в этом пакете в конфиге контейнера нужно просто забиндить дефолтную реализацию
СП
Получается всё субъективно и исходить из практики. Если разработчик пакета не видит кейсов, когда потребуется альтернативная реализация, то делать БЕЗ интерфейса. Если вдруг потом на практике понадобится интерфейс - то перепилить. Так?
DS
Получается всё субъективно и исходить из практики. Если разработчик пакета не видит кейсов, когда потребуется альтернативная реализация, то делать БЕЗ интерфейса. Если вдруг потом на практике понадобится интерфейс - то перепилить. Так?
Потом сложно будет прикрутить интерфейс, лучше сразу правильно проектировать.
СП
Потом сложно будет прикрутить интерфейс, лучше сразу правильно проектировать.
Так вот и вопрос) Как понять, что правильно интерфейс сделать. Если разработчик на текущий момент не знает вариантов, когда может потребоваться больше одной реализации... как понять что сделать: интерфейс или просто класс?
NO
Для этого нужен опыт и чутьë либо совет от опытного человека )
СП
Для этого нужен опыт и чутьë либо совет от опытного человека )
Это все что бы придумать кейс, когда потребуется несколько реализаций. И если не придумалось с опытом, чутьем и советами - то делать без интерфейса. Так выходит :)
NO
Это все что бы придумать кейс, когда потребуется несколько реализаций. И если не придумалось с опытом, чутьем и советами - то делать без интерфейса. Так выходит :)
Да, можно без него изначально. Позже добавить, когда понадобится.
СП
В целом я понял. Спасибо!
DS
Да, можно без него изначально. Позже добавить, когда понадобится.
Ага, особенно в пакете, который юзается во многих проектах. Херак такой, граждане теперь мы используем интерфейс, а класс я переимновал и сделал еще несколько реализаций, будь-те добры меняйте теперь все свой код))
AM
Я верно помню что @viperawa перевод для https://github.com/yiisoft/yii-cycle/pull/48 собирался делать?
Д
Я верно помню что @viperawa перевод для https://github.com/yiisoft/yii-cycle/pull/48 собирался делать?
ага. сегодня сделаю. вчера работал. сегодня вроде будет время.