Problème avec les modèles

**stouf()** · 19/05/2009, 22h14

Bonjour,

Je suis entrain de faire le tuto sur les modèle de "Jawher Moussa" et j'ai une petite erreur quand j'appuie sur le submit.

Voici le code du Login (pour le dao et la classe pojo c'est comme dans le tuto)

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
 
/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
 
package com.myapp.wicket;
 
import com.myapp.dao.IUserDao;
import com.myapp.dao.UserDao;
import com.myapp.pojo.User;
import org.apache.wicket.markup.html.form.Form;
import org.apache.wicket.markup.html.form.PasswordTextField;
import org.apache.wicket.markup.html.form.TextField;
import org.apache.wicket.model.CompoundPropertyModel;
import org.apache.wicket.model.LoadableDetachableModel;
 
/**
 *
 * @author stouf
 */
public class Login extends BasePage {
 
    private User user = new User();
 
    public Login() {
        super ();
        Form form = new Form( "loginForm", 
                new CompoundPropertyModel( new LoadableDetachableModel(){
            @Override
            protected Object load(){
                return new User();
            }
        } ) ) {
 
            @Override
            protected void onSubmit(){
                IUserDao userDao = new UserDao();
                userDao.insert( user );
            }
        };
 
        form.add( new TextField( "login" ) );
        form.add( new PasswordTextField( "password" ) );
        add(form);
    }
}

Voici l'erreur que j'ai eu

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
 
Error serializing object class com.myapp.wicket.Login [object=[Page class = com.myapp.wicket.Login, id = 2, version = 0]]
org.apache.wicket.util.io.SerializableChecker$WicketNotSerializableException: Unable to serialize class: com.myapp.pojo.User
Field hierarchy is:
  2 [class=com.myapp.wicket.Login, path=2]
    private com.myapp.pojo.User com.myapp.wicket.Login.user [class=com.myapp.pojo.User] <----- field that is not serializable
        at org.apache.wicket.util.io.SerializableChecker.check(SerializableChecker.java:349)
        at org.apache.wicket.util.io.SerializableChecker.checkFields(SerializableChecker.java:618)
        at org.apache.wicket.util.io.SerializableChecker.check(SerializableChecker.java:541)
        at org.apache.wicket.util.io.SerializableChecker.writeObjectOverride(SerializableChecker.java:687)
        at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:322)
        at org.apache.wicket.util.io.IObjectStreamFactory$DefaultObjectStreamFactory$2.writeObjectOverride(IObjectStreamFactory.java:127)
        at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:322)
        at org.apache.wicket.util.lang.Objects.objectToByteArray(Objects.java:1097)
        at org.apache.wicket.protocol.http.pagestore.AbstractPageStore.serializePage(AbstractPageStore.java:199)
        at org.apache.wicket.protocol.http.pagestore.DiskPageStore.storePage(DiskPageStore.java:814)
        at org.apache.wicket.protocol.http.SecondLevelCacheSessionStore$SecondLevelCachePageMap.put(SecondLevelCacheSessionStore.java:327)
        at org.apache.wicket.Session.requestDetached(Session.java:1390)
        at org.apache.wicket.RequestCycle.detach(RequestCycle.java:1113)
        at org.apache.wicket.RequestCycle.steps(RequestCycle.java:1385)
        at org.apache.wicket.RequestCycle.request(RequestCycle.java:498)
        at org.apache.wicket.protocol.http.WicketFilter.doGet(WicketFilter.java:444)
        at org.apache.wicket.protocol.http.WicketFilter.doFilter(WicketFilter.java:282)
        at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:246)
        at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:214)
        at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:313)
        at org.apache.catalina.core.StandardContextValve.invokeInternal(StandardContextValve.java:287)
        at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:218)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:648)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:593)
        at com.sun.enterprise.web.WebPipeline.invoke(WebPipeline.java:94)
        at com.sun.enterprise.web.PESessionLockingStandardPipeline.invoke(PESessionLockingStandardPipeline.java:98)
        at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:222)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:648)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:593)
        at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:587)
        at org.apache.catalina.core.ContainerBase.invoke(ContainerBase.java:1096)
        at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:166)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:648)
        at org.apache.catalina.core.StandardPipeline.doInvoke(StandardPipeline.java:593)
        at org.apache.catalina.core.StandardPipeline.invoke(StandardPipeline.java:587)
        at org.apache.catalina.core.ContainerBase.invoke(ContainerBase.java:1096)
        at org.apache.coyote.tomcat5.CoyoteAdapter.service(CoyoteAdapter.java:288)
        at com.sun.enterprise.web.connector.grizzly.DefaultProcessorTask.invokeAdapter(DefaultProcessorTask.java:647)
        at com.sun.enterprise.web.connector.grizzly.DefaultProcessorTask.doProcess(DefaultProcessorTask.java:579)
        at com.sun.enterprise.web.connector.grizzly.DefaultProcessorTask.process(DefaultProcessorTask.java:831)
        at com.sun.enterprise.web.connector.grizzly.DefaultReadTask.executeProcessorTask(DefaultReadTask.java:341)
        at com.sun.enterprise.web.connector.grizzly.DefaultReadTask.doTask(DefaultReadTask.java:263)
        at com.sun.enterprise.web.connector.grizzly.DefaultReadTask.doTask(DefaultReadTask.java:214)
        at com.sun.enterprise.web.connector.grizzly.TaskBase.run(TaskBase.java:265)
        at com.sun.enterprise.web.connector.grizzly.ssl.SSLWorkerThread.run(SSLWorkerThread.java:106)
