LL
Я к тому что такая функция, это скорее всего следствие
XYSize: a a a
2020 April 13
PG
Ну да, условие и его следствия довольно подозрительные
YS
ну это не чистая функция
PG
Ну и что?
V
clean_storage(key, keys) - начини сигнатуры функции, а то непонятно, что за key, что за keys, что за storage, в чём должно заключаться cleanYS
в контексте все очевидно, в данном случае просто не ясно как лучше написать
суть такая
проверить, есть ли ключ в хранилише
если да, то удалить его, другие не трогать
если нет - удалить все
суть такая
проверить, есть ли ключ в хранилише
если да, то удалить его, другие не трогать
если нет - удалить все
YS
удаление, дорогая операция
LL
Вспомнил почему люблю интернет. Даже на какую-нибудь маленькую проблему в чатике найдётся 1000 и 1 комментарий
V
если да, то удалить его, другие не трогать
если нет - удалить все
так это совершенно разный функционал, и я за его смешение в одной функции дал бы по рукам
YS
нет
YS
нет
YS
он не даст
YS
** (ArgumentError) invalid right argument for operator "in", it expects a compile-time proper list or compile-time range on the right side when used in guard expressions, got: keysYS
это на удаленный коммент
AD
defp clean_storage(key, keys), do: clear(key, keys, key in keys )
defp clear(key, _keys, true), do: key |> Storage.delete()
defp clear(_key, keys, false), do: keys |> Enum.each(&Storage.delete/1)
PG
single responsibility left the groupLL
single responsibility left the group
V
V
если да, то удалить его, другие не трогать
если нет - удалить все
так это совершенно разный функционал, и я за его смешение в одной функции дал бы по рукам
В случаях
key in keys и key not in keys различается смысл аргумента keys. Это настолько пхп-стайл, что подозреваю ошибку проектирования функций уровнем выше.V
и что получается?
в папке у тебя лежат "a.sass", "b.sass", ты добавляешь файл "c.sass" - и из стораджа удаляются два первых ключа (названия файлов)?
в папке у тебя лежат "a.sass", "b.sass", ты добавляешь файл "c.sass" - и из стораджа удаляются два первых ключа (названия файлов)?