170 похожих чатов

А какой код предпочтительней 1 или 2?

16 ответов

7 просмотров

Второй вариант с очень большой (подчёркнуто) натяжкой имеет право на жизнь.

Sergey-TS Автор вопроса

Первый вариант лучше получается или он совсем плох?

Sergey-TS Автор вопроса

А во втором варианте, что надо поправить чтоб было хорошо?

Нужно посмотреть на тип. Если у него открытый конструктор, то это проблема. Это первое. Второе: добавится новый подтип, фабрику надо будет править. Третье: нет валидации наличия ключа в массиве.

https://pastebin.com/qyhqqqJR

ты бы еще к названиям прикопался

а как ты в своем варианте избавился от второй проблемы что то я не вижу?

Где в моем варианте я буду править фабрику при добавление продукта?

А если у класса будут зависимости?

$this->app->bind(ProductFactory::class, function (Application $app): ProductFactory { return new ProductFactory([ 'coffee' => static function () use ($app): Type { return $app->get(Coffee::class); }, ... ]); });

М, логично)🌚

Sergey-TS Автор вопроса

Спасибо, все понятно. Про такие вещи в паттернах надо читать?

Нет. И паттерны читать ради того, чтобы найти место, где их применить, тоже не надо. Просто решай проблемы. Со временем начнёшь замечать, что решаешь проблемы похожим образом, и придёшь к паттернам через опыт. Или почитай книгу банды четырёх.

Sergey-TS Автор вопроса

Так я через if в начале делал. Потом кто-то подсказал, что можно сделать так чтобы объект сам себя обновлял. Вот от туда уже пришел к варианту с полиморфизмом.

Есть книжка, которая мне зашла, с картинками которая

А ну-ка, может читал уже..

Похожие вопросы

Обсуждают сегодня

Добрый день. Хочу сделать отрисовку по команде на панели. Почему-то рисуется только при втором вызове. С чем может быть связано, не подскажете? procedure TForm1.FormDblClick(...
Kirill Filippenok
20
а зачем этот вопрос для удаления из чата?
Mёdkinson Medvezhkin
63
Эх кто-то пришел и весь праздник испортил :( You need complex FBX scene importing setup to change things on import? good luck with that. You need navigation and pathfinding? g...
Serg Gini
5
Всем привет! Нужен совет от опытных. Переношу свой проект с Делфи 10.2 Токио на Лазарус 3.2 установленный через инсталлятор fpcupdeluxe-x86_64-win64. При импортировании проект...
Дмитрий Завгородний
2
Всем привет! Подскажите. Я написал приложение на Delphi 10.2 Tokyo под Windows 10. И передо мной стал вопрос о том чтобы сделать это приложение кроссплатформенным (под Linux и...
Дмитрий Завгородний
24
Какого хера? /Sources/App/Modules/User/Models/UserLinkApple.swift:21:20: warning: stored property '_id' of 'Sendable'-conforming class 'UserLinkApple' is mutable @ID(...
Alexander Sherbakov
14
Привет всем. Подскажите где можно посмотреть, какая версия электрон, поддерживает версии windows? Некий changelog. Мне бы желательно, поддержку 7,8,10... latest, как понимаю и...
Anonym Squad
21
Почему стало ломаться на D11? "739002.86400000' is not a valid timestamp" function IncDateTime(aStamp:TTimeStamp;aKind:TTriggerKind;aInterval:Integer):TDateTime; //aStamp = 2...
Катерина Свиридова
8
у меня программа тысяч на 10 строк. Там в основном моя собственная логика. А по содержанию она просто работает с файловой системой (мастер для бэкапов) и таблицей с данными о ...
Дмитрий Завгородний
5
У тебя в конфиге нигде нет deny all; или вообще любого deny?
Alexander Sherbakov
10
Карта сайта