条件式が長くなってしまうため短くしたい

Question

import java.util.ArrayList;
import java.util.Arrays;
import java.util.Scanner;

public class main {
	public static void main(String[] args) {
		ArrayList<String> fruitsPictorialPatternString = new ArrayList<String>();
		fruitsPictorialPatternString = new ArrayList<>(Arrays.asList("スイカ", "バナナ", "ブドウ", "ミカン", "イチゴ", "パイナップル", "リンゴ",
				"トマト", "ピーチ", "かぼちゃ", "チェリー", "セブン", "スターバー", "ベル", "コイン", "おばけ"));
		ArrayList<String> fruitsPictorialPatternPic = new ArrayList<String>();
		fruitsPictorialPatternPic = new ArrayList<>(Arrays.asList("????", "????", "????", "????", "????", "????", "????", "????", "????",
				"????", "????", "➐", "★", "????", "????", "????"));
		System.out.println("メダルを入れて下さい。＜所有メダル数　30枚＞");

		int numberOfMedalsOwned = 30;
		while (true) {
			System.out.println("Press Enter.");
			Scanner scanner = new Scanner(System.in);
			scanner.nextLine();
			int targetIndex01 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex02 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex03 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			System.out.println(fruitsPictorialPatternPic.get(targetIndex01) + "" + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex02) + "" + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex03));
			int targetIndex04 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex05 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex06 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			System.out.println(fruitsPictorialPatternPic.get(targetIndex04) + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex05) + "" + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex06));
			int targetIndex07 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex08 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			int targetIndex09 = (int) (Math.random() * fruitsPictorialPatternPic.size());
			System.out.println(fruitsPictorialPatternPic.get(targetIndex07) + "" + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex08) + "" + "" + "" + "" + "|"
					+ fruitsPictorialPatternPic.get(targetIndex09));
			if (targetIndex04 == 15 && targetIndex05 == 15 && targetIndex06 == 15) {
				System.out.println(fruitsPictorialPatternString.get(15) + "が3つ並びました。１０枚出てきました。");
				numberOfMedalsOwned += 10;
				System.out.println("＜所有メダル数" + numberOfMedalsOwned + "枚＞");
			} else if (targetIndex04 == 10) {
				System.out.println(fruitsPictorialPatternString.get(targetIndex09) + "が出ました。3枚出てきました。");
				numberOfMedalsOwned += 3;
				System.out.println("＜所有メダル数" + numberOfMedalsOwned + "枚＞");
				//当たりを上段・真ん中・下段・右斜め・左斜めで判定したい
			} else if(targetIndex01 == 1 && targetIndex02 == 1 && targetIndex03 == 1){
				System.out.println("外れです。");
				numberOfMedalsOwned --;
				System.out.println("＜所有メダル数" + numberOfMedalsOwned + "枚＞");
			}else {
				System.out.println("外れです。");
				numberOfMedalsOwned --;
				System.out.println("＜所有メダル数" + numberOfMedalsOwned + "枚＞");

			if (numberOfMedalsOwned == 0) {
				System.out.println("終了です。");
				System.exit(0);
			} else if (numberOfMedalsOwned == 100) {
				System.out.println("＜所有メダル数" + numberOfMedalsOwned + "枚＞");
				System.out.println("おめでとうございます！景品がもらえます。");
				System.exit(0);
			}
			}
			
		}
	}
}

実装したいこと
当たりを上段・下段・右斜め・左斜めで判定する。
１２３
４５６
７８９
パターンは１２３・７８９．１５９・３５７が上段・下段・右斜め・左斜めに該当する。
絵柄がfruitsPictorialPatternPic 分条件式を書くとかなり長い条件式になってしまうため、
パターンの上段・下段・右斜め・左斜めが同じ柄だった場合としたい。
targetIndex01 == 0 && targetIndex02 == 0 && targetIndex03 == 0
targetIndex01 == 1 && targetIndex02 == 1 && targetIndex03 == 1
・
・
・
・
targetIndex01 == 9 && targetIndex02 == 9 && targetIndex03 == 9
もしtargetIndexが１つしかない場合、それを指定できるが今回targetIndex09 まであるため０１から０９まで指定するとコードが
ながくなってしまうので、この方法を避けて
コードを簡潔に書く方法を考えているのですが、わからず教えていただきたいです。

Comments

[int32_t]

10番目から15番目までの柄が揃っても当たりにはしないのですか?

[jimbe]

kazuma-s さんが回答されていますが、変数名に番号を付けた時点で「これは配列に出来るのでは…」と気付いて頂きたい所です。

