ソースコードの見直しをお願いします!
1番目の大メソッドをboolean selectじゃなくてdto.selectにするべきだとか、
insertのメソッドのint resultにps.executeUpdate()を入れるべきだとか言われたのですが、具体的に何をすればいいのか全く分からないのでこちらに質問させていただきました。ご回答お願いします!
Java
1package com.internousdev.uesto.dao; 2 3import java.sql.Connection; 4import java.sql.PreparedStatement; 5import java.sql.ResultSet; 6import java.sql.SQLException; 7 8import com.internousdev.uesto.dto.LoginOauthDTO; 9import com.internousdev.uesto.util.DBConnector; 10 11 12/** 13 * Oauthでログインに必要な情報を取得する為のDAOクラス 14 * @author 15 * @since 2016/04/05 16 * @version 1.0 17 */ 18public class LoginOauthDAO{ 19 20 /** 21 * データベースに接続するためのインターフェース 22 */ 23 private Connection con; 24 25 /** 26 * SQL文 27 */ 28 private String sql; 29 30 /** 31 * SQL文を実行するためのインターフェース 32 */ 33 private PreparedStatement ps; 34 35 /** 36 * 結果を格納するインターフェース 37 */ 38 private ResultSet rs; 39 40 /** 41 * 結果を格納するDTOクラス 42 */ 43 private LoginOauthDTO dto=new LoginOauthDTO(); 44 /** 45 * <p>データベース(mysql)に接続し、取得した値(userUniqueId,oauthName)がテーブル(user)に 46 * 登録されているか判定するためのメソッド</p> 47 * @param userUniqueId OAuthのサービス先のユニークID 48 * @param oauthName OAuthのサービス名 49 * @return true/false <p>登録されていればtrueを返し、ユーザーID・ユーザーNAMEを取得しDTOに格納します。 50 * 登録されていなければfalseを返します。</p> 51 */ 52 public boolean select(String userUniqueId, String oauthName){ 53 DBConnector db = new DBConnector(); 54 boolean result = false; 55 con = db.getConnection(); 56 try{ 57 sql = "SELECT id, name FROM user WHERE unique_id = ? AND oauth_name = ? AND delete_flag = 0"; 58 ps = con.prepareStatement(sql); 59 ps.setString(1,userUniqueId); 60 ps.setString(2,oauthName); 61 rs = ps.executeQuery(); 62 63 if(rs.next()){ 64 dto.setUserId(rs.getInt(1)); 65 dto.setUserName(rs.getString(2)); 66 result = true; 67 } 68 69 }catch(SQLException e){ 70 e.printStackTrace(); 71 }finally{ 72 try{ 73 con.close(); 74 }catch(Exception e){ 75 e.printStackTrace(); 76 } 77 } 78 return result; 79 } 80 81 public LoginOauthDTO getDto() { 82 return dto; 83 } 84 85 public void setDto(LoginOauthDTO dto) { 86 this.dto = dto; 87 } 88 89 /** 90 * データベース(mysql)に接続し、テーブル(user)にユーザー情報をインサートできるか確認する為のメソッド 91 * @param uniqueId OAuthのサービス先のユニークID 92 * @param userName OAuthのサービス先のユーザー名 93 * @param oauthName OAuthサービス元の名前 94 * @return true/false <p>インサート成功時にtrueを、失敗時にfalse返します。</p> 95 */ 96 public boolean insert(String uniqueId, String userName, String oauthName) { 97 DBConnector db = new DBConnector(); 98 con = db.getConnection(); 99 int result; 100 sql = "INSERT INTO user(name, unique_id, oauth_name, created_at, updated_at) values (?,?,?,?,?)"; 101 try { 102 ps = con.prepareStatement(sql); 103 ps.setString(1, userName); 104 ps.setString(2, uniqueId); 105 ps.setString(3, oauthName); 106 result = ps.executeUpdate(); 107// int insertCount = ps.executeUpdate(); 108// if (insertCount > 0) { 109// result = true; 110// } 111 } catch (SQLException e) { 112 e.printStackTrace(); 113 } finally { 114 try { 115 con.close(); 116 } catch (SQLException e) { 117 e.printStackTrace(); 118 } 119 } 120 return false; 121 } 122 123 /** 124 * DTO取得メソッド 125 * @return dto 126 */ 127 public LoginOauthDTO getLoginOauthDTO() { 128 return dto; 129 } 130 131 /** 132 * DTO格納メソッド 133 * @param dto ログインユーザDTO 134 */ 135 public void setLoginOauthDTO(LoginOauthDTO dto) { 136 this.dto = dto; 137 } 138}
回答1件
あなたの回答
tips
プレビュー
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。