Discussion :

[samil]

Salut
j'ai un problème avec l'instruction UPDATE. En fait je fais une mis à jour dans une de mes tables voici ma fonction qui fait la modification

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
30
31
32
33
34
35
36
37
38
39
40
41
 
 public void MiseaJourControleur(String Matricule, String CertificatMed, String DateQualif1, String NomAgent, String PrenomAgent, 
                                     String Sexe, String DateNaissance, String Nationalite, String Fonction, String DateRecrutement, 
                                     String Niveau, String DateRetraite) throws SQLException, Exception
        {
            conex = new Connexion();
            con = conex.getConn();
            Statement stmt = con.createStatement();
            int resupdtagent, nivo = 0;
 
            java.sql.Date datenaissance = null;
            java.sql.Date daterecrutement = null;
            java.sql.Date dateretraite = null;
            java.sql.Date certificatmed = null;
            java.sql.Date datequalif1 = null;
 
            SimpleDateFormat sdf = new SimpleDateFormat("dd/MM/yyyy");
 
            datenaissance = new java.sql.Date(sdf.parse(DateNaissance).getTime()); 
            daterecrutement = new java.sql.Date(sdf.parse(DateRecrutement).getTime()); 
            dateretraite = new java.sql.Date(sdf.parse(DateRetraite).getTime()); 
            certificatmed = new java.sql.Date(sdf.parse(CertificatMed).getTime());
            datequalif1 = new java.sql.Date(sdf.parse(DateQualif1).getTime());
 
 
            // Initialisation Niveau Anglais
            if(Niveau != null)   nivo = Integer.parseInt(Niveau);
 
            String updtagent = "UPDATE CONTROLEUR"+
                               " SET  CERTIFICATMED = "+certificatmed+ ", DATEQUALIFICATION1 = "+datequalif1+", NOMAGENT = "+NomAgent+
                                   ", PRENOMAGENT = "+PrenomAgent+", SEXE = "+Sexe+", DATENAISSANCE = "+datenaissance+
                                   ", NATIONALITE = "+Nationalite+", FONCTION = "+Fonction+", NIVEAU = "+nivo+
                                   ", DATERECRUTEMENT = "+daterecrutement+", DATERETRAITE = "+dateretraite+
                              " WHERE MATRICULE = "+Matricule;
 
           resupdtagent = stmt.executeUpdate(updtagent);
 
         con.commit();
 
            System.out.println("Mise à jour OK!!");
        }// Fin MiseaJourControleur

j'appelle cette fonction dans une page jsp et lorsque j'execute cette page j'ai l'erreur suivante

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
30
31
32
33
34
35
36
37
 
javax.servlet.ServletException: ORA-00933: la commande SQL ne se termine pas correctement
 
	org.apache.jasper.runtime.PageContextImpl.doHandlePageException(PageContextImpl.java:848)
	org.apache.jasper.runtime.PageContextImpl.handlePageException(PageContextImpl.java:781)
	org.apache.jsp.ficheagent_jsp._jspService(ficheagent_jsp.java:608)
	org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:97)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:322)
	org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:291)
	org.apache.jasper.servlet.JspServlet.service(JspServlet.java:241)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	org.netbeans.modules.web.monitor.server.MonitorFilter.doFilter(MonitorFilter.java:362)
 
 
cause mère 
 
java.sql.SQLException: ORA-00933: la commande SQL ne se termine pas correctement
 
	oracle.jdbc.driver.DatabaseError.throwSqlException(DatabaseError.java:112)
	oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:331)
	oracle.jdbc.driver.T4CTTIoer.processError(T4CTTIoer.java:288)
	oracle.jdbc.driver.T4C8Oall.receive(T4C8Oall.java:743)
	oracle.jdbc.driver.T4CStatement.doOall8(T4CStatement.java:207)
	oracle.jdbc.driver.T4CStatement.executeForRows(T4CStatement.java:946)
	oracle.jdbc.driver.OracleStatement.doExecuteWithTimeout(OracleStatement.java:1168)
	oracle.jdbc.driver.OracleStatement.executeUpdateInternal(OracleStatement.java:1614)
	oracle.jdbc.driver.OracleStatement.executeUpdate(OracleStatement.java:1579)
	gestionformation.Controleur.MiseaJourControleur(Controleur.java:184)
	org.apache.jsp.ficheagent_jsp._jspService(ficheagent_jsp.java:278)
	org.apache.jasper.runtime.HttpJspBase.service(HttpJspBase.java:97)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	org.apache.jasper.servlet.JspServletWrapper.service(JspServletWrapper.java:322)
	org.apache.jasper.servlet.JspServlet.serviceJspFile(JspServlet.java:291)
	org.apache.jasper.servlet.JspServlet.service(JspServlet.java:241)
	javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	org.netbeans.modules.web.monitor.server.MonitorFilter.doFilter(MonitorFilter.java:362)

je ne vois pas l'erreur que j'ai pu faire dans ma requête de mise à jour!! Quelqu'un voit il où j'ai pu péché?
Merci d'avance!

[Garuda]

Faire afficher la chaine requête aprés construction (System.out.println updtagent) pour analyse.
C'est encore le plus simple !

[samil]

Voici la chaine affiché après construction :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
UPDATE CONTROLEUR 
SET  CERTIFICATMED = 2007-06-02, 
DATEQUALIFICATION1 = 2001-01-20, 
NOMAGENT = toto, 
PRENOMAGENT = tata, 
SEXE = H, 
DATENAISSANCE = 1974-06-13, 
NATIONALITE = malais, 
FONCTION = null, 
NIVEAU = 4, DATERECRUTEMENT = 2000-03-01, 
DATERETRAITE = 2029-05-30 
WHERE MATRICULE = 369

je ne vois toujours pas d'anomalies!
PS: le champ FONCTION n'est pas obligatoire.

[Garuda]

Il faut que ta requète ressemble à ca

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
UPDATE controleur
   SET certificatmed = to_date('2007-06-02','YYYY-MM-DD'),
       datequalification1 = to_date('2001-01-20','YYYY-MM-DD'),
       nomagent = 'toto',
       prenomagent = 'tata',
       sexe = 'h',
       datenaissance = to_date('1974-06-13','YYYY-MM-DD'),
       nationalite = 'malais',
       fonction = NULL,
       niveau = 4,
       daterecrutement = to_date('2000-03-01','YYYY-MM-DD'),
       dateretraite = to_date('2029-05-30','YYYY-MM-DD')
 WHERE matricule = 369

La, c'est la manière brute.
Sinon tu peux faire du 'binding' de variable + des prepared statement (faire une recherche)
C'est beaucoup plus propre et efficace (réutilisation du curseur, 1 seul parsing)

[Garuda]

Cf
preparedstatement

[samil]

Merci Garuda j'ai essayé avec les PreparedStatement ça marche sans problème.