Size: a a a

Clojure — русскоговорящее сообщество

2020 November 11

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
Sergey Trofimov
см. комментарии к гисту
там вообще не обязателен ни doall ни run!, можно всё mapcat-ить, join-ить через "\n" и выводить всё сразу
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
чтобы минимизировать количество мест с сайдэффектами
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
Никита
Проект первый, попытка в многопоточность тоже. Лишнее?
ну, если бы у тебя в config какой-нибудь ещё curl/get был, то было бы нужно, а так получилось лишнее
источник

KC

Kirill Chernyshov in Clojure — русскоговорящее сообщество
Sergey Trofimov
ну, если бы у тебя в config какой-нибудь ещё curl/get был, то было бы нужно, а так получилось лишнее
Ну там есть как минимум чтение из файла
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
Никита
Проект первый, попытка в многопоточность тоже. Лишнее?
(keyword valute) используется два раза, можно в let забиндить,
:RUB в rate (базовая валюта?) выглядит как что-то, что можно вынести в def или вообще в конфиг
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
Kirill Chernyshov
Ну там есть как минимум чтение из файла
формально да, а по факту я сомневаюсь в практическом приросте скорости выполнения скрипта
источник

KC

Kirill Chernyshov in Clojure — русскоговорящее сообщество
Скорость то тут при чем? Синхронизация и обработка ошибок важнее.
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
кстати, про конфиг, зачем его перечитывать при создании?
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
что касается советов вместо принтов формировать строку
это всё хорошо, но тут у нас просто шелл скрипт, написанный в алгоритмике шелл скрипта с кучей echo 😊
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
и (вкусовщина) не понимаю, почему бы не юзать edn вместо yaml
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
Sergey Trofimov
что касается советов вместо принтов формировать строку
это всё хорошо, но тут у нас просто шелл скрипт, написанный в алгоритмике шелл скрипта с кучей echo 😊
окей, можно сформировать список, а потом run!-ом его вывести :)
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
я считаю, что про идиоматику надо думать даже в таких скриптах
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
когда пишешь скрипт «на один раз», меньше всего думаешь про идиоматику 😊
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
Sergey Trofimov
когда пишешь скрипт «на один раз», меньше всего думаешь про идиоматику 😊
хз, всегда думаю про неё :)
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
привычка
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
собственно, в скрипте всего 3 взаимодействия с внешним миром - конфиг, curl и вывод результата, так что кроме 3х функций оно больше никуда протекать не должно. да и отлаживать так проще по идее
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
Dmytro Lispyvnyi '(🌲 🍺)
хз, всегда думаю про неё :)
примеры есть твоих идеоматических скриптов на бабашке 😊 ?
источник

DL

Dmytro Lispyvnyi '(🌲... in Clojure — русскоговорящее сообщество
Sergey Trofimov
примеры есть твоих идеоматических скриптов на бабашке 😊 ?
NDA :)
источник

Н

Никита in Clojure — русскоговорящее сообщество
Подправил. Теперь хорошо? https://gist.github.com/Arvantos/d20df52c677a3669e13b078e6c99c2da
источник

ST

Sergey Trofimov in Clojure — русскоговорящее сообщество
да вроде было не так чтобы плохо 😊
источник