Unhandled rejection TypeError: Cannot read property 'postedBy' of nullの解決

Question

Javascriptのコードのエラーの原因と対策を突き止めたい。

秘密の匿名掲示板の削除機能の実装途中にエラーが出ました。
削除ボタンUI作成はうまくいったのですが、削除処理の実装して
削除ボタンを押しても消えないため。端末を見たらエラーメッセージが
表示されてました。

発生している問題・エラーメッセージ

Unhandled rejection TypeError: Cannot read property 'postedBy' of null
    at Post.findById.then (/home/satuki/workspace/secret-board-3024/lib/posts-handler.js:66:32)
    at tryCatcher (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/promise.js:512:31)
    at Promise._settlePromise (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromise0 (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/promise.js:694:18)
    at _drainQueueStep (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/async.js:138:12)
    at _drainQueue (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/async.js:131:9)
    at Async._drainQueues (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/async.js:147:5)
    at Immediate.Async.drainQueues [as _onImmediate] (/home/satuki/workspace/secret-board-3024/node_modules/bluebird/js/release/async.js:17:14)
    at runCallback (timers.js:694:18)
    at tryOnImmediate (timers.js:665:5)
    at processImmediate (timers.js:647:5)

該当のソースコード

'use strict';
const pug = require('pug');
const Cookies = require('cookies');
const util = require('./handler-util');
const Post = require('./post');

const trackingIdKey = 'tracking_id';

function handle(req, res) {
  const cookies = new Cookies(req, res);
  addTrackingCookie(cookies);

  switch (req.method) {
    case 'GET':
      res.writeHead(200, {
        'Content-Type': 'text/html; charset=utf-8'
      });
      Post.findAll({ order: [['id', 'DESC']] }).then((posts) => {
        res.end(pug.renderFile('./views/posts.pug', {
          posts: posts,
          user: req.user
        }));
        console.info(
          `閲覧されました: user: ${req.user}, ` +
          `trackinId: ${cookies.get(trackingIdKey) },` +
          `remoteAddress: ${req.connection.remoteAddress}, ` +
          `userAgent: ${req.headers['user-agent']} `
          );
      });
      break;
    case 'POST':
      let body = [];
      req.on('data', (chunk) => {
        body.push(chunk);
      }).on('end', () => {
        body = Buffer.concat(body).toString();
        const decoded = decodeURIComponent(body);
        const content = decoded.split('content=')[1];
        console.info('投稿されました: ' + content);
        Post.create({
          content: content,
          trackingCookie: cookies.get(trackingIdKey),
          postedBy: req.user
        }).then(() => {
          handleRedirectPosts(req, res);
        });
      });
      break;
    default:
      util.handleBadRequest(req, res);
      break;
  }
}

function handleDelete(req, res){
  switch(req.method){
    case 'POST':
      let body = [];
      req.on('data', (chunk) => {
        body.push(chunk);
      }).on('end', () => {
        body = Buffer.concat(body).toString();
        const decoded = decodeURIComponent(body);
        const id = decoded.split('id=')[1];
        Post.findById(id).then((post) => {
          if(req.user === post.postedBy){
          post.destroy();
        }
        handleRedirectPosts(req, res);
      })
      });
      break;
      default:
        util.handleBadRequest(req, res);
        break;
    }
}

function addTrackingCookie(cookies) {
  if (!cookies.get(trackingIdKey)) {
    const trackingId = Math.floor(Math.random() * Number.MAX_SAFE_INTEGER);
    const tomorrow = new Date(new Date().getTime() + (1000 * 60 * 60 * 24));
    cookies.set(trackingIdKey, trackingId, { expires: tomorrow });
  }
}

function handleRedirectPosts(req, res) {
  res.writeHead(303, {
    'Location': '/posts'
  });
  res.end();
}

module.exports = {
  handle: handle,
  handleDelete: handleDelete
};

試したこと

console.log(post)でpostの内容を見たこと、でも理解出来なかった。

補足情報（FW/ツールのバージョンなど）

N予備校 プログラミングWebアプリ入門 テキスト24
Git
Pug
Javascipt
Node.js
PostgreSQL

Answer 1

下記コードのpostがnull、つまり該当する投稿が見つからないためにエラーが出てるのではないかと思います。

Post.findById(id).then((post) => {
  // null には postedBy メソッドが存在しないのでエラー
  if (req.user === post.postedBy) {
    post.destroy();
  }
  handleRedirectPosts(req, res);
});

フロントエンドで送信する段階でidが間違っているか、クエリー文字列からidを取り出した際に間違ったidになってしまっているか、Post.findByIdメソッドの実装に不具合があるかのどれかかなと思います。

とりあえずこのままだと id=***&hoge=fooのように他のパラメータを後ろに付けて送信することができなかったりするので、クエリー文字列をパースする際には query-string などのパッケージを使うのが無難だと思います。