メールが届かなくなりました

Question

public class MailAction extends  LookupDispatchAction {

	 protected Map<String,String> getKeyMethodMap() {
		// キーとメソッドの対応を登録する
		Map<String,String> map = new HashMap<String,String>();
		map.put("firster","lookupDispatchA");
		map.put("forgets","lookupDispatchB");
		return map;
	}



	private String dbName="staff";
	private String user = "root";
	private String pass = "";





	public ActionForward lookupDispatchA  (ActionMapping mapping
			,ActionForm form
			,HttpServletRequest request
			,HttpServletResponse response) throws Exception {
		request.setCharacterEncoding("UTF-8");

		Mailform frm = (Mailform)form;
		String name = frm.getName();
		String address = frm.getAddress();
		String pwd = frm.getPwd();

		System.out.println(pwd);



		 //actionMessaages作成
		ActionMessages errbox = new ActionMessages();
		ActionMessage errmsg1 = new ActionMessage("errors.empty","未入力");
		ActionMessage errmsg2 = new ActionMessage("errors.email","半角入力");
		ActionMessage errmsg3= new ActionMessage("errors.already","既出");


		if( name == null || address == null) {/*未入力*/
		errbox.add("empty",errmsg1);
		saveErrors(request,errbox);
		return mapping.findForward("back");
		}/*ActionMessage（string、String>はダメ*/


		/*半角入力*/
		 if (address != "[\\d[a-zA-Z]*]+@[\\d[a-zA-Z]*]"){

			errbox.add("half",errmsg2);
			saveErrors(request,errbox);
			return mapping.findForward("back");
			}

			/*configのsource取得*/

		 DBchinuki  db = new DBchinuki();
		 db.getDBConnection(dbName,user,pass);
		 int connect = db.getDBConnection(dbName,user,pass);
		 if(connect == 1) {
		 	return mapping.findForward("issue");
		 } else  {
		 	return mapping.findForward("miss");
		 }


		 String diagno = db.seldb(address);/*登録者かどうか検索*/
			System.out.println("/**********************************/"+diagno);

		 /*完了済み*/
		 if(address == diagno) {
			 errbox.add("already",errmsg3);

		  saveErrors(request,errbox);
		  return mapping.findForward("issue");

		 }

		 db.insdb(name, address,pwd);/*登録*/
		 db.throwDBConnection();  /*切断*/
		 SendMail sm = new SendMail(address, pwd);
	     int TS = sm.doMail();
	     if(TS == 1) {
	         return mapping.findForward("issue");
	     } else {
	         return mapping.findForward("miss");
	     }





	}

public ActionForward lookupDispatchB(ActionMapping mapping, /*忘れた方*/
ActionForm form,
HttpServletRequest request,
HttpServletResponse response) throws  Exception {
	Mailform frm = (Mailform)form;
	String address = frm.getAddress();
	String address1 = frm.getAddress1();
	String pwd = frm.getPwd();

	DBchinuki db = new DBchinuki();
	String diagno = db.seldb(address);/*登録者か確認*/



	 //actionMessaages作成
	ActionMessages errbox = new ActionMessages();
	ActionMessage errmsg1 = new ActionMessage("errors.empty","未入力");
	ActionMessage errmsg2 = new ActionMessage("errors.email","半角入力");
	ActionMessage errmsg4 = new ActionMessage("errors.unregist","未登録");

	if(address1 == null ) {/*未入力*/
	errbox.add("empty",errmsg1);
	saveErrors(request,errbox);
	return mapping.findForward("back");
	}/*ActionMessage（string、String>はダメ*/

	/*半角入力*/
	 if (address1 != "[\\d[a-zA-Z]*]+@[\\d[a-zA-Z]*]"){
		errbox.add("half",errmsg2);
		saveErrors(request,errbox);
		return mapping.findForward("back");

		}


	 db.getDBConnection(dbName,user,pass);
int connect = db.getDBConnection(dbName,user,pass);
if(connect == 1) {
	return mapping.findForward("issue");
} else  {
	return mapping.findForward("miss");
}






     /*jquery変数pwd受け渡し*/


     db.updb(pwd,address1);/*更新 addressは条件*/



	 /*未登録*/
	 if(address1 != diagno)
	 {
		 errbox.add("unregist",errmsg4);

		 saveErrors(request,errbox);
		 return mapping.findForward("back");

	 }

	 db.throwDBConnection();  /*切断*/
	/*configのsource取得*/
	 SendMail sm = new SendMail(address1, pwd);
     int TS = sm.doMail();
     if(TS == 1) {
         return mapping.findForward("issue");
     } else {
         return mapping.findForward("miss");
     }

}
}



```要点
困っている点
コンソールにはランダムパスワードが表示されているがメールが届かないしDBに登録されない

エラー 到達不能コード
String diagno = db.seldb(address);/A
 db.updb(pwd,address1);/*更新 addressは条件*/Bででてます
※除去してはいけない処理です



どこかまちがっているのかわかりません。
javamailのバージョンが表示されなくなりました。

添削とアドバイスお願いします

DBに接続してメールが届くように戻したいです
さらにエラーメッセージも条件の時のみ表示させたいです

Answer 1

処理の途中にも関わらずActionForwardを返している為、
到達不能エラーが発生しています。

メソッドが失敗？した時だけ、ActionForwardを返すようにコードを書き変えればいいかと。

if(connect == 1) {
             return mapping.findForward("issue");
         } else  {
             return mapping.findForward("miss");
}

Comments

[kentros511]

そうですよね
質問してから気づきなおしましたが　またとじ｛｝の数が合わなくなりました

[kentros511]

いつも回答してくれて感謝してます