🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
for

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

JavaScript

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

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

解決済

4回答

1634閲覧

5人のテストの得点から最低値の算出

roto_note

総合スコア11

for

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

JavaScript

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

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

1クリップ

投稿2019/12/16 16:17

編集2019/12/16 16:31

配列にテストの点数を入れて最低得点を表示したい

点数集計の問題を説いているのですが問題には5人の生徒の最高得点、最低得点、平均点を表示してくださいと書いてあって最高得点、平均点は表示できたのですが最低得点を表示しようとしても0点と表示されてしまいます。最高得点を出したときと同じようにコードを書いてみたのですが表示されません。

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

最低得点を表示しようとしても「最低得点は0点でさんです」と表示されてしまう。

### 該当のソースコード var students = ["山田", "高橋", "田中", "鈴木", "吉田"]; var totalScore = 0; var maxScore = 0; var maxPersonName = ""; var miniScore = 0; var miniPersonName = ""; var scoreArray = []; for (let i = 0; i < students.length; i++) { var score = parseInt(prompt(students[i] + "さんの点数")); scoreArray.push(score); totalScore = totalScore + score; if (score > maxScore) { maxScore = score; maxPersonName = students[i]; } if (score < miniScore) { miniScore = score; miniPersonName = students[i]; } } alert("平均点は" + (totalScore / students.length) + "点です"); alert("最高得点は" + maxScore + "点で" + maxPersonName + "さんです"); alert("最低得点は" + miniScore + "点で" + miniPersonName + "さんです");

試したこと

最高得点のコードを書いた時と同じように比較の記号を反対にして一番低い点数がminiScoreに入るようにした。
よろしくお願いします。

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

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

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

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

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

guest

回答4

0

ベストアンサー

回答は出揃っていると思いますので、少しだけ違う視点で回答させていただきます。

JavaScriptに限りませんが、プログラミングする場合、コードの処理の流れや変数の値の変化を頭の中で具体的にイメージできることが大事です。もしそれが難しい場合は、デバッガーを利用したり、変数の値を途中に表示するなどして、確認します。

今回のご質問の場合、最低得点を示す変数miniScoreは初期値として0です。生徒それぞれの得点が順番に[10, 20, 30, 40, 50]と取得できるとしたらどうでしょう。if (10 < miniScore)は成立しない、if (20 < miniScore)は成立しない、・・・ if (50 < miniScore)は成立しない、と、全て成立せず、miniScore0のままです。流れと変数の値の変化を頭の中でイメージできていれば、すぐに分かったと思います。あるいはscoreの値を途中でalertで表示するなどすれば。

比較の為の初期値ですが、値の取り得る範囲としては100点満点として最低0から最高100です。JavaScriptを使う限りにおいては別の回答でいただいたようにNumber.MIN_SAFE_INTEGERNumber.MAX_SAFE_INTEGERを使っておけば変数の取り得る範囲として保証されるので安全ですが、問題文に注目すると、最低0、最高100で充分とも言えます。

JavaScript

1// 初期値 2var maxScore = 0; 3var miniScore = 100;

対象としている問題の値の取り得る範囲と、その問題を解決するためのプログラミング言語の変数の型の取り得る範囲を意識しておくことも、実際のプログラミングにおいては大事だと思います。

投稿2019/12/17 00:24

dodox86

総合スコア9254

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

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

roto_note

2019/12/17 02:20

初期値が0で条件式のなかでずっとfalseだった為、値は変わらず0だったのですね。 解決しました。変数の変化は常にイメージするようにします。 ありがとうございました。
guest

0

kota_yamashitaさんの回答のコードも使えると思うけど、JSにはMAX_SAFE_INTEGERとMIN_SAFE_INTEGERというのがあります。要するに、minScoreを一番大きいintとして初期化することによって最初の点数がそれより低いからminScoreが必ず更新されます。maxScoreの場合は逆です。こういう定数があるから、これを使った方がより正しい気がします。

var students = ["山田", "高橋", "田中", "鈴木", "吉田"]; var totalScore = 0; var maxScore = Number.MIN_SAFE_INTEGER; var maxPersonName = ""; var miniScore = Number.MAX_SAFE_INTEGER; var miniPersonName = ""; var scoreArray = []; for (let i = 0; i < students.length; i++) { var score = parseInt(prompt(students[i] + "さんの点数")); scoreArray.push(score); totalScore = totalScore + score; if (score > maxScore) { maxScore = score; maxPersonName = students[i]; } if (score < miniScore) { miniScore = score; miniPersonName = students[i]; } } alert("平均点は" + (totalScore / students.length) + "点です"); alert("最高得点は" + maxScore + "点で" + maxPersonName + "さんです"); alert("最低得点は" + miniScore + "点で" + miniPersonName + "さんです");

投稿2019/12/16 19:37

kurapan

総合スコア79

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

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

roto_note

2019/12/17 02:37

回答ありがとうございます。 最大値で初期化してくれる定数があるんですね。。 勉強になりました。 ありがとうございます。
guest

0

miniScoreのデフォルト値が0で更新されてないからです。
仮にスコアを-1とかの0未満の値を入力するときちんと最低得点が出ると思います。

色んな修正の仕方があると思いますが一番分かりやすく修正が少なく済むのは、
テストということでマイナス値が入力されない前提となりますが、
miniScoreのデフォルト値を-1にして、条件分岐の前にminiScoreが-1のときだけ入力されたスコアを代入とかでしょうね。

コードにするとこんな感じでしょうか。

JavaScript

1var students = ["山田", "高橋", "田中", "鈴木", "吉田"]; 2 3var totalScore = 0; 4var maxScore = 0; 5var maxPersonName = ""; 6var miniScore = -1; //変更点 7var miniPersonName = ""; 8var scoreArray = []; 9 10for (let i = 0; i < students.length; i++) { 11 var score = parseInt(prompt(students[i] + "さんの点数")); 12 scoreArray.push(score); 13 14 if(miniScore == -1) miniScore = score; //変更点 15 16 totalScore = totalScore + score; 17 18 if (score > maxScore) { 19 maxScore = score; 20 maxPersonName = students[i]; 21 } 22 if (score < miniScore) { 23 miniScore = score; 24 miniPersonName = students[i]; 25 } 26} 27 28alert("平均点は" + (totalScore / students.length) + "点です"); 29alert("最高得点は" + maxScore + "点で" + maxPersonName + "さんです"); 30alert("最低得点は" + miniScore + "点で" + miniPersonName + "さんです");

投稿2019/12/16 16:46

kota_yamashita

総合スコア356

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

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

roto_note

2019/12/17 02:27

回答ありがとうございます。 デフォルト値が0で条件式が更新されていないことが問題だったことがわかりました。 参考にさせていただきます!
guest

0

js

1var miniScore = 0; 2// (中略) 3 if (score < miniScore) { // 0よりscoreが小さい場合

投稿2019/12/16 16:48

kei344

総合スコア69596

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.36%

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

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

質問する

関連した質問