Develop #1
Develop #1
Conversation
| http.HandleFunc("/api/tasks", handlers.GetTasksHandler(database)) | ||
| http.HandleFunc("/api/task/done", handlers.DoneTaskHandler(database)) | ||
|
|
||
| if err := http.ListenAndServe(":"+port, nil); err != nil { |
There was a problem hiding this comment.
Лучше перед стартом выводить лог, чтобы видеть, что приложение запустилось и на каком порту
internal/app/todo_app.go
Outdated
| if task.Repeat != "" { | ||
| nextDate, err := NextDate(now, task.Date, task.Repeat) | ||
| if err != nil { | ||
| return "", fmt.Errorf("invalid repeat rule: %v", err) |
There was a problem hiding this comment.
Для вывода в логи %v, а для оборачивания %w (wrap)
| return "", fmt.Errorf("invalid repeat rule: %v", err) | |
| return "", fmt.Errorf("invalid repeat rule: %w", err) |
| } | ||
| t.ID = strconv.FormatInt(id64, 10) | ||
| tasks = append(tasks, t) | ||
| } |
There was a problem hiding this comment.
После цикла нужно проверить курсор на наличие ошибок
| } | ||
| idInt, err := strconv.Atoi(id) | ||
| if err != nil { | ||
| return nil, fmt.Errorf("invalid task id '%s'", id) |
There was a problem hiding this comment.
Очень хорошо, что везде оборачиваешь, но лучше текст ошибок оставлять
| return nil, fmt.Errorf("invalid task id '%s'", id) | |
| return nil, fmt.Errorf("invalid task id '%s': %w", id, err) |
|
|
||
| var task app.Task | ||
| if err := json.NewDecoder(r.Body).Decode(&task); err != nil { | ||
| log.Printf("JSON Decode Error: %v", err) |
There was a problem hiding this comment.
Верно сделано, что ошибку в лог, а пользователю простое сообщение
| var task app.Task | ||
| if err := json.NewDecoder(r.Body).Decode(&task); err != nil { | ||
| log.Printf("JSON Decode Error: %v", err) | ||
| json.NewEncoder(w).Encode(map[string]string{ |
There was a problem hiding this comment.
Только перед записью ответа нужно выставить код ответа
There was a problem hiding this comment.
Если не указать код, то по умолчанию будет 200
Привет! Спасибо за ревью. Исправил по замечаниям |
| id := r.URL.Query().Get("id") | ||
| if err := app.DeleteTask(db, id); err != nil { | ||
| log.Printf("DeleteTask error: %v", err) | ||
| w.WriteHeader(http.StatusBadRequest) |
There was a problem hiding this comment.
А если это внутренняя ошибка коннекта к БД?
|
|
||
| if r.Method != http.MethodPost { | ||
| http.Error(w, "Method not allowed", http.StatusMethodNotAllowed) | ||
| w.WriteHeader(http.StatusMethodNotAllowed) |
There was a problem hiding this comment.
Тут все было верно, так тоже можно записывать ответ
Итоговый PR