掲示板入力フォームチェック後の投稿処理判定

##背景・実現したいこと
javaを使って、簡易掲示板を作成しています。
ところが、入力チェックのロジックは完成しているのですが、エラーメッセージがあるときだけ、
投稿処理を停止する方法がわかりません。どのようにロジックを書いたら良いでしょうか？
下記にソースコードを記載しておきます。アドバイスを下さい。よろしくお願い致します。

P.S ちなみに「doBBSWrite(request, response, bbsData);」をコメントアウトまたは
削除すると、投稿処理ができません。

--------ソースコード(一部)---------
import java.io.;
import javax.servlet.;
import javax.servlet.http.*;

public class BBSServlet extends HttpServlet {

///////////////////// メンバ変数 ///////////////////

ServletContext sc;
BBSManagerBeans bbsManager;

///////////////////// 初期化 ///////////////////

public void init(ServletConfig config)
throws ServletException
{
super.init(config);
sc = getServletContext();

String dataPath = config.getInitParameter("datapath");   
if (dataPath == null) {   
  dataPath = "";   
}

bbsManager = (BBSManagerBeans)sc.getAttribute("bbsmanager");   
if (bbsManager == null) {   
  bbsManager = new BBSManagerBeans(dataPath);   
  sc.setAttribute("bbsmanager",bbsManager);   
}

}

///////////////////// "POST"メソッド ///////////////////

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

///////////////////// "GET"メソッド ///////////////////

public void doGet(HttpServletRequest request, HttpServletResponse response)
throws IOException, ServletException
{
BBSDataBeans bbsData = new BBSDataBeans();

String data = request.getParameter("function");   
if (data == null) {   
  doBBSRead(request, response, bbsData);   
}   
else if (data.compareTo("WRITE") == 0) {   
  doBBSWrite(request, response, bbsData);   
}   
else if (data.compareTo("DELETE") == 0) {   
  doBBSDelete(request, response, bbsData);   
}

request.setAttribute("bbsdata",bbsData);   
sc.getRequestDispatcher("/bbs.jsp").forward(request, response);

}

bbsManager.DeleteItem(index, password);   
bbsManager.ReadArticles(0, bbsData);

}
///////////////////// メッセージの投稿 ///////////////////

public void doBBSWrite(HttpServletRequest request, HttpServletResponse response, BBSDataBeans bbsData)
throws IOException, ServletException
{
BBSItem item = new BBSItem();

item.name     = KanjiConv.ConvJStr(request.getParameter("name"));   
if (item.name == null || item.name.length() == 0){
  err = "名前が入力されていません" ;
  request.setAttribute("err",err);
  }

}

yuji38kwmt

2016/06/12 03:40

余談なんですが、ソースコードを```:java で括ると、色づけされて見やすいです。

行動規範の内容に同意します

回答1件

doBBSWriteメソッドでエラーならばfalseを返すようにして、呼び出し元で判定してエラーページに遷移させるのはいかがでしょうか？

java
1public boolean doBBSWrite(HttpServletRequest request, HttpServletResponse response, BBSDataBeans bbsData)   
2      throws IOException, ServletException   
3  {   
4    BBSItem item = new BBSItem();
5
6    item.name     = KanjiConv.ConvJStr(request.getParameter("name"));   
7    if (item.name == null || item.name.length() == 0){
8      err = "名前が入力されていません" ;
9      request.setAttribute("err",err);
10      return false;
11    }
12     return true;
13
14  }

java
1if (!doBBSDelete(request, response, bbsData)) { //入力チェック失敗
2  //エラーページに遷移
3  sc.getRequestDispatcher("/error.jsp").forward(request, response);
4}