Discussion :

[manguigs]

Bonjour,
Voulant illustrer le principe des exceptions personnalisé, je me permets d'écrire le code suivant qui fonctionne pas!
En fait j'ai une class livre2 avec comme propriété titre et prix.
je veux que si l'utilisateur entre un prix negatif une exceptions soit lévée

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
 
class prixexceptions extends Exception {
		public prixexceptions(){
		 System.out.println("le prix doit pas etre negatif");
		} 
}	
public class livre2{
	private String titre;
	private double prix;
 
	public livre2(String untitre, double unprix) throws prixexceptions{ 
			if(unprix<0){
				throw prixexceptions();
			} else {	
			   this.titre=untitre;
			   this.prix=unprix;
			  }
	}
	public livre2(){
	 this.titre="inconnu";
	 this.prix=0;
	} 
	public String toString(){
			String msg=" ";
			msg+="Le titre:"+" "+this.titre+("\n"); 
			msg+="Prix"+" "+this.prix;			
			return msg;
	}	
	public static void main(String[] args){ 
	livre2 l=null;
	 try{
	  l=new livre2("Booba",-200);
	 } catch (prixexceptions e){
	    l=new livre2();
	   }
 
	//livre l =new livre();
	System.out.println(l.toString());
		}
}

A la compilation j'ai deux erreurs que j'ai du mal à resoudre:
erreur1:

Code :

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

The serializable class prixexceptions does not declare a static final serialVersionUID field of type long

A la ligne 4 dont voici le code dans mon éditeur

Code :

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

	class prixexceptions extends Exception {

erreur2:

Code :

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

The method prixexceptions() is undefined for the type livre2

a la ligne 14 dont voici le code

Code :

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

	throw prixexceptions();

Bien merci pour votre aide

[Lucio_]

Essayes plutot :

Code :

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

throw new prixexceptions();

Edit: une meilleure facon de gérer une exception créé se trouve dans ce sujet:
http://www.developpez.net/forums/d88...eer-exception/

[Jidefix]

Bonjour,

Premièrement, ton exemple montre surtout l'importance de bien nommer ses classes:

Code :

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

throw prixexceptions()

devrait etre

Code :

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

throw new prixexceptions()

Tu confonds une méthode prixexceptions() qui n'existe pas avec le constructeur de la classe prixexceptions.
Je te conseille de renommer ta classe en PrixException (avec majuscules et sans "s").

Deuxièmement, ce qui illustre bien les exceptions, ce serait plus de try/catcher ta fonction et de faire différents traitements en fonction de l'exception, du style:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public livre2(String untitre, double unprix){ 
			try{
                            if(unprix<0){
				throw new PrixException();
			    } else {	
			    this.titre=untitre;
			    this.prix=unprix;
			  }
                          catch(PrixException pe){
                            //Faire quelque chose
                          }
	}

Troisièmement, en général on n'imprime pas dans le code d'une exception. Une exception n'a de sens que si elle est "throwée". Le code de base est plutôt celui-la:

Code :

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

1
2
3
4
5
class prixexceptions extends Exception {
		public prixexceptions(){
		 super("le prix doit pas etre negatif");
		} 
}

Enfin,

Code :

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

The serializable class prixexceptions does not declare a static final serialVersionUID field of type long

Ceci est un warning, pas une erreur. Ca ne t'empeche pas de compiler mais te signale juste qu'un truc de ta classe n'est pas tip top. En l'occurence, toute classe doit déclarée une variable qui ressemble à ça:

Code :

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

private static final long serialVersionUID = 1L;

M'enfin ça c'est pas important... enfin je crois pas en fait je sais pas à quoi ça sert...

[dingoth]

Mouais...


Le traitement en fonction des exceptions, c'est bien, mais c'est plutôt si on a un réel besoin d'avoir un autre comportement lorsqu'on attrape l'exception.

Dans ce cas-ci, c'était parfaitement correct (quoique j'aurais lancé une IllegalArgumentException plutôt qu'une PrixException).


Le besoin de définir un serialVersionUID ne se rencontre que pour les classes implémentant java.io.Serializable et ce dans le but de voir si les deux versions d'une même classe sont compatibles. En gros, ce serialVersionUID doit être changé dès que la signature d'une variable d'instance est ajoutée, supprimée ou modifiée.

[manguigs]

Bien merci
le

Code :

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

throw new prixexceptions()

Après cette modif le code fonctionne sans avoir eu à mettre

Code :

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

super

dans la classe prixexceptions au lieu de

Code :

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

System.out.println

Encore une fois bien merci

[Lucio_]

Le super n'est pas là pour permettre la compilation du programme, mais pour une meilleure utilisation de l'exception. Avec un System.println.out, toute exception entraînera la même réponse, alors qu'un "super" permettra de rendre l'exception plus spécifique. Sur un petit programme ça peut paraître inutile, mais sur un gros c'est assez important.

[tromaltsec]

Je rajouterai en plus que le System.out.println("") n'est pas terrible dans la mesure où si tu fais une application avec une interface graphique ton message ne s'affichera que dans une console (à condition que l'utilisateur ait lancé l'application dans la console). Donc en gros la seule chose que l'utilisateur verra c'est que le programme ne tourne pas