Discussion :

[hamzawhy]

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 :

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 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

[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-

Code :

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

catch(Exception e){}

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

A+.

[OButterlin]

Bonjour,<br>
<br>
1- Rien qu'avec la couleur syntaxique, tu peux apercevoir que la chaîne est tronquée à la ligne 6 du code posté.<br>
2- Ne met pas un ";" dans une requête à exécuter via jdbc.<br>
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.<br>
4- <br>
Tu dois afficher les erreurs dans un fichier log ou console pour faciliter le débogage.<br>
<br>
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 :

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

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.