СС
Ran 1 tests containing 14 assertions.Наконец-то я победил...
0 failures, 0 errors.
Size: a a a
2019 October 09
СС
может и коряво конечно
AR
я так хотел так хотел но там было оч сложно а на питоне 10 строчек получилось ¯\_(ツ)_/¯
Спасибо за мнение
V
Спасибо за мнение
хмл-соап <-> жисон
V
прокси хайлод
СС
а по поводу ревью актуально?
меня интересует это
https://github.com/gofaq/morse-clj/blob/master/src/morse/core.clj
и это
https://github.com/gofaq/morse-clj/blob/master/test/morse/core_test.clj
Хотя последнее просто тесты. Во всяком случае все зеленое. Хотелось бы услышать какие ошибки допустил и как их можно по-кложуриански исправить.
Из тестов, наверное, понятно будет, что делает приложение
меня интересует это
https://github.com/gofaq/morse-clj/blob/master/src/morse/core.clj
и это
https://github.com/gofaq/morse-clj/blob/master/test/morse/core_test.clj
Хотя последнее просто тесты. Во всяком случае все зеленое. Хотелось бы услышать какие ошибки допустил и как их можно по-кложуриански исправить.
Из тестов, наверное, понятно будет, что делает приложение
СС
подправил один тест
ИФ
насколько сильно придираться?
СС
вообще это моя первая задача такая написанная на clojure)))
но лучше придираться так сильно, как толкьо возможно. быстрее научусь. а научиться я хочу очень сильно
но лучше придираться так сильно, как толкьо возможно. быстрее научусь. а научиться я хочу очень сильно
ИФ
(cond
(= type :morse) characters
(= type :alpha) (str/join "" characters)
такие конды проще писать как case
(= type :morse) characters
(= type :alpha) (str/join "" characters)
такие конды проще писать как case
ИФ
(case type
:morse characters
…
ИФ
с кейсом осторожно, он работает только со литералами на месте, по-моему использование символов из других неймспейсов может не работать
ИФ
(defn get-single-character
[alpha character]
(get alpha (str character))) ; <- lint
ИФ
функции которые Вы не экспортируете лучше объявлять как
(defn- чтобы было понятно – ручка может сломаться/переехать/изменить форму/ужалить/украсть рубльИФ
(defn convert
[type phrase]
(get-characters type (str/split phrase #" ")))
я бы назвал type как source-type или from-type
ИФ
я вот подумал сначала что это целевой
V
хмл-соап <-> жисон
хотелось бы услышать решение на кложе)
V
ну или увидеть если повезет)
ИФ
можно решить докстрокой
(defn convert
“type of input”
[type phrase]
(get-characters type (str/split phrase #" ")))
СС
(defn get-single-character
[alpha character]
(get alpha (str character))) ; <- lint
это линтер какой-то ругается что ли?