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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Struts 2

Apache Struts 2は、Apache Strutsプロジェクトにて開発されているオープンソースのJavaベースのWebアプリケーションフレームワークです。Sturts1に比べ、設定ファイルの削減、依存性の注入、POJO等の改善がなされています。

Q&A

解決済

1回答

1632閲覧

OAuthコードの見直し DAO

yuiti926

総合スコア26

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Struts 2

Apache Struts 2は、Apache Strutsプロジェクトにて開発されているオープンソースのJavaベースのWebアプリケーションフレームワークです。Sturts1に比べ、設定ファイルの削減、依存性の注入、POJO等の改善がなされています。

0グッド

0クリップ

投稿2016/04/06 05:45

ソースコードの見直しをお願いします!
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}

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

自己解決

解決しました!
わけがわからなくて答えようなかったですねこれw 草1本!

投稿2016/04/06 07:17

yuiti926

総合スコア26

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問