掛け金設定まで戻らず再入力を促したい

Question

前提

Javaで掛け金システムのある丁半プログラムを作っています。

実現したいこと

System.out.println("半(=1)か丁(=2)か?");
の後に1,2以外の数字が入力された場合、【エラー】入力できるのは1と2です　と
出したいのですが、出力結果が
4(入力された数字)
//ここに1,2以外が入力されたときに「【エラー】入力できるのは1と2です」といれて"半(=1)か丁(=2)か?"と表示し、再び入力できるようにしたいです。
6(さいの目の数字)

発生している問題・エラーメッセージ

該当のソースコード

import java.util.Scanner;
public class Main {
 public static void main(String[] args) {
  int syojikin = 100;
  System.out.print("所持金＝" + syojikin);
  Scanner betsc = new Scanner(System.in);
  Scanner scanner = new Scanner(System.in);
  Scanner kake = new Scanner(System.in);
  while (!scanner.nextLine().equals("end")) {
   System.out.print("掛け金を入力:");
   String kakekinsc = kake.next();
   if (kakekinsc.equals("end")) {
    break;
   }
   int kakekin = Integer.parseInt(kakekinsc);
   if (kakekin > syojikin) {
    System.out.println("所持金以上の金額は賭けられません。");
    continue;
   }
   int zandaka = syojikin - kakekin;
   System.out.println("残高" + zandaka);
   while(){
   System.out.println("半(=1)か丁(=2)か?");
   String bet1sc = betsc.next();
   int bet = Integer.parseInt(bet1sc);
 if (bet <= -1 || bet >= 3){
		System.out.println("【エラー】入力できるのは1と2です");
	       continue;
	   }
	   boolean bb = bet1sc.chars().allMatch(Character::isDigit);
	   if (bb != true) {
			System.out.println("【エラー】入力できるのは整数だけです");
	    continue;
	  }
   int sai = 1 + (int)(Math.random() * 6.0);
   System.out.println(sai);
   boolean b = (sai % 2 == 0);
   boolean b1 = (bet % 2 == 0);
   if (b == true && b1 == true) {
    System.out.println("当たり!");
    zandaka = zandaka + (kakekin * 2);
    System.out.print("残高" + zandaka);
    syojikin = zandaka;
    continue;
   } else if (b == true && b1 != true) {
    System.out.println("はずれ!");
    System.out.print("残高" + zandaka);
    syojikin = zandaka;
    if (syojikin == 0) {
     break;
    }
    continue;
   } else if (b != true && b1 != true) {
    System.out.println("当たり!");
    zandaka = zandaka + (kakekin) * 2;
    System.out.print("残高" + zandaka);
    syojikin = zandaka;
    continue;
   } else if (b != true && b1 == true) {
    System.out.println("はずれ!");
    System.out.print("残高" + zandaka);
    syojikin = zandaka;
    if (syojikin == 0) {
     break;
    }
    continue;
   }
   scanner.close();
  }
  scanner.close();
 }
}
}

試したこと

・System.out.println("【エラー】入力できるのは1と2です");
continue;
と書きましたが

出力結果は
【エラー】入力できるのは「0～2」です

掛け金を入力:

と掛け金設定まで戻ってしまいます。

・while文の条件に何を書いたらいいかがわからず止まってしまいました。

補足情報（FW/ツールのバージョンなど）

Java８です

ご回答いただいたpig_vba様、jimbe様
アドバイスいただきありがとうございました。大変勉強になりました。
また皆さんに一つお聞きしたいことがありまして、このようなJavaやプログラム等で必要なmethodごとに分ける部分を見分ける方法やコツなどありますでしょうか？
java初心者で問題を見たとき、まずはどこから書いたらいいのかなと迷うことが多く、
コメントでいただけたら幸いです。

Comments

[jimbe]

とりあえず、Scanner は同じ元 (System.in) から複数は作らないほうが良いです。不具合の元になります。作る意味もありません。
1つ作って使い回してください。

[awz_9]

使いまわしができるものなのですね。ありがとうございます。冗長になっていました。

[jimbe]

>必要なmethodごとに分ける部分を見分ける

『部品として名前を付け易い塊』です。
回答のコードでは二つの入力をメソッドにしましたが、main のコードだけを見た時、 int kakekin = inputKAKEKIN(scanner); の記述だけで「カケキンを(スキャナから)インプットしてintとして返す」ということが読み取れるはずです。メソッド名自体が何をしているのかのコメントでもあるのです。
Integer.parseInt も、parse(解析)・int(整数)で、パラメータ(入力)が String、返り値(出力) int ですから、ドキュメントを読まなくても String→int 変換と察しが付くでしょう。(そこから更に『じゃあ変換出来なかったらどうなるんだ?』まで興味が繋がると、 isDigit の必要性まで実行しなくてもたどり着けます。 )

また、コピペでコードを作っていると、コピーして修正して…のコピーの部分をメソッドにし、修正する部分をパラメータとする感じも、プログラムを作り始めた初期頃には有効でしょう。

[awz_9]

返信が遅くなり申し訳ありません！
何から何までお答えしていただいてありがとうございます。
また、コピペでコードを作っていると、コピーして修正して…のコピーの部分をメソッドにし、修正する部分をパラメータとする感じ
これぜひ今後使って勉強していきます。

Answer 1

scanner はデータを取り出したら、もうそのデータは2度と取り出せません。

