Eingaben validieren

Teil des Abschnitts REST-APIs der Node.js-Journey von Coddy. Lektion 6 von 25.

Ein Client kann im Body alles Mögliche senden, und if (!name) hält nur leere Werte zurück. 123, true und ["x"] sind wahrheitswertig und kommen durch. Auch eine Zeichenfolge aus Leerzeichen kommt durch: Fange sie mit trim() ab, denn ' '.trim() ist ''. Prüfe den Typ jedes Feldes mit typeof und anschließend dessen Grenzen:

const { seats } = req.body ?? {};
if (typeof seats !== 'number' || !Number.isInteger(seats) || seats < 1) {
    return res.status(400).json({ error: 'seats must be a whole number from 1' });
}

Formuliere die Nachricht für die Person, die die Antwort liest: Nenne das Feld und sage, welchen Wert es haben muss, damit klar ist, was zu korrigieren ist. Bei einer PATCH-Anfrage werden nur die Felder geprüft, die mitgesendet wurden, da die übrigen unverändert bleiben.

Aufgabe

Einfach

Derzeit lässt if (!title) einen Titel wie 123 oder " " durch. Prüfe den Typ und die Größe jedes Feldes, bevor du es speicherst.

Prüfe in routes/tasks.js in der POST-Route title:
- Es muss eine Zeichenkette sein.
- Es darf nicht leer sein oder nur Leerzeichen enthalten.
- Es darf höchstens 100 Zeichen lang sein, einschließlich Leerzeichen.
Wenn eine dieser Bedingungen nicht erfüllt ist, antworte mit Status 400 und einer error-Meldung, die das Wort title enthält, zum Beispiel:
```
{ "error": "title must be a string of 1 to 100 characters" }
```
Prüfe in der PATCH-Route nur die Felder, die der Body sendet:
- Wenn er title sendet, gelten dieselbe Regel und dieselbe Art von Fehlermeldung.
- Wenn er done sendet, muss der Wert true oder false sein. Andernfalls antworte mit Status 400 und einer Fehlermeldung, die das Wort done enthält:
```
{ "error": "done must be true or false" }
```
Ein abgelehnter PATCH ändert nichts.
Gültige Bodys funktionieren weiterhin: POST gibt nach wie vor 201 zurück, und PATCH mit { "done": true } gibt weiterhin 200 zurück. Lass PUT unverändert; Schemas prüft das ebenfalls mit zod.

Probiere es aus: Sende POST { "title": 123 } an /api/tasks: Du erhältst 400 mit dem title-Fehler. Sende POST { "title": " " }: ebenfalls 400. PATCH /api/tasks/2 mit { "done": "yes" }: 400 mit dem done-Fehler, und GET /api/tasks/2 zeigt weiterhin "done": false. PATCH /api/tasks/2 mit { "done": true }: 200.

Probier es selbst

import { Router } from 'express';
import { nextId, tasks } from '../store.js';

const router = Router();

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 res.status(404).json({ error: 'Task not found' });
    }
    res.json(task);
});

router.post('/', (req, res) => {
    const { title } = req.body ?? {};
    if (!title) {
        return res.status(400).json({ error: 'title is required' });
    }
    const task = { id: nextId(), title, done: false, 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 res.status(404).json({ error: 'Task not found' });
    }
    const { title, done } = req.body ?? {};
    if (title !== undefined) task.title = title;
    if (done !== undefined) task.done = done;
    res.json(task);
});

router.put('/:id', (req, res) => {
    const index = tasks.findIndex((t) => t.id === Number(req.params.id));
    if (index === -1) {
        return res.status(404).json({ error: 'Task not found' });
    }
    const { title, done } = req.body ?? {};
    if (title === undefined || done === undefined) {
        return res.status(400).json({ error: 'PUT needs every field: title and done' });
    }
    const { id, createdAt } = tasks[index];
    tasks[index] = { id, title, done, 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 res.status(404).json({ error: 'Task not found' });
    }
    tasks.splice(index, 1);
    res.status(204).end();
});

export default router;