homework-08-ataranchiev #9
Conversation
| ; pretty useless I think | ||
| (deftest test-header | ||
| (is (= (header "Test") [:h2 {:style {:margin "8px 4px"}} "Test"]))) |
There was a problem hiding this comment.
Думаю тест со значением nil не был бы лишним.
| (are [input expected] (= (id->int input) expected) | ||
| "0" 0 | ||
| "z" 61 | ||
| "clj" 149031 | ||
| "Clojure" 725410830262 | ||
| "" 0)) |
There was a problem hiding this comment.
Я бы добавил тестов с некорректными аргументами. И сгруппировал все тесты на legal arguments и illegal arguments при помощи testing
| {:uri "/", | ||
| :request-method :post, | ||
| :params | ||
| {:url | ||
| "https://github.com/Clojure-Developer/Clojure-Developer-2022-10"}} | ||
| {:status 201, | ||
| :headers {"Location" "1", | ||
| "Content-Type" "application/json; charset=utf-8"}, | ||
| :body "{\"id\":\"1\"}"} |
There was a problem hiding this comment.
Неправильное форматирование. Зачем добавлять отступ, если это параметры одной функции?. Если хочешь их сгруппировать, то лучше добавь пустую строку между парами мап.
There was a problem hiding this comment.
я пользуюсь zprint как форматировщиком. Он сам все это делает автоматически. Я по сути код не форматирю.
| "Content-Type" "application/json; charset=utf-8"}, | ||
| :body "{\"id\":\"1\"}"} | ||
| ; Put clj | ||
| {:uri "/clj", :request-method :put, :params {:url "https://clojure.org"}} |
There was a problem hiding this comment.
ИМХО, каждую пару ключ-значение лучше помещать на отдельную строку. Аргументы в пользу такого code-style'а:
- Это препятствует появлению длинных строк.
- Это улучшает читаемость кода.
2 participants
No description provided.