Проверка входных данных

Часть раздела REST API путешествия по Node.js на Coddy. Урок 6 из 25.

Клиент может отправить в теле запроса что угодно, а if (!name) отсеивает только пустые значения. 123, true и ["x"] считаются истинными и проходят проверку. Строка из пробелов тоже проходит: отфильтруй её с помощью trim(), поскольку ' '.trim() — это ''. Проверь тип каждого поля с помощью typeof, а затем проверь ограничения:

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

Сформулируй сообщение для того, кто читает ответ: укажи имя поля и скажи, каким оно должно быть, чтобы человек знал, что исправить. Запрос PATCH проверяет только те поля, которые в нём переданы, поскольку остальные не изменяются.

Задание

Легко

Сейчас if (!title) пропускает заголовки вроде 123 или " ". Перед сохранением проверь тип и размер каждого поля.

В routes/tasks.js, в маршруте POST, проверь title:
- Это должна быть строка.
- Строка не должна быть пустой или состоять только из пробелов.
- Её длина должна быть не более 100 символов, включая пробелы.
Если любое из этих условий не выполнено, верни ответ со статусом 400 и текстом ошибки в поле error, содержащим слово title, например:
```
{ "error": "title must be a string of 1 to 100 characters" }
```
В маршруте PATCH проверяй только поля, переданные в теле запроса:
- Если передано title, применяются те же правила и такой же тип ошибки.
- Если передано done, его значением должно быть true или false. В противном случае верни ответ со статусом 400 и ошибкой, содержащей слово done:
```
{ "error": "done must be true or false" }
```
Отклонённый PATCH ничего не меняет.
Корректные тела запросов по-прежнему работают: POST возвращает 201, а PATCH с { "done": true } по-прежнему возвращает 200. Оставь PUT без изменений; в схемах zod тоже выполняет проверку.

Проверь: отправь POST-запрос с { "title": 123 } на /api/tasks: получишь 400 с ошибкой для заголовка. Отправь POST-запрос с { "title": " " }: также получишь 400. Отправь PATCH /api/tasks/2 с { "done": "yes" }: получишь 400 с ошибкой для поля done, а GET /api/tasks/2 по-прежнему покажет "done": false. Отправь PATCH /api/tasks/2 с { "done": true }: получишь 200.

Попробуйте сами

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;