JAVAでStringを、intに変える

Question

jspで入力された情報を、javaServletで受けとるコードを作っています。
Stringで来る情報をintに変える際に、数字以外が入力されていたら『入力エラー』を表示して返すことにしたいです。

ただ、Stringをintに変えるところまでは出来たのですが肝心な文字チェックがコーディング出来ず悩んでおります。
宜しくお願い申し上げます。

//Servlet
package ex;

import java.io.IOException;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class Question04Servlet extends HttpServlet {

    public void doPost(HttpServletRequest request, HttpServletResponse response)
            throws IOException, ServletException {

        // レスポンスの為のコンテンツタイプの設定。クライアントにどのようなデータを送るのかを指定
        response.setContentType("text/html; charset=utf-8");
        // リクエストのエンコードの設定。リクエスト・パラメータ読み込み時の文字コードを明示
        request.setCharacterEncoding("utf-8");

        //Beanクラスのインスタンス生成
        Question04FormBean q4 = new Question04FormBean();

        //リクエストからパラメータの取得
        String inputData1 = request.getParameter("inputData1");
        String inputData2 = request.getParameter("inputData2");
        String msg = "入力に不備があります。整数を入力してください。";

        //入力チェック(「null」或いは、空文字チェック)
        if (null == inputData1 || "".equals(inputData1)) {
            //入力値がない場合
            q4.setMsg(msg);
        } else if (null == inputData2 || "".equals(inputData2)) {
            //入力値がある場合
            q4.setMsg(msg);
        }

        //整数に変換 ※ここで同時にチェック判定を行いたい。
        int number1 = Integer.parseInt(inputData1);
        int number2 = Integer.parseInt(inputData2);

        //足算を実施
        int sum = number1 + number2;
        msg = String.valueOf(sum);
        q4.setMsg(msg);

        //リクエストに「msg」をキーに値をセット
        request.setAttribute("msg", msg);
    }

}

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[tatakauneko3]

申し訳ございません。スマホから投稿したら汚い状態になっておりました。きちんとではないですが、PCから再投稿したので少し見やすく変更致しました。

[m.ts10806]

コードを伴う場合はモバイルからの投稿はやめたほうがいいです。どういう環境下か知らないですが、基本的にコードもコピペになりませんし。

[tatakauneko3]

移動中に投稿したためスマホからになりました。今後はPCから行うようにします。

Answer 1

下記いずれかまたは両方で

• try-catchでNumberFormatExceptionを捕捉する
• 正規表現で「数値のみかどうか」を確認する（調べれば幾らでも出てきます）

Comments

[tatakauneko3]

ありがとうございます。今回はtry-catchにてNumberFormatExceptionにて解決出来ました。

Answer 2

try - catch で例外を捕捉するのは、
例外を発生されるのってちょっと重いって聞いたことがあるんですよね
（何万件も処理するのでなければ誤差の範囲かもしれませんが）

そうすると正規表現でチェックするのが良いのかなあと思います。
正規表現でチェックする場合は、
「マイナス記号は許容するのか」
「小数点はOK？」
あたりも気にすると良いかなと思います。

＃ まあ簡単さでいえば try - catch かなとは思いますけど。
＃ 変換できなかったらエラー投げてくれるわけですし...

---

余所様のサイトですが ぐぐったら出てきたやつ

• あえて言うほどではない　数値変換時の型チェック Java編
  https://www.techscore.com/blog/2012/12/20/あえて言うほどではない　数値変換時の型チェッ/

Comments

[tatakauneko3]

回答ありがとうございます。今回はtry-catchにしましたが、正規表現も学習致します。

Answer 3

