javaでクラス間で配列を受け渡すプログラムを作ろうとしています。

Question

java初心者です。

obj.standardscoreSet(a[l],num,average);
var[x] += ((a[x] - ave)*(a[x] - ave));
の所で
java.lang.ArrayIndexOutOfBoundsException

と指摘されているのですが、どのように直せばいいのかが分かりません。

ご教授お願い申しあげます。

Comments

[ygoooo]

コードを記載する際はマークダウン記法を使いましょう。「```」で囲んでください。

[退会済みユーザー]

ご指摘ありがとうございます。

[fiveHundred]

元のソースを消した意図はなんでしょうか？ また、私の回答をベストアンサーにした理由も聞きたいです。 正直申し上げますと、目的を達成できないから適当にベストアンサーをつけて、質問を無理やり終わらせたとしか思えないです。

[退会済みユーザー]

すいません。皆様のご指摘通り、自分の作りたいプログラムを作ると考え、元のソースは消去させていただきました。

[fiveHundred]

退会済みのため、何を言っても無駄でしょうが、システムからのメッセージにある通り、内容を消す行為は禁止されております。今回の場合、ソースに直接指摘している回答もあるので、ソースを消すことは望ましくありません。

[momon-ga]

そもそも、消去したって編集の履歴に残ってるので意味ないですけどねぇ・・・

Answer 1

ついでに私からも2点ほど

   double[] Answer=new double[n];

上記では、standardscoreSet()が呼び出される前にn個の配列が作成されます。
そのタイミングでは、nに対して代入が行われていないため、正しくない動作になるはずです。
（実行してないため、本当にそうかは分かりませんが、おそらく0個の配列が作成されると思います）

   var[ix] += ((a[ix] - iave)*(a[ix] - iave));

上記で+=が使われていますが、これはどういう意味でしょうか？
a += bはa = a + bと同義ですが、aに該当するvar[ix]は、配列の作成後に何も代入されていないため、このように使うのはおかしいです。
単純に代入したいのであれば、a = bで十分でしょう。

また、Subクラスの各配列はx番目しか使われていないのにも関わらず、配列にしておりますが、これはどういった意図でしょうか。
おそらく、追加の機能を作ることを前提にしているからだと思っていますが、そうでないなら普通の変数で十分です。

Answer 2

何から指摘したらいいかわからないぐらいひどいコードになっています。

あなたがやろうとしていることはあなたのレベルを数段階超えてます。
「配列渡したら合計を返す」ぐらいまで戻ったほうがいいです。

とりあえず指摘

1 .

mainのa[]の各要素は「点数」を表していて、
standardscoreSet()の第一引数ixに使われている。
そのixはstandardscoreSet()内で配列のインデックスに使われている。
その配列の要素数はa[]の要素数を渡されているので明らかにおかしい。

2 .

int[] a=new int[n]; // 全部0
int[] var=new int[n];
int[] bu=new int[n];
double[] stdev=new double[n];
double[] hensachi=new double[n];

var[ix] += ((a[ix] - iave)*(a[ix] - iave)); // 何がしたいのこれ?

Answer 3

java.lang.ArrayIndexOutOfBoundsException
というエラーが発生してしまいます。自分で色々と調べて見たのですが、よく分かりません。
ご教授お願い申しあげます。

調べたなら、なんとなくあたりはつくと思うのですが・・・
ソースは見ていませんが、Exceptionが発生した行数で、どの配列に問題があるかわかるはずです。

追記：コメントより

obj.standardscoreSet(a[l],num,average);
 var[x] += ((a[x] - ave)*(a[x] - ave));

> の所が指摘されている

まず、要素の１番目は、0というのは理解していますか？
x（ix？）が１個の場合は、a[1]でなくa[0]で、アクセスする必要があります。
あと、inとixの大小関係は正しいですか？

その添え字の値について確認すれば解決できるかと。

maiko0318さんより確認方法の提示
> その行の手前でxを表示してみてください。
> 有効範囲は０からnum-1までです。またnumが正しく設定されていることも確認しましょう

Comments

[退会済みユーザー]

```obj.standardscoreSet(a[l],num,average);```
``` var[x] += ((a[x] - ave)*(a[x] - ave));```
の所が指摘されているのですが、どのように直せばいいのかが分かりません。

[momon-ga]

ちょっと、変数名がわかりずらいのと無駄な代入が多くて理解しずらいですが
まず、要素の１番目は、0とうのは理解していますか？
x（ix？）が１個の場合は、x[1]でなくx[0]にアクセスします。
あと、inとixの大小関係は正しいですか？

[maiko0318]

その行の手前でxを表示してみてください。
有効範囲は０からnum-1までです。またnumが正しく設定されていることも確認しましょう。