Caused by: java.io.NotSerializableException: com.myapp.pojo.User
        at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1156)
        at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1509)
        at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1474)
        at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1392)
        at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1150)
        at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:326)
        at org.apache.wicket.util.io.IObjectStreamFactory$DefaultObjectStreamFactory$2.writeObjectOverride(IObjectStreamFactory.java:119)
        ... 39 more

Merci d'avance pour votre aide

Stouf()

**joseph_p** · 19/05/2009, 23h10

salut

Le message d'erreur dit "java.io.NotSerializableException: com.myapp.pojo.User", autrement dit ta classe User n'implémente pas l'interface Serializable.

Ceci dit, cela ne devrait pas poser de problème vu que tu utilises un LoadabaleDetachableModel, autrement dit ton utilisateur est chargé (load) à chaque premier appel de chaque "request cycle".

Aussi, d'où vient l'erreur ?

La réponse réside là :
private User user = new User();
En effet, tu attaches un user à ta page. Or ta page est sérialisée entre chaque appel (chaque request cycle), donc entre le moment où elle est créée, affichée, puis qu'une action utilisateur te la renvoie.

Aussi, si tu supprimes ce champ de ta page, ton erreur disparaitra.

Enfin, tu créés un nouvel utilisateur à chaque usage de ta page, est ce voulu ?
Dans le même ordre d'idée, je pense que ton TextField doit toujours être rempli à chaque validation, nous sommes d'accord ? Si oui, utilises plutot un RequiredTextField

++

**stouf()** · 20/05/2009, 08h27

Bonjour Zedros,

Merci pour ta réponse constructive et pleine de bon sens

juste une petite remarque, si je retire le "private User user = new User()" alors dans ma classe DAO ( userDao.insert( user ); ) je lui passe comment l'utilisateur ?

Pour les champs requis, je connais bien mais je faisais juste un test ou deux sur les modèles et donc j'ai fait au plus simple et plus rapide

Merci
Stouf()

**joseph_p** · 20/05/2009, 08h52

Envoyé par stouf()

Merci pour ta réponse constructive et pleine de bon sens

Envoyé par stouf()

juste une petite remarque, si je retire le "private User user = new User()" alors dans ma classe DAO ( userDao.insert( user ); ) je lui passe comment l'utilisateur ?

Bonne question

