Express res.json()で処理を終了する方法

Question

Express res.json()で処理を終了する方法

Expressを用いて以下のようなコードを実装しました。

router.post('/', (req, res) => {
  --省略--
  if (flag) {
    res.json('data1': data1)
  }

  res.json('data2': data2)
})

この処理が呼び出されたとき、flagがfalseであれば問題はないのですが、flagがtrueのときにres.json('data1': data1)が呼び出された後に、res.json('data2': data2)も呼び出され、

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

という2度レスポンスが送られた旨のエラーが発生します。

router.post('/', (req, res) => {
  --省略--
  if (flag) {
    res.json('data1': data1)
  } else {
    res.json('data2': data2)
  }
})

とすれば当然エラーは発生しなくなります。このような仕様になっているのはres.json()の後も処理が行われることが前提となっているためですが、res.json()の後に処理をする必要がなく、それらの処理を無視したい場合

router.post('/', (req, res) => { ・・・➀
  --省略--
  if (flag) {
    res.json('data1': data1)
    ➀を終了させる処理
  }

  res.json('data2': data2)
  ➀を終了させる処理
})

のように「➀を終了させる処理」は存在するのでしょうか。

追記が多くなってしまったので少しまとめます。
・returnを使う方法 → ➀内でさらに関数を（メソッドの引数などで）呼び出し、その関数内でres.json()を実行するときに➀を抜けられない。入れ子にしないなら使える。

この他に何かよい方法をご存知でしたらぜひお教えください

###追記
実際には

router.post('/', (req, res) => {
  --省略--
  if (flag) {
    return res.json('data1': data1)
  } else {
    return res.json('data2': data2)
  }
})

のように実装していたのですが、

router.post('/', (req, res) => { ・・・Ⅰ
  --省略--
  AAAA()
  .then((flag) => { ・・・Ⅱ
    if(flag) {
      return res.json('data1': data1) ・・・Ⅲ
    } else {
      return res.json('data2': data2) ・・・Ⅳ
    }
  })
  .then((DATA) => { //res.json()の戻り値
    --省略--
  })

  return res.json('data3': data3) ・・・Ⅴ
})

のようなコードを考えた時、
ⅢもしくはⅣが実行されるのはⅡの関数内であり、このreturnではres.jsonの戻り値がDATAとして次の関数に渡されるだけでⅠを抜けられないことからreturn以外の方法があればと思い質問しました。
説明が不足しており、大変申し訳ありませんでした。

###追記2
確かに上記のコードではⅤが先に実行されますね。
コードを修正します。

router.post('/', (req, res) => { ・・・Ⅰ
  --省略--
  AAAA()
  .then((flag) => { ・・・Ⅱ
    if(flag) {
      return res.json('data1': data1) ・・・Ⅲ
    } else {
      return res.json('data2': data2) ・・・Ⅳ
    }
    --省略--
  })
  .then((DATA) => { //res.json()の戻り値
    --省略--
    return res.json('data3': data3) ・・・Ⅴ
  })
})

このコードではⅢもしくはⅣが実行された後にⅤが実行されエラーになると思います。

#追記3

router.post('/', (req, res) => { ・・・Ⅰ
  --省略--
  AAAA()
  .then((flag) => { ・・・Ⅱ
    if(flag) {
      return res.json('data1': data1) ・・・Ⅲ
    } else {
      return res.json('data2': data2) ・・・Ⅳ
    }
    --省略--
  })
  .then((DATA) => { //res.json()の戻り値
    --省略--
    return res.json('data3': data3) ・・・Ⅴ
  })
})

この処理をreturnが使えない例として挙げたのはよくなかったかもしれません…
res.json()の実行後にⅠを抜ける共通処理的なものはあるのでしょうか。

Comments

[hoshi-takanori]

return じゃ駄目でしょうか？

[hoshi-takanori]

それだと V がまず実行されて、非同期で III または IV が実行されるのでは…。

最近 express 書いてないのでよく分かりませんが、返事を確実に 1 回だけ返すには async/await を使うのがいいんじゃないかという気がします。
https://qiita.com/yukin01/items/1a36606439123525dc6d

Answer 1

プログラミングの問題です。

res.jsonは一度しか実行してはいけません。
でも1度は確実に実行したいわけじゃないですか。
だったら関数の末尾に書けばよくね？って話です。

router.post('/', (req, res) => {
  let result = null;

  // --省略--
  if (flag) {
    result = {data1: data1};
  }

  // 初期の質問をベースにするならこういう逃げ方が良いかな
  if (result == null) {
    result = {data2: data2};
  }

  // 共通の終了処理があるならres.jsonの前後で指定すれば良いね
  res.json(result);
})

関数でまるっと包んでも良いと思います。

// 複雑な事をやっている検索システムという体で適当な関数名にした
const awesomeSearch = (connection) => {
  // --省略--

  if (flag) {
    return {data1: data1};
  }

  return {data2: data2};
}

router.post('/', (req, res) => {
  // 事前準備としてconnectionみたいなのを準備
  const connection = {};

  res.json(awesomeSearch(connection));
  // 終了処理を挟んでconnectionを処理
})

---

いやいや、発展した質問部分のようにPromiseを使って非同期でやってるから無理だよ！

async/awaitを使ってください。
「追記3」のコードをベースに改修するとこうなります。

// 関数の前にasyncキーワードを付与することで
// awaitが使えるasync関数になる
router.post('/', async (req, res) => {
  let result = {};

  // --省略--

  // Promiseインスタンスを返すAAAA関数の実行→resolveを取り出す処理はこう書ける
  const flag = await AAAA();
  if (flag) {
    result.data1 = data1;
  }

  // 初期のresultが{}なのでキーの数で空か否かを決定する逃げ方に変更してみた
  if (Object.keys(result).length == 0) {
    result.data2 = data2;
  }

  // 上記AAAA関数のthenを同期的に叩けるようになるので
  // 複雑な処理を経由した後のDATAもこのレベルに記述出来るはず
  if (DATA) {
    result.data3 = data3;
  }

  res.json(result);
})

Comments

[watter]

なるほど、resultを用意してからさいごにまとめてres.jsonする...思いつきませんでした。ご回答ありがとうございました。