java,ルーレットでの隣合う数字の最大値を求めるプログラム

###前提・実現したいこと
ルーレットにおいて、
連続するN個の和の最大値を求める。
Nが2~35の時のそれぞれの最大値をプログラムで計算して答えたい。
注意、配列の最後と最初は円形なので繋がっている事としています。

###発生している問題・エラーメッセージ
計算が想定通りになりません。
正解としては、
51、77、90、・・・・
となるはずなのですが、
51、124、209、・・・
となってしまします。
分からない状況です。
もし、分かる方いらっしゃれば、お教えください。
また、足りない情報がありましたら、補足致します。

###該当のソースコード

public class nyuumon6__10 {

public static void main(String[] args) {


	int[] EuroStyle = {0, 32, 15, 19, 4, 21, 2, 25, 17, 34, 6,27,13,36,11,30,8,
			23,10,5,24,16,33,1,20,14,31,9,22,18,29,7,28,12,35,3,26};

	int[] sum = new int[36];
	int[] Max_2 = new int[36];
	//繰り返し回数のループ
    for(int i=2;i<36;i++){
        for(int j=0;j<36;j++){
			
			
			//実際の繰り返し処理
			for(int k=0;k<i;k++){
				if(j+k<36){
				sum[j]+=EuroStyle[j+k];
				}else{
					//if(j+k!=72){
						sum[j]+=EuroStyle[j+k-36];
					//}else{
					//	sum[j]+=EuroStyle[0];
					//}
				}
				
				
			}
			//判定
        	if(Max_2[i]<sum[j]){
        			Max_2[i] =sum[j];
        	}
			//System.out.println(Max_2[i]);
           }
        System.out.println(Max_2[i]);	
        }	
    
}

}

行動規範の内容に同意します

回答2件

ループが複数存在する場合は、一気に作成するのではなく、順番に作っていって、メソッドなどで分割するとわかりやすくなります。

分割するポイントとしては以下のとおりで、これをそれぞれメソッド作るとわかりやすいでしょう。

配列 EuroStyleの配列から、連続して数値を取り出して合計する
合計したものの中から最大値を求める
連続で取り出す個数は2～35まで変わる

これを順次組み立てていきましょう。

配列 EuroStyleの配列から、連続して数値を取り出して合計する

これは取り出す個数に応じて合計することになりますので、メソッドの引数として受け取るようにしましょう。

java
1/**
2 * @param count 連続で取り出す個数
3 * @param position 開始位置
4 */
5public int sum(int count,int position) {
6  int total = 0;
7  // ここで合計をする処理を書く
8  return total;
9}

重要なポイントは「配列の最後に達したら、次は先頭の値を取り出す」ことですね。
そういう時には余り算を使います。
具体的にはEuroStyleから取り出す位置が超えてしまう=取り出す位置÷EuroStyleの長さで出る余りの値とすると良いでしょう。

java
1public int sum(int count,int position) {
2  int total = 0;
3  int length = EuroStyle.length; // 配列の長さ
4  for (int s = 0; s < count; s++) {
5    int target = (position + s) % length;  // 取り出す位置を決める
6    total += EuroStyle[target];
7  }
8  return total;
9}

合計したものの中から最大値を求める

Math.max( a,b ) で大きい方の値を取得できますので、ループの中で保持しておくを常にいれておけばよいでしょう。先ほどの「取り出す位置」はここで指定します。EuroStyleの最初から最後までを起点として合計を出すので、つまりEuroStyleの最初(0番目)から最後(EuroStyleの長さ未満まで)を繰り返します。

java
1  public int max(int count) {
2    int length = EuroStyle.length; // 配列の長さ
3    int max = 0;  // 最大値を格納する変数
4
5    for (int i = 0; i < length; i++) {
6      int sum = sum(count, i);
7      max = Math.max(max, sum); // 現在の最大値と、計算した合計値の大きい方を採用
8    }
9
10    return max;
11  }