テキストフィールドに何も入れない場合の動作について

Question

現在サーブレットについて勉強しているものです。

身近に聞ける人がおらず、お手数ですがご教授願います。

分からないところは、JSPでテキストフィールドがあるページを作成し、そこに何も入れないまま送信した場合の動作についてです。

以下に載せているサンプルAの通りに作成し、何も入れないで送信したところ、if( (numberStr == null) || (numberStr.length() == 0) )の条件文の中の処理が走るようで、”何かいれて”というメッセージが表示されるのを確認しております。

しかし、私が作成したファイルを使い、テキストフィールドに何も入れないまま送信した場合はエクセプション画面(例外レポート)に飛んでしまいます。

同じ条件を使い、同じように表示させているつもりなのですが、私が作成したソースが上手くいかず、参考書のソースが上手く動作している理由がどうしてもわかりません。

恐らくソースを見た方が早い、という方が多いと思いますので、見栄えは悪いのですが、ソースを張らせていただきます。

また、入力文字数の制限より、import等は全て削除しました。

また、何も入力しない場合に用いられている、try、catchの必要性についても、いまいち分かりません。
ifとelseで全て網羅するようにせず、なぜtry,catchを使うのでしょうか？

ご存じの方、いらっしゃいましたら、是非教えて下さい。

サンプルA-start---------------------------------------------------------------------
public class GuessServlet extends HttpServlet {

/* 当たりの数字をランダムに決定 */
private int hit = new Random().nextInt(10);

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

/* jspファイルで入力した数字をひっぱってくる */
String numberStr = request.getParameter("number");
int input = Integer.parseInt(numberStr);

response.setContentType("text/plain; charset=Windows-31J");
PrintWriter out = response.getWriter();

if( (numberStr == null) || (numberStr.length() == 0) )
{
request.setAttribute("msg","何か入れて");
RequestDispatcher dispatcher = request.getRequestDispatcher("/reqattr/guess.jsp");
dispatcher.forward( request, response);
}
else if(hit == input)
{
request.setAttribute("msg","当たり。数字=" + input);
RequestDispatcher dispatcher = request.getRequestDispatcher("/reqattr/guess_hit.jsp");
dispatcher.forward( request, response);
}
else{
request.setAttribute("msg","はずれ。数字=" + input);
RequestDispatcher dispatcher = request.getRequestDispatcher("/reqattr/guess.jsp");
dispatcher.forward( request, response);
}
}
}

サンプルA-end---------------------------------------------------------------------

私のソース-start---------------------------------------------------------------------

public class GuessServlet extends HttpServlet {

/* 当たりの数字をランダムに決定 */
private int hit = new Random().nextInt(10);

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

/* jspファイルで入力した数字をひっぱってくる */
String numberStr = request.getParameter("number");

String error = null;
int number = -1;

if( (numberStr == null) || (numberStr.length() == 0) )
{
error = "何か入れて";
}

if(error == null)
{
try{
number = Integer.parseInt(numberStr);
if(number < 0 || 9 < number)
{
error = "０～９を入れて";
}
}
catch(NumberFormatException e)
{
error ="数字を入れてください";
}
}

String path;

if(error != null)
{
request.setAttribute("msg", error);
path = "/reqattr/guess.jsp";
}
else
{
if(hit == number)
{
request.setAttribute("msg","当たり。数字=" + number);
path = "/reqattr/guess_hit.jsp";
}
else
{
request.setAttribute("msg","はずれ。数字=" + number);
path = "/reqattr/guess.jsp";
}
}
RequestDispatcher dispatcher = request.getRequestDispatcher(path);
dispatcher.forward( request, response);
}
}

私のソース-end---------------------------------------------------------------------

yamada_g · Accepted Answer

補足にある認識で間違っていません。

例外が発生したとき、原因がよくわからなかったら
とりあえずエラーメッセージや例外名で検索してみるといいですよ。
特殊なものでなければ、なにかしらのページが見つかると思います。

今回であれば、
java.lang.NumberFormatException: For input string: ""
とか、
java.lang.NumberFormatException
などで。

ただ、まずはエラーメッセージをちゃんと読むことが大事です。
今回も
>For input string: ""
から、なんとなくでも「入力が空文字なのがいけない」っていうのが分かるようになると
自分で解決できるようになっていくのではないかと思います。

yamada_g · Answer

もう一度確認して欲しいのですが、本当に"私のソース"と書いてある方（下に記載されているほうのソース）で
おっしゃるような例外が発生しますか？
当方の環境でそのまま動かしてみても問題ないですが。

先ほども書いたとおり、サンプルとなっている方のソースでは
>/* jspファイルで入力した数字をひっぱってくる */
>String numberStr = request.getParameter("number");
>int input = Integer.parseInt(numberStr);
の3行目でNumberFormatExceptionが発生します。
これって補足にあるスタックとレースの内容と一致していますよね。
try~catchについての補足を読んでも、サンプルとKoh1111さんが書いたソースが逆になっているように思います。

とりあえず、なぜ記載の例外が発生するかというと、
Integer.parseIntメソッドは引数に整数に変換できない文字列を渡すとNumberFormatExceptionを投げる仕様です。
http://java.sun.com/javase/ja/6/docs/ja/api/java/lang/Integer.html#parseInt(java.lang.String)
「number」に何も入力がないときには、
Integer.parseInt(null);
を実行することになります。なので上記例外が発生します。

「私のソース」と書いてある方では、
>if( (numberStr == null) || (numberStr.length() == 0) )
で「number」が入力されているかを確認し、入力されているときだけInterger.parseInt()するようになっています。
ですので例外が発生しません。

「私のソース」にあるtry~catchについてですが、
1.仮にtry~catchを使わないとしたら入力された文字列が全て数字であるか等を確認する必要があります。
2.try~cathcの場合は、Integer.parseInt()を実行してNumberFormatExceptionが発生したら数字ではないと判断できます。
自分はとっては2.の方法で済むならそっちの方が楽だと思いますがいかがでしょうか。
>if文で全ての状態を網羅するのではなく
の意味がはっきりと分からなかったのですが、一度に全ての条件をチェックする必要はなくて
ここでは「入力された値が数値であるか」「その数値は0~9であるか」を確認する処理をするっていうことだと思います。
「入力があるかないか」は他の箇所で別にやればいいと思いますし。
実際「私のソース」ではそうなっていますね。

なんかダラダラと長くなってしまいましたが、参考になれば幸いです。

yamada_g · Answer

見た感じは「number」の値がnullでも問題ないように見えますけどね。
むしろ、サンプルの場合だと
>int input = Integer.parseInt(numberStr);
でNumberFormatExceptionが発生すると思いますけど。

文字数制限で例外の内容を書けなかったのだと思いますが、
本当に「numberがnullだから」発生している例外なのでしょうか？

>また、何も入力しない場合に用いられている、try、catchの必要性についても、いまいち分かりません。
これは、どの部分のことを言っているのか分かりませんでした。
記載のコードで言えば、try~catchしているのは何かしら入力されている場合だけですよね。

テキストフィールドに何も入れない場合の動作について

補足にある認識で間違っていません。

もう一度確認して欲しいのですが、本当に"私のソース"と書いてある方（下に記載されているほうのソース）で

見た感じは「number」の値がnullでも問題ないように見えますけどね。

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング