BB
Ну можно завернуть в Экто Мулти
Size: a a a
2019 December 16
BB
Или как нужно?
ВК
не нужно. ибо название метода не соотв полностью его сути
ВК
он делает что то типа insert_or_ignore
ŹR
Есть известные наукам концепции типа find-or-create
BB
А ну да
BB
Туплю
ŹR
В экто это называется create_if_not_exists
ŹR
Название может не супер, но зато оно универсиализировано
ВК
компромиссное
ŹR
Но если говорить про S в слове SOLID, то тут нужен контекст
ŹR
Потому что add_city которое принимает {:ok, че-то} - имеет неправильный API чтобы быть универсальным
ВК
мы можем жертвовать любой буквой в разумных пределах, ради читаемости кода, и наоборот
ŹR
И значит кто-то должен получать {:ok, что-то}, а дергать create_if_not_exist, а потом результат помещать в результирующую мапу и заварачивать в ок тупл
ВК
И значит кто-то должен получать {:ok, что-то}, а дергать create_if_not_exist, а потом результат помещать в результирующую мапу и заварачивать в ок тупл
а это значит, что интерфейс медота зависим уже от результата другого,
ŹR
Не, это у тебя он зависит
ŹR
Потому что он принимает только ок тупл
ВК
либо nil
ŹR
А это очевидно не имеет отношения к созданию-или-получению города
ŹR
Для создания города не нужен ок