Реализация приложения - #1
Conversation
По факту фильтр работает, но тест пытается получить данные с неверной текущей датой
|
Сортировка нарушена, сверху должны быть самые актуальные 
|
ivannizh
left a comment
ivannizh
left a comment
There was a problem hiding this comment.
Эдуард, привет!
Отличная выпускная работа, всё работает, тесты проходят.
Код хорошо написан и верно разложен по файлам!
Работу принимаю!
Поздравляю с окончанием курса и желаю успехов в дальнейшем освоении Go!
| } | ||
|
|
||
| if err = db.DeleteTask(id); err != nil { | ||
| writeJson(w, ErrorResponse{err.Error()}, http.StatusUnprocessableEntity) |
There was a problem hiding this comment.
Тут или 404 если нет такой задачи, или 500 если ошибка БД, надо смотреть на ошибку и выставлять код
|
|
||
| task.Date = date | ||
| if err = db.UpdateTask(task); err != nil { | ||
| writeJson(w, ErrorResponse{err.Error()}, http.StatusInternalServerError) |
There was a problem hiding this comment.
Ошибку от БД (или внутренних сервисов) лучше не отдавать наружу, там могут быть чувствительные данные (адрес БД, названия таблиц, запросы и тд).
Лучше всего такую ошибку просто залогировать, а пользователю вернуть заглушку "Внутренняя ошибка"
|
|
||
| <-sig | ||
| s.Shutdown() | ||
| db.Close() |
There was a problem hiding this comment.
Лучше всегда закрывать через дефер сразу после проверки ошибки, так как дефер срабатывает и на панике, а если так писать, то при панике этот код не выполниться
2 participants
No description provided.