problème de la connexion à la base de donnés oracle 10g

Version imprimable

Bonsoir,
je travaille sous oracle 10g,et je doit se connecter à la base de donnée oracle mais j'ai toujours un problème au niveau de la méthode ExecuteQuery(),voici le code pour se connecter à la base:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 public void delete() {
 
        try 
{ 
    String url = "jdbc:odbc:thin@localhost:1521:XE";
Connection connexion = DriverManager.getConnection("jdbc:oracle:thin:@localhost:1521:XE [why on WHY,"why","why"); 
 
Statement instruction = (Statement) connexion.createStatement();   
 
instruction = null;
            ResultSet rs = null;
            String e=jTextField31.getText();
            String SQL = "delete id from PRODUIT where id='"+e+ "'  ;'"; 
            instruction = (Statement) connexion.createStatement();
            instruction.executeQuery(SQL);
 
 
    }catch(Exception e){}
    }

merci pour l'aide

14/04/2014, 07h49
andry.aime

Bonjour,

1- Rien qu'avec la couleur syntaxique, tu peux apercevoir que la chaîne est tronquée à la ligne 6 du code posté.
2- Ne met pas un ";" dans une requête à exécuter via jdbc.
3- Utilise un PreparedStatement et une méthode set* pour passer le paramètre dans la requête.
4-

Citation:

Code:

catch(Exception e){}

Tu dois afficher les erreurs dans un fichier log ou console pour faciliter le débogage.

A+.
14/04/2014, 09h11
OButterlin

Citation:

Envoyé par andry.aime

Bonjour, 
 
1- Rien qu'avec la couleur syntaxique, tu peux apercevoir que la chaîne est tronquée à la ligne 6 du code posté. 
2- Ne met pas un ";" dans une requête à exécuter via jdbc. 
3- Utilise un <a href="http://docs.oracle.com/javase/6/docs/api/java/sql/PreparedStatement.html" target="_blank">PreparedStatement</a> et une méthode set* pour passer le paramètre dans la requête. 
4- 
Tu dois afficher les erreurs dans un fichier log ou console pour faciliter le débogage. 
 
A+.

Une précision pour le point 3- pour que tu sois bien conscient des dangers de l'injection SQL, dans ton code initial

Code:

String SQL = "delete id from PRODUIT where id='"+e+ "';'";

il suffirait que la valeur passée de "e" soit "0' or '1'='1" pour te vider ta table...
Avec un PreparedStatement, tu ne risques pas ce genre de désagrément.