2015-07-19 4 views
0

Я работаю над простым веб-приложением Java, которое принимает данные от пользователя через форму и сохраняет его в базе данных SQL через сервлет. Теперь, когда я пишу код, все работает отлично, но я получаю сообщение об ошибке в строке, где я от записиИсключенное исключение SQL в строке 'ps = con.prepareStatement();'

str="insert into ..." 
ps= con.createStatement(str); <-- Error here 
ps.executeUpdate();    <--error here 

Это says- нерегулируемого исключение SQL должен быть улов или выброшен.

Итак, я окружаю заявление с помощью try и catch block, но теперь, когда я запускаю программу, я получаю это: java.sql.SQLException: [Microsoft] [ODBC SQL Server Driver] [SQL Server] Не удалось найти сохраненный процедура str

Я застрял и не могу найти решение в любом месте. Я создал базу данных и таблицу, а значения вставляются через SQL Query. Я создал User DSN с именем 'mydata' через odbcad32. Пожалуйста, помогите мне!

import java.io.IOException; 
import java.io.PrintWriter; 
import java.sql.Connection; 
import javax.servlet.ServletException; 
import javax.servlet.http.HttpServlet; 
import javax.servlet.http.HttpServletRequest; 
import javax.servlet.http.HttpServletResponse; 
import java.sql.*; 


protected void processRequest(HttpServletRequest request, HttpServletResponse response) 
     throws ServletException, IOException { 
    response.setContentType("text/html;charset=UTF-8"); 
    PrintWriter out = response.getWriter(); 
    String type="",name="",pw="",city="",country="",contact="",sal=""; 
    type=request.getParameter("ddltype"); 
    name=request.getParameter("txtname"); 
    pw=request.getParameter("txtpwd"); 
    city=request.getParameter("txtcity"); 
    sal=request.getParameter("txtsal"); 
    country=request.getParameter("txtcountry"); 
    contact=request.getParameter("txtcontact"); 
    try { 
     conn(); 
     String str="insert into details values('"+type+"','"+name+"','"+pw+"','"+city+"','"+country+"','"+contact+"','"+sal+"')"; 
     ps=con.prepareStatement("str"); 
     ps.executeUpdate(); 

     /* TODO output your page here. You may use following sample code. */ 
     out.println("<!DOCTYPE html>"); 
     out.println("<html>"); 
     out.println("<head>"); 
     out.println("<title>Servlet regsev</title>");    
     out.println("</head>"); 
     out.println("<body> INSERTED SUCCESSFULLY"); 
     out.println("<h1>Servlet regsev at " + request.getContextPath() + "</h1>"); 
     out.println("</body>"); 
     out.println("</html>"); 

    } 
    catch(SQLException e) 
    { 
     out.print(""+e); 
    } 
} 

// <editor-fold defaultstate="collapsed" desc="HttpServlet methods. Click on the + sign on the left to edit the code."> 
/** 
* Handles the HTTP 
* <code>GET</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 doGet(HttpServletRequest request, HttpServletResponse response) 
     throws ServletException, IOException { 
    processRequest(request, response); 
} 

/** 
* 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 { 
    processRequest(request, response); 
} 

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

}

+2

Можете ли вы добавить свой полный код здесь (или как можно больше). –

+1

Также обратите внимание на важность достаточной проводки кода: ваш реферат (** ps = con.createStatement (str); **) отличается от реального кода. –

+0

Ха-ха! Мне жаль, что это был мой первый раз. –

ответ

1

Многие замечания здесь.

String str="insert into details values('"+type+"','"+name+"','"+pw+"','"+city+"','"+country+"','"+contact+"','"+sal+"')"; 
    ps=con.prepareStatement("str"); 
    ps.executeUpdate(); 

Все 3 неверных. Подготовленные операторы помогают повысить производительность (в большинстве случаев), упрощают код и предотвращают внедрение SQL. Ваш код не делает 2 из 3.

Ниже пример того, как это должно быть:

String str="insert into details (type, name,pwd) 
    values(?,?,?)"; 
    ps=con.prepareStatement(str); // so no "" around str. 
    ps.setString(1,type); // Sets the content of the first ?, all safe against SQL Injection 
    ps.setString(2,name); // Sets the content of the second ? 
    ps.setString(3,pwd); // Sets the content of the third ? 
    ps.execute(); // Execute instead of executeUpdate. 

База данных может теперь также повторно использовать подготовленный план выполнения оператора, экономя несколько 100s второй вставки.

+0

О, боже мой! Я просто не заметил ошибки. "" Вокруг ул. Большое вам спасибо и извините! :) –

+0

Ps: Отметьте ответ как принятый :) –

+0

@AakankshaPandey: Есть больше проблем, чем просто двойные кавычки вокруг 'str'. Пример в этом ответе не уязвим для SQL Injection. Исходный код уязвим. Этот пример не вызывает 'createStatement'. Исходный код вызывает 'createStatement' с недопустимым параметром. Все замечания в этом ответе уместны. – spencer7593

Смежные вопросы