前提・実現したいこと
簡易掲示板を作成し、ログイン後に登録情報を設定画面で変更する際、パスワードやIDが未入力の場合は現状維持(現在のパスワードやIDを保持)させたいです。現在は、エラーメッセージが出るようになっています。1000文字以内ということなのでサーブレットは載せていません。
発生している問題・エラーメッセージ
エラーメッセージを出さないようにすることはできたのですが、未入力の場合空文字がパスワードに設定されてしまいます。SQLのUpdateやgetUserのメソッドの書き換えをしているうちに迷走してしまいました。
該当のソースコード
package chapter6.service; import static chapter6.utils.CloseableUtil.*; import static chapter6.utils.DBUtil.*; import java.sql.Connection; import chapter6.beans.User; import chapter6.dao.UserDao; import chapter6.utils.CipherUtil; public class UserService { public void register(User user) { Connection connection = null; try { connection = getConnection(); String encPassword = CipherUtil.encrypt(user.getPassword()); user.setPassword(encPassword); UserDao userDao = new UserDao(); userDao.insert(connection, user); commit(connection); } catch (RuntimeException e) { rollback(connection); throw e; } catch (Error e) { rollback(connection); throw e; } finally { close(connection); } } public User getUser(int userId) { Connection connection = null; try { connection = getConnection(); UserDao userDao = new UserDao(); User user = userDao.getUser(connection, userId); commit(connection); return user; } catch (RuntimeException e) { rollback(connection); throw e; } catch (Error e) { rollback(connection); throw e; } finally { close(connection); } } public void update(User user) { Connection connection = null; try { connection = getConnection(); String encPassword = CipherUtil.encrypt(user.getPassword()); user.setPassword(encPassword); UserDao userDao = new UserDao(); userDao.update(connection, user); commit(connection); } catch (RuntimeException e) { rollback(connection); throw e; } catch (Error e) { rollback(connection); throw e; } finally { close(connection); } } }
// DAOクラス package chapter6.dao; import static chapter6.utils.CloseableUtil.*; import java.sql.Connection; import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; import java.sql.Timestamp; import java.util.ArrayList; import java.util.List; import chapter6.beans.User; import chapter6.exception.NoRowsUpdatedRuntimeException; import chapter6.exception.SQLRuntimeException; public class UserDao { public void insert(Connection connection, User user) { PreparedStatement ps = null; try { StringBuilder sql = new StringBuilder(); sql.append("INSERT INTO users ( "); sql.append("account"); sql.append(", name"); sql.append(", email"); sql.append(", password"); sql.append(", description"); sql.append(", created_date"); sql.append(", updated_date"); sql.append(") VALUES ("); sql.append("?"); // account sql.append(", ?"); // name sql.append(", ?"); // email sql.append(", ?"); // password sql.append(", ?"); // description sql.append(", CURRENT_TIMESTAMP"); // created_date sql.append(", CURRENT_TIMESTAMP"); // updated_date sql.append(")"); ps = connection.prepareStatement(sql.toString()); ps.setString(1, user.getAccount()); ps.setString(2, user.getName()); ps.setString(3, user.getEmail()); ps.setString(4, user.getPassword()); ps.setString(5, user.getDescription()); ps.executeUpdate(); } catch (SQLException e) { throw new SQLRuntimeException(e); } finally { close(ps); } } public User getUser(Connection connection, String accountOrEmail, String password) { PreparedStatement ps = null; try { String sql = "SELECT * FROM users WHERE (account = ? OR email = ?) AND password = ?"; ps = connection.prepareStatement(sql); ps.setString(1, accountOrEmail); ps.setString(2, accountOrEmail); ps.setString(3, password); ResultSet rs = ps.executeQuery(); List<User> userList = toUserList(rs); if (userList.isEmpty() == true) { return null; } else if (2 <= userList.size()) { throw new IllegalStateException("2 <= userList.size()"); } else { return userList.get(0); } } catch (SQLException e) { throw new SQLRuntimeException(e); } finally { close(ps); } } public User getUser(Connection connection, int id) { PreparedStatement ps = null; try { String sql = "SELECT * FROM users WHERE id = ?"; ps = connection.prepareStatement(sql); ps.setInt(1, id); ResultSet rs = ps.executeQuery(); List<User> userList = toUserList(rs); if (userList.isEmpty() == true) { return null; } else if (2 <= userList.size()) { throw new IllegalStateException("2 <= userList.size()"); } else { return userList.get(0); } } catch (SQLException e) { throw new SQLRuntimeException(e); } finally { close(ps); } } private List<User> toUserList(ResultSet rs) throws SQLException { List<User> ret = new ArrayList<User>(); try { while (rs.next()) { int id = rs.getInt("id"); String account = rs.getString("account"); String name = rs.getString("name"); String email = rs.getString("email"); String password = rs.getString("password"); String description = rs.getString("description"); Timestamp createdDate = rs.getTimestamp("created_date"); Timestamp updatedDate = rs.getTimestamp("updated_date"); User user = new User(); user.setId(id); user.setAccount(account); user.setName(name); user.setEmail(email); user.setPassword(password); user.setDescription(description); user.setCreatedDate(createdDate); user.setUpdatedDate(updatedDate); ret.add(user); } return ret; } finally { close(rs); } } public void update(Connection connection, User user) { PreparedStatement ps = null; try { StringBuilder sql = new StringBuilder(); sql.append("UPDATE users SET"); sql.append(" account = ?"); sql.append(", name = ?"); sql.append(", email = ?"); sql.append(", password = ?"); sql.append(", description = ?"); sql.append(", updated_date = CURRENT_TIMESTAMP"); sql.append(" WHERE"); sql.append(" id = ?"); ps = connection.prepareStatement(sql.toString()); ps.setString(1, user.getAccount()); ps.setString(2, user.getName()); ps.setString(3, user.getEmail()); ps.setString(4, user.getPassword()); ps.setString(5, user.getDescription()); ps.setInt(6, user.getId()); int count = ps.executeUpdate(); if (count == 0) { throw new NoRowsUpdatedRuntimeException(); } } catch (SQLException e) { throw new SQLRuntimeException(e); } finally { close(ps); } } }
試したこと
教材を読み進めてここまできました。サーブレットのif文を消すと空文字が設定できるのは分かるのですが、DAOとServiceクラスの変更点がいまいち分かりません。また、JSPのpostmethodから受け取った値はどこへいっているのでしょうか?
初心者の質問ですが、よろしくお願いします。
補足情報(FW/ツールのバージョンなど)
Tomcat7.0
java 8
回答1件
あなたの回答
tips
プレビュー