Discussion :

[Ceubex]

Bonjour,

Il y a un truc que je ne pige vraiment pas.

J'ai une classe que je déclare avec :
public class Grille implements Serializable,Cloneable

Dans une autre classe du projet je tappe :
Grille gr=new Grille(liste);
Grille grBis=gr.clone();

Et Eclipse me surlinge gr.clone(); en me disant que la methode clone n'est pas disponible pour cette objet !
Qu'est ce que je fais de mal ?

Merci d'avance

[avtonio]

salut
ta méthode clone est protected?

[tchize_]

par défaut clone est protected. Il faut donc la rendre publique:

Code :

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

1
2
3
4
5
6
public class Grille implements Serializable,Cloneable {
public Object clone(){
    return super.clone();
}
 
}

[adiGuba]

Salut,

Plus précisément, clone() est définie de la manière suivante :

Code :

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

protected Object clone() throws CloneNotSupportedException;

Donc comme indiqué tu dois la redéfinir en public si tu souhaites l'utiliser depuis d'autres classes :

Code :

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

1
2
3
4
5
public class Grille implements Cloneable {
	public Object clone() throws CloneNotSupportedException {
	    return super.clone();
	} 
}

Dans le même temps, je te conseillerai de "masquer" l'exception (qui ne devrait normalement "jamais" se produire) :

Code :

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

1
2
3
4
5
6
7
8
9
public class Grille implements Cloneable {
	public Object clone() {
	    try {
			return super.clone();
		} catch (CloneNotSupportedException e) {
			throw new UnsupportedOperationException(e); 
		}
	} 
}

Cela permet d'utiliser la méthode plus facilement, sans avoir à s'embêter d'une exception inutile (UnsupportedOperationException est non-checked, donc le compilateur ne t'obligera pas à la traiter).



Enfin, à partir de Java 5.0 tu peux également changer le type de retour, afin d'éviter les multiples cast inutile :

Code :

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

1
2
3
4
5
6
7
8
9
public class Grille implements Cloneable {
	public Grille clone() {
	    try {
			return (Grille) super.clone();
		} catch (CloneNotSupportedException e) {
			throw new UnsupportedOperationException(e); 
		}
	} 
}

Et voilà une méthode clone() bien plus pratique à utiliser

a++

[Ceubex]

Merci beaucoup. J'ai tout ce qui me faut et ça marche.

[Ceubex]

En fait j'ai encore un dernier soucis. Je souhaitais à la base cloner Grille pour prendre un de ses élements mais en fait je clone la réference et l'element n'est pas cloné.

Enfin bref, ça n'est plus grille que je veux cloner mais un Container.
Alors j'ai eu l'idée de faire une nouvelle classe :

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
class Container2 extends Container implements Cloneable
{
	Container2()
	{
		super();
	}
 
	public Container2 clone(){
	    try {
			return (Container2) super.clone();
		} catch (CloneNotSupportedException e) {
			System.out.println(e.toString());
		}
		return null;
	}
}

ou même simplement :

Code :

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

1
2
class Container2 extends Container implements Cloneable
{}

Mais à chaque fois lorsque je fais le cast :
(Container2) maGrille.sParent
Eclipse me met que
"main" java.lang.ClassCastException: javax.swing.JSplitPane cannot be cast to Container2
Je précise que maGrille.sParent est un JSplitPane en l'occurence parfois c'est autre chose mais c'est toujours un Component et la fonction finale demande un Container