Impossible de faire un update MySql

**tonymx15** · 05/06/2012, 09h42

Bonjour à tous,

Voilà je cherche à faire un update MySql lors du lancement d'un servlet. J'ai auparavant réussi à faire un update mais lorsque je fais un update simple sur un seul champs ça ne marche pas...

Ce code ne marche pas (servlet permettant de rajouter +1 au nombre de clique sur la vidéo dans la base de données):

Code :

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

1
2
3
4
 
int id = Integer.parseInt(req.getParameter("id"));        
Query q = em.createQuery("UPDATE Video v SET v.click = v.click+1 WHERE v.id = "+id);
q.executeUpdate();

Ce code marche (correspond à un autre servlet, servlet permettant d'éditer une vidéo):

Code :

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

1
2
3
4
5
6
7
8
9
 
        String title = req.getParameter("title");
        String url = req.getParameter("url");
        String description = req.getParameter("description");
        int id = Integer.parseInt(req.getParameter("id"));
        Category c = em.find(Category.class, Integer.parseInt(req.getParameter("category")));
 
        Query q = em.createQuery("UPDATE FROM Video v SET v.click = v.click+1, v.title = '"+title+"', v.url = '"+url+"', v.description = '"+description+"', v.category = "+c.getId()+" WHERE v.id = "+id);
        q.executeUpdate();

La page me renvoie un message d'erreur au niveau de "q.executeUpdate()"...

Etat HTTP 500 -

type Rapport d''exception

message

description Le serveur a rencontré une erreur interne () qui l'a empêché de satisfaire la requête.

exception

javax.persistence.TransactionRequiredException: Executing an update/delete query
org.hibernate.ejb.AbstractQueryImpl.executeUpdate(AbstractQueryImpl.java:96)
fr.zeubeuz.servlet.showVideo.doGet(showVideo.java:33)
javax.servlet.http.HttpServlet.service(HttpServlet.java:621)
javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
note La trace complète de la cause mère de cette erreur est disponible dans les fichiers journaux de Apache Tomcat/7.0.23.

Apache Tomcat/7.0.23

**karement** · 05/06/2012, 11h09

salut,
tu as essayé de tester ta requête directement en base de donnée?

**tonymx15** · 05/06/2012, 11h21

Oui et elle marche !

**DevServlet** · 05/06/2012, 11h28

Enlèves le

Code :

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

FROM

dans ta requête et ça marchera mieux

**tonymx15** · 05/06/2012, 11h33

Le FROM n'est pas obligatoire pour les UPDATE...

Code :

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

1
2
3
 
        Query q = em.createQuery("UPDATE FROM Video v SET v.click = v.click + 1 WHERE v.id = "+id);
        q.executeUpdate();

Toujours pareille :

Etat HTTP 500 -

type Rapport d''exception

message

description Le serveur a rencontré une erreur interne () qui l'a empêché de satisfaire la requête.

exception

javax.persistence.TransactionRequiredException: Executing an update/delete query
org.hibernate.ejb.AbstractQueryImpl.executeUpdate(AbstractQueryImpl.java:96)
fr.zeubeuz.servlet.showVideo.doGet(showVideo.java:34)
javax.servlet.http.HttpServlet.service(HttpServlet.java:621)
javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
note La trace complète de la cause mère de cette erreur est disponible dans les fichiers journaux de Apache Tomcat/7.0.23.

Apache Tomcat/7.0.23

**DevServlet** · 05/06/2012, 11h34

Code :

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

Query q = em.createQuery("UPDATE  Video v SET v.click := v.click + 1 WHERE v.id := "+id);

Il fallait enlever la clause "FROM" , elle y' était toujours, et ensuite la syntaxe de tes paramètres n'est pas bonne, il fallait mettre les ":" aussi

**DevServlet** · 05/06/2012, 11h42

En clair voici ce qui doit marcher :

Code :

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

1
2
3
4
5
6
Query q = em.createQuery("UPDATE Video v SET v.click := paramClick, v.title :=paramTitle, v.url := paramUrl, v.description := paramDescription, v.category := paramCategory WHERE v.id := paramId");
q.setParameter("paramClick",taValeurParamClickJava)
q.setParameter("paramTitle",taValeurParamJava);
//Tous les autres set parameter ici
...
        q.executeUpdate();

Il ne doit y avoir aucune concaténation.

**tonymx15** · 05/06/2012, 11h46

Code :

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

1
2
3
4
5
6
7
8
9
 
        int id = Integer.parseInt(req.getParameter("id"));
 
        int click = em.find(Video.class, id).getClick()+1;
 
        Query q = em.createQuery("UPDATE FROM Video v SET v.click := paramClick WHERE v.id := paramId");
        q.setParameter("paramId", id);
        q.setParameter("paramClick", click);
        q.executeUpdate();

Comme ceci ça ne marche pas non plus ...

**DevServlet** · 05/06/2012, 11h56

Ah excuse moi, j'ai inversé le coté des ":", essaies ca :

Code :

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

1
2
3
4
5
6
7
8
    int id = Integer.parseInt(req.getParameter("id"));
 
        int click = em.find(Video.class, id).getClick()+1;
 
        Query q = em.createQuery("UPDATE FROM Video v SET v.click = :paramClick WHERE v.id = :paramId");
        q.setParameter("paramId", id);
        q.setParameter("paramClick", click);
        q.executeUpdate();

**tonymx15** · 05/06/2012, 11h57

Code :

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

1
2
3
4
5
6
7
8
9
 
        int id = Integer.parseInt(req.getParameter("id"));
 
        int click = em.find(Video.class, id).getClick()+1;
 
        Query q = em.createQuery("UPDATE Video v SET v.click = :paramClick WHERE v.id = :paramId");
        q.setParameter("paramId", id);
        q.setParameter("paramClick", click);
        q.executeUpdate();

Etat HTTP 500 -

type Rapport d''exception

message

description Le serveur a rencontré une erreur interne () qui l'a empêché de satisfaire la requête.

exception

javax.persistence.TransactionRequiredException: Executing an update/delete query
org.hibernate.ejb.AbstractQueryImpl.executeUpdate(AbstractQueryImpl.java:96)
fr.zeubeuz.servlet.showVideo.doGet(showVideo.java:37)
javax.servlet.http.HttpServlet.service(HttpServlet.java:621)
javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
note La trace complète de la cause mère de cette erreur est disponible dans les fichiers journaux de Apache Tomcat/7.0.23.

Apache Tomcat/7.0.23

Toujours pareille ...

**DevServlet** · 05/06/2012, 11h59

Il doit y avoir une log hibernate quelque part qui donne plus de détails sur l'erreur, peut on voir ce fichier et l'entité video aussi ?

**tonymx15** · 05/06/2012, 12h01

Entité vidéo :

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package fr.zeubeuz.entity;
 
import javax.persistence.*;
 
@Entity
@Table(name="video")
public class Video {
 
    @Id
    @GeneratedValue
    private int id;
 
    private String title;
    private String code;
    private String thumbnail;
 
    @Lob
    private String embed;
 
    private String description;
    private int click;
 
    @OneToOne
    @JoinColumn(name="category_id")
    private Category category;
 
    public Video(){}
 
    /*
    public  Map<String, String> getQueryMap(String query){  
        String[] params = query.split("&");  
        Map<String, String> map = new HashMap<String, String>();  
        for (String param : params)  
        {  
            String name = param.split("=")[0];  
            String value = param.split("=")[1];  
            map.put(name, value);  
        }
        return map;
    }
 
    public void test(URL url){
        String query = url.getQuery();
        Map<String, String> map = getQueryMap(query);  
        Set<String> keys = map.keySet();  
        for (String key : keys){
            System.out.println("Name=" + key);  
            System.out.println("Value=" + map.get(key));  
        }
    }
    */
 
    public String getReduceTitle(String title){
        return title.substring(0, 8)+"...";
    }
 
    public String getThumbnailYouTube(String code){
        return "http://img.youtube.com/vi/"+code+"/1.jpg";
    }
 
    public String getEmbedYouTube(String code){
        return "<object style=\"height: 390px; width: 640px\"><param name=\"movie\" value=\"http://www.youtube.com/v/"+code+"?version=3&feature=player_detailpage\"><param name=\"allowFullScreen\" value=\"true\"><param name=\"allowScriptAccess\" value=\"always\"><embed src=\"http://www.youtube.com/v/"+code+"?version=3&feature=player_detailpage\" type=\"application/x-shockwave-flash\" allowfullscreen=\"true\" allowScriptAccess=\"always\" width=\"640\" height=\"360\"></object>";
    }
 
    public int getId() {
        return id;
    }
 
    public void setId(int id) {
        this.id = id;
    }
 
    public String getTitle() {
        return title;
    }
 
    public void setTitle(String title) {
        this.title = title;
    }
 
    public String getDescription() {
        return description;
    }
 
    public void setDescription(String description) {
        this.description = description;
    }
 
    public Category getCategory() {
        return category;
    }
 
    public void setCategory(Category category) {
        this.category = category;
    }
 
    public int getClick() {
        return click;
    }
 
    public void setClick(int click) {
        this.click = click;
    }
 
    public String getCode() {
        return code;
    }
 
    public void setCode(String code) {
        this.code = code;
    }
 
    public String getThumbnail() {
        return thumbnail;
    }
 
    public void setThumbnail(String thumnail) {
        this.thumbnail = thumnail;
    }
 
    public String getEmbed() {
        return embed;
    }
 
    public void setEmbed(String embed) {
        this.embed = embed;
    }
 
}

Log (erreur sur la ligne q.executeUpdate()):

5 juin 2012 12:01:09 org.apache.catalina.core.StandardWrapperValve invoke
GRAVE: "Servlet.service()" pour la servlet showVideo a généré une exception
javax.persistence.TransactionRequiredException: Executing an update/delete query
at org.hibernate.ejb.AbstractQueryImpl.executeUpdate(AbstractQueryImpl.java:96)
at fr.zeubeuz.servlet.showVideo.doGet(showVideo.java:37)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:621)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:722)
at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:305)
at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:210)
at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:224)
at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:169)
at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:472)
at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:168)
at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:98)
at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:928)
at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:118)
at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:407)
at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:987)
at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:539)
at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:300)
at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
at java.lang.Thread.run(Thread.java:680)

