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

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

ただいまの
回答率

88.58%

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

解決済

回答 4

投稿

  • 評価
  • クリップ 0
  • VIEW 2,366

homel15k

score 19

こんにちは。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>


よろしくお願いします。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 4

checkベストアンサー

+4

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>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/02/08 12:44

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

    キャンセル

+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>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/02/08 12:45

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

    キャンセル

+3

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
   }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/02/08 12:46

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

    キャンセル

+2

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)];

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.58%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る