try-catch文 サーブレットと他クラス

Question

前提・実現したいこと

フォーム画面で年齢や身長、体重を入力しBMIなどを測定するものを作りたいと思っています。
フォーム画面では年齢などを必須入力としたいため未入力チェックを行っているのですが、現在の状態では全角や半角文字に対してエラーメッセージを表示できません。そのため全角や半角文字に対してもエラーメッセージが出るような処理を行いたいのでtry-catch(NumberFormatException)を使用しようと思っています。また、メッセージの内容は、未入力だった場合と全角や半角文字が入力された場合とで分けたいと思っています。ですがdoGetメソッドが長くなってしまうので他クラス別メソッドで未入力チェックを実装しようと思っています。ですが、他クラスでtry-catchを使用してその値をサーブレットに渡す？イメージが全くつきません。

ageのみなどでも構いませんので、修正箇所を教えていただけないでしょうか。

またtry-catchやNumberFormatException以外のよい方法などありましたらお願いします。

該当のソースコード

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

        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");

        Health health = new Health();

        health.setAge(age);
        health.setGender_flag(gender_flag);
        health.setHeight(height);
        health.setWeight(weight);
        health.setFat_percentage(fat_percentage);
        health.setWaist(waist);
        health.setActivity(activity);

        // 未入力チェック（年齢、身長、体重のみ必須）
        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);
        }

package validators;

import java.util.ArrayList;
import java.util.List;

import models.Health;

public class HealthValidator {
    public static List<String> validate(Health h) {
        List<String> errors = new ArrayList<String>();

        String age_error = _validateAge(h.getAge());
        if(!age_error.equals("")) {
            errors.add(age_error);
        }

        String height_error = _validateHeight(h.getHeight());
        if(!height_error.equals("")) {
            errors.add(height_error);
        }

        String weight_error = _validateWeight(h.getWeight());
        if(!weight_error.equals("")) {
            errors.add(weight_error);
        }

        return errors;
    }

    private static String _validateAge(Integer age) {
        if(age == null || age.equals("")) {
            return "年齢を入力してください";
        }

        return "";
    }

    private static String _validateHeight(Double height) {
        if(height == null || height.equals("")) {
            return "身長を入力してください";
        }

        return "";
    }

    private static String _validateWeight(Double weight) {
        if(weight == null || weight.equals("")) {
            return "体重を入力してください";
        }

        return "";
    }
}

Comments

[jimbe]

> 他クラス別メソッドで未入力チェックを実装しようと思っています。ですが、他クラスでtry-catchを使用してその値をサーブレットに渡す？イメージが全くつきません。

ご提示のコードはそれっぽくなっているようですが, 何か問題なのでしょうか.
if 文で errors に add する代わりに, try-catch で NumberFormatException 発生時に errors に add するだけかと思います.

[m.ts10806]

>全角や半角文字に対してエラーメッセージを表示できません
正規表現でバリデーションするのはダメなのでしょうか。
「入力チェック」の範疇であればtry-catchで拾うよりきちんとブラックリスト方式で「ダメなものを指定して弾く」方が理にかなっていると思います。

[退会済みユーザー]

jimbe様>
例えばtry{!age_error.equals("")
}catch{(NumberFormatException e)
{errors.add(age_error);}
ということでしょうか？

[退会済みユーザー]

m.ts10806様>
正規表現でバリデーション、またブラックリスト方式というのが理解が及ばないのですが、jimbe様がおっしゃっていることではないのですよね？
「ダメなものを指定」というのは全角や文字を指定するということでしょうか？

[m.ts10806]

はい。「～～に該当するものがあったらNG」という方式です。
よく聞く場所はスパムメール対策ですね。
https://www.kddi.com/yogo/%E3%83%A2%E3%83%90%E3%82%A4%E3%83%AB/%E3%83%96%E3%83%A9%E3%83%83%E3%82%AF%E3%83%AA%E3%82%B9%E3%83%88%E6%96%B9%E5%BC%8F.html

[m.ts10806]

そもそもバリデーションはほとんどがブラックリスト方式で入力のチェックを行うものです。
ご自身でも「入力が空だったら」でエラーとしていますが、それはブラックリスト方式です。
受け入れられないものをチェックすることですね。
Exceptionはあくまで「異常処理」の範疇で、「意図しない事態が起きた時」というのが通例です。
「自由に入力できる欄」からくる情報は「受け入れられるものだけ受け入れる」とするべきで、
そうすると「受け入れられないもの」は「想定できる範囲」になるので、Exceptionは不適当かなと思っています。

[退会済みユーザー]

プログラミング思考の点で、ということですね！
ありがとうございます。参考にさせていただきます。

[jimbe]

> 例えば ～
try-catch の記述の仕方をお分かりになっていないということでしょうか.
だとすれば NumberFormatException がどのような場合に発生するのか等からお調べになれば, try-catch の例文も見られるかと思います.

Answer 1

jimbe様がおっしゃったように、if文をtry-catchに変えてひとまずは思った通りの動作をさせることができました。ありがとうございます。
m.ts様の視点で改めて改良していこうかと思います。