質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.61%

DAOでUPDATE文を使うが、反映されない

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 660

初めに

データベースにアクセスして、既存のテーブルの日付を入れる部分のnullに、updateで更新したい。

環境

windows10
eclipse
Tomcat 9

ソースコード

サーブレット

public class DocumentInfoInServlet extends HttpServlet {
    private returnInfoInEntity entity;
    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        try {
            //文字コード指定
            request.setCharacterEncoding("UTF-8");
            //actionを取得
            String action = request.getParameter("action");

            //資料返却DAO
            DocumentInfoInDAO documentInfoIndao = new DocumentInfoInDAO();
            //確認ボタン
            if(action.equals("documentInfo")) {
                int member_id = Integer.parseInt(request.getParameter("member_id"));
                int doc_id = Integer.parseInt(request.getParameter("doc_id"));
                String return_date_year = request.getParameter("return_date_year");
                String return_date_month = request.getParameter("return_date_month");
                String return_date_day = request.getParameter("return_date_day");

                //日付の連結とsql型の日付に変換
                String date = return_date_year+"-"+return_date_month+"-"+return_date_day;
                Date return_date = Date.valueOf(date);

                entity = new returnInfoInEntity(member_id,doc_id,return_date);
                request.setAttribute("returnInfoIn",entity);
                gotoPage(request, response, "returnInfoConfirmation.jsp");
            }else if(action.equals("returnDocument")) {
                documentInfoIndao.returnDocument(entity.getMember_id(),entity.getDoc_id(),entity.getReturn_date());
                //request.setAttribute("confirm",entity2);
                gotoPage(request, response, "documentInfoInComp.jsp");
            }else if(action.equals("documentInfoback")) {
                gotoPage(request, response, "lind.jsp");
            }else if(action.equals("returncomp")) {
                gotoPage(request, response, "lind.jsp");
            }


        }catch(NumberFormatException e) {
            e.printStackTrace();
            JOptionPane.showMessageDialog(null, "入力してください","エラー",JOptionPane.ERROR_MESSAGE);
            gotoPage(request, response, "documentInfoIn.jsp");
            }
    }
    protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        // TODO Auto-generated method stub
        doGet(request, response);
    }
    private void gotoPage(HttpServletRequest request, HttpServletResponse response, String page) throws ServletException, IOException {
        RequestDispatcher rd = request.getRequestDispatcher(page);
        rd.forward(request, response);

    }

DAO

import java.sql.Connection;
import java.sql.Date;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.SQLException;

public class DocumentInfoInDAO {
    private Connection con;
    public DocumentInfoInDAO(){
        getConnection();
    }
    public void returnDocument(int member_id,int doc_id,Date return_date) {
        if(con == null)
            getConnection();
        PreparedStatement st = null;
        try {
            //返却日を入力

            String sql="UPDATE rental set return_date = ? where member_id = ? AND doc_id = ?;";
            st=con.prepareStatement(sql);
            st.setDate(1, return_date);
            st.setInt(2, member_id);
            st.setInt(3, doc_id);
            }catch(Exception e) {
            e.printStackTrace();
        }finally {
            try {
                if(st != null) st.close();
                close();
            }catch(Exception e) {
                e.printStackTrace();
            }
        }
    }
    private void getConnection(){
        try {
            Class.forName("org.postgresql.Driver");

            String url = "jdbc:postgresql:sample";
            String user = "postgres";
            String pass = "sample";

            con = DriverManager.getConnection(url, user, pass);

        }catch(Exception e) {

        }
    }

    private void close() throws SQLException{
        if(con!=null) {
            con.close();
            con=null;
        }
    }

試したこと

このコードを実行後、コマンドプロンプトでselect文で繁栄されているかを確認すると、
更新されていない。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

パラメータをセットした後に実行していないため、UPDATEが行われていないのだと思います。

String sql="UPDATE rental set return_date = ? where member_id = ? AND doc_id = ?;";
st=con.prepareStatement(sql);
st.setDate(1, return_date);
st.setInt(2, member_id);
st.setInt(3, doc_id);

// この行が抜けている
st.executeUpdate();

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/06/16 23:20

    こんなことに気づかなかったなんて、、
    有り難うございました。。
    思い込みで確認は良くないですね、、

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る