while (!scanner.nextLine().equals("end")) {

などとやってしまっては、 "end" 以外の文字列が入力されていたとしても、もうその文字列は次に next() しても取り出せません。

半丁の入力では、Integer.parseInt して 1,2 かチェックした後に isDigit をチェックしても意味はありません。
数値で無ければ parseInt が先に例外を発します。

当たり/外れの if が 4 つ並んでますが、当たりは b == b1 とし、 外れは else にすればそんなに必要ありません。

scanner は close したらもう使えません。

エラーで入力を繰り返すような場合は、入力部分をメソッドとして取り出してしまうほうが、全体の入れ子が深くならずに済みます。

package teratail_java.q_5vhgagiqxg5chr;

import java.util.Random;
import java.util.Scanner;

public class Main {
  public static void main(String[] args) {
    Random random = new Random(System.currentTimeMillis());
    Scanner scanner = new Scanner(System.in);

    int syojikin = 100;
    System.out.println("所持金＝" + syojikin);
    do {
      int kakekin = inputKAKEKIN(scanner, syojikin);
      if(kakekin == 0) break;

      syojikin -= kakekin;
      System.out.println("残高＝" + syojikin);

      int bet = inputBet(scanner);

      int sai = random.nextInt(6) + 1;
      System.out.println(sai);
      if(sai%2 == bet%2) {
        System.out.println("当たり!");
        syojikin += (kakekin * 2);
      } else {
        System.out.println("はずれ!");
      }

      System.out.println("残高＝" + syojikin);
    } while(syojikin > 0);
  }

  private static int inputKAKEKIN(Scanner scanner, int syojikin) {
    while(true) {
      System.out.print("掛け金を入力:");
      String kakekinsc = scanner.nextLine();
      if(kakekinsc.equals("end")) return 0; //end

      int kakekin = Integer.parseInt(kakekinsc);
      if(kakekin > syojikin) {
        System.out.println("所持金以上の金額は賭けられません。");
        continue;
      }
      return kakekin;
    }
  }

  private static int inputBet(Scanner scanner) {
    while(true) {
      try {
        System.out.println("半(=1)か丁(=2)か?");
        String betsc = scanner.nextLine();
        int bet = Integer.parseInt(betsc);
        if(bet != 1 && bet != 2) {
          System.out.println("【エラー】入力できるのは1と2です");
          continue;
        }
        return bet;
      } catch(NumberFormatException e) {
        System.out.println("【エラー】入力できるのは整数だけです");
      }
    }
  }
}

実行結果

所持金＝100
掛け金を入力:50
残高＝50
半(=1)か丁(=2)か?
1
2
はずれ!
残高＝50
掛け金を入力:10
残高＝40
半(=1)か丁(=2)か?
3
【エラー】入力できるのは1と2です
半(=1)か丁(=2)か?
0
【エラー】入力できるのは1と2です
半(=1)か丁(=2)か?
1.5
【エラー】入力できるのは整数だけです
半(=1)か丁(=2)か?
abc
【エラー】入力できるのは整数だけです
半(=1)か丁(=2)か?
1
6
はずれ!
残高＝40
掛け金を入力:end

Comments

[awz_9]

大変お粗末なコードから読み取っていただきありがとうございます。
お恥ずかしい限りです。
parseIntで例外が出て本当に困っており本当に助かりました。
また入力部分をメソッドとして取り出すところは本当に勉強になりました。
ベストアンサーにさせてください。

[jimbe]

>parseIntで例外が出て本当に困っており

そのようなことは質問に書かれていなかったと思いますが…。
例外で止まるのは、止まるべくして止まるのであって、超常現象とかではありません。プログラマがすべきなのは「何か出て動かない、困った」では無く、例外は何を言っているのか・何故出るのか・その根本の原因は何なのか・その対策はどうするかを調べて、実際に直すことです。

parseInt で例外が出たのであれば、まずは parseInt のドキュメント(仕様書)を見るべきです。そこにはどのような場合にどのような例外が発するのか書いてあります。

https://docs.oracle.com/javase/jp/8/docs/api/java/lang/Integer.html#parseInt-java.lang.String-
>例外:
>NumberFormatException - 文字列が解析可能な整数型を含まない場合。

この時点で「Integer.parseInt して 1,2 かチェックした後に isDigit をチェックしても意味は無い」が分かるはずです。
対応は、 parseInt する『前に』 isDigit でチェックするか、 NumberFormatException の発生を isDigit の代わりとして捉えるか、で、回答は後者を取っています。

Answer 2

私はjava触ってないので明確に答えを提示できないのですが、

   while(1){
   System.out.println("半(=1)か丁(=2)か?");
   String bet1sc = betsc.next();
   int bet = Integer.parseInt(bet1sc);
　if (bet <= -1 || bet >= 3){
		System.out.println("【エラー】入力できるのは1と2です");
   }else{
       break;
   }

   boolean bb = bet1sc.chars().allMatch(Character::isDigit);
   if (bb != true) {
    break;
  }

こんな感じに「正しい数字が入力されるまでループする」とすればいいのでは？

Comments

[awz_9]

ありがとうございます。一度試してみたところ
while文の条件に何を書いたらいいかがわからず止まってしまいました。

[pig_vba]

「正しい数値が入力されるまでループする」パターンと「正しい数値が入力されたらbreakする(whileの条件式は無限ループ)」の2パターンがあると思います。前者はエラー条件の逆を書けばいいですし、後者ならwhile(true){でいいはずです。

[awz_9]

trueなのですね！今後頭に入れておきます。