【Java】DAOとBeanを用いてUPDATE機能を実装したいのですが...

Question

いつもお世話になっております。

結論から申し上げますと、DAOのwhile文内およびModelの「dao.execute();」以降の記述方法が分からない状況です。
Output用のBeanも間違っているような気もしております...。

システムとしては、IDを入力して検索ボタンを押せばデータベース上にある既存の情報が表示され、その後修正したい項目を書き直して更新ボタンを押せば情報が更新される...といったものを実装しようとしております。

上記のシステムについては、Modelのみであれば既に実装は出来たのですが、更に実践的にするため、DAOとBeanを用いて実装しようとしているものの、更新機能云々の前にID検索後に既存情報が表示できないというところで詰まっています。

丸投げの質問になり大変恐縮ではございますが、ご教授いただける方がいらっしゃいましたら幸いです。
もし他に必要な情報がございましたらお申し付けください。

以下、汚いコードで大変申し訳ございませんが、一読いただければと思います。

該当のソースコード

package estimatePackage.bean;

public class CompanyUpdateInputBean {
	
	// Input
	private String company_update_id = "";

	public String getCompany_update_id() {
		return company_update_id;
	}
	public void setCompany_update_id(String cui) {
		this.company_update_id = cui;
	}

}

package estimatePackage.bean;

public class CompanyUpdateOutputBean {
	
	// Output
	private String company_update_id;
	private String company_update_name;
	private String company_update_phone;
	private String company_update_mail;
	private String company_update_address;

	public String getCompany_update_id() {
		return company_update_id;
	}
	public void setCompany_update_id(String cui) {
		this.company_update_id = cui;
	}
	public String getCompany_update_name() {
		return company_update_name;
	}
	public void setCompany_update_name(String cun) {
		this.company_update_name = cun;
	}
	public String getCompany_update_phone() {
		return company_update_phone;
	}
	public void setCompany_update_phone(String cup) {
		this.company_update_phone = cup;
	}
	public String getCompany_update_mail() {
		return company_update_mail;
	}
	public void setCompany_update_mail(String cum) {
		this.company_update_mail = cum;
	}
	public String getCompany_update_address() {
		return company_update_address;
	}
	public void setCompany_update_address(String cua) {
		this.company_update_address = cua;
	}
	
}

package Dao;

import java.sql.ResultSet;
import java.sql.Statement;

import cmmon.CommonConnection;
import estimatePackage.bean.CompanyUpdateInputBean;
import estimatePackage.bean.CompanyUpdateOutputBean;

public class CompanyUpdateDao {
	
	private CompanyUpdateInputBean input = new CompanyUpdateInputBean() ;
	private CompanyUpdateOutputBean output = new CompanyUpdateOutputBean();

	public CompanyUpdateDao() {
		
	}
	
	public void execute() throws Exception {
		try {
			
			// データベースに接続
			CommonConnection con = new CommonConnection();
			// ステートメント作成
			Statement stmt = con.getStmt();
			//SQL文の実行結果を格納
			ResultSet rs = stmt.executeQuery(getSqlPreview());
			
			while (rs.next()) {
				CompanyUpdateOutputBean bean = new CompanyUpdateOutputBean();
				bean.setCompany_update_id(rs.getString("ID"));
				bean.setCompany_update_name(rs.getString("NAME"));
				bean.setCompany_update_phone(rs.getString("PHONE"));
				bean.setCompany_update_mail(rs.getString("MALL"));
				bean.setCompany_update_address(rs.getString("ADLESS"));
			}
			
			rs.close();
			stmt.close();
			
		} catch (Exception e) {
			e.printStackTrace();
			throw e;
		}
	}
	
	private String getSqlPreview() {
		StringBuffer sb = new StringBuffer();
		
		sb.append("SELECT ");
		sb.append("ID, NAME, PHONE, MALL, ADLESS ");
		sb.append("FROM COMPANY_TABLE ");
		sb.append("WHERE NAME ");
		sb.append("LIKE '%" + input.getCompany_update_id() + "%'");
		
		return sb.toString();
	}
	
