אימות קלט

חלק מהיחידה REST APIs במסלול ה-Node.js של Coddy. שיעור 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 תווים, ולא להכיל רווחים בלבד, אחרת יוחזר קוד 400 עם error המציין את title.
PATCH: אותו הדבר לגבי title כשנשלח, וגם done, כשנשלח, חייב להיות true או false.

שלחו POST עם { "title": 123 }, ולאחר מכן PATCH /api/tasks/2 עם { "done": "yes" }.

נסו בעצמכם

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;