必須項目に値が入らなかった場合エラーにする

Question

javaの勉強をしながら名刺を管理するwebアプリケーション開発をしています。

ユーザーの情報を受け付ける画面で、会社名、姓名を入れなかった場合は
会社名、姓名を入れてくださいというエラー画面に流したいと考えています。
会社名、姓名を入れた場合は確認画面に流したいと思いservletを作ったのですが
エラーが起きています。

//リクエストパラメーターの文字コードを指定
		request.setCharacterEncoding("UTF-8");

		//リクエストパラメーターを取得した後、Account型インスタンスに情報格納
		Account account = new Account();
		account.setBusinessUnit(request.getParameter("business_unit"));
		account.setDivision(request.getParameter("devision"));
		account.setSection(request.getParameter("section"));
		account.setLastName(request.getParameter("last_name"));
		account.setFirstName(request.getParameter("first_name"));
		account.setLastNameKana(request.getParameter("last_name_kana"));
		account.setFirstNameKana(request.getParameter("first_name_kana"));
		account.setCompanyName(request.getParameter("company_name"));
		account.setCompanyNameKana(request.getParameter("company_name_kana"));
		account.setZipCode(request.getParameter("zip_code"));
		account.setMunicipality(request.getParameter("municipality"));
		account.setAddress(request.getParameter("address"));
		account.setTelDirect(request.getParameter("tel_direct"));
		account.setTelMain(request.getParameter("tel_main"));
		account.setEmail(request.getParameter("email"));
		account.setFax(request.getParameter("fax"));
		account.setUrl(request.getParameter("url"));
		account.setCorpNote(request.getParameter("corp_note"));


		//HttpSessionインスタンスの取得
		HttpSession session = request.getSession();
		session.setAttribute("Account", account);


		//フォワード先
		String forwardPath = null;

		forwardPath = "WEB-INF/jsp/RegisterCheck.jsp";

		//リクエストパラメーターをチェック
		String errorMsg = "";
		if(companyName == null || companyName.length() == 0){
			errorMsg += "会社名が入力されてません";
			forwardPath = "WEB-INF/jsp/Err.jsp";

		}
     if(lastName == null || lastName.length() == 0){
			errorMsg += "姓が入力されてません";
			forwardPath = "WEB-INF/jsp/Err.jsp";
		}

		if(firstName == null || firstName.length() == 0){
			errorMsg += "名が入力されてません";
			forwardPath = "WEB-INF/jsp/Err.jsp";
		}

この3個所でを変数に～解決できません
～を解決できません。と表示されてしまいます。

if(companyName == null || companyName.length() == 0){
if(lastName == null || lastName.length() == 0){
if(firstName == null || firstName.length() == 0){

エラーを無くして、必須項目に値が入らなかった場合エラーにしたい場合どう変更すれば宜しいでしょうか？
ご教示願えると幸いです。

*環境
mysql 5.7
windows7

Answer 1

ローカル変数 forwardPath の値は使用されていません
上記はエラーではなく警告ですね？

これは、宣言し、値の代入はしているが、値を使用していない場合に表示されるものです。

forwardPathを使用して遷移先を指定する処理を実施していないのではないでしょうか。

Comments

[kumatarou]

＞上記はエラーではなく警告ですね？
警告ですね・・・申し訳ございません

RequestDispatcher dispatch = request.getRequestDispatcher(forwardPath);
dispatch.forward(request, response);
にしたら無事に出来ました。ありがとうございました！

Answer 2

companyNameはAccountというモデルのプロパティとのことなので、入力チェックのプログラムではAccountモデルのインスタンスaccountを経由してアクセスしないと取得できませんね。

例えば、
if(companyName == null || companyName.length() == 0){

==>

if(account.getCompaneName() == null|| account.getCompanyName().length() == 0){

Comments

[kumatarou]

そうです！そう書きたかったのです！ありがとうございます！
また、わかりやすくコードを書いて頂きありがとうございます

エラーそのものは無くなったのですが、
ローカル変数 forwardPath の値は使用されていません
と出てきて、実行すると当たり前ですがforwardPathが効いていません・・・

String forwardPath = null;の下で値を指定しているにも関わらずなぜこのエラーが起きるのでしょうか？
表題とは違った質問になってしまいますが、ご教示して頂けると幸いです。

Answer 3

Web画面の入力項目から、lastName, firstName, companyNameを取得する部分がありません。

String lastName= request.getParameter("lastName");
String firstName= request.getParameter("firstName");
String companyName= request.getParameter("companyName");

このようにして、リクエストパラメーターから入力フォームのデータを取得して下さい。
getparameterの引数に渡す文字列は、inputタグのnameの値です。

Comments

[kumatarou]

回答ありがとうございます

この部分を
account.setCompanyName(request.getParameter("company_name"));
account.setLastName(request.getParameter("last_name"));
account.setFirstName(request.getParameter("first_name"));

　　　　　　　　　　　　　　　　　　　↓このように変更すれば宜しいですか？

String lastName= request.getParameter("lastName");
String firstName= request.getParameter("firstName");
String companyName= request.getParameter("companyName");

勉強を始めたばかりなので理解が追いつかず申し訳ございません。

[shanxia]

いえ、accountの部分を見ていませんでした。
今回の場合だと、次の様な3行を追加すると良いかと思います。

String companyName = account.getCompanyName();
String lastName = account.getLastName();
String firstName = account.getFirstName();

[kumatarou]

回答ありがとうございます。

載せて頂いた3行を追加しますとエラーは消えるのですが

forwardPath = "WEB-INF/jsp/RegisterCheck.jsp";←この部分でローカル変数 forwardPath の値は使用されていませんと表示されてしまいます。

String型にforwardPathを入れているため反応してくれないのでしょうか？

[shanxia]

forwardPathには、遷移先のURLを代入しているのですね？
それならば必須項目チェックの後に、dispatcherへセットして下さい。

RequestDispatcher dispatch = request.getRequestDispatcher(forwardPath);
dispatch.forward(request, response);

[kumatarou]

エラーも無くなり無事にエラーページに飛べました！

ありがとうございました！

Answer 4

lastName, firstName, companyNameという３つの変数は宣言されていますか？

Comments

[kumatarou]

回答ありがとうございます

Accountというmodelで宣言しています。

private String companyName; //会社名
private String lastName; //姓
private String firstName; //名

this.companyName = companyName;
this.lastName = lastName;
this.firstName = firstName;

public String getCompanyName() {
return companyName;
}

public void setCompanyName(String companyName) {
this.companyName = companyName;
}

public String getLastName() {
return lastName;
}

public void setLastName(String lastName) {
this.lastName = lastName;
}

public String getFirstName() {
return firstName;
}

public void setFirstName(String firstName) {
this.firstName = firstName;
}

長くなってしまいましたがお願い致します