javaでおみくじを上手く作動させていです

Question

前提・実現したいこと

こんにちは
javaを初めて数日になる初心者です。

javaでおみくじを作っています。

「やる」「やらない」をそれぞれコンソール入力し、
「やる」→運勢を占う
「やらない」→「やめとく」と表示されるようにしたいです。

こんな感じで書いてます
「やる」を入力したら運勢を占うようになったのですが、「やめとく」を入力した際のメッセージが表示されなくて困っています。
もしよろしければどなたか教えていただけないでしょうか

また、おみくじを作るのであればもっと簡潔にできるよ！という方法もあれば教えていただけると大変うれしいです

コード

//////以下ソース
package hello;

import java.util.Scanner;

public class Omikuji {

public static void main(String[] args) {
	System.out.println("おみくじ！");
	System.out.println("やる？やらない？");
	System.out.println("やるときは「やる」を入力");
	System.out.println("やらないは「やめとく」を入力");

	Scanner scanner = new Scanner(System.in);



	boolean a =false;
	boolean b =false;
	int i =0;
	while(true) {
		if(scanner.next().equals("やる")) {
			System.out.println("あなたの運勢は......");
			a=true;
			break;
		}else if(scanner.next().equals("やめとく")) {
			b=true;
			break;
		}
	}

	if(a) {int r = new java.util.Random().nextInt(5);
		switch(r) {
			case 0:
			System.out.println("大吉");
			break;
		case 1:
			System.out.println("中吉");
			break;
		case 2:
			System.out.println("末吉");
			break;
		case 3:
			System.out.println("吉");
			break;
		case 4:
			System.out.println("凶");
			break;
		case 5:
			System.out.println("大凶");
			break;
		}
	}else if(b) {
		System.out.println("またね”!");
		}

}

}

Answer 1

簡潔にしましたｗ

     public static void main(String[] args) {
        String[] Ans = {"大吉", "中吉", "末吉", "吉", "凶", "大凶"};
        System.out.println("おみくじ！");
        System.out.println("やる？やらない？");
        System.out.println("やるときは「やる」を入力");
        System.out.println("やらないは「やめとく」を入力");

        Scanner scanner = new Scanner(System.in);

        while (true) {
            String　sel = scanner.next();
            if (sel.equals("やる")) {
                System.out.println("あなたの運勢は......");
                int r = new java.util.Random().nextInt(5);
                System.out.println(Ans[r]);
            } else if (sel.equals("やめとく")) {
                System.out.println("またね”!");
                break;
            }
        }

    }

run:
おみくじ！
やる？やらない？
やるときは「やる」を入力
やらないは「やめとく」を入力

やる
あなたの運勢は......
末吉
やる
あなたの運勢は......
中吉
やめとく
またね”!
ビルド成功(合計時間: 3分 1秒)

環境はLinux mint 19.1+NetBeans IDE 8.2

Comments

[95Y.K.]

回答ありがとうございます！
配列を使って、やぱりそこから出したほうが綺麗ですよね。

Answer 2

若干無理矢理感はありますが.

package hello;

import java.util.*;

public class Omikuji {

  public static void main(String[] args) {
    System.out.println("おみくじ！");
    System.out.println("やる？やらない？");
    System.out.println("やるときは「やる」を入力");
    System.out.println("やらないは「やめとく」を入力");

    String[] omikuji={"大吉","中吉","末吉","吉","凶","大凶"};
    Scanner scanner = new Scanner(System.in);
    Random random = new Random();
    HashMap<String,Boolean> map = new HashMap<String,Boolean>(){{put("やる",true);put("やめとく",false);}};

    Boolean play = null;
    do { play = map.get(scanner.next()); } while(play == null);
    System.out.println(play?"あなたの運勢は......\n"+omikuji[random.nextInt(5)]:"またね!");
  }
}

Comments

[95Y.K.]

回答ありがとうございます！
はじめてみるものが多かったのでたくさん勉強していきます

Answer 3

if(scanner.next().equals("やる")) {
...
}else if(scanner.next().equals("やめとく")) {
...
}

条件を判定する度にトークンを読み込んでいるのが問題です。
先に一回だけ読み込んでおいて、それを判定に用いれば良いでしょう。

String input = scanner.next();
if(input.equals("やる")) {
    ...
}else if(input.equals("やめとく")) {
    ...
}

コードの書き方について

teratailには、コードを見やすく表示する機能があります。
質問編集画面を開き、コードを選択した状態で<code>ボタンを押して下さい。

Comments

[95Y.K.]

回答、ご教授ありがとうございます！
日が浅いため、頂いた回答への理解度もまだ浅いのですが、これからたくさん勉強して納得できるように努めていきます！

Answer 4

実際のおみくじに近い動作をさせるようにしてみました。
つまり、箱に大吉,小吉,,, などの紙を適当な枚数をいれ、それをかき混ぜ順番に引いていくいう方法です。
大吉,小吉の各出現率も直感的に制御ができます。

package teratail;

import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Scanner;

public class Omikuji {
    static private List<String> cards = new ArrayList<String>();

    public static void main(String[] args) {
        opening();
        play();
    }
    
   static void opening() {
        System.out.println("おみくじ！");
        System.out.println("やるときは「やる」を入力");
        System.out.println("やらないは「やめとく」を入力");
    }

    static void play() {
        Scanner scanner = new Scanner(System.in);
        while (true) {
            System.out.print("やる？やらない？");
            if (scanner.next().equals("やる")) {
                System.out.println(draw());
            } else {
                System.out.println("またね”!");
            }
        }
    }

    static String draw() {
        if (cards.size() == 0) {
            cards.addAll(Collections.nCopies(10, "大吉") ) ;
            cards.addAll(Collections.nCopies(20, "中吉") ) ;
            cards.addAll(Collections.nCopies(20, "末吉") ) ;
            cards.addAll(Collections.nCopies(40, "吉") ) ;
            cards.addAll(Collections.nCopies(8, "凶") ) ;
            cards.addAll(Collections.nCopies(2, "大凶") );
 
            Collections.shuffle(cards);
        }
        return cards.remove(0);
    }
}