**DevServlet** · 05/06/2012, 12h20

Qu'est ce qui te fait penser que ça plante sur la requête? Peux tu faire du pas à pas (mode debug) pour determiner exactement où ça plante ? Tes logs ne donnent aucun renseignement exploitable

**tonymx15** · 05/06/2012, 12h30

Quand je commente le "q.executeUpdate()" ça marche...

Même avec les points d'arrêt je n'arrives pas à voir d'où peut venir ce problème qui me semble assez basique...

**DevServlet** · 05/06/2012, 13h43

Peut on voir le code complet ta servlet showVideo? Les logs hibernate doivent être tracés quelque part, c'est l'intérieur de ceux ci que tu dois trouver le détail exact de la cause de l'erreur.

**tonymx15** · 05/06/2012, 14h07

showVideo :

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package fr.zeubeuz.servlet;
 
import fr.zeubeuz.entity.Category;
import fr.zeubeuz.entity.Video;
import java.io.IOException;
import java.util.Calendar;
import java.util.List;
import javax.persistence.EntityManager;
import javax.persistence.EntityManagerFactory;
import javax.persistence.Persistence;
import javax.persistence.Query;
import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
 
public class showVideo extends HttpServlet{
 
    EntityManagerFactory emf;
 
    @Override
    protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
        EntityManager em = emf.createEntityManager();
 
