rabotaet #220
rabotaet #220
Conversation
dirtymew
left a comment
dirtymew
left a comment
There was a problem hiding this comment.
Привет
стало лучше но PR оформлен все еще неверно - он должен быть в твой форк а не репозиторий задания
по реализации есть замечания - см комментарии
| } | ||
| w.Header().Set("Content-Type", "application/json") | ||
| w.WriteHeader(http.StatusOK) | ||
| w.Write(resp) |
There was a problem hiding this comment.
_ , _ = w.Write(resp)
тут стоит либо явно проигнорировать ошибку или ее логировать
There was a problem hiding this comment.
Поясните, пожалуйста. Распишите подробно, что нужно сделать, с примерами, пожалуйста.
There was a problem hiding this comment.
@artwovk прошу прощения упустил вопрос потому, что PR был другой отправлен в следующую итерацию ревью
работа с ошибками это очень важная особенность программирования на go
если функция возвращает ошибку ее принято обработать например
if _ , err := w.Write(resp); err != nil {
...
}
в данной конкретно ситуации если ошибка из функции w.Write мы уже не сможем вернуть ответ на запрос пользователя - так как ошибка при записи тела ответа
поэтому либо явно игнорируем ее _ , _ = w.Write(resp) - такая конструкция говорит о том что разработчик в курсе что функция возвращает ошибку и сознательно ее игнорирует
либо логируем например так
if _ , err := w.Write(resp); err != nil {
log.Print(err)
}
| } | ||
|
|
||
| func postZad(w http.ResponseWriter, r *http.Request) { | ||
| var zad Task |
There was a problem hiding this comment.
названия переменным нужно выбирать так чтобы они несли какой то смысл
не стоит использовать сокращения и транслитерацию
var task Task
| w.Write(resp) | ||
| } | ||
|
|
||
| func postZad(w http.ResponseWriter, r *http.Request) { |
| w.WriteHeader(http.StatusCreated) | ||
| } | ||
|
|
||
| func getZad(w http.ResponseWriter, r *http.Request) { |
| // Ниже напишите обработчики для каждого эндпоинта | ||
| // ... | ||
|
|
||
| func getVse(w http.ResponseWriter, r *http.Request) { |
| w.Write(resp) | ||
| } | ||
|
|
||
| func delZad(w http.ResponseWriter, r *http.Request) { |
| http.Error(w, err.Error(), http.StatusBadRequest) | ||
| return | ||
| } | ||
| tasks[zad.ID] = zad |
There was a problem hiding this comment.
сначала нужно проверить что такой задачи еще нет - и если есть вернуть ошибку
There was a problem hiding this comment.
Можно, пожалуйста, ссылку на урок, где нас этому учат?
Я не нашёл информации.
There was a problem hiding this comment.
@artwovk чему? как проверить есть ли ключ в мапе? ну так я понимаю с этим нет проблем на 96 ты это делаешь.
или тому добавить не равно изменить? тут ответ ожидается http.StatusCreated - что подразумевает добавление чего то - да и в задании так описано
2 participants
No description provided.