AR
Size: a a a
2020 February 28
AR
вот там на 12 странице!
T
Спасибо, прочитаю
ДН
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
ST
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
более читаемо разбить на функции
ST
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
и похоже update-in напрашивается
ДН
и похоже update-in напрашивается
в result нету :address, update-in не подойдет
ST
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
твой 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)))
ST
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
->> можно записать как some->>ДН
твой 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)))
точно.
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
ST
точно.
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
имеет
ST
точно.
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
а имеет смысл get-in запихивать в отдельную функу и ее вызывать ради 2х случаев?
получится и короче, и более читаемо, и эту функцию ты можешь отладить/протестировать отдельно
IG
> Как думаете, почему в кложуре сделано не так же?
будет не код, а минное поле. Сейчас если не указал аргумент, компилятор ругнется. А если смолчит, то бахнет совсем в другом месте.
будет не код, а минное поле. Сейчас если не указал аргумент, компилятор ругнется. А если смолчит, то бахнет совсем в другом месте.
ДН
спасибо, я попробую
IG
ты вычислил число, а на самом деле это функция, из-за того, что не указал аргумент. Потом передал его десять раз в словаре, и кто-то прибавил к нему число, а там функция. Хорошей отладки.
ST
получится и короче, и более читаемо, и эту функцию ты можешь отладить/протестировать отдельно
впрочем, я тебе так и присылал примеры с вынесенной отдельно функцией
AI
В продолжение моего вопроса. Можно ли это укоротить? Или так более читаемо?
(assoc result :address
(if-let
[physical (->>
(get-in value [:organization :resource :address])
(some #(when (#{"physical"} (:type %)) %))
(:text))]
physical
(->> (get-in value [:organization :resource :address])
(some #(when (#{"postal"} (:type %)) %))
(:text))))
Жесть какая-то... Причем, дублированная...
(let [m (group-by (comp keyword :type)
(get-in value [:organization :resource :address]))]
(-> (or (:physical m) (:postal m)) first :text))
AI
С предложением выносить функции уже похоже на квест угадай автора по рекомендации
ДН
Жесть какая-то... Причем, дублированная...
(let [m (group-by (comp keyword :type)
(get-in value [:organization :resource :address]))]
(-> (or (:physical m) (:postal m)) first :text))
В практической clojure только месяц, поэтому и спрашиваю) спасибо за ещё вариант, с group-by ещё не встречался
AI
(if-let [хрень (калкулэйтед-хрень)] хрень другая-хрень) =>
(or (калкулэйтед-хрень) другая-хрень)