Discussion :

[tonytrua]

Bonjour,
J’essaie d’insérer des données dans ma BD , la connexion est bonne mais quand j’exécute le code une exception est levée car la syntaxe est mauvaise mais je ne comprend pas pourquoi si vous pouviez m'indiquer l'erreur :/
Merci d'avance

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

"INSERT INTO client VALUES('anthony@mail.fr','nom','prenom','anée','M','France','ville','adresse','CP','motPasse'")

[karement]

salut,
ça doit venir de la parenthèse à la fin

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

"INSERT INTO client VALUES('anthony@mail.fr','nom','prenom','anée','M','France','ville','adresse','CP','motPasse')"

[tonytrua]

merci beaucoup mais en faite il doit y avoir une autre erreur car malgré ma correction lors de la compilation j'obtiens cette erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Exception in thread "main" com.mysql.jdbc.exceptions.jdbc4.MySQLSyntaxErrorException: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '"INSERT INTO client (mail,nom,prenom,age,sexe,pays,ville,adresse,cp,password) VA' at line 1
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
"INSERT INTO client (mail,nom,prenom,age,sexe,pays,ville,adresse,cp,password) VALUES('palmieri.anthony@hotmail.fr','Palmieri','Anthony','1991','M','France','Contes','191 montée du serre','06390','motPasse')"	at java.lang.reflect.Constructor.newInstance(Constructor.java:525)
	at com.mysql.jdbc.Util.handleNewInstance(Util.java:409)
	at com.mysql.jdbc.Util.getInstance(Util.java:384)
	at com.mysql.jdbc.SQLError.createSQLException(SQLError.java:1054)
	at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3566)
	at com.mysql.jdbc.MysqlIO.checkErrorPacket(MysqlIO.java:3498)
	at com.mysql.jdbc.MysqlIO.sendCommand(MysqlIO.java:1959)
	at com.mysql.jdbc.MysqlIO.sqlQueryDirect(MysqlIO.java:2113)
	at com.mysql.jdbc.ConnectionImpl.execSQL(ConnectionImpl.java:2562)
	at com.mysql.jdbc.StatementImpl.executeUpdate(StatementImpl.java:1664)
	at com.mysql.jdbc.StatementImpl.executeUpdate(StatementImpl.java:1583)
	at javaapplication1.Client.insertBase(Client.java:99)
	at javaapplication1.JavaApplication1.main(JavaApplication1.java:18)
Java Result: 1
BUILD SUCCESSFUL (total time: 0 seconds)

voila mon code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
package javaapplication1;
 
import java.sql.*;
 
public class JavaApplication1 {
 
    public static void main(String[] args) throws Exception {
       Connection cnx = null;
        Class.forName("com.mysql.jdbc.Driver");
        cnx = DriverManager.getConnection("jdbc:mysql://127.0.0.1:3306/mabase", "root", "anthony1991");
        Statement req = cnx.createStatement();
        Statement b = cnx.createStatement();
        ResultSet rs = req.executeQuery("select * FROM client");
        Client clt = new Client("palmieri.anthony@hotmail.fr,Palmieri,Anthony,1991,M,France,Contes,191 montée du serre,06390,motPasse");
    String requete="INSERT INTO Client VALUES('";
 
        System.out.println(clt);
    clt.insertBase(cnx,b);
      while (rs.next()) 
       {
            System.out.println(rs.getString(1));
rs.close();}
 
ResultSet rx = req.executeQuery("show tables");
       while(rx.next()){
               System.out.println(rx.getString(1));
        }
    }        
}