En fait, quand tu associes directement un modèle avec un composant, dans ton cas via le constructeur, tu définies le DefaultModel lié au composant. Cela peut aussi se faire directement via un setDefaultModel(model) voir un setDefaultModelObject(object). Pour récupérer le defaultModel, tu fais ensuite un getDefaultModel() sur ton composant en question (soit ton formulaire là) (NB : regarde via l'autocomplémention les différentes variantes proposées).

Seul défaut à cette approche : tu perds le type en chemin, lors de tes getDefaultModel* tu devras alors caster. Pour éviter la chose, tu peux passer par un champ de ta classe.

Par contre, je te conseille fortement de mettre un modèle dans ton champ. Et tu peux toujours utilier le defaultmodelobject. Par exemple dans ton cas ça donnerait ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Foo{
LoadableDetachableModel myBarModel ;
 
public Foo(String id){
            super(id);
            myBarModel = new LoadableDetachableModel(){
            @Override
            protected Object load(){
                return new User();
            }
             setDefaultModel(new CompoundPropertyModel(myBarModel));
      }
}

Dernière remarque : avec le temps, je deviens moins emballé par le CompoundPropertyModel. Il y a déjà ce cast obligatoire dès que tu accèdes au defaultobjectmodel. Cela peut arriver assez fréquemment et c'est pas tip top.

Il t'impose également une correspondance parfaite entre ton objet et tes wicket:id, pas top pour le refactoring.

De plus, si un de tes composants doit avoir un différent modèles, il est vite inutile. De même, pour avoir un composant sans modèle, tu dois alors le dire explicitement, via :

Code :

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

new FileUploadField("myUpload",new Model(null));

Bref, je suis un peu mitigé

J'aime bien les new Composant("id", new PropertyModel(myBarModel,"chemin")); en fait.

voili voilou
++

**stouf()** · 20/05/2009, 09h14

dans le cas ou je fais un PropertyModel alors ma classe User devra faire un "implements Serializable" ?

Pas facile ces modèles

Merci Zedros

**joseph_p** · 20/05/2009, 09h27

Envoyé par stouf()

dans le cas ou je fais un PropertyModel alors ma classe User devra faire un "implements Serializable" ?

Hum, mon exemple n'est peut être pas le mieux codé/pensé/expliqué : tu peux très bien mettre un LoadableDetachableModel dans le champ associé à ta classe, puis seulement après l'accéder via des propertymodel. Tient, d'ailleurs, mon code est faux, il aurait fallu écrire :

Code :

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

new Composant("id", new PropertyModel(myBarModel.getObject(),"chemin"));

Envoyé par stouf()

Pas facile ces modèles

Je dirai plutôt "pas si facile"

lol En fait, je suis d'accord, c'est, du moins perso, un des rares points que j'ai trouvé/trouve encore parfois un peu ardu avec Wicket. Ceci dit, ces modèles aident énormément à construire aisément des pages hautement dynamiques, leur plus value vaut donc bien quelques efforts.

Perso, ce qui m'a beaucoup aidé est cette petite phrase : "un modèle n'est qu'une indirection (un wrapper) autour des données afin de déterminer comment on y accède".

Cette phrase à quelques corrolaires : ne pas utiliser sans réfléchir/comprendre des modèles de partout, mettant un modèle dans un autre afin de satisfaire tel ou tel attribut d'une méthode.

Au contraire, ne pas hésiter à aller cherche l'objet, via IModel.getObject, puis à le remettre dans un autre modèle. Après tout, ce modèle ne fait que dire s'il s'agit de lecture seule, d'une lecture unique, de nouvel accès à ton objet à chaque lecture ou d'un chargement initial suivi d'un accès à l'objet chargé (NB : je laisse au lecteur, à titre d'exercice, l'association avec les modèles correspondants

).

Pour finir, tu peux utiliser tes propres modèles au besoin, et notamment dans les cas où tu utilises des entités. Là où je bosse on a d'ailleurs un EntityModel tip top, qui s'occupe de charger la bête au besoin puis de le réattacher si on n'est plus dans le même request cycle (les DAO étant souvent thread locaux, un nouveau request cycle indique un nouveau thread). Mon tech lead est absent cette semaine, mais je vais voir avec lui la semaine pro si je peux publier plus de détails de la bête

++

NB : un petit résolu serait bienvenu

**joseph_p** · 21/05/2009, 10h07

Envoyé par ZedroS

La réponse réside là :
private User user = new User();
En effet, tu attaches un user à ta page. Or ta page est sérialisée entre chaque appel (chaque request cycle), donc entre le moment où elle est créée, affichée, puis qu'une action utilisateur te la renvoie.

Aussi, si tu supprimes ce champ de ta page, ton erreur disparaitra.
++

hey, j'l'avais dit là moué

lol

++

Problème avec les modèles

Wicket Java

Vue hybride

Discussions similaires

Partager

Partager