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

Question

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

点数集計の問題を説いているのですが問題には５人の生徒の最高得点、最低得点、平均点を表示してくださいと書いてあって最高得点、平均点は表示できたのですが最低得点を表示しようとしても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に入るようにした。
よろしくお願いします。

Answer 1

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

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

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

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

// 初期値
var maxScore = 0;
var miniScore = 100;

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

Comments

[roto_note]

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

Answer 2

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 + "さんです");

Comments

[roto_note]

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

Answer 3

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

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

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

var students = ["山田", "高橋", "田中", "鈴木", "吉田"];

var totalScore = 0;
var maxScore = 0;
var maxPersonName = "";
var miniScore = -1; //変更点
var miniPersonName = "";
var scoreArray = [];

for (let i = 0; i < students.length; i++) {
  var score = parseInt(prompt(students[i] + "さんの点数"));
  scoreArray.push(score);

  if(miniScore == -1) miniScore = 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 + "さんです");

Comments

[roto_note]

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

Answer 4

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