回答編集履歴
1
コードレビュー追記
test
CHANGED
@@ -45,3 +45,31 @@
|
|
45
45
|
・[prepareStatementの使用](http://www.javadrive.jp/servlet/database/index10.html)
|
46
46
|
|
47
47
|
更に[sql2o](http://www.sql2o.org/)のようなDBアクセスを簡単に書けるライブラリを利用するとコード量と手間を減らせます。
|
48
|
+
|
49
|
+
|
50
|
+
|
51
|
+
---
|
52
|
+
|
53
|
+
コードレビュー追記
|
54
|
+
|
55
|
+
|
56
|
+
|
57
|
+
insertの時にtitleのサイズを越えているとエラーが発生します。
|
58
|
+
|
59
|
+
事前にサイズチェックする方がよいです。
|
60
|
+
|
61
|
+
|
62
|
+
|
63
|
+
SELECTはinsertで追加したレコードのPKを取得していると思われます。
|
64
|
+
|
65
|
+
このクエリでは同じtitleのレコードが既に存在した場合に古いレコードのroom_idを取得してしまいます。
|
66
|
+
|
67
|
+
直前にinsertしたレコードのPKを取得するならSELECT last_insert_id()が適切です。
|
68
|
+
|
69
|
+
|
70
|
+
|
71
|
+
create tableは同じテーブル名が存在するとエラーが発生します。
|
72
|
+
|
73
|
+
おそらくroom_idはauto incrementされたものなので問題ないと思いますが、create table if not existsで確実にエラーが発生しないコードにする方がよいです。
|
74
|
+
|
75
|
+
|