通常リクエストパラメーターの妥当性チェックを通過したあとに
変換かけるのが普通ですが、まぁパラメーター2個くらいならこんな感じでしょうか。

  public void doPost() {
    /**
      割愛
    */

    //整数に変換 ※ここで同時にチェック判定を行いたい。
    Optional<Integer> number1 = this.parse(inputData1);
    Optional<Integer> number2 = this.parse(inputData2);
    if (number1.isEmpty() || number2.isEmpty()) {
      final String invalidMessage = "入力に不備があります。整数を入力してください。";
      q4.setMsg(invalidMessage);
      request.setAttribute("msg", invalidMessage);
      return;
    }

    //足算を実施
    int sum = number1.get() + number2.get();

  }

  /**
   * String型をInteger型へ変換する.
   *
   * @param value 文字列
   * @return 数値
   */
  public Optional<Integer> parse(final String value) {
    try {
      return Optional.of(Integer.parseInt(value));
    } catch (NumberFormatException e) {
      return Optional.empty();
    }
  }

尚、q4(Question04FormBean)がどこにも生かされていないので
どう取り扱っていいかよくわからないです。

Comments

[tatakauneko3]

回答ありがとうございます。おっしゃるとおりBeanはきちんと生かされておりませんでした。底の部分は別途修正して改善しました。

Answer 4

Character.isDigit で文字列のすべての文字が数字かどうかを見るやり方もあります。

import java.util.Scanner;

class Main {
	static boolean invalid(String s) {
		if (s == null || s.isEmpty()) return true;
		for (int i = 0; i < s.length(); i++)
			if (!Character.isDigit(s.charAt(i))) return true;
		return false;
	}

	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		for (;;) {
			System.out.print("s1>> ");
			if (!sc.hasNextLine()) break;
			String s1 = sc.nextLine();
			System.out.print("s2>> ");
			if (!sc.hasNextLine()) break;
			String s2 = sc.nextLine();

			if (invalid(s1) || invalid(s2))
				System.out.println("不正入力");
			else {
				int n1 = Integer.parseInt(s1);
				int n2 = Integer.parseInt(s2);
				int sum = n1 + n2;
				String msg = String.valueOf(sum);
				System.out.println(msg);
			}
		}
	}
}

Comments

[Zuishin]

https://docs.oracle.com/javase/jp/8/docs/api/java/lang/Character.html#isDigit-char-

余計な文字が多数含まれると思います。逆にマイナス符号は含まれないんじゃないですか？

[kazuma-s]

例えば、全角数字を不正文字とするかどうかということですね。
Integer.parseInt は全角数字でも NumberFormatException をスローしないので
try-catch も使えないことになります。
符号を許すかどうかは正規表現でも注意が必要です。
int の範囲を超える数字列の処理も必要でしょうか？

[Zuishin]

どのみち parseInt で変換するのであれば、その前段階でのチェックは parseInt に合わせないと不整合が起こります。
parseInt で使用できない文字を通してしまうとハンドルされていない例外が起きてしまうということですね。

逆に、合わせているのであれば、事前チェックが必要かどうかは要検討と思います。

ついでにこれは好みの問題かもしれませんが、負の数や範囲外の値を排するのであれば、変換後に行う方が可読性が上がったり親切なエラーメッセージが出せるのかなと思います。
(int の範囲外であれば parseInt で例外になると思いますが)

[tatakauneko3]

お二人とも回答ありがとうございます。今回はintの範囲内でしたのでその点は問題ありませんでした。書き方・順番などは改めて参考にさせて頂きます。

Answer 5

正規表現でチェックするやり方があります。

        //整数に変換 ※ここで同時にチェック判定を行いたい。
        Pattern p = Pattern.compile("^[0-9]+$");
        Matcher m1 = p.matcher(inputData1);
        Matcher m2 = p.matcher(inputData2);
        boolean result = m1.matches() && m2.matches();
        if(!result) {
            // 数値以外の場合　※msgの使い方あってるかわからないけど、、、
            msg = "どっちかに数値以外が入力されています。";
            request.setAttribute("msg", msg);
            return;
        }
        int number1 = Integer.parseInt(inputData1);
        int number2 = Integer.parseInt(inputData2);

参考元：https://techacademy.jp/magazine/28637

Comments

[tatakauneko3]

ありがとうございます。正規表現は学習不足でした。きちんと見直して学びます。