Problème de vérification de chaîne de type entier

Version imprimable

Ce code est supposé doit permettre à terme de vérifier que nous sommes devant un n° de Secu
La vérification de la clé n'y est pas encore codée.

Code:

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
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
package autressources2;
// numsecsoc.java
 
/* 
 Voici le programme source numsecsoc.java écrit en langage Java qui correspond à
l’algorithme du numéro de Sécurité sociale. Aucun caractère accentué n’est utilisé
dans le nom des variables.
*/
 
 
class Num_Sec_Soc {
	private String ch;
 
	public String getCh() {
		return ch;
	}
 
	public void setCh(String ch) {
		this.ch = ch;
	}
 
	public void Affiche() {
		this.interprete_sexe();
		this.interprete_annee();
		this.interprete_mois();
		this.interprete_departement();
		this.interprete_commune();
		this.interprete_ordre();
 
	}
 
	@Override
	public String toString() {
		String sexe = ch.substring(0, 1);
		String annee = ch.substring(1, 3);
		String mois = ch.substring(3, 5);
		String departement = ch.substring(5, 7);
		String commune = ch.substring(7, 10);
		String cle = ch.substring(10, 12);
		return "Num_Sec_Soc :" + sexe + "." + annee + "." + mois + "." + departement + "." + commune + "." + cle;
	}
 
 
 
	public boolean verifier_sexe() {
		/*
		 * référence wikipedia sexe : 1 pour les hommes, 2 pour les femmes, 3 ou
		 * 7 pour les personnes étrangères de sexe masculin en cours
		 * d'immatriculation en France , 4 ou 8 pour les personnes étrangères de
		 * sexe féminin en cours d'immatriculation en France
		 */
 
		if (TestErreur(ch.substring(0, 1))) {
			int sexe = Integer.parseInt(ch.substring(0, 1));
 
			switch (sexe) {
			case 1:
				return true;
			case 2:
				return true;
			case 3:
				return true;
			case 4:
				return true;
			case 7:
				return true;
			case 8:
				return true;
			default:
				return false;
			}
		} else {
			return false;
		}
	}
 
	public void interprete_sexe() {
 
		/*
		 * référence wikipedia sexe : 1 pour les hommes, 2 pour les femmes, 3 ou
		 * 7 pour les personnes étrangères de sexe masculin en cours
		 * d'immatriculation en France 4 ou 8 pour les personnes étrangères de
		 * sexe féminin en cours d'immatriculation en France
		 */
		int sexe = Integer.parseInt(ch.substring(0, 1));
		if (verifier_sexe()) {
			switch (sexe) {
			case 1:
				System.out.println("Bonjour Monsieur");
				break;
			case 2:
				System.out.println("Bonjour Madame");
				break;
			case 3:
				System.out.println("Bonjour Monsieur");
				break;
			case 4:
				System.out.println("Bonjour Madame");
				break;
			case 7:
				System.out.println("Bonjour Monsieur");
				break;
			case 8:
				System.out.println("Bonjour Madame");
				break;
			    } 
			}
		else {
			System.out.printf("Code sexe %d incorrecte \n", sexe);
		}
 
	}
 
	public boolean verifier_annee() {
		String  annee_c =  ch.substring(1, 3);
		if(TestErreur(annee_c)) {
			return true;
		}
		return false;
	}
 
	public void interprete_annee() {
		String  annee_c =  ch.substring(1, 3);
		if(TestErreur(annee_c)) {
			int annee = Integer.parseInt(ch.substring(1, 3)) + 1900;		
			System.out.println("Vous êtes né en : " + annee);	
		}
 
	}
 
 
 
	public boolean verifier_mois() {
		/* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */
		int mois = Integer.parseInt(ch.substring(3, 5));
		if(mois >=1 || mois >=12 || mois == 62  || mois == 63) {
			return true;
		} else {
			return false;
		} 
 
	}
 
	public void interprete_mois() {
		/*
		 * de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu
		 * 
		 */
		int mois = Integer.parseInt(ch.substring(3, 5));
		String Nom_mois = new String("");
 
		if(this.verifier_mois()) {
		switch (mois) {
		case 1:
			Nom_mois = "Janvier";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 2:
			Nom_mois = "Février";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 3:
			Nom_mois = "Mars";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 4:
			Nom_mois = "Avril";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 5:
			Nom_mois = "Mai";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 6:
			Nom_mois = "Juin";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 7:
			Nom_mois = "Juillet";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 8:
			Nom_mois = "Août";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 9:
			Nom_mois = "Septembre";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 10:
			Nom_mois = "Octobre";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 11:
			Nom_mois = "Novembre";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 12:
			Nom_mois = "Décembre";
			System.out.println("Au mois de : " + Nom_mois);
			break;
		case 62:
			System.out.println("Référence de mois inconnu pour personne d'origine étrangère");
			break;
		case 63:
			System.out.println("Référence de mois inconnu pour personne d'origine étrangère");
			break;
		}
		} else {
			System.out.println("Erreur de code mois");
		}
	}
 
