lool #182
lool #182
Conversation
| http.Error(w, err.Error(), http.StatusBadRequest) | ||
| return | ||
| } | ||
| tasks[task.ID] = task |
There was a problem hiding this comment.
Перед этой операцией полезно добавить проверку - есть ли уже такой ID. И если да - вернуть ошибку пользователю. Иначе задача может быть перезатерта
precode.go
Outdated
| if err != nil { | ||
| fmt.Printf("Ошибка записи: %v", err) | ||
| } | ||
| json.NewEncoder(w).Encode(task) |
There was a problem hiding this comment.
Это здесь нужно? Стоит проверить как выглядит ответ этой ручки
There was a problem hiding this comment.
Согласен. Можно было и через Chat GPT проверить. Сразу выдал ошибку. Глаза замылились в процессе написания)
There was a problem hiding this comment.
Согласен. Можно было и через Chat GPT проверить. Сразу выдал ошибку. Глаза замылились в процессе написания)
Потестить через Postman лучше
| } | ||
| w.Header().Set("Content-Type", "application/json") | ||
| w.WriteHeader(http.StatusOK) | ||
| _, err = w.Write(resp) |
There was a problem hiding this comment.
Класная телега у тебя. кстати. Подписался)
There was a problem hiding this comment.
Класная телега у тебя. кстати. Подписался)
Рада. Добро пожаловать!
| @@ -73,6 +73,11 @@ func postTask(w http.ResponseWriter, r *http.Request) { | |||
| http.Error(w, err.Error(), http.StatusBadRequest) | |||
| return | |||
| } | |||
| if _, ex := tasks[task.ID]; ex { | |||
| http.Error(w, "Задача с таким ID уже существует", http.StatusConflict) | |||
There was a problem hiding this comment.
Форматирование съехало, рекомендую добавить go fmt в настройки IDE, в автозапуск при сохранении
There was a problem hiding this comment.
Форматирование съехало, рекомендую добавить
go fmtв настройки IDE, в автозапуск при сохранении
Спасибо за фичу. Учту в следующих проектах.
| @@ -99,7 +104,7 @@ func getTaskForId(w http.ResponseWriter, r *http.Request) { | |||
| if err != nil { | |||
| fmt.Printf("Ошибка записи: %v", err) | |||
| } | |||
| json.NewEncoder(w).Encode(task) | |||
|
|
|||
No description provided.