[lop000p]

10番目から15番目まで＞すみません、条件見落としてました。。。
追加いたします。
そうなのですね、、配列とリストがほぼ同じとしか認識できておりませんでした。。。

Answer 1

new Scanner(System.in) をループの中で実行してはいけません。

変数を配列にすればよいのではありませんか？

import java.util.Scanner;

class main {
    public static void main(String[] args) {
        String[] name = {
			"スイカ", "バナナ", "ブドウ", "ミカン", "イチゴ", "パイナップル",
			"リンゴ", "トマト", "ピーチ", "かぼちゃ", "チェリー", "セブン",
			"スターバー", "ベル", "コイン", "おばけ"
		};
        String[] pic = {
			"WM", "BA", "GR", "OR", "SB", "PA", "AP", "TO",
			"PE", "PK", "CH", "SE", "★", "BL", "CO", "GS"
		};
        int nm = 30;  // number of medals owned
        Scanner scanner = new Scanner(System.in);

        System.out.println("メダルを入れて下さい。＜所有メダル数　" + nm + "枚＞");
        while (true) {
            System.out.println("Press Enter.");
            scanner.nextLine();
			int[] ti = new int[9];  // target index
			for (int i = 0; i < 9; i++)
				ti[i] = (int) (Math.random() * pic.length);
			for (int i = 0; i < 9; i += 3)
				System.out.println(pic[ti[i]] + "|"
					+ pic[ti[i+1]] +  "|" + pic[ti[i+2]]);
            if (ti[3] == 15 && ti[4] == 15 && ti[5] == 15) {
                System.out.println(name[15] + "が3つ並びました。10枚出てきました。");
                nm += 10;
                System.out.println("＜所有メダル数" + nm + "枚＞");
            } else if (ti[0] == ti[1] && ti[1] == ti[2] ||  // 上段
                       ti[6] == ti[7] && ti[7] == ti[8] ||  // 下段
                       ti[0] == ti[4] && ti[4] == ti[8] ||  // 右斜め
                       ti[2] == ti[4] && ti[4] == ti[6]) {  // 左斜め
                System.out.println("上段、下段、斜めが揃いました。3枚出てきました。");
                nm += 3;
                System.out.println("＜所有メダル数" + nm + "枚＞");
            } else {
                System.out.println("外れです。");
                nm --;
                System.out.println("＜所有メダル数" + nm + "枚＞");
            }
            if (nm == 0) {
                System.out.println("終了です。");
                break;
            } else if (nm >= 100) {
                System.out.println("＜所有メダル数" + nm + "枚＞");
                System.out.println("おめでとうございます！景品がもらえます。");
                break;
            }
        }
    }
}

Comments

[lop000p]

ご教授頂きありがとうございます。配列とリストはほぼできることは同じと考えていたのですが、配列でこんなやり方ができるとは思いませんでした。
１点配列の動きで不明点がありまして、おききしたいのですが、
for (int i = 0; i < 9; i += 3)
iに+= 3をされておりますが、この+= 3とした場合、
for (int i = 0; i < 9;+= 3)
ti[i] = (int) (Math.random() * pic.length);
Math.random()で生成されたインデックスとコンソールに出力された
文字は一致が確認できました。
値の一致は確認できたのですが、なぜ+= 3とした場合、値がforでループさせた値と一致できたのかわかりません。

for (int i = 0; i < 9; i ++)
i ++とした場合、コンソールに出力された値は
for (int i = 0; i < 9; i++)
ti[i] = (int) (Math.random() * pic.length);
で生成されたインデックス番号　
[0][1][2][3][4][5][6][7][8]
[7][5][2][1][2][0][6][0][1]

i ++としてコンソール出力された値
[7][5][2][5][2][1][2][1][2][1][2][0][2][0][5][0][5][0][5][0][1]Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 9 out of bounds for length 9
at main.main(furupara_hairetsu.java:26)

752|521|212
120❘205❘050
501|....

i++とした場合一部一致して一部一致が確認できなく、配列の動きが不明なのですが、
どうして　　+=3　とされたのか教えていただけないでしょうか？

[jimbe]

ご本人で無くてすいません。
その for ループは、ti を 3 つずつ 3 段表示するためのループです。
ですのでループ自体は 3 回でなければなりません。
一方、 ループのカウンタ i は、ループ内での処理の関係上、 0・3・6 と変化する必要があります。
その為に "i を 0 から 9 まで 3 ずつ加算する" ループとなっています。

[lop000p]

＞ti を 3 つずつ 3 段表示するためのループ
jimbeさん、ご教授頂きましてありがとうございます！！