node.JSのif orもしくは正規表現書き方について

Question

20x,30xステータス以外の条件つけるaws lambdaスクリプトをNode.JSで作ってます。
この場合、どのように条件つければよろしいでしょうか。

以下のように書いてみたのですが、200ステータスも含まれていたので何か間違ってる気がします。

  if (Number(status_code) == 503 || 500 || 400 ||404) {
       
      snsParams.Message += line.split(' ')[0].split('.')[0] + '\t' + status_code + '\t' + source_ip+ '\t' 
       + line.split('"')[1] + '\t' + line.split('"')[3] + '\n' ;
       console.log(snsParams.Message);
    }

しょぼい質問で申し訳ないですが、よろしくお願いします。

Answer 1

以前も似たような質問がありましたが、そう書くと「Number(status_code) == 503」または「500がtrue」というような条件と解釈されてしまって、必ずtrue判定されます。

書き方としては、2通り考えられます。

その1：わかりやすく書く

// これで数値化できる
var status_num = +status_code;
if(status_num === 503 || status_num === 500 || status_num === 403 || status_num === 404)

その2： 配列を参照

var codes = [403, 404, 500, 503];
if(codes.indexOf(+status_code)!== -1)

余談ですが、HTTPステータスコードの400番台、500番台はすべてエラー時に起きるものなので、特別にハンドリングしたいものがなければ>= 400でまとめて拾うのもありです。

Comments

[Kimsehwa]

ありがとうございます！おかげさまで解決できました