javascript 出力した値の要素数を出したい

Question

ご訪問ありがとうございます。プログラミング初心者で、学習教材の問題と日々戦っているのですが、どうしてもわからない部分があり、質問させていただきました。

２
２５
１４
６７
５０

という数値があった場合に、３０以下の数字がいくつあるかを出力する（上の場合は３と出力したい）という問題で悩んでおります。

var reader = require('readline').createInterface({
  input: process.stdin,
  output: process.stdout
});
reader.on('line', (line) => {
  lines.push(line);
});
reader.on('close', () => {
    
 for (  var i = 0;  i < lines.length;  i++  ) {
    if((lines[i])<=30){
            console.log(lines[i]); 
            
        
    }
}
});
コード

これで、
２
２５
１４
とは出るのですが、３を出すために.lengthをつけても、文字数の長さを数値化してしまい、うまくいきません。

この場合、この行数を配列にして、要素数を導くのでしょうか？
それとも行数を出力することができるのでしょうか。。
どちらにしても理解が出来ないため、教えていただけると幸いです。よろしくお願いいたします。
（何言ってるかわからない場合お手数ですがコメントいただきたいです）

Answer 1

全角の数値を例示しているところが気になりますが
配列に数値が保持されているなら、filterしてlengthを確認すればよいです

var a=[2,25,14,67,50];
var b=a.filter(x=>x<=30).length;
console.log(b);

Comments

[kumo]

迅速なご返答ありがとうございます。
この問題は入力値をこちらで指定することが出来ず、教材の方で指定されており、
例２
３
５６
８７
２２
といった別の入力値にも対応した出力をしなければならず、
yambejp様の1行目の文aを空にして一度試してみましたがうまくいきませんでした。。
詳細不足ですみません。

ですがfilter というメソッドは知らなかったのでとても為になりました。ありがとうございます。

[yambejp]

これってNode.jsですか？
もしそうならタグ追加したほうがよいでしょう
（ちがうなら環境にあわせたタグを追加してください）

[papinianus]

pushするときにNumberに変えてた上で、回答のaを全部linesに変えればできるはず

[papinianus]

paizaとかatcoderとかcodewarsとかではないですかね

[kumo]

papinianus様
ご回答ありがとうございます。返答が遅くなってしまいすみません。
おかげさまで解くことが出来ました！！

問題は変えてますがpaizaです。他の問題でも同じような理由で解けない問題が多く困っておりました、この文を参考にしながら解いていきたいと思います。本当にありがとうございました！！

[kumo]

papinianus様
papinianus様のコメントにも評価させていただきたかったのですが、terateilにそのようなシステムがないようなので、この一連の回答をもってベストアンサーとさせていただきました。
もし評価が一つも上がらなければすみません。ですがとても助かりましたので、再度感謝の旨お伝えさせていただきます。ありがとうございました！

Answer 2

この場合、この行数を配列にして、要素数を導くのでしょうか？

それとも行数を出力することができるのでしょうか。。

そんなややこしいことはせずに、出力を行う場所でカウントを行っていきましょう。出力1回を行うごとに、回数をカウントする変数に1を足す、そして配列の出力が終わってから、カウントした回数を出力する、という形となります。

---

別解として、Array.prototype.filterで「条件を満たす配列」を一気に作ってしまう方法がありますが、そちらを使うのはまだ早いかと思います。

Comments

[kumo]

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

回答を参考にcountを入れてみた（★が追加行数）のですが、出力値は
0
1
2
となってしまい、うまくこの解決方法を検索できずにいます。。
すみませんが、具体的にどこがややこしくなってしまっているのかまたご返答いただきたいです。

process.stdin.resume();
process.stdin.setEncoding('utf8');

★var count=0;
var lines = [];
var reader = require('readline').createInterface({
input: process.stdin,
output: process.stdout
});
reader.on('line', (line) => {
lines.push(line);
});
reader.on('close', () => {

for ( var i = 0; i < lines.length; i++ ) {
if((lines[i])<=30){
★ console.log(count++);

}
}
});

[maisumakun]

console.log(count++); ではなく、ここはlines[i]をそのまま出力してください。というより、console.logが何をやっているものなのか理解していれば、置き換えてしまうようなことはしないかと思うのですが。

----

「検索できずにいます」とありますが、このようなピンポイントな状況を検索して解決しようとしても、多くの場合はおぼつきません。コードが何をしているのかを理解することから始めましょう。

[kumo]

そちらも試してみたところ元と同じようになってしまい、うまくいきませんでした。

lines[i]の値ではなく総個数を出したいのですが、
console.log(lines[i].length);とした場合になぜ文字数を出力してしまうのかがわかりません。

そうですよね。。理解不足ですみません。問題よりインプットの方頑張ります。
お忙しい中お時間割いてくださりありがとうございました。

[maisumakun]

個数は後で別に出力しましょう。

[hide0128]

kumoさん
console.log(lines[i]);
の後に
count++
for を抜けたところで
console.log(count);
して

[kumo]

hide0128様
ご回答ありがとうございます。返信が遅くなってしまい申し訳ございません。
修正いたしましたところ、無事解くことができました・・・！！
本当にありがとうございます！

[kumo]

hide0128様
できれば２つベストアンサーを押したかったのですが、僅差で早く回答してくださった方にベストアンサーをつけさせていただきました。
ですがhide0128様の回答もとても為になりましたので、その旨お伝えさせていただきます。