	private  boolean TestErreur(String cc ) {
		char[] s = cc.toCharArray();
		boolean testerreurfrappe = true;
		for (int i = 0; i < s.length; i++) {
			if (!Character.isDigit(s[i])) {
				testerreurfrappe = false;
			}
		}
		return testerreurfrappe;
	} 
 
	public boolean verifier_departement() {
		String departement = ch.substring(5, 7);
		int numdep;
		//de 01 à 19 ou 21 à 95 ou 2A ou 2B, 96 à 99 pour des naissances à l'étrange
 
		switch (departement) {
		case "2A" :
			departement ="20";
		case "2B" :
			departement ="20";
			}	
		return false;
	}
	public void interprete_departement() {
		// aucun département ne comporte plus de 999 communes
		//de 01 à 19 ou 21 à 95 ou 2A ou 2B, 96 à 99 pour des naissances à l'étrange
 
		int departement = Integer.parseInt(ch.substring(5, 7));
		System.out.println("Dans le département : " + departement);
	}
 
 
	public boolean verifier_commune() {
		if (TestErreur( ch.substring(7, 10))) {
			return true;
		}
		return false;
	}
 
	public void interprete_commune() {
		String commune = ch.substring(7, 10);
		if (TestErreur(commune)) {
			System.out.println("Dans la commune : " + commune);
		} else{
 
		}
	}
 
	public void interprete_ordre() {
 
		int ordre = Integer.parseInt(ch.substring(10, 12));
		System.out.println("Avec le numéro d’ordre : " + ordre);
	}
	public void interprete_cle() {
		/*
		 * clé de contrôle = complément à 97 du nombre 
		 * formé par les 13 premiers chiffres du NIR modulo 97 F
		 */
		int ordre = Integer.parseInt(ch.substring(13, 15));
		System.out.println("Avec le numéro d’ordre : " + ordre);
	}
}

Le lanceur

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
package autressources2;
import java.io.*;
 
public class Start {
	// programme principal
	public static void main(String[] args) {
		// TODO Auto-generated method stub
		 Num_Sec_Soc Numero = new Num_Sec_Soc() ;
		 String ch= Saisie.lire_String("Entrez votre numéro de Sécurité sociale : ");
		 Numero.setCh(ch);
		 System.out.println(Numero);
 	     Numero.interprete_sexe();
		 Numero.interprete_annee();
		 Numero.interprete_mois();
		 Numero.interprete_departement();
		 Numero.interprete_commune();
		 Numero.interprete_ordre();
		 Numero.interprete_cle();
 
 
	}
 
}

Citation:

voici la saisie d'un code incorrecte : but vérifier que les tests de validation fonctionne
Num_Sec_Soc :a.15.65.46.545.65
le code erreur d'éclipse