et la méthode présente dans la classe client

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
public void insertBase(Connection cnx,Statement req) throws SQLException{
           String requete="\"INSERT INTO client (mail,nom,prenom,age,sexe,pays,ville,adresse,cp,password) VALUES('";
            StringBuilder sb = new StringBuilder(requete);
            sb.append(this.mail).append("\',\'");sb.append(this.nom).append("\',\'");sb.append(this.prenom).append("\',\'");sb.append(this.age).append("\',\'");sb.append(this.sexe).append("\',\'");sb.append(this.pays).append("\',\'");sb.append(this.ville).append("\',\'");sb.append(this.adresse).append("\',\'");sb.append(this.codePostal).append("\',\'");sb.append(this.motDePasse).append("\')\"");
            System.out.print(sb.toString());
            req.executeUpdate(sb.toString());

merci d'avance. J'ai lu que cela pouvais venir d'une erreur de version mais je ne pense pas vu que je fonctionne sur NetBeans et que mes requêtes SQL fonctionnent.
Merci d'avance pour votre aide

[andry.aime]

Bonjour,

String requete="\"INSERT INTO client (mail,nom,prenom,age,sexe,pays,ville,adresse,cp,password) VALUES('";

Tu ne dois pas mettre les requêtes dans des doubles quotes.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

String requete="INSERT INTO client (mail,nom,prenom,age,sexe,pays,ville,adresse,cp,password) VALUES('";

A+.

[tonytrua]

Merci beaucoup pour ta correction

[tonytrua]

j'ai un nouveau problème pour un getString une exception SQL est levé , à la suite d'un select , je ne comprend pas pourquoi mon résultat est vide , car lorsque je tape la commande simplement a la main il y a un résultat quand même :/
Merci d'avance

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public boolean ExistBase(Connection cnx, String ValueMail, Statement req) throws SQLException {
        String requete = "SELECT * FROM Client WHERE mail='ValueMail'";
        ResultSet kx = req.executeQuery(requete);
        String v = kx.getString("Mail");
        System.out.println(v);
        //if ((rs.getString("mail")).equals(ValueMail)) {
        kx.close();
        return true;
        //} else {
        //  req.close();
        // return false;
        //}
    }
}

mon main

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public class JavaApplication1 {
 
    public static void main(String[] args) throws Exception {
       Connection cnx = null;
        Class.forName("com.mysql.jdbc.Driver");
        cnx = DriverManager.getConnection("jdbc:mysql://127.0.0.1:3306/mabase", "root", "anthony1991");
        Statement req = cnx.createStatement();
        Statement b = cnx.createStatement();
        Statement a = cnx.createStatement();
 
        Client clt = new Client("testGetString,Palmieri,Anthony,1991,M,France,Contes,191 montée du serre,06390,motPasse");
 
if(clt.ExistBase(cnx, "zaz", b)==true){
 
        System.out.println("goooood");}
 
}     
    }

[andry.aime]

Re,

1- N'insère pas directement la valeur dans la requête. Utilise PreparedStastement.
2- Peut-on voir le log d'erreur?

A+.

[tchize_]

tu dois toujours appeler la méthode next() sur ton ResulteSet avant de lire. Cette méthode positionnera la lecture sur le premier row et, au passage, te retournera false si il n'y a pas de premier row.

[tonytrua]

Merci pour vos réponse , en faite mon problème et que je souhaiterai voir si l'email en paramètre existe sur la BD.
Et je suis sur que les mails sont uniques car c'est une clés primaire de ma BD.
Donc suis obligé de parcourir le résultat avec un kx.next() ?
Et quand on a pas commencé a faire kx.next() et qu'il n'y a qu'un seul résultat cela renvoie false non ? et au premier appel de kx.next() cela renvoie true et le résultat non ?
Et j'ai corrigé mon erreur en faite j'avais tout simplement oublier de concaténer le ValueMail ..
Donc dans mon cas le mieux serait de faire un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if(kx.next()==true){//je recupère la String et la compare a value mail ?}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public boolean ExistBase(Connection cnx, String ValueMail, Statement req) throws SQLException {
        String requete = "SELECT * FROM Client WHERE mail="+ValueMail;
        ResultSet kx = req.executeQuery(requete);
        String v = kx.getString("Mail");
        System.out.println(v);
        //if ((kx.getString("mail")).equals(ValueMail)) {
        kx.close();
        return true;
        //} else {
        //  req.close();
        // return false;
        //}
    }
}

[tchize_]

initialement, un resultset pointe avant le premier row (cf documentation de ResultSet.next()). Il est donc bien nécessaire de faire un appel à next() avant de lire les données.
Dans ton cas, vu la logique de la méthode, tu devrais faire un truc du genre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public boolean ExistBase(Connection cnx, String ValueMail, Statement req) throws SQLException {
        String requete = "SELECT * FROM Client WHERE mail="+ValueMail;
        ResultSet kx = req.executeQuery(requete);
        try{
            return kw.next(); // as-t-on au moins une ligne ?
        } finally {
            kx.close();
        }
 
}

Maintenant, regarde du coté des prepared statements, parce que c'est un très très très mauvaise idée de concaténer comme tu le fais une requête SQL

Et renseigne toi aussi sur les conventions de nommage en java, parce que ta méthode et tes paramètres les violent gentillement