5人のテストの得点から平均値と最大値の算出

Question

こんにちは。Js初心者のものです。

5人のテストの得点から平均値と最大値の算出するプログラムを作っています。
表示は「○○さんの平均値は○○点」のように人の名前と点数を表示させたいです。
まず、人の名前を配列に入れてnum=parseInt(tokuten);で点数を数値に変換しています。
下のプログラムだと

平均値はNaN点です
最高得点はundefined点でundefinedさんです

となってしまいます。

<script>
var people=new Array(5);
people[0]="山田"
people[1]="高橋"
people[2]="田中"
people[3]="鈴木"
people[4]="吉田"

sum=0;
for(i=0;i<5;++i){
  tokuten=prompt(people[i]+"さんの点数");
  num=parseInt(tokuten);
  max=num[0];
  sum=num[i]+sum;
  if( max < num[i] ){
  		max = num[i];
  	}
}
// 平均
console.log("平均値は"+sum/5+"点です");
// 最大値
console.log("最高得点は"+max+"点で"+people[i]+"さんです");

</script>

よろしくお願いします。

Answer 1

tokutenを受けたnumをなぜか配列と勘違いされているようです
普通に変数で処理すればよいでしょう

<script>
var people=new Array(5);
people[0]="山田"
people[1]="高橋"
people[2]="田中"
people[3]="鈴木"
people[4]="吉田"

sum=0;
max=0;
for(i=0;i<5;i++){
  tokuten=prompt(people[i]+"さんの点数");
  num=parseInt(tokuten);
  sum+=num;
  if( max < num ){
    max = num;
  }
}
alert("最大="+max+",平均="+sum/5);
</script>

Comments

[homel15k]

回答ありがとうございます。numを変数にして、if文にmaxPersonName=people[i];を追加したらできました。

Answer 2

var people=new Array(5);
people[0]="山田"
people[1]="高橋"
people[2]="田中"
people[3]="鈴木"
people[4]="吉田"

得点はどこ！？！？！？
下記のようにオブジェクトの配列を作ってください。

var students = [
  {name: "山田", points: 80},
  {name: "高橋", points: 75},
  {name: "田中", points: 85},
  {name: "鈴木", points: 100},
  {name: "吉田", points: 65}
];

---

お小言を言っておくと、peopleは単数形なので1人の人を指します。
複数形にしてください、得点という単語からより正しくはstudentsでは？

maxは名前を使うので人そのものを入れた方が良いでしょう。
5というリテラルな数値は使っては駄目です。array.lengthを利用してください。

"平均値は"+sum/5+"点です"に関しては/の結合が優先度が高いので動作しそうですが、
数値計算の場合は変な動きをすることがあります。
ここは明示的に括弧で括って"平均値は"+(sum/5)+"点です"と記載した方が分かりやすいです。

下記を反映してリファクタリングかけるとこうなりました。

var students = [
  {name: "山田", points: 80},
  {name: "高橋", points: 75},
  {name: "田中", points: 85},
  {name: "鈴木", points: 100},
  {name: "吉田", points: 65}
];
var sum = 0;
var max = {name: "", points: 0};
for (var i = 0; i < students.length; i++) {
  var student = students[i];
  sum += student.points;
  if (max.points < student.points) {
    max = student;
  }
}
// 平均
console.log("平均値は" + (sum / students.length) + "点です");
// 最大値
console.log("最高得点は" + max.points + "点で" + max.name + "さんです");

---

【おまけ】 Lodashによるスタイリッシュな解決法

Lodashというライブラリはこういったオブジェクトの配列を計算するのが超得意です。
平均得点も最高得点もたったの1行です。
この為にオブジェクトの配列にしろと言ったといっても過言ではありません。

下記のコードをhttps://codepen.io/travist/full/jrBjBz/に貼り付けて結果を確認してみてください

var students = [
  {name: "山田", points: 80},
  {name: "高橋", points: 75},
  {name: "田中", points: 85},
  {name: "鈴木", points: 100},
  {name: "吉田", points: 65}
];
var avg = _.meanBy(students, function(it){ return it.points });
var max = _.maxBy(students, function(it){ return it.points });

result = {avg: avg, max: max};

{
   "avg": 81,
   "max": {
      "name": "鈴木",
      "points": 100
   }
}

Comments

[homel15k]

詳しい回答ありがとうございます！参考になりました。

Answer 3

いくつか問題があります。

num=parseInt(tokuten);

max=num[0];
sum=num[i]+sum;

変数numは配列ではありませんので添字でアクセスできません。

console.log("最高得点は"+max+"点で"+people[i]+"さんです");

変数iはforループの中でしか使えません。[変数のスコープ]について学習してください。

<script>
var people=new Array(5);
people[0]="山田"
people[1]="高橋"
people[2]="田中"
people[3]="鈴木"
people[4]="吉田"

sum=0; // 合計得点
max=0; // 最高得点
maxPersonName="名無し" // 最高点を取った人の名前
tokutenArray = [];    // 得点を保存する配列
for(i=0;i<people.length;++i){
  tokuten=prompt(people[i]+"さんの点数");
  num=parseInt(tokuten);
  tokutenArray.push(num); // このプログラムでは使ってないですが、結果を保存したいときはこのように配列にデータを追加します
  sum = sum + num;
  if( num > max )
  {
    max = num;
    maxPersonName = people[i];
  }
}
// 平均
alert("平均値は"+sum/5+"点です");
// 最大値
alert("最高得点は"+max+"点で"+maxPersonName+"さんです");
</script>

Comments

[homel15k]

回答ありがとうございます。変数のスコープをもう一度復讐したいと思います。

Answer 4

var sum=0;
var nums=[];
for(i=0;i<5;++i){
  tokuten=prompt(people[i]+"さんの点数");
  nums[i]=parseInt(tokuten);
  sum+=nums[i];
}
var max=Math.max.apply({},nums);
var max_people=people[people.indexOf(max)];