	public CompanyUpdateInputBean getInput() {
		return input;
	}

	public void setInput(CompanyUpdateInputBean input) {
		this.input = input;
	}

	public CompanyUpdateOutputBean getOutput() {
		return output;
	}

	public void setOutput(CompanyUpdateOutputBean output) {
		this.output = output;
	}
}

以下のModel内のコメントアウト(※try文)部分が今回DAOとBeanで実装したい部分になります。
※SQL文内の誤字(MALLやADLESS等)は無視してください。
※文字数制限のため、一部の箇所は削除しておりますのでご了承ください

package estimatePackage;

import Dao.CompanyUpdateDao;
import cmmon.CommonConnection;
import estimatePackage.bean.CompanyUpdateInputBean;


public class PartnerUpdatePageFormAction extends EditFormAction {
	private String input_company_id;
	private String partnerUpdateResultPage;
	private String backTopPage;
	private String editId;
	private String editName;
	private String editPhone;
	private String editMail;
	private String editAddress;
	private String partnerEdit;
	private String messageArea;

	public String execute() throws Exception {

		
		String result = "";
		if (isNotNull(partnerUpdateResultPage)) {
			result= "returnPartnerEditResultPage1";
		}

		if (isNotNull(partnerEdit)) {
			result= "returnPartnerEditResultPage2";
		}

		if (isNotNull(backTopPage)) {
			result= "returnBackTopPage";
			return result;
		}

		// 検索 or 更新処理
		if (isNotNull(partnerUpdateResultPage)) {
			// 検索の場合
			partnerSelect();

		} else if (isNotNull(partnerEdit)) {
			// 更新の場合
			partnerUpdate();
		}

		return result;

	}

    private void partnerSelect() throws Exception {
    	
    	CompanyUpdateInputBean companyUpdateInputBean = new CompanyUpdateInputBean();
    	companyUpdateInputBean.setCompany_update_id(input_company_id);
    	
    	CompanyUpdateDao dao = new CompanyUpdateDao();
    	dao.setInput(companyUpdateInputBean);
    	dao.execute();
    	
    	
//		try {
////			// データベースに接続
//			CommonConnection con = new CommonConnection();
//			Statement stmt = con.getStmt();
//
//			// SQL文発行
//			String SQL = "SELECT ID, NAME, PHONE, MALL, ADLESS FROM COMPANY_TABLE WHERE ID =" + input_company_id;
//
//			//SQL文の実行結果を格納
//			ResultSet rs = stmt.executeQuery(SQL);
//
//			while (rs.next()) {
//				//        String[] companyist
//				editId = "" + rs.getInt("ID");
//				editName = rs.getString("NAME");
//				editPhone = rs.getString("PHONE");
//				editMail = rs.getString("MALL");
//				editAddress = rs.getString("ADLESS");
//			}
//
//			rs.close();
//			stmt.close();
//
//		}
//		catch (Exception e) {
//			e.printStackTrace();
//			throw e;
//		}
    }

	public String getInput_company_id() {
		return input_company_id;
	}

	public void setInput_company_id(String input_company_id) {
		this.input_company_id = input_company_id;
	}

	public String getPartnerUpdateResultPage() {
		return partnerUpdateResultPage;
	}

	public void setPartnerUpdateResultPage(String purp) {
		this.partnerUpdateResultPage = purp;
	}

	public String getBackTopPage() {
		return backTopPage;
	}

	public void setBackTopPage(String btp) {
		this.backTopPage = btp;
	}

	public String getEditId() {
		return editId;
	}

	public void setEditId(String editId) {
		this.editId = editId;
	}

	public String getEditName() {
		return editName;
	}

	public void setEditName(String editName) {
		this.editName = editName;
	}

