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

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

ただいまの
回答率

89.13%

データベースの内容をjspから受け取ったデータに更新したい

解決済

回答 2

投稿 編集

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

kokok

score 111

<%@ page language="java" contentType="text/html; charset=UTF-8"
    pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>password</title>
</head>
<body>
<form action="/webkensyu/pass"  method="get">
<p><label>id<input type="text" name="text1"></label></p><br>
<p><label>ユーザー名<input type="text" name="text2"></label></p><br>
<p><label>新しいパスワード<input type="text" name ="text3"></label></p>
<button type="submit">確認</button>
</form>
</body>
</html>
package servlet;

import java.io.IOException;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

import dao.DBIO;

/**
 * Servlet implementation class pass
 */
@WebServlet("/pass")
public class pass extends HttpServlet {
    private static final long serialVersionUID = 1L;


    public pass() {
        super();
        // TODO Auto-generated constructor stub
    }


    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        request.setCharacterEncoding("UTF-8");
        String text1 = request.getParameter("text1");
        String text2 = request.getParameter("text2");
        String text3 = request.getParameter("text3");
        DBIO dbio = new DBIO();
        boolean result = dbio.Passchange(text1,text2,text3);

        if(result == true) {
            //成功すればパスワード変更

            RequestDispatcher dispatcher = request.getRequestDispatcher("/passchange.jsp");
            dispatcher.forward(request,  response);
        }else {
            //失敗すればもう一度パスワード変更画面へ
            RequestDispatcher dispatcher = request.getRequestDispatcher("/password.jsp");
            dispatcher.forward(request,  response);
        }
    }

    protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        // TODO Auto-generated method stub
        doGet(request, response);
    }

}
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;

public class DBIO {

    private static final String DBNAME = "jdbc:mysql://localhost:3306/webkensyu?useUnicode =true&characterEncoding=utf8";
    private static final String DBUSER = "user";
    private static final String DBPASS = "pass";
    private Connection conn;


    public DBIO(){
    this.connect();
}

    public void connect() {
    try {

        Class.forName("com.mysql.jdbc.Driver");

        conn = DriverManager.getConnection(DBNAME,DBUSER,DBPASS);


    }    catch(SQLException e) {

        e.printStackTrace();

    }catch(ClassNotFoundException e) {

        e.printStackTrace();

    }
  }


    //ユーザー一覧表示
    public ArrayList<String> getUser(){

        ArrayList<String> result = new ArrayList<String>();

        try {
        this.connect();

        String sql ="SELECT user_name FROM users";
        PreparedStatement pstmt = conn.prepareStatement(sql);
        ResultSet rs = pstmt.executeQuery();

        while(rs.next()) {

            result.add(rs.getString("user_name"));    
        }        
        }catch(SQLException e) {

            e.printStackTrace();
        }finally {

            this.close();
        }


        return result;
    }



    //パスワード変更
    public boolean Passchange(String id, String name, String pass){

        try{
            this.connect();
            String sql ="UPDATE users SET password = ? WHERE user_id = ? && user_name = ?";
            PreparedStatement pstmt = conn.prepareStatement(sql);
            pstmt.setString(1, pass);
            pstmt.setString(2, id);
            pstmt.setString(3, name);
            ResultSet rs = pstmt.executeQuery();
        if(rs != null) {
            return true;
        }else {
            return false;
        }
        }catch (SQLException e) {

            e.printStackTrace();
        }finally {

            this.close();
        }        

        return false;
    }

    //ログインで入力されたユーザー名、パスワードの確認
    public boolean Authentication(String name, String pass) {
        try {

            this.connect();    

            String sql ="SELECT user_name,password FROM users WHERE user_name = ? && password = ? ";
            PreparedStatement pstmt = conn.prepareStatement(sql);
            pstmt.setString(1, name);
            pstmt.setString(2, pass);
            ResultSet rs = pstmt.executeQuery();

        if(rs != null) {
            return true;
        }else {
            return false;
        }
        }catch (SQLException e) {

            e.printStackTrace();
        }finally {

            this.close();
        }        
        return false;
    }



