condition "if" Fermé

Question

Bonsoir, j'ai fait une page inscription ,est celle la est securisé c'est a dire c'est pas n'importe qui peut s'inscrire il faut que le nom de la personne existe sur la base de donnée de l'adminictrateur s'il existe en lui crée un compte si non en lui renvoie un message d'erreur, mais ma condition ne fonctionne c'est tout le monde qui passe ce qui est faut : voici ma page jsp:

Bienvenue dans L'espace Numérique De Travail de l'UMMTO

et ma servlet: public class InscrEtudiantServlet extends HttpServlet { @Override protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { String nom_etu=request.getParameter("nom_etu"); String PRENOM_ETU= request.getParameter("prenom_etu"); String ADRES_ETU=request.getParameter("adress_etu"); String LIEU_NAISS_ETU= request.getParameter("lieu_naiss_etu"); String EMAIL_ETU=request.getParameter("email_etu"); String SEXE_ETU= request.getParameter("sexe_etu"); String LOGIN_ETU= request.getParameter("login_etu"); String PASWRD_ETU= request.getParameter("pswrd_etu"); Etudiant e = new Etudiant(); try { if ((!nomEtuExist(nom_etu))) { Compte c= new Compte(1,LOGIN_ETU,PASWRD_ETU); addEtud(c); request.getRequestDispatcher("espace_Etudiant.jsp").forward(request, response); } else{ request.getRequestDispatcher("erreur.jsp").forward(request, response); } } catch (ClassNotFoundException ex) { Logger.getLogger(InscrEtudiantServlet.class.getName()).log(Level.SEVERE, null, ex); } catch (SQLException ex) { Logger.getLogger(InscrEtudiantServlet.class.getName()).log(Level.SEVERE, null, ex); } } @Override public String getServletInfo() { return "Short description"; }// public boolean nomEtuExist(String nom_etu) throws ClassNotFoundException, SQLException { Class.forName("com.mysql.jdbc.Driver"); Connection cn=DriverManager.getConnection("jdbc:mysql://localhost:3306/bdd","root",""); ResultSet rs = cn.createStatement().executeQuery("SELECT * FROM etudiant WHERE NOM_ETU ='"+nom_etu+"'"); return rs.next(); } public void addEtud(Compte c) { try { Class.forName("com.mysql.jdbc.Driver"); Connection cn=DriverManager.getConnection("jdbc:mysql://localhost:3306/bdd","root",""); PreparedStatement pr=cn.prepareStatement("INSERT INTO compte VALUES(NULL,?,?)"); pr.setString(1, c.getLogin_etu()); pr.setString(2, c.getPswrd_etu()); pr.execute(); } catch (Exception ex) { ex.printStackTrace(); } }} pouvez vous m'aider svp Merci.

KX · Answer

Bonjour, Désolé je t'ai oublié sur l'autre discussion :-( return rs.next(); c'est un peu faible comme condition d'acceptation. Ça veut dire que tu acceptes l'étudiant dès que ta requêtes SQL renvoie au moins un résultat, peu importe le contenu du résultat... Ce serait intéressant d'afficher le résultat de la requête au moment où tu acceptes l'étudiant pour voir ce qu'elle contient réellement, et voir pourquoi elle contient des résultats alors que tu t'attendrais à ce qu'il n'y en ait pas. public static List> readAllRows(ResultSet rs) throws SQLException { List> rows = new ArrayList>(); ResultSetMetaData meta = rs.getMetaData(); while (rs.next()) { Map row = new LinkedHashMap(); for (int i=1; i<=meta.getColumnCount(); i++) { String column = meta.getColumnName(i); row.put(column, rs.getObject(column)); } rows.add(row); } return rows; } // la méthode ne dépend d'aucun état, elle devrait donc être static public static boolean nomEtuExist(String nom_etu) throws SQLException { // inutile : doit être fait une seule fois dans tout le programme, pas à chaque requête // Class.forName("com.mysql.jdbc.Driver"); // il vaudrait mieux faire des constantes ou mieux des fichiers .properties pour stocker les données de connexions Connection cn=DriverManager.getConnection("jdbc:mysql://localhost:3306/bdd","root",""); // Attention à l'injection SQL, il vaut mieux faire un PreparedStatement qu'une concaténation de String avec le paramètre dedans PreparedStatement stmt = cn.prepareStatement("SELECT * FROM etudiant WHERE NOM_ETU = ?"); stmt.setString(1, nom_etu); ResultSet rs = stmt.executeQuery(); // Affichage des résultats de la requête List> rows = readAllRows(rs); System.out.println("nomEtuExist("+nom_etu+")="+rows); // ne pas oublier de fermer tes connexions à la base de données rs.close(); stmt.close(); cn.close(); return !rows.isEmpty(); }

Condition "if"

1 réponse

Discussions similaires