dispatch action のエラーで困っています

Question

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



	private String url="jdbc:mysql:localhost/staff";
	private String user = "root";
	private String pass = "";





	public ActionForward lookupDispatchA  (ActionMapping mapping
			,ActionForm form
			,HttpServletRequest request
			,HttpServletResponse response) throws Exception {

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


		MailForm2 frm1 = (MailForm2)form;
		String address1 = frm1.getAddress1();

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

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

		/*半角入力*/
		 if (address != "[\\d[a-zA-Z]*]+@[\\d[a-zA-Z]*]"){
			errbox.add("half",errmsg2);}


		 /*完了済み*/
		 if(address == address1) {
			 errbox.add("already",errmsg3); }
		 /*未登録*/
		 if(address1 !=address)
		 {
			 errbox.add("unregist",errmsg4);

		  saveErrors(request,errbox);

		 }
		/*configのsource取得*/

		 DBooo db = new DBoooo();
		 db.getDBConnection(url,user,pass);

         db.seldb(address);/*登録者かどうか検索*/
         db.insdb(name, address);/*登録*/

/*メール送信*/


		try{/*start*/
			//property
			Properties prop = new Properties();
			prop.setProperty("mail.smtp.host", "localhost");
			//SMTPサーバーのセット
			prop.setProperty("mail.smtp.port", "25");

			//メールセッションを確立
			Session session = Session.getInstance(prop);

			//送信メッセージ
			MimeMessage mimeMessage1 = new MimeMessage(session);


			//ToAddressのセット
			InternetAddress toAddress =new InternetAddress("333.ppp.co.jp","ゲストさん");
			mimeMessage1.setRecipient(Message.RecipientType.TO,toAddress);

			//FRomAdressnoセット
			InternetAddress fromAddress = new InternetAddress("[e-mail]","管理者");
			mimeMessage1.setFrom(fromAddress);

			new String("Subject");
			mimeMessage1.setSubject("パスワード発行","iso-2022-jp");/*件名*/
			new String ("Text");
			mimeMessage1.setText("あなたのパスワードは" + pwd  + "です","iso-2022-jp");

			//保存
			 mimeMessage1.saveChanges();


			//メールの送信
			Transport.send(mimeMessage1);
		}catch(MessagingException e){/*すべての例外*/
		      e.printStackTrace();
		     return null;
		    }


		/*configのsource取得*/


        db.throwDBConnection();  /*切断*/
        return mapping.findForward("first");

}
public ActionForward lookupDispatchB(ActionMapping mapping, /*忘れた方*/
ActionForm form,
HttpServletRequest request,
HttpServletResponse response) {
Aと同じ
return mapping.findForward("forget");
}  ← ここでエラー発生  ｝が足りません 
｝← ここに足すとさらにエラーが出ます

エラーがあと一つなので 適切な }の位置を見つけたいです
どこに ｝をいれたらよいでしょうか

Comments

[izkn]

こちらの質問が他のユーザから「質問の範囲が広すぎる」という評価を受けています
わからない点を明確にし、調査したこと・試したことと共に記入していただくと、回答が得られやすくなります。

Answer 1

public class MailAction extends  LookupDispatchAction {

この括弧が閉じていませんね。一番最後にもう一つ}が必要です。
インデントが揃っていませんしブロックの書き方にも統一感がありません。見た目を整える習慣を付けるとこのようなミスは起こりにくくなります。

---

コードが編集されていたので追記です。
最後の｝が全角記号ですが、元のソースコードではちゃんと半角の}になっていますか？

Answer 2

まずはメソッドの実装を return null; のみにしてみましょう。
その後に、実装部分をある程度のまとまり毎に張り付けてみてはどうでしょうか。