Konflikty

Część sekcji API REST ścieżki Node.js w Coddy. Lekcja 13 z 25.

Drugie zadanie o nazwie „Wyślij fakturę” sprawia, że lista jest myląca, ale jego treść jest prawidłowa. To 409 Conflict: żądanie koliduje z tym, co jest już zapisane. Kod 400 oznacza, że sama treść żądania jest nieprawidłowa.

const taken = users.some((u) => u.name.toLowerCase() === name.toLowerCase());
if (taken) {
    return sendError(res, 409, 'conflict', 'That username is taken');
}

Porównuj zapisane małymi literami, ponieważ ludzie odczytują „Ada” i „ada” jako to samo imię. Gdy żądanie zmienia nazwę czegoś, pomiń sam element podczas wyszukiwania. W przeciwnym razie użytkownik, który zapisze formularz bez zmiany nazwy, spowoduje konflikt z samym sobą.

Wyzwanie

Łatwy

Dwa zadania o nazwie "Send the invoice" sprawiają, że lista staje się nieczytelna. Odrzucaj tytuł, który ma już inne zadanie.

W routes/tasks.js porównuj tytuły bez rozróżniania wielkości liter: "Book the meeting room" i "book the MEETING room" to ten sam tytuł.
W POST, po pomyślnym przejściu weryfikacji schematu: jeśli jakiekolwiek zadanie ma już ten tytuł, użyj sendError, aby odpowiedzieć statusem 409, i niczego nie dodawaj:
```
{ "error": { "code": "conflict", "message": "A task with this title already exists" } }
```
Zrób to samo w PATCH (tylko wtedy, gdy jego treść zawiera title) i w PUT. W tych dwóch trasach pomiń w wyszukiwaniu zmieniane zadanie, aby mogło zachować własny tytuł lub zmienić tylko jego wielkość liter. Odrzucone żądanie PATCH lub PUT niczego nie zmienia.

Wypróbuj: wyślij POST { "title": "book the meeting room" } do /api/tasks: otrzymasz 409 z kodem conflict, ponieważ zadanie 2 nazywa się "Book the meeting room". Wyślij PATCH /api/tasks/3 z treścią { "title": "Write the Project Brief" }: otrzymasz 409, a GET /api/tasks/3 nadal pokaże "Send the invoice". Wyślij PUT /api/tasks/2 z treścią { "title": "Book the meeting room", "done": true }: otrzymasz 200, ponieważ to własny tytuł zadania 2. Wyślij POST { "title": "Order new chairs" }: otrzymasz 201.

Spróbuj swoich sił

import { Router } from 'express';
import { z } from 'zod';
import { sendError, sendInvalid } from '../errors.js';
import { nextId, tasks } from '../store.js';

const router = Router();

// Kompletne zadanie w postaci przesyłanej przez PUT. .strict() odrzuca pola, których nie wymienia.
const Task = z
    .object({
        title: z.string().trim().min(1).max(100),
        done: z.boolean(),
    })
    .strict();
const NewTask = Task.partial({ done: true });
const TaskChanges = Task.partial();

router.get('/', (req, res) => {
    res.json(tasks);
});

router.get('/:id', (req, res) => {
    const task = tasks.find((t) => t.id === Number(req.params.id));
    if (!task) {
        return sendError(res, 404, 'not_found', 'Task not found');
    }
    res.json(task);
});

router.post('/', (req, res) => {
    const result = NewTask.safeParse(req.body);
    if (!result.success) {
        return sendInvalid(res, result.error);
    }
    const { title, done = false } = result.data;
    const task = { id: nextId(), title, done, createdAt: new Date().toISOString() };
    tasks.push(task);
    res.status(201).location(`/api/tasks/${task.id}`).json(task);
});

router.patch('/:id', (req, res) => {
    const task = tasks.find((t) => t.id === Number(req.params.id));
    if (!task) {
        return sendError(res, 404, 'not_found', 'Task not found');
    }
    const result = TaskChanges.safeParse(req.body);
    if (!result.success) {
        return sendInvalid(res, result.error);
    }
    Object.assign(task, result.data);
    res.json(task);
});

router.put('/:id', (req, res) => {
    const index = tasks.findIndex((t) => t.id === Number(req.params.id));
    if (index === -1) {
        return sendError(res, 404, 'not_found', 'Task not found');
    }
    const result = Task.safeParse(req.body);
    if (!result.success) {
        return sendInvalid(res, result.error);
    }
    const { id, createdAt } = tasks[index];
    tasks[index] = { id, ...result.data, createdAt };
    res.json(tasks[index]);
});

router.delete('/:id', (req, res) => {
    const index = tasks.findIndex((t) => t.id === Number(req.params.id));
    if (index === -1) {
        return sendError(res, 404, 'not_found', 'Task not found');
    }
    tasks.splice(index, 1);
    res.status(204).end();
});

export default router;