javaの掲示板について

質問

サーブレットの削除機能の抜粋なのですが

public void doGet(HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException {

try {
MessageDao dao = new MessageDao();
String sendMessage = request.getParameter("sendMessage");
String deleteMessage = request.getParameter("deleteMessage");

//削除
if(deleteMessage != null){
String sql = "SELECT * FROM MESSAGE_TABLE where ID =";
String rsPassword = request.getParameter("rsPassword");
String deleteid = request.getParameter("id");
String deletepassword = request.getParameter("deletepassword");
if (deletepassword.equals(sql) || deleteid.equals(sql)){

dao.deleteMessage(delete);
success = "削除しました。";
request.setAttribute("success", success);
}
if (!deletepassword.equals(sql) || deleteid.equals(sql)){
errorpass = "パスワードが違います。";
request.setAttribute("errorpass", errorpass);
}
}
------------------------------------------------------------------------------------
jsp部分
<input type="deletepassword" size="20">
<input type="hidden" name="id" value="<%= message.getId() %>">
<input type="hidden" name="rsPassword" value="<%= message.getPassword() %>">
<input type="submit" name="deleteMessage" value="削除">

それぞれをこの様な書き方で記述した場合、削除ボタンを押された際に削除フォームに入力されたパスワードとidをデータベース内に格納されているパスワードとidと比較して削除、もしくはエラーという様にできているでしょうか？できていなければ変更点の御指摘をお願いします。
------------------------------------------------------------------------------------
<%
String errorpass = (String)request.getAttribute("errorpass");
if(errorpass != null){
out.println(errorpass);
}
%>

またjspの表示部分を上記のようにしているのですが、out.printlnを使わないで記述する場合どのように書けばよいのでしょうか?

よろしくお願いします。

回答No.1

teketon · ベストアンサー

＞String sql = "SELECT * FROM MESSAGE_TABLE where ID =";
SQLを使用してDBへアクセスして結果を受け取ってください。
MessageDaoクラスで参考になるようなことをやっているのではないですか？

＞<input type="hidden" name="rsPassword" value="<%= message.getPassword() %>">
hiddenはHTMLのソースから参照できるので、
パスワードの値をユーザ側へ送信してはいけません。
これは技術的にはできるけど、セキュリティ的には普通やらないよねといった指摘です。

お礼

ありがとうございます。この様に改良してみたのですがコンパイルエラーが起こります。
-------------------------------------------------------
Connection conn = null;
PreparedStatement pstmt = null;
ResultSet rs = null;

try {
conn = this.getConnection();
String sql = "SELECT * FROM MESSAGE_TABLE where ID =" + deleteid;
pstmt = conn.prepareStatement(sql);

rs = pstmt.executeQuery();

while(rs.next()) {
String password = rs.getString("PASSWORD");
}

} finally {
this.close(rs);
this.close(pstmt);
this.close(conn);
}
---------------------------------------------------
エラー: シンボルを見つけられません
conn = this.getConnection();
　　　　　　^
と表示されます。これはなぜ起こるのでしょうか？

回答No.2

teketon

＞エラー: シンボルを見つけられません
＞conn = this.getConnection();
　　　　　　^
＞と表示されます。これはなぜ起こるのでしょうか？
貼り付けたクラスのインスタンスがgetConnectionメソッドを持っていないから。