Телеграмм чат группы prophp7 страница 9667

Нет чёткого правила 10 плохо, а 9 хорошо. Просто чаще всего это ошибка в проектировании, скорее всего ты складываешь вместе данные, которые вместе хранить не нужно

Разумеется, что нет чёткого. Но, например, а если в энтити накидано зеркально как в таблице в БД много всяких полей, хранящих простую инфу, например? Лень выдумывать пример из конкретной предметной области сейчас. Какая-нибудь инфа о товаре, или книге тем (как в библиотеке), например? Много строк разных, которые потом просто выводить и сортировать/фильтровать по ним. Мне кажется, не так страшно, что они вместе.

источник

20:35пожаловаться #4

AD

Andrey Dembitskyi in PHP

Константин Грачев

Если нет, то архитектура не гибкая, о чём тебе и написали. нарушение SPR и Open/Close

Для фабрик это ок.

источник

20:37пожаловаться #5

OA

Oleg Arkhipov in PHP

Но я бы полностью согласился, если речь бы шла о «10 полях», которые все участвуют в условно сложной логике, например. В т.ч. не у энтити, а у какого-то иного класса, сервиса, и т.д.

источник

20:37пожаловаться #6

КГ

Константин Грачев in PHP

Oleg Arkhipov

Разумеется, что нет чёткого. Но, например, а если в энтити накидано зеркально как в таблице в БД много всяких полей, хранящих простую инфу, например? Лень выдумывать пример из конкретной предметной области сейчас. Какая-нибудь инфа о товаре, или книге тем (как в библиотеке), например? Много строк разных, которые потом просто выводить и сортировать/фильтровать по ним. Мне кажется, не так страшно, что они вместе.

Мы тут вроде проектирование обсуждаем, а ты аргументируешь отзеркаливанием базы

источник

20:37пожаловаться #7

S

SarcasmIO in PHP

Daniil

хорошо как создавать тогда стораджи?

я смотрел только StorageFactory

представь у тебя есть некий StorageInterface который имплементят все стораджи

@Preemiere тебе правильно объяснил за SRP/OCP

думаю если бы ты заменил её на, штуку принимающую все реализации StorageInterface в конструктор через !tagged где спецификациями опрашивал какую вернуть, то тебе не пришлось бы изменять старый код, когда ты ввел новый сторедж

источник

20:37пожаловаться #8

D

Daniil in PHP

Константин Грачев

Если нет, то архитектура не гибкая, о чём тебе и написали. нарушение SPR и Open/Close

как по типу стораджа переданому в метод создавать разные типы стораджей без изменения кода

источник

20:37пожаловаться #9

OA

Oleg Arkhipov in PHP

Константин Грачев

Мы тут вроде проектирование обсуждаем, а ты аргументируешь отзеркаливанием базы

Не вижу, чтобы я что-то аргументировал этим.

источник

20:37пожаловаться #10

D

Daniil in PHP

Andrey Dembitskyi

Для фабрик это ок.

я тоже так думаю

источник

20:38пожаловаться #11

S

SarcasmIO in PHP

Daniil

как по типу стораджа переданому в метод создавать разные типы стораджей без изменения кода

$storageResolver->resolve($userInput->type) //an StorageInstance returned

источник

20:38пожаловаться #12

КГ

Константин Грачев in PHP

Oleg Arkhipov

Не вижу, чтобы я что-то аргументировал этим.

Если тебе лень, дальше можно ничего не обсуждать.
Для описанных тобою полей есть EAV

источник

20:39пожаловаться #13

D

Daniil in PHP

SarcasmIO

$storageResolver->resolve($userInput->type) //an StorageInstance returned

ну у тебя в resolve тоже будут if

источник

20:40пожаловаться #14

S

SarcasmIO in PHP

Daniil

ну у тебя в resolve тоже будут if

нет

источник

20:40пожаловаться #15

OA

Oleg Arkhipov in PHP

Константин Грачев

Если тебе лень, дальше можно ничего не обсуждать.
Для описанных тобою полей есть EAV

То есть ты считаешь, что лучше влезать в EAV в таком случае, нежели создать ряд скучных полей, лежащих в одном классе?

источник

20:40пожаловаться #16

S

SarcasmIO in PHP

Daniil

ну у тебя в resolve тоже будут if

там будет итерация по всем реализациям, у каждой из которой я спрошу, ты это умеешь, нет, тогда дальше, и так пока не найду нужную

источник

20:40пожаловаться #17

OA

Oleg Arkhipov in PHP

@oneerror можешь пояснить, что значит «через !tagged»?

источник

20:41пожаловаться #18

А

Антон in PHP

Daniil

ну у тебя в resolve тоже будут if

foreach ($resolvers as $resolver) {
    if ($resolver->supports($storage) {
          $resolver->resolve($type);
    }
}

источник

20:41пожаловаться #19

КГ

Константин Грачев in PHP

SarcasmIO

там будет итерация по всем реализациям, у каждой из которой я спрошу, ты это умеешь, нет, тогда дальше, и так пока не найду нужную

можно хешмап держать, если выбор всегда по одной строке

источник

20:41пожаловаться #20