Insertion données java dans Mysql Résolu/Fermé

Question

Bonjour, 

Je code en java sous et Eclipse et utilise Mysql(serveur version 5.7.24) . Souhaitant envoyer des données vers ma BD.

J'ai fait la requête suivante afin d'insérer des données dans ma base de données :

 String reqclient = ("insert into 'Client' ('numeroCartefidelite', nom, 'prenom', 'mail', 'codePostal', 'nbPointfidelite')
	 values("' +numeroCartefidelite+ "' , '" +nom+ "','"  +prenom+  "','" +mail+  "' , '" +codePostal+ "' , '"  +nbPointfidelite+ "');");


Après compilation j'ai l'erreur suivante : 

Exception in thread "main" java.lang.Error: Unresolved compilation problem: 
	String literal is not properly closed by a double-quote


Merci d'avance àtous ceux qui pourraient me dépanner.

jordane45 · Answer

Bonjour
Tu as mal écrit ta requête.... Et mal concaténé ta string...

KX · Answer

Bonjour,

Il ne faut jamais faire de concaténation de String pour écrire une requête SQL.

Dans le cas général ça fera ce que tu voudras :
insert into Client (numeroCartefidelite, nom, prenom, mail, codePostal, nbPointfidelite)
values('numeroCartefidelite', 'nom', 'prenom', 'mail', 'codePostal', 'nbPointfidelite');
Mais si le nom contient une apostrophe par exemple, ça va faire n'importe quoi :
insert into Client (numeroCartefidelite, nom, prenom, mail, codePostal, nbPointfidelite)
values('numeroCartefidelite', 'Giscard d'Estaing', 'Valéry', 'mail', 'codePostal', 'nbPointfidelite');
Cela pourrait même permettre de l'injection SQL, une faille de sécurité qui permettrait de mettre du code SQL dans l'un de tes String et qui viendrait se concaténer à ta requête pour exécuter une toute autre requête.

Exemple avec ce nom : "','','',''); drop Client; -- "
Ce qui donnerait cette requête, qui va détruire ta base de données :
insert into Client (numeroCartefidelite, nom, prenom, mail, codePostal, nbPointfidelite)
values('numeroCartefidelite', '','','',''); drop Client; -- ', 'prenom', 'mail', 'codePostal', 'nbPointfidelite');
Pour corriger ça en Java c'est très simple en utilisant des PreparedStatement. Ta requête est écrite avec des paramètres "?" qui seront remplacés par des setString (ou équivalent selon le type des données).
PreparedStatement pstmt = con.prepareStatement("insert into Client (numeroCartefidelite, nom, prenom, mail, codePostal, nbPointfidelite) values(?,?,?,?,?,?)");
pstmt.setString(1, numeroCartefidelite);
pstmt.setString(2, nom);
pstmt.setString(3, prenom);
pstmt.setString(4, mail);
pstmt.setString(5, codePostal);
pstmt.setString(6, nbPointfidelite);

Shielded · Answer

Le message : The method prepareStatement(String) is undefined for the type Bd

CODE DE  LA CLASSE CLIENT: AU LIEU DE Bd

package memo;

import java.sql.PreparedStatement;
import java.sql.SQLException;

public class Client {
 
 private String nom, prenom, mail;
 private int numeroCartefidelite, codePostal, nbPointfidelite;
 
 public Client(int numFidelite, String nom, String prenom, String mail,  int cp, int pointFidelite) {
  
  this.nom = nom;
  this.prenom = prenom;
  this.mail = mail;
  this.numeroCartefidelite = numFidelite;
  this.codePostal = cp;
  this.nbPointfidelite = pointFidelite;
 }

public void  ajouterClient(){
   
  Bd connexion1 = new Bd();
   
  try {
   
  String reqclient = "insert into Client (numeroCartefidelite , nom, prenom, mail, codePostal, nbPointfidelite) values(?, ?, ?, ?, ?, ?)";
        
        PreparedStatement prepare = connexion1.prepareStatement(reqclient);
        prepare.setInt(1, 123);
        prepare.setString(2, "Test");
        prepare.setString(3, "prenom");
        prepare.setString(4, "test@test.fr");
        prepare.setInt(5, 78140);
        prepare.setInt(6, 10);
        
        prepare.executeUpdate();

System.out.println("insertion faite");
        connexion1.fermerconnexion(); 
      
        } catch (SQLException e) {
            System.out.println("la connexion a la bd a echoue");
            e.printStackTrace();
        
        }  
       
 }

}

L'idée étant d'appeler la methode ajouterClient() lors de la creation d'une instance dans le main

Shielded · Answer

package memo;

import java.sql.Connection;

import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

public class Bd{
    private  Statement stmt;
    public static Connection con;
    static ResultSet resultat;
    private static Statement state;
    private static ResultSet results = null; 
    private static String url = "jdbc:mysql://localhost:3306/commerce?zeroDateTimeBehavior=CONVERT_TO_NULL&serverTimezone=UTC";
    
    private static String username = "root";
    private static String pwd = "root";
    
     public  void Connecter(){ 
       
        try {
         
            Class.forName("com.mysql.cj.jdbc.Driver");
            
        } catch (ClassNotFoundException e) {
            System.out.println("echec !");
        }
        try {
             
           con = DriverManager.getConnection(url, username, pwd);
            System.out.println(con);
         
        } catch (SQLException e) {
            System.out.println("la connexion à la bd a echoue");
            e.printStackTrace();

}

}
     
  public int prepareStatement(String string) {
  int j = 0;
  String reqclient = ("insert into Client (numeroCartefidelite , nom, prenom, mail, codePostal, nbPointfidelite) values(?, ?, ?, ?, ?, ?, ?)");
  try{
   PreparedStatement prepare = con.prepareStatement(reqclient);

} catch (SQLException e) {
   j = 1;
   e.printStackTrace();
   
  }
  return j;
    
 }
}

C'est le contenu de la classe Bd.

Shielded · Answer

Mon problème est reglé : il fallait prendre Connection pour type de retour de la methode Connecter() dans Bd.

package memo;

import java.sql.Connection;


import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;


public class Bd{
    private  Statement stmt;
    public static Connection con;
    static ResultSet resultat;
    private static Statement state;
    private static ResultSet results = null; 
    private static String url = "jdbc:mysql://localhost:3306/commerce?zeroDateTimeBehavior=CONVERT_TO_NULL&serverTimezone=UTC";
    
    private static String username = "root";
    private static String pwd = "root";
    
     public  Connection Connecter(){ 
    	  
        try {
        	
            Class.forName("com.mysql.cj.jdbc.Driver");
            
        } catch (ClassNotFoundException e) {
            System.out.println("echec !");
        }
        try {
             
           con = DriverManager.getConnection(url, username, pwd);
            System.out.println(con);
           
         
        } catch (SQLException e) {
            System.out.println("la connexion à la bd a echoue");
            e.printStackTrace();
 
        
        }
 
        return con;
    }
     
}

Enfin   la classe Client:

public class Client {
	
	private String nom, prenom, mail;
	private int numeroCartefidelite, codePostal, nbPointfidelite;
	
	public Client(int numFidelite, String nom, String prenom, String mail,  int cp, int pointFidelite) {
		
		this.nom = nom;
		this.prenom = prenom;
		this.mail = mail;
		this.numeroCartefidelite = numFidelite;
		this.codePostal = cp;
		this.nbPointfidelite = pointFidelite;
	}

	public void  ajouterClient(){
		 
		Bd connexion1 = new Bd();
		Connection con = connexion1.Connecter();
		 
		try {
			
		String reqclient = "insert into Client (numeroCartefidelite , nom, prenom, mail, codePostal, nbPointfidelite) values(?, ?, ?, ?, ?, ?)";
        
        PreparedStatement prepare = con.prepareStatement(reqclient);
        prepare.setInt(1, this.codePostal);
        prepare.setString(2, this.nom);
        prepare.setString(3, this.prenom);
        prepare.setString(4, this.mail);
        prepare.setInt(5, this.codePostal);
        prepare.setInt(6, this.nbPointfidelite);
        
        prepare.executeUpdate(); 
		          
        System.out.println("insertion faite");
        connexion1.fermerconnexion(); 
      
        } catch (SQLException e) {
            System.out.println("la connexion a la bd a echoue");
            e.printStackTrace();
        
        }  
	     
		
	} 
	

}  

Merci encore une fois...

Insertion données java dans Mysql

5 réponses

Discussions similaires