ST
я не предлагал решения, только указал что тут есть проблема в делении
как это решать — на усмотрение автора
как это решать — на усмотрение автора
ну вот автор смотрит, думает «не знаю, как это решать», и решает не решать 😊
Size: a a a
2020 November 12
KC
пока круг пользователей не выйдет за рамки автора скрипта, вопрос не имеет однозначного ответа 😊
ок, понимаю что могу говорить только за себя.
для меня как пользователя и автора важно знать что та вундервафля что я написал пару месяцев назад внезапно не стрельнет в меня чем-то что я знал что может стрельнуть
для меня как пользователя и автора важно знать что та вундервафля что я написал пару месяцев назад внезапно не стрельнет в меня чем-то что я знал что может стрельнуть
KC
ну вот автор смотрит, думает «не знаю, как это решать», и решает не решать 😊
и при этом просит ревью от сообщества которое ему начинает ссать в уши про кодстаил и идеоматичность
ST
ок, понимаю что могу говорить только за себя.
для меня как пользователя и автора важно знать что та вундервафля что я написал пару месяцев назад внезапно не стрельнет в меня чем-то что я знал что может стрельнуть
для меня как пользователя и автора важно знать что та вундервафля что я написал пару месяцев назад внезапно не стрельнет в меня чем-то что я знал что может стрельнуть
вот ты когда «видишь деление» или любую арифметеческую операцию, защищаешься от того, что аргумент может случиться не числовой?
KC
для справки — у меня бомбит не от отсутствия проверок, а от того какие советы после ревью тут появляются
KC
ок, кажется отпустило, сорян если задел кого, пойду форточку открою
ST
ну ты же сразу не посоветовал
кто что может/хочет, то и ревьюит
можешь сделать вывод, что большинство счастливы с happy path
кто что может/хочет, то и ревьюит
можешь сделать вывод, что большинство счастливы с happy path
ST
для справки — у меня бомбит не от отсутствия проверок, а от того какие советы после ревью тут появляются
то есть было бы красиво, если бы ты просто пришёл, и добавил своё ревьюерский совет, а не стал бы обсуждать советы других участников обсуждения 😊
а дальше автор пусть решает, что с полученной информацией делать
а дальше автор пусть решает, что с полученной информацией делать
KC
1. тут и без меня полно умных инженеров
2. время на вчитывание в скрипт у меня появилось только сейчас
3. решил проверить обсуждение что бы не дублировать советы, увидел только пространные обсуждения идеоматичности и кодстайла (даже про отступы что-то было)
4. бомбануло
2. время на вчитывание в скрипт у меня появилось только сейчас
3. решил проверить обсуждение что бы не дублировать советы, увидел только пространные обсуждения идеоматичности и кодстайла (даже про отступы что-то было)
4. бомбануло
ST
ну вот непонятно про 4, которое напрямую не следует из 1-2-3 😊
может не выспался? 😊
может не выспался? 😊
KC
и это тоже
OR
(def output-strings
(mapcat
(fn [{:keys [base currencies]}]
(conj
(->> currencies
(filter (fn [{currency :name}]
(if-not (contains? currency-map (keyword currency))
(prn (str "OMAGAD, NO " currency))
currency)))
(map #(format-rate (assoc % :base base))))
"---------------------"))
config))
всё, все рады
OR
Это, конечно, не трансдусеры, но зато очень человечно сообщает, что не срослось.
ST
(def output-strings
(mapcat
(fn [{:keys [base currencies]}]
(conj
(->> currencies
(filter (fn [{currency :name}]
(if-not (contains? currency-map (keyword currency))
(prn (str "OMAGAD, NO " currency))
currency)))
(map #(format-rate (assoc % :base base))))
"---------------------"))
config))
всё, все рады
а если contains, но лажу?
OR
Ну, началось 💩
OR
Как вылезет — так и починим
OR
Ясно же, что можно нож точить долго, а можно — эффективно.
ST
Ну, началось 💩
ну поэтому я и предлагал, что просто надо спеку на инпут сделать, и ругаться что и как криво в инпуте, а сам код не мучать дурацкими проверками
OR
ну поэтому я и предлагал, что просто надо спеку на инпут сделать, и ругаться что и как криво в инпуте, а сам код не мучать дурацкими проверками
Это лучший вариант, да. Но ресурсоёмкость )
ST
Ясно же, что можно нож точить долго, а можно — эффективно.
ну тогда просто обложить весь кусок try/catch и сказать OMAGAD