AI
одинокие атомы в тред-макросах в скобки не обязательно -
(:text) => :textSize: a a a
2020 February 28
AI
ну и гет-ин одинаковый читаемее в лет выносить
AI
точно.
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
не имеет конечно же
AG
С предложением выносить функции уже похоже на квест угадай автора по рекомендации
👍
ST
Жесть какая-то... Причем, дублированная...
(let [m (group-by (comp keyword :type)
(get-in value [:organization :resource :address]))]
(-> (or (:physical m) (:postal m)) first :text))
ну то есть чтобы найти первый элемент в большой коллекции мы сделаем кучу ненужной работы по группировке коллекции...
ST
С предложением выносить функции уже похоже на квест угадай автора по рекомендации
С предложением не писать код, делающий ненужных вычислений — тоже 😊
AG
Сергей видимо просто большой поклонник Алана Перлиса ;)
"It is better to have 100 functions operate on one data structure than 10 functions on 10 data structures." —Alan Perlis
"It is better to have 100 functions operate on one data structure than 10 functions on 10 data structures." —Alan Perlis
AI
Мой предыдущий опыт показывает, что обычно у кастомеров не более 3 адресов - почтового, физического и запасного. Но конечно в бенчмарке со сферическим кастомером со 100500 адресами твой кот выиграет. В бенчмарке со 100500 кастомерами с несколькими адресами каждый - может ненамного тоже. Но имхо это ловля блох на пустом месте и попытка оправдать нечитаемого кота )
ST
Мой предыдущий опыт показывает, что обычно у кастомеров не более 3 адресов - почтового, физического и запасного. Но конечно в бенчмарке со сферическим кастомером со 100500 адресами твой кот выиграет. В бенчмарке со 100500 кастомерами с несколькими адресами каждый - может ненамного тоже. Но имхо это ловля блох на пустом месте и попытка оправдать нечитаемого кота )
совсем неочевидно из контекста задачи, что в векторе всегда 2-3 элемента
[{:text "123" :type "postal"}
{:text "789" :type "physical"}]ST
совсем неочевидно из контекста задачи, что в векторе всегда 2-3 элемента
[{:text "123" :type "postal"}
{:text "789" :type "physical"}]для меня это выглядит, что полей там может быть дофига
ST
и начинающему желательно понимать алгоритмический смысл решения, а не просто заучить подходящие функции из коре
а смысл здесь простой — есть некая последовательность объектов, нужно найти первый попадающий под условие
варианты с фильтрами и групбаями просто не подходят по смыслу
а смысл здесь простой — есть некая последовательность объектов, нужно найти первый попадающий под условие
варианты с фильтрами и групбаями просто не подходят по смыслу
ST
твой if-let можно записать как
(or
(->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text)))
и
or засунуть в some, чтобы находилось за один проход, конечноAI
будет неправильно работать
ST
да сорри, неправильно
ST
да, здесь group-by лучше подходит
AG
Поделитесь, кто что использует для генерации документации по исходникам (clj/cljs).
AI
да, здесь group-by лучше подходит
поздно, я уже оптимальный вариант для часто переезжающего кастомера накостылил
(loop [physical nil
postal nil
[{:keys [type text]} :as r] (get-in value [:organization :resource :address])]
(cond
physical physical
(empty? r) postal
:else (recur ({"physical" text} type)
(or postal ({"postal" text} type))
(rest r))))
IG
Поделитесь, кто что использует для генерации документации по исходникам (clj/cljs).
Codox, ранее lein autodoc
AG
Codox, ранее lein autodoc
Cobox - как-то не умеет спеки парсить и это прискорбно. а с autodoc вообще ерунда какая-то, я его даже запустить не могу. походу lein plugin сломан…
KC
Поделитесь, кто что использует для генерации документации по исходникам (clj/cljs).
