質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.46%
for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

2回答

826閲覧

javascript 要素の大小比較 for文のエラー 標準入力

shinji389

総合スコア8

for

for文は、様々なプログラミング言語で使われている制御構造です。for文に定義している条件から外れるまで、for文内の命令文を繰り返し実行します。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2021/10/05 23:51

編集2021/10/06 00:18

前提・実現したいこと

1行目の二桁の数字と比較し
3行目以降の二桁の数字とどちらが大きいかを調べたい。

毎回複数の要素が2行目の数字の数だけ以降の行に
ランダムで入力されるので入力内容に合わせて比較結果を出力させたい。

比較方法としては1つ目の数字同士の大小を比較し、
同じ数字の場合は2つ目の数字の大小を比較する。
比較後に一行目の数字が大きい場合は"High",小さい場合は"Low"と出力する。

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

B = lines[i].split(" ");
^
TypeError: Cannot read property 'split' of undefined

該当のソースコード

条件例
7 3 // 比較する親要素 lines[0]
4 // 以降入る子要素の数 lines[1]
7 1 // 以降 比較する子要素 lines[2]...
7 4
5 3
10 1

javascript

var A = lines[0].split(" "); for(var i = 2; i <= lines[1]+1 ; i++){ B = lines[i].split(" "); if(A[0] > B[0]){ console.log("High"); } else if(A[0] < B[0]) { console.log("Low"); } else if(A[0] == B[0] ){ if(A[1] > B[1]){ console.log("High"); } else if(A[1] < B [1]){ console.log("Low"); } } }

出力結果

Low
High
High
High

試したこと

出力結果が2行目以降同じものを出力してしまっているので
エラーメッセージの内容から無限ループが起きてしまっているので
for文の範囲を変更してみたが同様のエラーとなり解決できなかった。
その他の部分でも間違いがあるか探したがわからなかったので
改善点についてご教授ください。
よろしくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

maisumakun

2021/10/05 23:52

linesの定義はどこですか?
shinji389

2021/10/06 00:20

sの記載が漏れていました。 7 3 // 比較する親要素 lines[0] 以降の行をlines[1]、[2]、[3]...と定義していってます。
guest

回答2

0

入力の仕方次第ですね

javascript

1const str=`7 3 24 37 1 47 4 55 3 610 1`; 7const lines=str.split(/[\r\n]+/).map(x=>x.split(' ').map(x=>Number(x))); 8 9for(var i=2;i<2+lines[1];i++){ 10 const flg=lines[0][0]<lines[i][0] || 11 lines[0][0]==lines[i][0] && 12 lines[0][1]<lines[i][1]; 13 console.log(flg?"Low":"High"); 14}

投稿2021/10/06 00:22

yambejp

総合スコア115012

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

4 // 以降入る子要素の数 line[1]

for(var i = 2; i <= lines[1]+1 ; i++){

もし lines[1] に入っているのが数値の 4 ではなく文字列の "4" だった場合、lines[1]+1 は文字列 "41" になります。

投稿2021/10/06 00:18

編集2021/10/06 00:19
int32_t

総合スコア21016

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

shinji389

2021/10/06 02:30

回答ありがとうございます。 文字列になっていたため数値に直すことで2行目の内容を繰り替えさず 各行の内容を比較できるようになりましたが 同じエラー内容が出たままとなっています。 なにか別の問題、解決策はありますでしょうか。
int32_t

2021/10/06 02:37 編集

これ以上は、問題を再現させるlinesの正確な内容を開示してもらわないとわかりません。 lines[1]が文字列かも、というのも憶測でしたし、問題を再現できる完全なコードを示さないと、的確なアドバイスは難しいです。
shinji389

2021/10/06 02:46

文字列を数値に変換できていない箇所があり、 再度全ての要素を数値に変換することで解決しました。 何度も回答いただきありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問