Discussion :

[jc63]

Bonjour,
voila je ne suis pas une fèche en java et je commence à m'énerver sur un problème de cast ( ou peut être que je n'utilise pas la bonne méthode ).

Alors voila j'ai un objet A qui contient une collection d'objet B et/ou B1 et/ou B2 (B1 et B2 héritant de B) Ce que je sui en train de faire c'est une méthode pour cloner certain élément de B. J'ai bien implémenter la méthode clone de mes classes B, B1, B2.

A -> Gabarit
B -> GabaritElement
B1 -> GabaritNote
B2 -> GabaritCumul

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
 
public void cloneGabaritVague(Gabarit gabarit, Vague source, Vague cible) {
 
	// collection chargée par hibernate
	Set<GabaritElement> ges = gabarit.getElementsByVague(source);
	for (GabaritElement element : ges) {
		log.debug( "Object : " + element + " | " + element.getClass().getName() );
		// on clone l'objet - et c'est la que ca plante 
                // model.GabaritElement cannot be cast to model.GabaritNote
		GabaritElement ge = (GabaritElement) element.clone();
		// on modifie l'objet cloné
		ge.setVague(cible);
		// on sauvegarde avec Hibernate
		gabaritElementManager.save( ge );
	}
}

l'erreur compléte :

-------------------------------------------------------------------------------
Test set: service.GlobalManagerTest
-------------------------------------------------------------------------------
Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 12.141 sec <<< FAILURE!
testCloneGabaritVague(service.GlobalManagerTest) Time elapsed: 0.484 sec <<< ERROR!
java.lang.ClassCastException: model.GabaritElement cannot be cast to model.GabaritNote
at model.GabaritNote.clone(GabaritNote.java:51)
at service.impl.GlobalManagerImpl.cloneGabaritVague(GlobalManagerImpl.java:123)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:304)
at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:172)
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:139)
at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:107)
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:161)
at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:204)
at $Proxy38.cloneGabaritVague(Unknown Source)
at service.GlobalManagerTest.testCloneGabaritVague(GlobalManagerTest.java:119)



Un peu d'aide ne serait pas de refus. ca commence vraiment àme prendre le chou et biensur je suis déjà à la bourre.

Merci d'avance, Julien.

[billynirvana]

Peux-t-on voir la méthode clone dans GabaritNote.java, surtout vers la ligne 51

[jc63]

Effectivement je ne suis même pas allé voir...

Code :

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

1
2
3
4
5
6
7
8
9
 
public Object clone(){
 
	GabaritNote gabaritNote = (GabaritNote) super.clone(); // <-- ligne 51
	gabaritNote.setGroupeNote(groupeNote);
 
	return gabaritNote;
 
}

[billynirvana]

D'après l'erreur capturée, la méthode super.clone() retourne un objet GabaritElement, alors que tu fais un

Code :

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

GabaritNote gabaritNote = (GabaritNote) super.clone();

Tu peux te débrouiller tout seul maintenant?

[adiGuba]

Salut,


Ce que je ne comprend pas c'est que tu manipules des GabaritElement et tu te retrouves dans des GabaritNote...

Il y a une relation entre ces deux classes ???

a++

[jc63]

D'après l'erreur capturée, la méthode super.clone() retourne un objet GabaritElement, alors que tu fais un

Code :

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

GabaritNote gabaritNote = (GabaritNote) super.clone();

Tu peux te débrouiller tout seul maintenant?

j'aimerais mais je ne vois pas bien comment faire. est-ce que je peux me sservir de la méthode clone de la classe mère. ou faut il que je réimplémente tout.

Salut,

Ce que je ne comprend pas c'est que tu manipules des GabaritElement et tu te retrouves dans des GabaritNote...

Il y a une relation entre ces deux classes ???
a++

ui GabaritNote est une sous classe de GabaritElement.

[adiGuba]

j'aimerais mais je ne vois pas bien comment faire. est-ce que je peux me sservir de la méthode clone de la classe mère. ou faut il que je réimplémente tout.
(...)
ui GabaritNote est une sous classe de GabaritElement.

Fait voir la méthode clone() de GabaritElement dans ce cas. Le problème vient surement de là

a++

[jc63]

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
 
public Object clone(){
 
	GabaritElement ge = new GabaritElement();
 
	ge.setElement(element);
	ge.setGabarit(gabarit);
	ge.setVague(vague);
	ge.setUsed(used);
	ge.setCommentaire(commentaire);
	ge.setDescription(description);
	ge.setNc(nc);
	ge.setTypeElement(typeElement);
 
	return ge;
}

[adiGuba]

Ok je comprend le problème...

La méthode clone() de GabaritElement est mal implémenté.
Pour faire une vrai méthode clone() il faut obligatoirement appelé la méthode super.clone() (ce qui revient à appeler la méthode clone de Object). Cette dernière crée un objet du même type que le type réel de l'objet...

Dans ton code la méthode GabaritNote.clone() appelle GabaritElement.clone() qui retourne un objet de type GabaritElement qui ne peut logiquement pas être caster en GabaritNote...

Bien sûr pour cela il faut implémenter Cloneable, et ta méthode clone() devrait plutôt ressembler à ceci :

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
	public Object clone() {
		try {
			// On appele la méthode clone() de la classe parente,
			// qui doit elle même faire de même jusqu'à appeler
			// la méthode clone de Object :
			Object cloned = super.clone();
 
			// On fait eventuellement quelques traitements sur l'objet cloné,
			// Mais le système de cloneage a théoriquement initialisé tous
			// les champs de la nouvelle instance donc c'est souvent inutile
 
			// On retourne l'instance cloné :
			return cloned;
		} catch(CloneNotSupportedException e) {
			// Ne devrait jamais arriver si on implémente Cloneable :
			throw new RuntimeException(e);
		}
	}

Plus d'info : Cloner en Java

a++

[jc63]

Merci pour vos réponses et surtout la dernière. maintnant je n'ai plus de pb de cast et mes objets on l'air d'être clonné correctement.

j'ai toujours un problème mais cette fois c'est hibernate qui ne m'enregistre pas mes nouveaux objets clonnés. par hasard, auriez vous une idée.
En fait une fois mon objet cloné, je met à null mon attribut Id (clé primaire) et j'enregistre mon objet.