но вообще может быть имело бы смысл встроить :base в каждую валюту конфига при считывании, чтобы не плясать потом

`Может это превратить в функцию, которая получает урл и называется
"get-cbr-currency-map"?`
up to you 😊

Наверное, curl сразу умеет парсить json параметром {:as :json} или аналогично

Не, это код прямо из доки курла бабашки

ну, по стилю отступов единого мнения нет
я при переходе с других языков сначала писал в столбик, потом традиционная лисповая запись вытягивать цепочки в строку мне понравилась больше
поэтому я не пишу -> на отдельной строке, никогда 😊

И затем передавать её в функцию rate как аргумент (ну а туда из format-rate как аргумент)

точнее не to you, а к автору скрипта, сорри

вы тут немного упоролись по идеоматичности и кодстайлу полностью проигнорировав обработку ошибок

❯ bb rates.bb
----- Error --------------------------------------------------------------------
Type:     java.lang.NullPointerException
Location: /private/tmp/rates.bb:30:10

----- Context ------------------------------------------------------------------
26:   ([currency]
27:    (if (= (keyword currency) :RUB)
28:        1.0
29:        (let [{:keys [Value Nominal]} ((keyword currency) currency-map)]
30:          (/ Value Nominal))))
             ^---
31:   ([currency base]
32:    (/ (rate currency) (rate base)))
33:   ([currency base nominal]
34:    (* nominal (rate currency base))))
35:

----- Locals -------------------------------------------------------------------
currency: "IDR"
map__2:   nil
Nominal:  nil
Value:    nil

----- Stack trace --------------------------------------------------------------
clojure.core//      - <built-in>
user/rate           - /private/tmp/rates.bb:30:10
user/rate           - /private/tmp/rates.bb:25:1
user                - /private/tmp/rates.bb:32:7
clojure.core//      - <built-in>
...
user                - /private/tmp/rates.bb:44:27
clojure.core/into   - <built-in>
user                - /private/tmp/rates.bb:44:15
clojure.core/mapcat - <built-in>
user                - /private/tmp/rates.bb:42:3

ну для этого, наверное, скрипт нужно запускать, а нам всем это лень 😊

❯ bb rates.bb
----- Error --------------------------------------------------------------------
Type:     java.lang.NullPointerException
Location: /private/tmp/rates.bb:30:10

----- Context ------------------------------------------------------------------
26:   ([currency]
27:    (if (= (keyword currency) :RUB)
28:        1.0
29:        (let [{:keys [Value Nominal]} ((keyword currency) currency-map)]
30:          (/ Value Nominal))))
             ^---
31:   ([currency base]
32:    (/ (rate currency) (rate base)))
33:   ([currency base nominal]
34:    (* nominal (rate currency base))))
35:

----- Locals -------------------------------------------------------------------
currency: "IDR"
map__2:   nil
Nominal:  nil
Value:    nil

----- Stack trace --------------------------------------------------------------
clojure.core//      - <built-in>
user/rate           - /private/tmp/rates.bb:30:10
user/rate           - /private/tmp/rates.bb:25:1
user                - /private/tmp/rates.bb:32:7
clojure.core//      - <built-in>
...
user                - /private/tmp/rates.bb:44:27
clojure.core/into   - <built-in>
user                - /private/tmp/rates.bb:44:15
clojure.core/mapcat - <built-in>
user                - /private/tmp/rates.bb:42:3

Разве так не всегда?

хм… достаточно внимательно на скрипт посмотреть и отметить в голове места которые могут взорваться

а их тут много

happy path наше всё

Я не смотрел, честно скажу

Но здесь опять вопрос про то, насколько это критично

а потом появляются статейки типа Software disenchantment (

серьезно? надо смотреть насколько критично появление NPE?

🤷 Ошибки важно обрабатывать, но если это скрипт для себя и это не критично — тогда можно проработать happy path и не мучаться. Конкретно здесь всё упирается в ресурсы и приоритеты.

по твоему выходит — лучше думать про соответствие кодстайлу а не про то насколько корректна программа?

Откуда ты вывел кодстайл?