Discussion :

[adiGuba]

L'important est de remonter l'exception et de ne surtout pas l'ignorer, avec l'exception source comme parent...

Pour le reste c'est à l'appréciation de chacun. Perso j'ai pris l'habitude d'utiliser RuntimeException mais on pourrait utiliser n'importe quoi, y compris une exception perso.



Mais en fait le top du top serait de remonter directement CloneNotSupportedException comme si c'était une unchecked-exception, afin que le compilateur ne bronche pas lorsqu'on ne la traite pas...
Ce qui n'est malheureusement pas autoriser par le langage pour le moment... (même s'il existe un "hack" permettant cela - j'essayerais de détailler cela dans un billet blog si j'ai un peu de temps).


a++

[alex2746]

Merci pour vos conseils mais j'ai un petit soucis pour implémenter Cloneable.

Ma classe Personne implémente déjà autre chose...
Est il possible de faire deux implements pour une même classe? Comment faire?

Code :

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

1
2
 
public class Person implements uneInterface implements Cloneable {...}

ne fonctionne pas je suppose...

Merci d'avance.

[dingoth]

Bien sûr :

Code :

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

public class Person implements uneInterface, Cloneable {...}

[hibour]

À y réfléchir, ne serait-il pas plus correct de lancer une Error dans ce cas ?

Oui lancer une erreur c'est plus judicieux je pense et c'est plus clair pour tout le monde, un InternalError encapsulant l'exception ne laissera pas d'ambiguïté..
La java doc de UnsupportedOperationException dit qu'on lance cette excpetion si l'opération n'est pas supportée alors que dans notre cas il s'est passé une erreur qu'on s'attendait pas du tout si on implémente bien Cloneable et on autorise le clonage:

Code :

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

1
2
3
4
5
6
7
public Object clone() {
   try{
	return super.clone();
   }catch(Exception e) {
       throw new XXXException(e); //une unchecked exception
   }
}

Voila