未入力チェック 文字と数字両方適応させた処理

Question

前提・実現したいこと

入力フォームで未入力チェックをしたいのですが、現在半角数字を入力したときにはエラーメッセージが表示されるようになっていますが、全角や文字列が入力されるとエラーが出てしまいます。ユーザーが全角や文字を入力したときにもエラーの処理を行いたいです。

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

try,catchの例外処理がうまく使えていない。

エラーメッセージ

該当のソースコード

protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        // TODO Auto-generated method stub
        EntityManager em = DBUtil.createEntityManager();


        →パラメータを取得する際に、ここにtry？

        String age = request.getParameter("age");
        String gender_flag = request.getParameter("gender_flag");
        String height = request.getParameter("height");
        String weight = request.getParameter("weight");
        String fat_percentage = request.getParameter("fat_percentage");
        String waist = request.getParameter("waist");
        String activity = request.getParameter("activity");

        →取得後catch(NumberException)？

        Health health = new Health();

        health.setAge((age != null && age.equals("")) ? null : Integer.parseInt(age));
        health.setGender_flag(gender_flag);
        health.setHeight((height != null && height.equals("")) ? null : Double.parseDouble(height));
        health.setWeight((weight != null && weight.equals("")) ? null : Double.parseDouble(weight));
        health.setFat_percentage((fat_percentage != null && fat_percentage.equals("")) ? 0 : Double.parseDouble(fat_percentage));
        health.setWaist((waist != null && waist.equals("")) ? 0 : Double.parseDouble(waist));
        health.setActivity(activity);

        →それともerrorチェック前？

        // 未入力チェック（年齢、身長、体重のみ必須）
        List<String> errors = HealthValidator.validate(health);
        if(errors.size() > 0) {
            em.close();
            request.setAttribute("_token", request.getSession().getId());
            request.setAttribute("health", health);
            request.setAttribute("errors", errors);
            RequestDispatcher rd = request.getRequestDispatcher("/WEB-INF/views/health/healthCheck.jsp");
            rd.forward(request, response);
        }

        HealthCheckLogic healthCheckLogic = new HealthCheckLogic();

        healthCheckLogic.execute(health);

        request.setAttribute("health", health);

        RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/views/health/healthCheckResult.jsp");
        dispatcher.forward(request, response);
    }
}

試したこと

try,catchとNumberExceptionを使用すれば数字以外を処理できると調べたのですが、使いどころ（コードの挿入箇所）が理解できず苦しんでいます。初心者な質問で申し訳ありませんがご助力お願いします。

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

eclipse
Tomcat8

Answer 1

タイミングとしては、
→取得後catch
→それともerrorチェック前？
の間にある Health クラスにリクエストパラメータを格納する箇所でしょう。

ServletのdoPostメソッドの中にチェック処理を全部実装するのはメソッドが長くなってしまうので、

• 別メソッドを作って Health クラスにリクエストパラメータを格納する処理を実装するか
• ないしはHealthクラスそのものに実装する

として、例外の制御をdoPostの中で実装するのが良いでしょうか。

Comments

[退会済みユーザー]

具体的な箇所を明示していただきありがとうございます。
自分の思う完全な処理ではありませんが、一応の解決ができました。
また、可読性を考えた実装方法も提案していただけたので、参考にさせていただきます。
本当にありがとうございました。

Answer 2

try-catch の基本は

try {
 A
} catch( B ) {
 C
}

という書き方で, A で B という例外が発生した場合に C にフローが移るというものです.
ですので, 「全角や文字を入力したとき」どこで, どのような例外が発生するのかを明確にし, その発生元が A となるように try-catch で囲み, 発生する例外 を B に(例外クラス名 変数の形で)書き, C でエラー処理を行ってください.

なお, 変換時の例外でエラーを得る方法の他, 変換前に半角数字以外が含まれていないかをチェックする方法もあります.

Comments

[退会済みユーザー]

回答ありがとうございます。
自分の知識不足のため、どこが例外発生個所かすらわかっていませんでした。
ですが、catchやA,B,Cの意味などは理解することができました。
本当にありがとうございました。