質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.48%

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

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 2,664
退会済みユーザー

退会済みユーザー

前提・実現したいこと

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

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

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

0

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/18 19:08

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

    キャンセル

0

try-catch の基本は

try {
 A
} catch( B ) {
 C
}


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

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/18 19:11

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る