    public void close() {

        if(conn != null) {
            try {
                conn.close();
            }catch(SQLException e) {
                e.printStackTrace();            
            }
        }
    }        
}

jspから渡されたパスワードでデータベースのパスワードを更新したいのですが(ログインしている人の)
実行すると

at dao.DBIO.Passchange(DBIO.java:82) ResultSet rs =pstmt.executeQuery();
at servlet.pass.doGet(pass.java:38)boolean result =dbio.Passchange(text1,text2,text3);

とエラーが出てしまいます。
解決出来なかったのでアドバイス頂けると助かります

    //パスワード変更
    public boolean Passchange(String id, String name, String pass){

        try{
            this.connect();
            String sql ="UPDATE users SET password = ? WHERE user_id = ? && user_name = ?";
            PreparedStatement pstmt = conn.prepareStatement(sql);
            pstmt.setString(1, pass);
            pstmt.setString(2, id);
            pstmt.setString(3, name);
            ResultSet rs = pstmt.executeQuery();
        if(rs != null) {
            return true;
        }else {
            return false;
        }
        }catch (SQLException e) {

            e.printStackTrace();
        }finally {

            this.close();
        }        

        return false;
    }

--追記--

//ユーザーの作成
    public int Make_user(String id, String name, String pass) {
        try {

            this.connect();
            String sql ="INSERT INTO users(user_id,user_name,password)VALUES(?,?,?)";
            PreparedStatement pstmt = conn.prepareStatement(sql);
            pstmt.setString(1, id);
            pstmt.setString(2, name);
            pstmt.setString(3, pass);
            int rs = pstmt.executeUpdate(sql);
            if(rs == 1) {
                return 0;
            }else {

                return -1;
            }

        }catch (SQLException e) {

            e.printStackTrace();
        }finally {

            this.close();
        }        

        return -1;
    }


新しくユーザー作成機能を作ろうとしたのですが

at dao.DBIO.Make_user(DBIO.java:137) int rs = pstmt.executeUpdate(sql);

at servlet.Makeuser.doGet(Makeuser.java:40) int result = dbio.Make_user(text1,text2,text3); (サーブレット呼び出し側)

のところでエラーが出てしまいます。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • tetutetu

    2019/09/04 13:51

    at dao.DBIO.Passchange...(略
    はどこでエラーが発生しているかを示しているだけで、何のエラーか示されている部分ではありません。
    エラーメッセージの読み方も学んでいきましょう。

    キャンセル

回答 2

+2

ResultSet rs = pstmt.executeQuery();


executeQueryはselect文で使います。
UPDATEやDELETEの際はexecuteUpdate()です。

executeUpdateは戻り値に数値を返しますので、ResultSetも必要ありません。
更新や削除などの処理が行われた件数が返ってきます。

int hoge = pstmt.executeUpdate();

ついでに

ユーザー認証が必要な何らかのWEBアプリを作成しているものとお見受けしますが
パスワード変更処理をdoGetメソッドにやらせるのは流石に如何なものかと思います。
Get,Postでそれぞれどのような処理をすべきか調べてみましょう。
(あと、変更するにあたって旧パスワードの入力は求めないのでしょうか)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/04 13:57

    回答ありがとうございます。
    executeQueryをexecuteUpdate()に変更したところ実行できました。

    パスワードはpostにするべきでした。
    旧パスワードの入力も作成思います。
    ありがとうございます。

    キャンセル

check解決した方法

-2

executeUpdate() は 引数なしでした。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/04 15:07

    最初からそのように書いてあるのですが。

    キャンセル

  • 2019/09/04 15:17

    蛇足ですがもう一点。
    恐らく研修中とのことで、ローカル環境で開発をされているものかと思われますが
    クラス名やメソッド名、変数名等には、ネーミング規約(コーディング規約、命名規約)というものが存在します。
    今のうちに規約に沿った書き方ができるようにしましょう。

    キャンセル

  • 2019/09/04 16:44

    申し訳ないです。自分で色々書いてたら間違えてしまいました。

    勉強していきたいと思います

    キャンセル

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

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