Convalidare l'input

Fa parte della sezione API REST del percorso Node.js di Coddy. Lezione 6 di 25.

Un client può inviare qualsiasi cosa nel corpo e if (!name) blocca solo i valori vuoti. 123, true e ["x"] sono valori truthy e passano. Anche una stringa composta da spazi passa: intercettala con trim(), poiché ' '.trim() è ''. Controlla il tipo di ogni campo con typeof, poi i suoi limiti:

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' });
}

Scrivi il messaggio pensando a chi legge la risposta: indica il nome del campo e spiega cosa deve essere, così saprà cosa correggere. Una richiesta PATCH controlla solo i campi che le sono stati inviati, poiché gli altri non stanno cambiando.

Sfida

Facile

Al momento if (!title) lascia passare titoli come 123 o " ". Controlla il tipo e la lunghezza di ogni campo prima di salvarlo.

In routes/tasks.js, nella route POST, controlla title:
- Deve essere una stringa.
- Non deve essere vuoto né contenere solo spazi.
- Deve essere lungo al massimo 100 caratteri, spazi inclusi.
Se una qualsiasi di queste condizioni non è soddisfatta, rispondi con lo stato 400 e un messaggio di error che contenga la parola title, per esempio:
```
{ "error": "title must be a string of 1 to 100 characters" }
```
Nella route PATCH, controlla solo i campi inviati nel corpo della richiesta:
- Se invia title, si applicano la stessa regola e lo stesso tipo di errore.
- Se invia done, il valore deve essere true o false. Altrimenti, rispondi con lo stato 400 e un errore che contenga la parola done:
```
{ "error": "done must be true or false" }
```
Una richiesta PATCH rifiutata non modifica nulla.
Le richieste valide continuano a funzionare: POST restituisce ancora 201 e PATCH con { "done": true } restituisce ancora 200. Lascia PUT così com'è; anche gli schemi con zod vengono controllati.

Prova: invia POST con { "title": 123 } a /api/tasks: ottieni 400 con l'errore relativo a title. Invia POST con { "title": " " }: anche in questo caso ottieni 400. Invia PATCH /api/tasks/2 con { "done": "yes" }: ottieni 400 con l'errore relativo a done, e GET /api/tasks/2 mostra ancora "done": false. Invia PATCH /api/tasks/2 con { "done": true }: ottieni 200.

Provalo tu

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;