Java二次元配列 平均点を求めるメソッドの作成

Question

3人分の4科目の得点の平均点を求め、実行結果のようになるように３つのメソッドget_ave、get_hyouka、print_dataを作成する。尚、4桁に満たない番号は０を付加して4桁にすること。
public class Text {
public static void main(String[] args) {

//定数の宣言・設定
final int NINZU = 3;

int[] bango = {1,102,1003};
	int[][] ten = {{95,83,45,56},
			{25,36,66,48},
			{89,80,79,90}};
	double	ave;
	char	hyouka;
	System.out.println("No.  kokugo  sansuu  rika  syakai  heikin  hyouka");
	for (int i=0; i< NINZU; i++){
		ave = get_ave(ten[i]);
		hyouka = get_hyouka(ave);
		print_data(bango[i],ten[i],ave,hyouka);
	}
}

//平均点を求めるメソッド
private static double get_ave(int[] i) {

	**int[][] ten={};
	double sum = 0.0;
	int x = 0;
	for(;x<ten.length;x++){

		for(int y =0;y<ten[x].length;y++){
			sum+=ten[x][y];
		}
		
	}
	return (sum/ten[x].length);**

}

//評価を求めるメソッド
private static char get_hyouka(double d) {
//この部分の処理を作成
}

//表示を行うメソッド
private static void print_data(int b ,int[] t ,double a ,char h) {
	//この部分の処理を作成
}

}
[実行結果]
No kokugo snsuu rika syakai heikin hyouka
0001 95 83 45 56 69.75 C
0102 25 36 66 48 43.75 D
1003 89 80 79 90 84.50 A

平均点メソッドを書いたんですか？正しいですか？

Answer 1

get_aveメソッドの中で使っているtenは，このメソッド内で定義した空の配列を指しているので，
このままではこのメソッドは0.0しか返しません．
あるいは例外が発生するかもしれません．

mainメソッドから各人のten配列が渡されているので，渡されたiを使うべきです．

Answer 2

get_aveメソッドの中で、渡された「i」が使用されていないように見受けられます。

    private static double get_ave(int[] i) {
        double sum = 0.0;
        int x = 0;
        for (; x < i.length; x++) {

            sum += i[x];

        }
        return (sum / i.length);
    }
これでいかがでしょうか？

Comments

[kimi0000]

できました、ありがとうございます。ちなみに表示を行うメソッド どう書けばいいですか？何度もすみません

[SatoshiMashino]

引数のtでfor文を回し、「System.out.println()」で出力すればよいと思いますよ。
ヘッダーの表示、空白の数、gの小数点に気を付けてください。

[swordone]

横から失礼．
学校の課題か何かだと思うのでヒントを．
まずメソッドには「番号」「各教科の点数の配列」「平均点」「評価のアルファベット」が渡されている．
基本的にはこれをすべて文字列化して出力するか，それぞれを順次出力する．
条件になっている「番号を0埋めで4桁表示」は，
1. 番号の桁数(or値の範囲)を判別して，先頭に必要な数"0"を加える
2. 0埋めするフォーマットを使う
のいずれかを行う．

[kimi0000]

//表示を行うメソッド
private static void print_data(int b, int[] t, double a, char h) {
int bango[] = {};
for(b=0;b<bango.length;b++){

System.out.println(bango[b]);

}
何度もすみません、書いたんですがどこがおかしいですか？

[SatoshiMashino]

bは上位メソッドから与えられた「番号」そのものなので、ループでは使用いたしません。

private static void print_data(int b, int[] t, double a, char h) {

System.out.print(b + " ");

for (int i = 0; i < t.length; i++) {

System.out.print(t + " ");
}

System.out.print(a + " ");

System.out.print(h + " ");
}

こんな感じでしょうか。
細かい表示のフォーマットなどはそちらで何とかしてみてください。

[kimi0000]

ありがとうございました。