        int id = Integer.parseInt(req.getParameter("id"));
 
        int click = em.find(Video.class, id).getClick()+1;
 
        Query q = em.createQuery("UPDATE Video v SET v.click = :paramClick WHERE v.id = :paramId");
        q.setParameter("paramId", id);
        q.setParameter("paramClick", click);
        //q.executeUpdate();
 
 
        int nbVideos = em.createQuery("SELECT v FROM Video as v").getResultList().size();
        Calendar calendar = Calendar.getInstance();
        int day = calendar.get(Calendar.DATE);
        int videoIdOfDay = Math.abs(nbVideos / day);
 
        List<Video> theVideoOfDay = em.createQuery("SELECT v FROM Video as v WHERE v.id = "+videoIdOfDay).getResultList();
        while ( theVideoOfDay.isEmpty() ){
            theVideoOfDay = em.createQuery("SELECT v FROM Video as v WHERE v.id = "+videoIdOfDay++).getResultList();
        }
        List<Video> video = em.createQuery("SELECT v FROM Video as v WHERE v.id = "+id).getResultList();
        List<Category> categories = em.createQuery("SELECT c FROM Category as c WHERE c.id != 12 AND c.id != 13 AND c.id != 14").getResultList();
 
        req.setAttribute("theVideoOfDay", theVideoOfDay);
        req.setAttribute("categories", categories);
        req.setAttribute("video", video);
        RequestDispatcher rd = req.getRequestDispatcher("showVideo.jsp");
        rd.forward(req, resp);
 
    }
 
    @Override
    public void destroy() {
        emf.close();
    }
 
    @Override
    public void init() throws ServletException {
        emf = Persistence.createEntityManagerFactory("PU");
    }
 
 
 
}

**DevServlet** · 05/06/2012, 14h13

Petite question, mise à part cette requête update ne marchant pas, t'en as d'autres requêtes dans d'autres servlets qui marchent ? Je ne vois pas le nom du contexte de persistence chargé par ton entitmanaggerFactory

**tonymx15** · 05/06/2012, 14h14

Oui toutes les autres marchent !

**DevServlet** · 05/06/2012, 15h03

Il faut absolument que tu loggues les traces hibernate, peux tu rajouter ça dans ton log4J.xml :

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
<appender name="hibernateAppender" class="org.apache.log4j.FileAppender">
    <param name="File" value="target/hibernateLog.txt" />
    <param name="Append" value="false" />
    <layout class="org.apache.log4j.PatternLayout">
      <param name="ConversionPattern" value="%d [%t] %p - %c - %m%n" />
    </layout>
  </appender>
 
  <category name="org.hibernate.type" additivity="false">
    <priority value="warn" />
    <appender-ref ref="hibernateAppender" />
  </category>
 
  <category name="org.hibernate" additivity="false">
    <priority value="warn" />
    <appender-ref ref="hibernateAppender" />
  </category>

. Tu relances ton serveur , tu retestes et tu nous dis quele est la trace d'erreur.