Exception in thread "main" java.lang.NumberFormatException: For input string: "a"
at java.lang.NumberFormatException.forInputString(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at autressources2.Num_Sec_Soc.interprete_sexe(numsecsoc.java:85)

Je cherche ou est l'erreur.
voic un autre test le N° de sécu pourrait correspondre

Citation:

Entrez votre numéro de Sécurité sociale : 215890544565415
Num_Sec_Soc :2.15.89.05.445.65
Bonjour Madame
Vous êtes né en : 1915
Dans le département : 5
Dans la commune : 445
Avec le numéro d’ordre : 65
Avec le numéro d’ordre : 15

Salut,

L'endroit où se trouve l'erreur est indiquée dans la stacktrace :

Code:

autressources2.Num_Sec_Soc.interprete_sexe(numsecsoc.java:85)

Ligne 85, dans le fichier numsecsoc.java, classe Num_Sec_Soc, méthode interprete_sexe.

Soit la ligne

Code:

int sexe = Integer.parseInt(ch.substring(0, 1));

forcément si tu saisis : a15654654565, le premier caractère est a et le transformer en int est impossible puisque c'est une lettre. D'où exception.

Alors, oui, tu fais un test pour tester si la saisie ne contient pas que des chiffres avec TestErreur, mais la méthode, appelée dans verifier_sexe, est appelée après la ligne 85 :
Code:

1 2 int sexe = Integer.parseInt(ch.substring(0, 1)); if (verifier_sexe()) {
Donc ça ne sert à rien. C'est avant qu'il faut tester.
Code:

1 2 3 4 if (verifier_sexe()) { int sexe = Integer.parseInt(ch.substring(0, 1)); /* ... traitement */
Ton code manque globalement de logique : tu fais les contrôles à l'affichage, selon cette séquence :

extraire un bout de chaîne
le convertir en int
test si le bout de chaîne est valide

tester si le bout de chaîne contient des chiffres
le convertir en int
tester si le int est valide
si le test est valide alors afficher des trucs en fonction de la valeur convertie en 1

Comme le test est fait à l'affichage : ça veut dire qu'on peut affecter un code de sécurité sociale (par la méthode setCh) et ne savoir qu'il est incorrect que quand on l'affiche par la méthode affiche().

Dans setCh, tu pourrais faire le test et soulever une exception (ce qui une manière plus courante en Java de traiter les cas d'erreur que retourner un booléen). Si tu testes tout de suite que le code fait une longueur de 13 caractères (ou de 15 si tu inclus le traitement de la clé), puis que tu testes que ça contient que des chiffres ensuite, plus besoin de faire ce test ensuite, pour chaque partie du code. Cela dit, on pourrait largement se passer de ce test, parce que le numéro de sécurité social est surtout un code, pas un nombre, même s'il n'est constitué que de chiffres.

On pourrait faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public void verifierSexe(char c) { switch(c) { case '1': case '2': case '3': case '4': case '7': case '8': break; default: throw new IllegalArgumentException("Code de sexe incorrecte"); } }
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 public void setCh(String ch) { if ( ch.length!=13 && ch.length!=15 ) { throw new IllegalArgumentException("Il devrait y avoir 13 chiffres (ou 15 avec la clé)"); } for(int i=0; i<ch.length; i++) { char c=ch.charAt(i); if ( c<'0' || c>'9' ) { if ( i!=7 || !(ch.charAt(i-1)=='2' && (c=='A'||c=='B')) ) { // cas de la Corse throw new IllegalArgumentException("Caractère incorrect en position " + (i+1) +": " + c); } } } } verifierSexe(ch.charAt(0)); // verifier le reste if ( ch.length==15 ) { // verifier la clef } this.ch=ch; }
Code:

1 2 3 4 5 6 7 8 9 10 11 boolean ok=false; do { String ch= Saisie.lire_String("Entrez votre numéro de Sécurité sociale : "); try { Numero.setCh(ch); ok=true; } catch(IllegalArgumentException e) { System.err.println("Erreur de saisie : " + e.getMessage()+"\nRecommencez..."); } } while(!ok);
Par ailleurs, normalement une classe doit être mise dans un fichier qui porte le même nom, casse comprise. Faire ce que tu fais, créer un fichier numsoc.java pour y mettre une classe Num_Sec_Soc, est un détournement possible du système mais ça va te faire prendre de mauvaises habitudes et ensuite tu seras gêné pour importer tes classes dans un autre package. Oublie même cette possibilité de faire des classes non publiques dans les fichiers Java : ça ne sert qu'à rendre confus le code.
Enfin, il y a des conventions d'écriture en Java, qui facile la lecture du code par tous, parce que ça fait un système commun. Aussi nommer ses méthodes ou variable avec un nom qui a du sens facilite aussi cette lecture et la compréhension du code. Un petit peu de logique également : appeler une méthode TestErreur qui retourne true lorsqu'il n'y a pas d'erreur n'est pas logique (estValide(), nestPasEnErreur()...) ou alors testErreur() retourne true lorsqu'il y a erreur.

04/09/2017, 20h54
dedalios

Réflection pertinente

Merci je vais reprendre tous cela.
Un N° de sécu n'est pas uniquement numérique 2A 2B pour le code de département sont un codage possible selon wikipedia...
Étrangement je n'en ai jamais vue...

Le code est une récupération qui ne contenais que les affichages sans aucune forme de contrôle je cherche donc a intégrer les processus de contrôle
04/09/2017, 21h01
joel.drigo

Citation:

Envoyé par dedalios

Un N° de sécu n'est pas uniquement numérique 2A 2B pour le code de département sont un codage possible selon wikipedia...

Oui, oui, c'est pourquoi j'ai inclus ce cas dans mon exemple.

Citation:

Envoyé par dedalios

Étrangement je n'en ai jamais vue...

Personnellement, je n'ai pas vu des masses de numéro de sécu, à part le mien, et n'étant pas natif de Corse... jamais vu non plus.

Probleme

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
  public void setCh(String ch) {
    if ( ch.length!=13 && ch.length!=15 ) {
         throw new IllegalArgumentException("Il devrait y avoir 13 chiffres (ou 15 avec la clé)");
    }
    for(int i=0; i<ch.length; i++) {
         char c=ch.charAt(i);
         if ( c<'0' || c>'9' ) {
              if ( i!=7 || !(ch.charAt(i-1)=='2' && (c=='A'||c=='B')) )  { // cas de la Corse
                 throw new IllegalArgumentException("Caractère incorrect en position " + (i+1) +": " + c);
              }
            }
         }
    }
    verifierSexe(ch.charAt(0));
    // verifier le reste
    if ( ch.length==15 ) {
        // verifier la clef
    }
    this.ch=ch;
}

le code a un problème

Code:

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
 
class Numsecsoc {
  private String ch; 
 
  public String getCh() {
		return ch;
  }
 
  public void verifierSexe(char c) {
	   switch(c) {
	      case '1':
	      case '2':
	      case '3':
	      case '4':
	      case '7':
	      case '8':
	         break;
	      default:
	         throw new IllegalArgumentException("Code de sexe incorrecte");
	         }
	   }
 
  public void setCh(String ch) {
	    if ( ch.length()!=13 && ch.length()!=15 ) {
	         throw new IllegalArgumentException("Il devrait y avoir 13 chiffres (ou 15 avec la clé)");
	    }
 
	    for(int i=0; i<ch.length(); i++) {
	         char c=ch.charAt(i);
	         if ( c<'0' || c>'9' ) {
	              if ( i!=7 || !(ch.charAt(i-1)=='2' && (c=='A'||c=='B')) ) 
	              	{ // cas de la Corse
	                 throw new IllegalArgumentException("Caractère incorrect en position " + (i+1) +": " + c);
	              	}
	            }
	         }
	    verifierSexe(ch.charAt(0));
	    // verifier le reste
	    if ( ch.length()==15 ) {
	        // verifier la clef
	    }
	    this.ch=ch;
	}
}

04/09/2017, 23h01
joel.drigo

Citation:

Envoyé par dedalios

le code a un problème

Tu l'as corrigé par toi même :ccool:

J'ai un doute sur ce code

Code:

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
 
  public void verifier_mois(String chaine) {
		/* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */
		String mois = chaine.substring(3, 5);
		char c0 = mois.charAt(0);
		char c1 = mois.charAt(1);
 
		boolean test=true;
		  switch(c0) {
		  case '0':
				if (c1 < '1'  || c1 >'9') {
					test = false;
				} 
				else {
					test = true;
					}
			  break;
		  case '1':
				if (c1 < '0'  || c1 >'2') {
					test = false;
				} 
				else {
					test = true;
					}
			  break;
		  case '6':
			  if (c1 != '2'  || c1 != '3') {
					test = false;
				} 	
			  else {
					test = true;
					}
			  break;
 
		  }
 
		if(test) {
			throw new IllegalArgumentException("Erreur sur le mois : ");
		}
		//mois >="01" || mois >="12" || mois == "62"  || mois == "63"
  }

est-il cohérent j'avoue le perdre entre le terme "ou" ou ceci ou cela est le ou logique ? je me croise les yeux sesoir

Salut,

Les OU me semblent correct.
Par contre tu t'embête un peu à traiter des chaines...

:arrow: Converti le tout en int et tu pourras traiter çà plus simplement :
Code:

1 2 3 4 5 6 7 8 public void verifier_mois(String chaine) { /* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */ String mois = chaine.substring(3, 5); int m = Integer.parseInt(mois); if ( m<1 || (m>12 && m!=62 && m!=63) ) { throw new IllegalArgumentException("Erreur sur le mois : "); } }

Ne pas oublier le cas où la chaine n'est pas convertible en int, sinon on aura le même souci qu'au début : une NumberFormatException.

Au plus simple, on peut simplement attraper l'exception :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public void verifier_mois(String chaine) {
	/* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */
	String mois = chaine.substring(3, 5);
        int m;
        try {
	 m = Integer.parseInt(mois);
        }
        catch(NumberFormerException e) {
		throw new IllegalArgumentException("Erreur sur le mois : " + mois,e);
        }
	if ( m<1 || (m>12 && m!=62 && m!=63) ) {
		throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
	}
}

ou en inversant :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
public void verifier_mois(String chaine) {
	/* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */
	String mois = chaine.substring(3, 5);
        try {
	         int m = Integer.parseInt(mois);
	         if ( (m>=1 && m<=12) || m==62 || m==63) ) {
                    return;
	         }
        }
        catch(NumberFormerException e) {
        }
	throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
}

Ou tester :

Code:

1
2
3
4
5
6
7
8
String mois = chaine.substring(3, 5);
        if ( mois.charAt(0)<'0' || mois.charAt(0)>'9' || mois.charAt(1)<'0' || mois.charAt(1)>'9' ) {
		    throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
        }
        int m = Integer.parseInt(mois);
 	if ( m<1 || (m>12 && m!=62 && m!=63) ) {
		throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
	}

Il y a également les expressions régulières, mais c'est peut-être prématuré par rapport à ton niveau ;

Code:

1
2
3
4
5
6
7
String mois = chaine.substring(3, 5);
         if ( !mois.matches("\\d+") {
		    throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
         }
         if ( m<1 || (m>12 && m!=62 && m!=63) ) {
		throw new IllegalArgumentException("Erreur sur le mois : "+ mois);
	}

06/09/2017, 07h24
adiGuba

Citation:

Envoyé par joel.drigo

Ne pas oublier le cas où la chaine n'est pas convertible en int, sinon on aura le même souci qu'au début : une NumberFormatException.

Une exception si la chaine est incorrect : cela me semble tout indiqué ;)

C'est vrai qu'on peut l'intercepter pour remonter une autre exception, mais c'est pas forcément utile.

Par contre je ne comprend pas trop l'intérêt de faire des vérifications AVANT le parseInt().
Ce dernier renverra une erreur si ce n'est pas un entier valide. C'est amplement suffisant et bien plus court...
06/09/2017, 09h02
joel.drigo

Citation:

Envoyé par adiGuba

Ce dernier renverra une erreur si ce n'est pas un entier valide. C'est amplement suffisant et bien plus court...

C'est un choix fonctionnel : soit des exceptions spécifiques pour chaque cas, soit une exception unique qui regroupe tous les cas (une vision plus métier). Laisser la NumberFormatException est plus court dans l'implémentation de la méthode, moins dans le traitement du cas à l'appel, ou il faudra penser d'une part à signaler l'ensemble des exceptions dans la JavaDoc et faire un multicatch dans l'appelant, donc moins court qu'un catch unique (on pourrait catcher RuntimeException, certe, mais je n'aime pas trop ce genre de généralisation dans le traitement de l'exception à l'appel, pourquoi pas catch Throwable partout tant qu'on y est). Si je suis plutôt partisan des exceptions spécifiques (et laisser les exceptions existantes se soulever telles quelles) pour les méthodes techniques, je suis plutôt partisan des exceptions uniques pour les cas métier.
06/09/2017, 19h50
adiGuba

D'accord sur le principe : il faut documenter.

Pour le reste NumberFormatException est déjà une IllegalArgumentException.
Il n'y a rien de choquant à remonter une classe fille ;)
06/09/2017, 20h02
joel.drigo

Citation:

Envoyé par adiGuba

Pour le reste NumberFormatException est déjà une IllegalArgumentException.

Mince, j'avais oublié ça :oops:. Effectivement, dans ce cas, on pourrait la laisser remonter. Reste le message qui ne dit pas sur quelle partie du numéro de sécu il y'a un problème. Si l'appelant veut pouvoir dire sur quoi il y a l'erreur, il ne pourra pas (il ne va pas analyser la stacktrace pour ça:P). Dans une optique métier, il faudrait sûrement faire une exception spécifique, un peu à la ParseException, dans laquelle on aurait des attributs permettant d'indiquer à l'utilisateur quelle partie est incorrecte (sous forme métier : code de sexe incorrect, département invalide, etc).

HUMM

Citation:
Envoyé par adiGuba
Salut,

Les OU me semblent correct.
Par contre tu t'embête un peu à traiter des chaines...

:arrow: Converti le tout en int et tu pourras traiter çà plus simplement :
Code:

1 2 3 4 5 6 7 8 public void verifier_mois(String chaine) { /* de 01 (janvier) à 12 (décembre) ou 62 ou 63 pour un mois non connu */ String mois = chaine.substring(3, 5); int m = Integer.parseInt(mois); if ( m<1 || (m>12 && m!=62 && m!=63) ) { throw new IllegalArgumentException("Erreur sur le mois : "); } }
je suppose que IllegalArgumentException prendrais en compte la problèmatique de valeur non numérique . si la saisie est 1xx2a321ze68ze1ze132 (soit quelque chose d'absude en soit )