配列を利用した平均の求め方

Question

前提・実現したいこと

プログラミング初心者です。
引数で配列と要素の個数を受け取り，受け取った配列の要素の平均を返すユーザ定義関数getAryAveを定義し、主プログラムでは、getAryAveを利用し，標準入力から入力された複数の数値の平均を求めて表示します。標準入力から入力された数値は配列に蓄えます。配列への数値の入力は最大10件までとし，件数が10件なるかEOFを読み込むまで繰り返すものとします。

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

最後の値が出てしまう(?)

該当のソースコード

<?php
function getAryAve($x,$y){
    $h = new SplFixedArray($y);
    $total = 0;
    for($i = 0; $i < $y; $i++){
        $h[$i] = $x;
        $total += $x;
    }
    $ave = $total / $y;
    return $ave;
}
//main
print('数値を入力してください');
$a = trim(fgets(STDIN));
$i = 0;
while($a != NULL and $i < 10){
    print('数値を入力してください');
    $a = trim(fgets(STDIN));
    $i++;
}
print('平均は'.getAryAve($a,10)."\n");

Comments

[cerfweb]

$h[$i] = $x; <-- これは何をしようとしているのでしょうか。
また、STDINにはどんな値が入りますか。
提示されたソース以外に関係している部分があれば全て表記してもらえますか。

[退会済みユーザー]

数値であれば特に指定はないです。
他に何か足りない部分などあればご指摘頂きたいです。

Answer 1

いくつか問題がありそうなので、平均を計算するのは後回しにして、まずは、「入力された数値を最大10個まで配列に格納する」を目指したほうが良さそうです。

print('数値を入力してください');
$a = trim(fgets(STDIN));
$i = 0;
while($a != NULL and $i < 10){
    print('数値を入力してください');
    $a = trim(fgets(STDIN));
    $i++;
}
var_export($a);

最後のvar_export()で、入力された数値がすべて表示されればOKですが、このコードだと、うまくいきません。
なぜななら、

• $aは配列じゃない
• whileループが回るたびに、前回の入力値が上書きされてしまう

からです。

別途、配列を用意して、そこへ$aの内容を転記(追記)していく必要があります。

$values = [];      // この配列に蓄積する。
print('数値を入力してください');
$a = trim(fgets(STDIN));
$i = 0;
while($a != NULL and $i < 10){
    $values[] = $a;          // 蓄積。
    print('数値を入力してください');
    $a = trim(fgets(STDIN));
    $i++;
}
var_export($values);

これでもまだ問題があるのですが、まずはご自分で動かして遊んでみてください。

Comments

[退会済みユーザー]

とてもわかり易い説明ありがとうございます。
上記コードは動かせましたが、最後の入力が表示できてませんね…

Answer 2

何をしているか分からないプログラムになっています。

まずは、「入力された数値を配列に順次格納し、var_dump($配列名);でそのデータを表示する」というプログラムを作ってみて下さい。
配列はSplFixedArrayとかじゃなくて、普通の配列を使います。

Comments

[退会済みユーザー]

回答ありがとうございます。ご指摘どおり、まずは表示だけでもやってみたいと思います。

[退会済みユーザー]

$values = [];
while(count($values) < 10){
print('数値を入力してください');
$a = trim(fgets(STDIN));
$values[] = $a;
}
var_dump($values);
このように作ってみましたがいかがでしょうか。

[otn]

データの個数が10個前提なら、良いと思います。

> 件数が10件なるかEOFを読み込むまで繰り返す
なので、EOFだと途中で繰り返しを止めないといけません。
EOFの時はfgetsがFALSEを返すので、
$a = fgets(STDIN);
if($a===FALSE) { break; }

また、配列に入れる段階で数値化した方が良いでしょう。
整数だけとは書いてないので、floatに変換して、
$values[] = (float)$a;

あとは、getAryAve($values, count($values)); で呼べるように関数を書きます。

[退会済みユーザー]

ありがとうございます。

Answer 3

<?php
function getAryAve($x,$y){
    $total = 0;
    for($i = 0; $i < $y; $i++){
        $total += $x[$i];
    }
    $ave = $total / $y;
    return $ave;
}
//main
$values = [];
print('数値を入力してください');
$a = trim(fgets(STDIN));
$values[] = $a;
while($a != NULL and count($values) < 10){
    print('数値を入力してください');
    $a = trim(fgets(STDIN));
    $values[] = $a;
}
print ('平均は'.getAryAve($values,count($values))."\n");

Comments

[otn]

これだと、ちょうど10個の場合じゃ無いとダメですね。
テストしてない？

Answer 4

$aですが、最後に入力された値を保持しているだけで、それ以前の入力は捨てられています。
getAryAve()では、配列を作っていますが、全ての要素に$xを設定しています。

そもそも、

引数で配列と要素の個数を受け取り

となっているのに、$xが配列でない時点で、要求を満たしていません。

コードを作成される前に、設計はされましたか？
各変数は、何を格納するのか、きちんと考えましたか？
初心者なら、コードを作成する前に、上記のような項目を書き出すなどして、明確にするようにしてください。

Comments

[退会済みユーザー]

回答ありがとうございます。書き出しですね。参考になります。