java SQL 構文について

###前提・実現したいこと
簡単なチャットを作成しているのですが表示画面に表示される一件目のデータがDBから得た情報ではなく入力された情報を表示してしまい、

DBでは文字数やnullを制限しているのに最初の投稿ボタン押下後では表示されてしまい、
再び投稿ボタン押下か再読み込みをすると消えるという現象が起きています。
データを表示する時にcontributionのgetでとってきているのですがどのように訂正すればよいでしょうか？

ご教授尾お願い致します。

DBは

CREATE TABLE CHAT (
ID VARCHAR (8)NOT NULL ,
TITLE VARCHAR(20) NOT NULL,
BODY VARCHAR (100)NOT NULL,
DATE VARCHAR (20) NOT NULL,
);

で作りました。

###投稿データをDBから取得するクラス

java
1
2public class ChatDataDAO_CHAT implements ChatDataDAO {
3
4
5    ArrayList<Contribution> dataList = new ArrayList();
6
7    private final String DRIVER_NAME = "org.h2.Driver";
8
9    private final String JDBC_URL = "jdbc:h2:~/test";
10
11    private final String DB_USER = "Me";
12
13    private final String DB_PASS = "";
14
15
16    public ChatDataDAO_CHAT() {
17
18
19        Connection conn = null;
20
21        try {
22            //データベースへ接続
23            Class.forName( DRIVER_NAME );//JDBCドライバーダウンロード
24
25            conn = DriverManager.getConnection( JDBC_URL, DB_USER, DB_PASS );//データベース接続
26
27            String sql = "SELECT * FROM CHAT"; //SELECT文の準備
28
29            PreparedStatement pStmt = conn.prepareStatement( sql );//preparedstatement作成
30
31            ResultSet result = pStmt.executeQuery();
32
33
34            String id = "";
35            String title = "";
36            String body = "";
37            String datetime = "";
38
39            while ( result.next() ) {
40                id = result.getString( "ID" );
41                title = result.getString( "TITLE" );
42                body = result.getString( "BODY" );
43                datetime = result.getString( "DATE" );
44
45                dataList.add( new Contribution( id, title, body, datetime ) );
46
47
48            }
49
50
51        } catch ( SQLException e ) {
52            e.printStackTrace();
53        } catch ( ClassNotFoundException e ) {
54            e.printStackTrace();
55        } finally {
56            try {
57                if ( conn != null ) {
58                    conn.close(); //データベースの切断
59
60
61                }
62            } catch ( SQLException e ) {
63                e.printStackTrace();
64            }
65        }
66    }
67
68
69    //投稿情報をdataListメンバに追加し、DBへ保存する
70    @Override
71    public void addContribution( Contribution c ) {
72
73
74        dataList.add( c );
75
76        Connection conn = null;
77
78        try {
79            //データベースへ接続
80            Class.forName( DRIVER_NAME );//JDBCドライバーダウンロード
81
82            conn = DriverManager.getConnection( JDBC_URL, DB_USER, DB_PASS );//データベース接続
83
84            String sql = "INSERT INTO CHAT VALUES(? , ? , ? ,? )"; //INSERT文の準備
85
86            PreparedStatement pStmt = conn.prepareStatement( sql );//preparedstatement作成
87
88
89            pStmt.setString( 1, c.getId() );
90            pStmt.setString( 2, c.getTitle() );
91            pStmt.setString( 3, c.getBody() );
92
93            pStmt.setString( 4, c.getDatetime() );
94
95            int result = pStmt.executeUpdate();
96
97
98        } catch ( SQLException e ) {
99            e.printStackTrace();
100        } catch ( ClassNotFoundException e ) {
101            e.printStackTrace();
102        } finally {
103            try {
104                if ( conn != null ) {
105                    conn.close(); //データベースの切断
106
107
108                }
109            } catch ( SQLException e ) {
110                e.printStackTrace();
111            }
112        }
113    }
114
115
116    //dataListからcount件数文の投稿データを保持したListに返却する
117    @Override
118    public ArrayList<Contribution> getContribution( int count ) {
119
120        ArrayList<Contribution> aList = new ArrayList<Contribution>();
121        int DLS = dataList.size();
122
123        if ( DLS < count ) {
124            for ( int j = 0; j < DLS; j++ )
125                aList.add( dataList.get( j ) );
126        } else {
127            for ( int j = DLS - count; j < DLS; j++ ) {
128                aList.add( dataList.get( j ) );
129
130            }
131        }
132        return aList;
133
134    }
135
136}
137
138

