JavaScriptで、繰り返し処理と条件分岐を同時に行う方法を教えていただきたいです

Question

前提・実現したいこと

JavaScriptで、繰り返し処理と条件分岐を同時に行う時のやり方を教えていただきたいです。

現在、標準入力から受け取った値が、閏年に該当するかどうかのプログラムを作成しています。

標準入力では、一つ目に、何個の西暦を閏年と判断するか、それ以降に閏年かを判定する西暦を入力しています。

例）
2, 1996, 2010

標準入力で取得した値は、整数化してnum_lines()という配列に格納しています。
一つ目の標準入力は変数Nとして取得しています。

繰り返し文（for)を使って、num_lines[1]からnum_lines[N]までの値をひとつづつ取得し、
条件分岐(switch命令）を使って、それぞれが閏年に該当するか調べていこうとしているのですが、コードを動かしても正しく動いてくれません。

出力結果

'num_lines[i] is a leap year'

とだけ出力されてしまいます。

おそらく、繰り返し文を使ってnum_lines[i]に、num_lines[1]からnum_lines[N]までを一つづつ入れていく処理が
switch命令にうまく反映されていないのが原因なのではないかと思うのですが、改善方法がわかりません。

間違っている点を指摘していただきたく思います。

よろしくお願いいたします。

該当のソースコード

//標準入力を取得
process.stdin.resume();
process.stdin.setEncoding('utf8');
var lines = [];
var reader = require('readline').createInterface({
  input: process.stdin,
  output: process.stdout
});
reader.on('line', (line) => {
  lines.push(line);
});

reader.on('close', () => {
//受け取った標準入力を整数化
    var num_lines = lines.map(function(num) { return num; });
//閏年かどうかを検討する西暦の数
    var N = num_lines[0];
    
    for(i=1; i<=N;  i++){
    
     switch(num_lines[i]){
         case 'num_lines[i] % 400 === 0':
            console.log('num_lines[i] is a leap year');
         break;
        case 'num_lines[i] % 100 === 0':
            console.log('num_lines[i] is not a leap year');
         break;
         case 'num_lines[i] % 4 === 0':
             console.log('num_lines[i] is a leap year');
        break;
        default:
            console.log('num_lines[i] is not a leap year')
         break;
    }
    } 
});

Answer 1

おそらく、繰り返し文を使ってnum_lines[i]に、num_lines[1]からnum_lines[N]までを一つづつ入れていく処理が

switch命令にうまく反映されていないのが原因なのではないかと思うのですが

違います。switchの条件の書き方が適当ではありません。

単に文字列を書いているだけなので、判定の条件が入力された文字列が、この文字列と一致するかになってしまっています。条件式として動作していません。

Comments

[maisumakun]

まずはループを回す前に、「1つの」日付に対するうるう年判定の処理を作ってください。

[moritat-222]

回答ありがとうございます。

数式で処理すべきところを文字列として処理してしまっていたことに気付けました。
ただ、コードを修正しても、やはりうまくいかず、閏年として処理すべきところが「閏年ではない」と判定されてしまいます。全てswitch文のdefaultで処理されてしまっているところまでは分かったのですが、それがなぜなのか、自分では解決できませんでした。

改善したコードはこちらです。
よろしければ、間違っている点を指摘していただきたく思います。
よろしくお願いいたします。

//標準入力を取得
process.stdin.resume();
process.stdin.setEncoding('utf8');
var lines = [];
var reader = require('readline').createInterface({
input: process.stdin,
output: process.stdout
});
reader.on('line', (line) => {
lines.push(line);
});

reader.on('close', () => {
//受け取った標準入力を整数化
var num_lines = lines.map(function(num) { return num; });
//閏年かどうかを検討する西暦の数
var N = num_lines[0];

for(var i=1; i<=N; i++){

switch(num_lines[i]){
case num_lines[i] % 400 === 0:
console.log(num_lines[i] + '　is a leap year');
break;
case num_lines[i] % 100 === 0:
console.log(num_lines[i] + ' is not a leap year');
break;
case num_lines[i] % 4 === 0:
console.log(num_lines[i] + ' is a leap year');
break;
default:
console.log(num_lines[i] + ' is not a leap year');
break;
}
}
});

[moritat-222]

解決できました！！！

switch(num_lines[i]){

ではなく

switch(true){

ですね！！

ありがとうございます