Удалить MySQL несколько записей с помощью флажка и кнопку отправки - PullRequest
Новая
       24

Удалить MySQL несколько записей с помощью флажка и кнопку отправки

0 голосов
/ 02 февраля 2012

Мне не удается обработать эту часть моего приложения.Я должен удалить некоторые записи (правильно загруженные из БД) в базе данных mysql со страницы jsp, установив флажки и нажав кнопку «Отправить».Даже если данные отображаются правильно, из БД ничего не удаляется. Вот код:

Вот класс 

/* ArticoliManager.java */
public class ArticoliManager {

public void cancellaArticolo(String chboxArticoliDaCancellare[]) throws SQLException{
Connection con = DBConnectionPool.getConnection();
PreparedStatement ps = null;
try {
    for(String deleteThem:chboxArticoliDaCancellare){
    String query = "DELETE * FROM articoli WHERE id='"+deleteThem+"'";
    ps = con.prepareStatement(query);
    ps.executeUpdate();
    con.commit();
}
}
finally {
    if (ps != null) {
        try {
            ps.close();
        }
        catch (SQLException ignored) {
        }
    }
    try {
        con.close();
    }
    catch (SQLException ignored) {
    }
}

}
}

Вот сервлет 

/* CancellaArticolo.java
*/
public class CancellaArticoloServlet extends HttpServlet {

protected void processRequest(HttpServletRequest request, HttpServletResponse response)
        throws ServletException, IOException, SQLException {
    response.setContentType("text/html;charset=UTF-8");
    PrintWriter out = response.getWriter();
    HttpSession session = request.getSession();
    int idArticoloDaCancellare = 0;
    try {
        ArticoliManager am = new ArticoliManager();
        String chboxArticoliDaCancellare[] = request.getParameterValues("chbox");
        am.cancellaArticolo(chboxArticoliDaCancellare);
        request.getRequestDispatcher("gestione_admin.jsp").forward(request, response);
    } finally {            
        out.close();
    }
}

@Override
protected void doGet(HttpServletRequest request, HttpServletResponse response)
        throws ServletException, IOException {
    try {
        processRequest(request, response);
    } catch (SQLException ex) {
        Logger.getLogger(CancellaArticoloServlet.class.getName()).log(Level.SEVERE, null, ex);
    }
}

/** 
 * Handles the HTTP <code>POST</code> method.
 * @param request servlet request
 * @param response servlet response
 * @throws ServletException if a servlet-specific error occurs
 * @throws IOException if an I/O error occurs
 */
@Override
protected void doPost(HttpServletRequest request, HttpServletResponse response)
        throws ServletException, IOException {
    try {
        processRequest(request, response);
    } catch (SQLException ex) {
        Logger.getLogger(CancellaArticoloServlet.class.getName()).log(Level.SEVERE, null, ex);
    }
}

/** 
 * Returns a short description of the servlet.
 * @return a String containing servlet description
 */
@Override
public String getServletInfo() {
    return "Short description";
}// </editor-fold>

Вот частьстраница jsp 

/* gestione_admin.jsp */
<%    
                            for (int i=0; i<al.size(); i++){
                            out.println("<table>");
                            out.println("<tr>");
                            out.println("<td>");
                            %>
                            <form action="CancellaArticolo">
                            <input type="checkbox" name="chbox" value="<%=+al.get(i).getId()%>"/>
                            <%
                            out.println("<b>Autore: </b>"+al.get(i).getAutore()+"                    <b>Articolo: </b>"+al.get(i).getTitolo()+"</td>");
                            out.println("</tr>");
                            out.println("</table>");
                            %>
                            </form>
                            <%
                            }
                            %>
                            <input type="submit" value="Cancella Articoli Selezionati"></input>
                            </form>

кажется, почти все в порядке ... в чем проблема?

1 Ответ

0 голосов
/ 02 февраля 2012

Значение флажка должно быть идентификатором элемента.Примерно так: 

<input type="checkbox" name="chbox" value="<%=al.get(i).getId()%>"/>

Вы уже должны были обнаружить это, когда потрудились отладить значения chboxArticoliDaCancellare.Как и у вас, они все "chkbox".

. Вы также должны убедиться, что все элементы ввода находятся внутри того же <form>, что и кнопка отправки, которую предполагается отправитьжелаемые данные вместе.Итак, в основном: 

<form action="yourServletURL" method="post">
    ...
    <input type="checkbox" ... />
    ...
    <input type="checkbox" ... />
    ...
    <input type="checkbox" ... />
    ...
    <input type="submit" ... />
    ...
</form>

Не связано с конкретной проблемой, вы неправильно используете PreparedStatement.У вас все еще есть дыра SQL-инъекции, потому что вы объединили контролируемое пользователем значение параметра запроса внутри строки SQL вместо использования заполнителя ? с вызовом PreparedStatement#setXxx().Кроме того, обратите внимание на JSTL / EL, это сделает ваш код презентации чище.

...