	public String getEditPhone() {
		return editPhone;
	}

	public void setEditPhone(String editPhone) {
		this.editPhone = editPhone;
	}

	public String getEditMail() {
		return editMail;
	}

	public void setEditMail(String editMail) {
		this.editMail = editMail;
	}

	public String getEditAddress() {
		return editAddress;
	}

	public void setEditAddress(String editAddress) {
		this.editAddress = editAddress;
	}

	public String getPartnerEdit() {
		return partnerEdit;
	}

	public void setPartnerEdit(String partnerEdit) {
		this.partnerEdit = partnerEdit;
	}

	public String getMessageArea() {
		return messageArea;
	}

	public void setMessageArea(String messageArea) {
		this.messageArea = messageArea;
	}
}

<%@ page language="java" contentType="text/html; charset=UTF-8"
	pageEncoding="UTF-8"%>
<%@ taglib prefix="s" uri="/struts-tags"%>

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>企業検索2</title>
<link rel="stylesheet" type="text/css"
	href="/EstimateSystem/StyleSheet/style.css">
</head>

<body>
	<header>
		<jsp:include page="header_parts.jsp" flush="true" />
	</header>

	<section>
		<div class="save-message-wrapper">
			<div class="save-message-container">
				<h2>
					<s:property value="messageArea" />
				</h2>
			</div>
		</div>

		
	</section>

	<s:form action="PartnerUpdatePageFormAction">
		<section>
			<div class="partner-wrapper">

				<div class="partner-container">
					<div class="partner-name">会社ID</div>
					<div class="partner-input">
						<s:textfield class="input-form" name="input_company_id" />
					</div>

					<div class="partner-btn-area">
						<s:submit value="検索" name="partnerUpdateResultPage" class="btn" />
						<s:submit value="戻る" name="backTopPage" class="btn" />
					</div>
				</div>
			</div>
		</section>

		<section>
			<div class="partner-info-save-wrapper">
				<div class="partner-info-save-container">
					<div class="partner-input-area">
						<table>
							<tr>
								<th>ID</th>
								<td><s:textfield class="input-form" name="editId"
										placeholder="ID" /></td>
							</tr>
							<tr>
								<th>会社名</th>
								<td><s:textfield class="input-form" name="editName"
										placeholder="会社名" /></td>
							<tr>
							<tr>
								<th>電話番号</th>
								<td><s:textfield class="input-form" name="editPhone"
										placeholder="電話番号" /></td>
							</tr>
							<tr>
								<th>メールアドレス</th>
								<td><s:textfield class="input-form" name="editMail"
										placeholder="メールアドレス" /></td>
							</tr>
							<tr>
								<th>住所</th>
								<td><s:textfield class="input-form" name="editAddress"
										placeholder="住所" /></td>
							</tr>
						</table>
					</div>

					<div class="partner-btn-area">
						<s:submit value="更新" name="partnerEdit" class="btn" />
					</div>
				</div>
			</div>
		</section>
	</s:form>

	<footer>
		<jsp:include page="footer_parts.jsp" flush="true" />
	</footer>

</body>
</html>

Comments

[dodox86]

そもそもですが、DAOやBeanにする前のコメント化されたモデル中のデータ取得部分で、これはちゃんと要望通り動いていたのでしょうか。
SELECTのSQLで複数のレコードが返るようなコードで、while (rs.next()) でループして回っているにしてはrs.getほにゃらら()で上書きして、最後の1件しか取得できていない気がするのですが。その問題はそのまま質問文中の書きかけのDAO、Beanに引き継がれているように見えます。

Answer 1

SQL の WHERE 節が変わってしまっていますし、output の bean は作っているだけでどこにも行きません。

どのようにデータを取得しどのように結果まで伝えていくかはまず当事者が決めることですから、二つのコードで「こっちとそっちはココが違う」とは指摘できますが「ここからどうすれば…」は無関係の他人には決められません。