BO、DAOのサンプルコードで使用されているメソッドの戻り値型について

Question

前提・実現したいこと

java初心者です。

下記コードで、
import元のDAOではcreate()の戻り値型はbooleanになっているのですが、
使用されているBOのexecute()はなぜ戻り値がないのでしょうか？

ものすごく初歩的な所でつまずいてる様な気もしますが、
今は、思い当たりません。

ご教示お願いいたします。

その他、
記載方法に改善点がある場合は、ご指摘下さいませ。

該当のソースコード

MainのdoPostで使用するBO

package model;

import dao.MutterDAO;

public class PostMutterLogic {
  public void execute(Mutter mutter) { // 引数を1つに変更
    MutterDAO dao = new MutterDAO();
    dao.create(mutter);
  }
}
```DB保存された投稿内容をリスト格納して返す **findAll()** と
投稿内容を**DB**に**INSERT**する **create()** を持つ**DAO**
コード
```java
package dao;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.List;

import model.Mutter;

public class MutterDAO {

  private final String DRIVER_NAME = "org.h2.Driver";
  private final String JDBC_URL =
      "jdbc:h2:file:C:/data/docoTsubu";
  private final String DB_USER = "sa";
  private final String DB_PASS = "";

  public List<Mutter> findAll() {
    Connection conn = null;
    List<Mutter> mutterList = new ArrayList<Mutter>();
    try {
      Class.forName(DRIVER_NAME);
      conn = DriverManager.getConnection(
          JDBC_URL, DB_USER, DB_PASS);

      // SELECT文の準備
      String sql =
          "SELECT ID,NAME,TEXT FROM MUTTER ORDER BY ID DESC";
      PreparedStatement pStmt = conn.prepareStatement(sql);

      // SELECTを実行
      ResultSet rs = pStmt.executeQuery();

      // SELECT文の結果をArrayListに格納
      while (rs.next()) {
        int id = rs.getInt("ID");
        String userName = rs.getString("NAME");
        String text = rs.getString("TEXT");
        Mutter mutter = new Mutter(id, userName, text);
        mutterList.add(mutter);
      }
    } catch (SQLException e) {
      e.printStackTrace();
      return null;
    } catch (ClassNotFoundException e) {
      e.printStackTrace();
      return null;
    } finally {
      // データベース切断
      if (conn != null) {
        try {
          conn.close();
        } catch (SQLException e) {
          e.printStackTrace();
          return null;
        }
      }
    }
    return mutterList;
  }

  public boolean create(Mutter mutter) {
    Connection conn = null;
    try {
      // データベースへ接続
      conn = DriverManager.getConnection(
          JDBC_URL, DB_USER, DB_PASS);

      // INSERT文の準備(idは自動連番なので指定しなくてよい）
      String sql = "INSERT INTO MUTTER(NAME, TEXT) VALUES(?, ?)";
      PreparedStatement pStmt = conn.prepareStatement(sql);
      // INSERT文中の「?」に使用する値を設定しSQLを完成
      pStmt.setString(1, mutter.getUserName());
      pStmt.setString(2, mutter.getText());

      // INSERT文を実行
      int result = pStmt.executeUpdate();

      if (result != 1) {
        return false;
      }
    } catch (SQLException e) {
      e.printStackTrace();
      return false;
    } finally {

      // データベース切断
      if (conn != null) {
        try {
          conn.close();
        } catch (SQLException e) {
          e.printStackTrace();
        }
      }
    }
    return true;
  }
}

Comments

[m.ts10806]

ご自身で組まれたのではないのでしょうか。
出所を明記してください。

[退会済みユーザー]

mts 10806 様
失礼致しました、
インプレス
スッキリわかるサーブレット＆JSP入門
です。

[m.ts10806]

質問本文に追記願います
書籍であれば版も

[jimbe]

なぜ execute に戻り値が必要とお考えなのでしょうか.

Answer 1

おそらく参考されている書籍のコードの問題ではあると思うんですが、

JDBCから実行したSQLのINSERT処理は、追加に成功したレコード数が返されます。
https://docs.oracle.com/javase/jp/8/docs/api/java/sql/PreparedStatement.html#executeUpdate--

明らかに1件のみ追加したのに0件が返される場合は、殆どの場合が一意制約違反か、SQLの誤りか、DB接続の例外でしょう。
そのため当該ソースでは、0件が返ってくるときに false を、例外のときも false を返しているので、特に true/false かを判断せず、結果も変わらないと扱って 戻りをvoidにしているのではないかと。

Comments

[退会済みユーザー]

A-pZ 様
コメントが遅くなり申し訳ありません！
丁寧にご回答いただき有難うございました。
DAO,BO,サーブレットとの繋がりがイメージ出来ておらず、
意味のわからない質問ですみませんでした。
お陰様で腑に落ちて理解する事が出来ました。
有難うございました。