Javaで指定された文字以外を受け付けない

Question

つい先日からjavaを学び始めました。
例外処理について詰まってしまったので質問させて頂きたいです。

プログラムは標準入力で受け取った数字までのフィボナッチ数列を表示させる仕様になっています。

この標準入力の時、"exit"と"quit"が入力されるとプログラムが終了させることは出来たのですが
他の文字列が入力されしまうとエラーを吐いてしまいます。

エラー文

Exception in thread "main" java.lang.NumberFormatException: For input string: "e"
	at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
	at java.base/java.lang.Integer.parseInt(Integer.java:652)
	at java.base/java.lang.Integer.parseInt(Integer.java:770)
	at Main.main(Main.java:11)

どのような例外処理を取れば良いのか分かりませんのでヘルプを頂きたく投稿させてもらいました。
よろしくお願い致します。

全体のコード

import java.util.Scanner;
class Main {
  public static void main(String[] args) {
    Scanner scan = new Scanner(System.in);
    while(true) {
      System.out.print("数字を入力してください(0< N <= 70) :");
      String input = scan.next();
      if (input.equals("quit") || input.equals("exit")) {
        break;
      } else {
        int num = CheckNum(Integer.parseInt(input));
        System.out.println(num + "番目のフィボナッチ数列: " + Fibonacci.calculate(num));
        System.out.println(num + "番目までのフィボナッチ数列: ");
        for(int i=1; i<=num; i++) {
          System.out.println(Fibonacci.calculate(i));
        }
      }
    }
  }
  static int CheckNum(int target) {
    while(true) {
      try {
        if (target>=1 && target<=70) return target;
      }
      catch(NumberFormatException e) {}
    } 
  }
}

class Fibonacci {
  public static long calculate(int num) {
    if(num == 1) return 0;
    if(num == 2) return 1;
    long fx = 0,
         f1 = 0,
         f2 = 1;
    for(int i=3; i<= num; i++) {
      fx = f1 + f2;
      f1 = f2;
      f2 = fx;
    }
    return fx;
  }
}

Answer 1

数値に変換できるかチェックしてから、数値に変換したいのだと思います。
だとすると、CheckNum メソッドと Integer.parseInt メソッドの呼び出しの順番が逆なのではないでしょうか？

int num = Integer.parseInt(CheckNum(input));

Comments

[m.ts10806]

CheckNumが引数の型をintで取ってますがこの作りだとtry-catchで拾えないのでは？

[nskydiving]

そうですね、CheckNumメソッドは引数もそうですが、この形に合わせて修正が必要なりますね。

[m.ts10806]

ただ回答でのご指摘通りそもそもの処理順の間違いは認識するところからですね。

[maskedrider5556]

皆さまご丁寧にご回答頂き誠にありがとうございました!

Answer 2

問題は数値に変換できない文字列がparseInt()に突っ込まれてしまうことが原因なので、parseInt()に突っ込む前に正規表現などで数字かどうかをチェックする、または、「数字以外は異常処理」と捉えてえelseを丸々try-catchでNumberFormatException()を拾うという手もあると思います。

Comments

[maskedrider5556]

ご回答頂きありがとうございました!!

Answer 3

いくつか気になる点があるので、プログラムを修正してみました。
できるだけ元の流れを維持するように書いたので不自然な部分もありますがご了承下さい。

変更点に関して記述します。

★1 Scannerが閉じられていない

今回は、たまたま実害は無いですが、ファイル等、閉じ忘れると痛い目に遭うことがあるので
意識だけはしておいた方がいいです。

★2

入力された文字列が、プログラムの動作上想定されたものかどうかチェックする処理を
追加しました。
チェックして、想定通りであれば実処理をする。という流れです。
元々のCheckNumは、色々誤りがあったので使用しないようにしました。（コードご参照ください）
※ Javaのメソッド名は先頭小文字で命名することが多いです。

import java.util.Scanner;
class Main {
  public static void main(String[] args) {
    Scanner scan = new Scanner(System.in);
    while(true) {
      System.out.print("数字を入力してください(0< N <= 70) :");
      String input = scan.next();
      if (input.equals("quit") || input.equals("exit")) {
        scan.close();    // ★1
        break;
      } else {
        if (isValidNum(input)) {  // ★2
          int num = Integer.parseInt(input);  // 既にチェック済みなので問題は起きない
          System.out.println(num + "番目のフィボナッチ数列: " + Fibonacci.calculate(num));
          System.out.println(num + "番目までのフィボナッチ数列: ");
          for(int i=1; i<=num; i++) {
            System.out.println(Fibonacci.calculate(i));
          }
        }
      }
    }
  }

  static int CheckNum(int target) {
    while(true) {
      try {
        if (target>=1 && target<=70) return target;
        // バグ 範囲外の数字を入力されると無限ループします
      } catch(NumberFormatException e) {
        // バグ ここで例外は発生しません。（ここが実行されることはありません）
        // catch句を空にするのはあまり良くない書き方です
      }
    }
  }

  static boolean isValidNum(String targetStr) {
    try {
      int target = Integer.parseInt(targetStr); // 数字以外であれば例外発生
      if (target>=1 && target<=70) return true;

      System.out.println("範囲外の数字です。");
      return false;
    } catch(NumberFormatException e) {
      System.out.println("数字ではありません。");
      // e.printStackTrace();
      return false;
    }
  }
}

Comments

[maskedrider5556]

丁寧にご回答頂きありがとうございます!!
非常に分かりやすかったです。
try&catchでの例外処理含めしっかりと理解しきれてない箇所がいくつか見つかったのでしっかりと復習しようと思います。
ありがとうございました！