入力の検証

CoddyのNode.jsジャーニー「REST API」セクションの一部。レッスン 6/18。

クライアントは本文にどんな値でも送信でき、if (!name)で防げるのは空の値だけです。123、true、["x"]は真と評価され、そのまま処理されます。また、空白だけの文字列も問題ないように見えます。各フィールドの型を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では、送信されたフィールドだけを確認します。ほかのフィールドは変更されないためです。

チャレンジ

簡単

POST: title は1～100文字の文字列でなければならず、空白のみであってはなりません。そうでない場合は、title を示す error とともに400を返します。
PATCH: title が送信された場合も同様です。また、done が送信された場合は、true または false でなければなりません。

{ "title": 123 } を指定して POST を送信し、次に { "done": "yes" } を指定して PATCH /api/tasks/2 を送信します。

自分で試してみよう

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;