###サーブレットクラス

java
1public class Chat1 extends HttpServlet {
2
3
4    private static final long serialVersionUID = 1L;
5
6
7    public Chat1() {
8
9        super();
10
11    }
12
13
14    @Override
15    protected void doGet( HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException {
16
17        this.doPost( request, response );
18
19    }
20
21
22    @Override
23    protected void doPost( HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException {
24
25
26        response.setContentType( "text/html; charset=UTF-8" );//HTML出力
27        request.setCharacterEncoding( "UTF-8" );
28
29
30        try {
31            ServletContext application = this.getServletContext();//ServletContextインスタンスの取得
32
33            ServletConfig sconfig = getServletConfig();
34            String primary = sconfig.getInitParameter( "chat" );
35
36            if ( primary == null ) {
37                primary = (String) application.getAttribute( "初期値" );
38            }
39
40            ChatDataDAO_CHAT CDC = (ChatDataDAO_CHAT) Class.forName( primary ).newInstance();
41
42            ArrayList<Contribution> chatList = new ArrayList<Contribution>();//ArrayListの生成
43            String id = request.getParameter( "Id" );
44            String title = request.getParameter( "Title" );
45            String body = request.getParameter( "Body" );
46            Date date = new Date();
47
48            String datetime = dateString( date );
49
50
51            if ( title != "" && body != "" && title != null && body != null ) {
52
53
54                CDC.addContribution( new Contribution( id, title, body, datetime ) );
55            }
56
57            int number = 5;
58            String num = request.getParameter( "num" );
59
60            if ( num != null ) {
61                number = Integer.parseInt( num );
62            }
63            chatList = CDC.getContribution( number );
64            int aList = CDC.dataList.size();
65
66            application.setAttribute( "chatList", chatList );//アプリケーションスコープにインスタンスを保存
67            application.setAttribute( "初期値", primary );//アプリケーションスコープにインスタンスを保存
68            application.setAttribute( "aList", aList );
69
70            RequestDispatcher dispatch = request.getRequestDispatcher( "/ChatOutput.jsp" );
71            dispatch.forward( request, response );
72
73            // System.out.println( chatList.size() );
74
75        } catch ( InstantiationException e ) {
76
77            e.printStackTrace();
78        } catch ( IllegalAccessException e ) {
79
80            e.printStackTrace();
81        } catch ( ClassNotFoundException e ) {
82
83            e.printStackTrace();
84        } finally {
85            System.out.println( "finally処置" );
86        }
87
88    }
89
90    static public final String DATE_PATTERN = "yyyy/MM/dd HH:mm:ss";
91
92
93    public static String dateString( Date date ) {
94
95        String str;
96        if ( date == null ) {
97            str = null;
98        } else {
99            str = new SimpleDateFormat( DATE_PATTERN ).format( date );
100        }
101        return str;
102    }
103
104}

###Contributionクラス

java
1
2public class Contribution {
3
4
5    private String id = "";
6
7    private String title = " ";
8
9    private String body = "";
10
11    private String date = "";
12
13
14
15
16    public Contribution( String id, String title, String body, String date ) {
17
18        this.id = id;
19        this.title = title;
20        this.body = body;
21        this.date = date;
22    }
23
24
25    public String getId() {
26
27        return this.id;
28    }
29
30
31    public String getTitle() {
32
33        return this.title;
34    }
35
36
37    public String getBody() {
38
39        return this.body;
40    }
41
42
43    public String getDate() {
44
45        return this.date;
46
47    }
48 
49         
50     }
51
52
53

###JSP

java
1
2
3<%ArrayList<Contribution> chatList = (ArrayList<Contribution>) application.getAttribute( "chatList" ); %>
4<% int aList =(Integer)application.getAttribute( "aList" ); %>
5<% %>
6<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
7<html>
8	<head>
9		<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
10		
11		<title>チャット</title>
12			<h1>チャットDAO</h1>
13	</head>
14
15<body>
16	<form action="/Test/Chat1" method="post">
17		ID：<input type="text" name = "Id" value = "" size=20>
18		タイトル<input type="text" name="Title" value="" size=20>
19		<br>
20		<input type="text" name = "Body" value = "" size=40>
21		<input type="submit" value=" 投稿 " name="do">
22		<input type="button" onclick="location.replace(location.href)" value="ページを更新">
23		<br>
24		表示行数：<input type="number" name="num" value="5" min="1" max="100" step="1">行
25		<input type="submit" value=" 設定 " name="setting">
26	</form>
27
28		<% int CLS =chatList.size();
29 for( int i = 0; i < chatList.size(); i++ ) {%>
30		
31
32	<%="ID:"%><%=chatList.get(CLS-1-i).getId()%>
33	<%="   タイトル："%><%=chatList.get( CLS - 1 - i ).getTitle()%>
34	<%="   "%><%=chatList.get( CLS-1-i ).getDatetime(  ) %>
35	<br>
36	<%=chatList.get(CLS - 1 - i ).getBody()%>
37	<br>
38<%}
39%>
40</body>
41</html>
42`
43
44
45###補足情報(言語/FW/ツール等のバージョンなど)
46eclipse  4.3 
47JDK 1.7 
48Tomcat 7 
49DB = H２

KiyoshiMotoki

2016/05/25 03:21

どのようにうまくいかないのかを追記してください。なにを書けばよいか分からない場合、まずは以下を一読することをお勧めします。 https://teratail.com/help/question-tips#questionTips3-4

khr0404

2016/05/30 02:22

KiyoshiMotokiさんこれからの質問では気をつけて質問していきます。助言ありがとうございました。

行動規範の内容に同意します

回答3件

ベストアンサー

Chat1クラスの

Date data = new Date();
String date = request.getParameter( "" );

if ( title != "" && body != "" && title != null && body != null ) {
CDC.addContribution( new Contribution( id, title, body, date ) );
}

ここで指定しているのが下の変数の「requestのパラメータから取得したdate」だからだと思います。
想定しているのは、上の変数 Date型のdataではないですか？

紛らわしい変数名をつけると、バグの元になりかねません。
分かりやすい名前をつけましょう。

また、発言した日時を登録したいのですよね？
わざわざAPサーバー上のシステム日付を渡すのではなく、DBサーバ上のCURRENT_TIMESTAMPを指定して登録してしまえばいいのではないでしょうか？

投稿2016/05/25 01:59

編集2016/05/25 02:01

kaputaros

総合スコア1844

khr0404

2016/05/30 02:29

kaputarosさん DBでのTIMESTAMPだとどうしてもうまくいかなかったので結局DBでは全部VARCHARで作成し日時はjavaのDATEでとってくる方法にしました。ですがDBでもNOTNULLで設定しているのにも関わらず一回目の投稿ボタン押下だとnullの情報も一度表示結果に表示されてしまい、もう一度更新するとその投稿は消えます。・nullが入っている内容で一回目の投稿ボタン押下後は表示はされるがDBへは追加されていない・二回目の投稿or更新ボタンを押下後はDB通り先ほど表示されていたnullの内容は表示されないなので一回目の投稿ボタン押下の時点で表示される一件目の情報はDBから取得した情報ではなくDBへ登録する前の情報だと思うのですがどこが原因がわかりません。。。

khr0404

2016/05/30 02:50

お騒がせしてすみませんサーブレットクラスのif分に記述が抜けていました。 if ( title != "" && body != "" && title != null && body != null && id != "" && id != null ) { で訂正したところ表示もされなくなりました。

行動規範の内容に同意します

お騒がせしてすみません
サーブレットクラスのif分に記述が抜けていました。

if ( title != "" && body != "" && title != null && body != null && id != "" && id != null ) {

で訂正したところ表示もされなくなりました。

投稿2016/05/30 02:49

khr0404

総合スコア43

・ ChatDataDAO_CHATクラスの
pStmt.setString( 4, c.getDate() );
を
pStmt.setTimestamp( 4, new Timestamp( System.currentTimeMillis() ) );
に変更したら
時間はでるようになったのですが
ID:aa aa：a null
ID:fff fff：fff 2016-05-25 10:29:38.53
ID:sss sss：sss 2016-05-25 10:28:06.262
ID:ttt ttt：ttt 2016-05-25 10:28:00.14
ID:ff ff：ff 2016-05-25 10:27:54.064