appel de methodes dans une meme classe

Version imprimable

12/11/2017, 18h00
BaDinh

appel de methodes dans une meme classe

Bonjour,

J'ai un exercice qui me demande d'appeler, a partir d'une methode statique string, une autre methode staique booleen dans une meme classe.

Soit :

public static String saisirEtValiderCodePostal (String invite, int noCivique) appelle public static boolean positionLettresEtChiffresAlternee (String codePostal).

Le paramètre invite est le message qui sera affiché pour demander à l'utilisateur
d'entrer un code. Le paramètre noCivique servira lors de la validation du code.
Retourne le code valide.

Honnetement, je ne sais pas le faire et je ne le vois nulle part dans mon cours de Java pour debutant.

Merci pour votre aide.

Commence donc par écrire ta classe avec ses deux méthodes statique dedans.
Ensuite tu verras que ce n'est pas très différent (dans ce cas précis) de l'invocation de méthodes non-statiques (que tu as deja du voir en cours j'imagine).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public class Truc { public static void faitKekchose(String unPremierParametre, int unSecondParametre) { [...] boolean leResultatDeLAppel = faitUnTruc(leBonParametre); [...] } public static boolean faitUnTruc(String unParametre) { [...] return unResultaKelconke; } }

Bonsoir,

Merci pour votre aide.

Voicie ce que je propose :

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
 
public static String saisirEtValiderCodePostal(String invite, int noCivique)
    {
        System.out.println(invite);
 
        // ...
        String codePostal;
 
        // appel de la methode public static int saisirEtValiderNoCivique 
        //(String invite, int min, int max)
        noCivique = saisirEtValiderNoCivique (invite, 1, 9999);
        //
 
 
        //appel de la methode public static boolean 
        //positionLettresEtChiffresAlternee (String codePostal)
        boolean result = positionLettresEtChiffresAlternee(codePostal);
        //
 
        int strlen;
        strlen= codePostal.length();
 
        while (codePostal.length() == 6) {
            codePostal = Clavier.lireString();
 
            // verifie et valide la longueur du code postal
            do  {
                System.out.println("ERREUR! La longueur du code postal doit" +
                    "etre egale a 6!");
                codePostal = Clavier.lireString();
 
            } while (strlen!=6) ; 
            //
 
            //Valide le code postal, parite pair et impair entre 
            //le num civique et la troisieme lettre alphabet du code postal
            if(codePostal.indexOf(codePostal.charAt(4))%2 ==0 && noCivique%2 !=0){
                System.out.println ("ERREUR! La parite de la troisieme lettre du code"+ 
                    "postal doit correspondre a celle du numero civique!" );
 
            }
 
            if(codePostal.indexOf(codePostal.charAt(4))%2 !=0 && noCivique%2 ==0){
                System.out.println ("ERREUR! La parite de la troisieme lettre du code"+ 
                    "postal doit correspondre a celle du numero civique!" );
 
            }
            //
        }
        return codePostal;
    }

Par contre je ne sais pas quoi faire avec le result.

J'ai un autre probleme : dans le main je souhaiterai faire appel a cette methode mais il m'affiche une erreur :

Code:

String codePostal = saisirEtValiderCodePostal("Entrez un code postal SANS espaces pour la PREMIERE adresse", NoCivique);

J'ai tout mis, codePostal, result..rien a faire !!!

Merci pour votre aide.

Cordialement.

14/11/2017, 00h34
bouye

Citation:

Envoyé par BaDinh

mais il m'affiche une erreur :

Erreur qui est ?
14/11/2017, 00h38
BaDinh
dans le main :
Code:

1 2 String codePostal = saisirEtValiderCodePostal("Entrez un code postal SANS" + "espaces pour la PREMIERE adresse", noCivique);
il m'affiche " Cannot find Symbol - variable noCivique". Si je l'enleve, une autre erreur s'affiche pour "saisirEtValiderCodePostal" : cannot be applied to given types...".

Merci.
14/11/2017, 00h59
bouye

L'erreur est plutôt claire (bien sur si tu ne maîtrise pas l'anglais je suis conscient que ce n'est peut-être pas le cas pour toi, pas de soucis de ce cote-la) : il n'existe aucune variable noCivique a cet endroit (PS : dans le 1er code que tu as copie il y a un n majuscule, vérifie ton code, il y a peut-être une faute de frappe ou tu as nommé la variable différemment).

De ce cote j'ai verifie, pas d'erruer de frappe.

Voici ce que j'ai dans le main au complet:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public static void main (String[] params) { int no1 = saisirEtValiderNoCivique("Entrez un numero" + "civique pour la PREMIERE adresse : ", 1, 9999); int no2 = saisirEtValiderNoCivique("Entrez un numero" + "civique pour la DEUXIEME adresse : ", 1, 9999); String codePostal = saisirEtValiderCodePostal("Entrez un code postal SANS" + "espaces pour la PREMIERE adresse", noCivique ); boolean saisie = continuer("Voulez-vous traiter des combinaisons" + "de codes postaux et de numeros civiques (oui / non)?"); } // main
1. Si je remplace No1 par noCivique, l'erreur disparait .
2. Une autre erreur apparait dans la methode String saisirEtValiderCodePostal(String invite, int noCivique) :
boolean result = positionLettresEtChiffresAlternee(codePostal); /// "variable codePostal might not have been initialized".
A moins que je fasse dans cette methode : codePostal = Clavier.lireString(); qui lui est deja repete dans le while dans cette methode !!!!

Bizarre non !!!

Merci

Salut,

Citation:

Envoyé par BaDinh

1. Si je remplace No1 par noCivique, l'erreur disparait .

C'est normal, le message d'erreur "Cannot find Symbol - variable noCivique" dit qu'aucune variable du nom "noCivique" n'est accessible à l'endroit où il y a l'erreur. Si tu renommes une variable accessible à cet endroit "noCivique", elle existe alors, donc le message n'a plus lieu d'être.

Ce n'est pas parce que la signature de la méthode saisirEtValiderCodePostal(String invite, int noCivique) a un second paramètre de nom "noCivique" qu'il faut que dans l'appel la variable s'appelle comme ça. Il faut simplement passer la valeur attendue, éventuellement par une variable qui existe.

De ce cote j'ai verifie, pas d'erruer de frappe.
Citation:
Envoyé par BaDinh
Voici ce que j'ai dans le main au complet:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public static void main (String[] params) { int no1 = saisirEtValiderNoCivique("Entrez un numero" + "civique pour la PREMIERE adresse : ", 1, 9999); int no2 = saisirEtValiderNoCivique("Entrez un numero" + "civique pour la DEUXIEME adresse : ", 1, 9999); String codePostal = saisirEtValiderCodePostal("Entrez un code postal SANS" + "espaces pour la PREMIERE adresse", noCivique ); boolean saisie = continuer("Voulez-vous traiter des combinaisons" + "de codes postaux et de numeros civiques (oui / non)?"); } // main
On voit que la variable noCivique n'est pas déclarée dans la méthode main. Tu as deux variables, no1 et no2, c'est l'une de celle-ci qu'il faut passer, si l'une de celle-ci correspond bien à l'information attendue en paramètre (ce qu'il semble être le cas, d'après le nom, il s'agit du numéro civique, ce qu'on a bien l'air de saisir dans no1 et no2. Et comme il s'agit du code postal correspondant à la première adresse, il semblerait que ça soit no1 qu'il faille passer. Remplacer no1 par noCivique, ou remplacer noCivique par no1 dans le code précédent en bref.

Citation:

Envoyé par BaDinh

2. Une autre erreur apparait dans la methode String saisirEtValiderCodePostal(String invite, int noCivique) :
boolean result = positionLettresEtChiffresAlternee(codePostal); /// "variable codePostal might not have been initialized".
A moins que je fasse dans cette methode : codePostal = Clavier.lireString(); qui lui est deja repete dans le while dans cette methode !!!!
Bizarre non !!!

Non. Pour pouvoir utiliser la valeur d'une variable, ce qui est fait quand tu la passes en paramètre dans un appel de méthode, il faut qu'elle ait une valeur, donc qu'elle ait été affectée au moins une fois de façon sure. Si la variable est affectée dans un while, le compilateur peut ne pas être sûr qu'on rentre bien dans le while, donc que l'affectation dans le while soit bien effectuée, et donc considère que la variable peut ne pas avoir de valeur et refuse donc de compiler. A toi de t'arranger pour qu'il y ait forcément une valeur affectée, quitte à mettre une valeur bidon, et que le reste du code garantisse qu'au final il y ait une valeur valide.

Le code suivant ne compile pas :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 public static void main(String[] args) { String valeur; boolean saisieInvalide=true; while(saisieInvalide) { System.out.println("Saisir oui ou non:"); valeur = Clavier.lireString(); if ( "oui".equalsIgnoreCase(valeur) || "non".equalsIgnoreCase(valeur) ) { saisieInvalide=false; } } System.out.println(valeur); }
Le code suivant compile, parce qu'on a affecté une valeur (null) à valeur, qui n'est bien sûr pas une valeur qu'on veut utiliser après, mais comme on entre forcément dans le while, et qu'on en sort que si on a une valeur valide, pas de problème pour la suite...
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 public static void main(String[] args) { String valeur=null; boolean saisieInvalide=true; while(saisieInvalide) { System.out.println("Saisir oui ou non:"); valeur = Clavier.lireString(); if ( "oui".equalsIgnoreCase(valeur) || "non".equalsIgnoreCase(valeur) ) { saisieInvalide=false; } } System.out.println(valeur); }

Bonjour,

merci pour votre aide.

Au sujet de :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public static void main(String[] args) {
 
 
		String valeur=null;
 
		boolean saisieInvalide=true;
		while(saisieInvalide) {
			System.out.println("Saisir oui ou non:");
			valeur = Clavier.lireString();
			if ( "oui".equalsIgnoreCase(valeur) || "non".equalsIgnoreCase(valeur) ) {
				saisieInvalide=false;
			}
		}
 
		System.out.println(valeur);
 
	}

J'avais deja cree auparavant une methode pour la saisie oui non, dont le voici :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public static boolean continuer (String msgInvite){
        String saisie = null;  
 
        System.out.println(msgInvite);
 
        while(true)
        {            
            saisie = Clavier.lireString();
 
            if(saisie.equalsIgnoreCase("oui") || saisie.equalsIgnoreCase("o"))
                return true;
 
            if(saisie.equalsIgnoreCase("non") || saisie.equalsIgnoreCase("n"))
                return false;
            System.out.println("ERREUR! La reponse doit etre oui, non , o ou n" );
 
        }
    }

et dans le main :

Code:

1
2
3
 
boolean saisie = continuer("Voulez-vous traiter des combinaisons" + 
                "de codes postaux et de numeros civiques (oui / non)?");

Qu'en pensez vous ?

14/11/2017, 16h07
joel.drigo

Citation:

Envoyé par BaDinh

Qu'en pensez vous ?

Que c'est une bonne solution à ton exercice... mais que ça n'a rien à voir avec le problème que tu exposes dans la présente discussion, et pour lequel j'ai donné juste un exemple de principe pour illustrer ma réponse.

Merci,

Mais c'est bon a savoir aussi.

Desole du detournement.

Au sujet de la derniere discussion , c'est regle !!!

Par contre quand je compile et que je teste, ca crashe lorsque je valide le code postal.

Pour utiliser la methode "public static String saisirEtValiderCodePostal(String invite, int noCivique)", j'ai aussi fait appel a la methode "public static boolean positionLettresEtChiffresAlternee (String codePostal)", dont voici le 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
42
43
44
45
46
47
48
49
50
51
public static boolean positionLettresEtChiffresAlternee (String codePostal){
 
        // au cas où il a été entré en minuscules
        codePostal=codePostal.toUpperCase();
 
        boolean result;
 
        // valeurs permises
        String a= "ABCDEFGHIJKLMNOPQRSTUVWXYZ" ;        
        String c= "0123456789" ;
 
        while (result = false){
            // compare, une par une, les valeurs entrées à celles permises
            if(a.indexOf(codePostal.charAt(0))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ;
            }
 
            if(c.indexOf(codePostal.charAt(1))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ; 
            }
 
            if(a.indexOf(codePostal.charAt(2))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ; 
            }
 
            if(c.indexOf(codePostal.charAt(3))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ; 
            }
 
            if(a.indexOf(codePostal.charAt(4))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ;
            }
 
            if(a.indexOf(codePostal.charAt(5))<0){
                System.out.println ("ERREUR! Le format du code postal doit etre :" + 
                    "Lettre Chiffre Lettre Chiffre Lettre Chiffre. " );
                return false ; 
            }
        }
        return result;
    }

Cette methode, qui retourne faux si le format nest pas respecte, ne verifie pas la longueur du code postal et ne fait pas de saisie ni d'affichage.

Merci pour votre aide.

14/11/2017, 16h42
joel.drigo
Citation:

Envoyé par BaDinh

Par contre quand je compile et que je teste, ca crashe lorsque je valide le code postal.

Ne jamais utiliser d'expression comme "ça crashe", ou "ça marche pas". Nous dire "j'ai telle erreur : "le message affiché", ou "j'ai tel résultat xxx alors que je devrais avoir yyy", ou ça affiche 3 fois tel truc alors que je voudrais que ça l'affiche qu'une fois, etc.
1. Code:
  
  if(a.indexOf(codePostal.charAt(0))<0){
  
  ne jamais appeler charAt(une constante) sans avoir testé la taille de la chaîne avant. Si la chaîne est vide, ça fait une erreur "StringIndexOutOfBountException" parce que le caractère de position 0 n'existe pas dans la chaîne.
2. Code:
  
  while (result = false) {
  
  Attention, en Java l'opérateur = est TOUJOURS l'opérateur d'affectation même dans une condition. Le code que tu écris mets donc false dans result, puis teste s'il faut entrer dans le while : result étant false, alors la réponse est non, donc on n'exécute pas le while. Et on retourne, à la fin de la méthode, result, donc false. Ta méthode retourne donc toujours false, quelque soit la saisie.
  
  L'opérateur de test d'égalité pour les types primitifs, comme boolean, est
  Code:
  
  1 2 3 while (result == false) { /*...*/ }
  Ou alors :
  Code:
  
  1 2 3 while (!result) { /*...*/ }
3. En l’occurrence, ton while ne sert à rien, puisque tu testes toutes les positions par des constantes de 0 à 5. Une boucle est une bonne idée, vu qu'on a trois fois de suite la même condition, mais un peu plus difficile à mettre au point pour une débutante.
4. A noter qu'à la place d'un indexOf(), pour tester l'appartenance à un intervalle de caractères, on peut comparer directement le caractère avec des caractères :
  Code:
  
  1 2 3 if ( 'A'<=c && c<='Z' ) { // c est une lettre comprise entre A et Z, incluses }
  Code:
  
  1 2 3 if ( c<'A' || 'Z'<c ) { // c n'est pas une lettre comprise entre A et Z, incluses }
14/11/2017, 17h18
BaDinh

Citation:

1.

Code:

if(a.indexOf(codePostal.charAt(0))<0){

ne jamais appeler charAt(une constante) sans avoir testé la taille de la chaîne avant. Si la chaîne est vide, ça fait une erreur "StringIndexOutOfBountException" parce que le caractère de position 0 n'existe pas dans la chaîne.

Ok, je sais que le codePostal doit etre : lettreChiffreLettreChiffrelettreChiffre. Donc n'est-il pas suffisant d'initialiser les string a et c. De plus le le string codePostal est deja declare.

A moins que je transforme les String a et c en constantes ou que je definisse le length du codePostal !!!

Merci
14/11/2017, 17h24
joel.drigo

Citation:

Envoyé par BaDinh

Ok mais n'est-il pas suffisant d'initialiser les string a et c. De plus le le string codePostal est deja declare.

Le fait que codePostal soit vide n'a strictement rien à voir ni avec a ni avec c. Et le fait qu'une variable soit déclarée ne garantit rien au sujet de son contenu, qui plus est si la valeur est issue d'une saisie clavier.

Citation:

Envoyé par BaDinh

A moins que je transforme les String a et c en constantes ou que je definisse le length du codePostal !!!

Aucun rapport non plus, constantes ou pas ça ne change rien. Et la longueur du code postal est aussi à tester à priori : si le code postal doit être trois alternances successives d'une lettre et d'un chiffre, un code postal qui n'est pas de longueur 6 n'est pas valide !
14/11/2017, 17h57
BaDinh

Donc il suffit juste de mettre un :

codePostal = Clavier.lireString();

dans le while.
14/11/2017, 18h32
joel.drigo

Citation:

Envoyé par BaDinh

Donc il suffit juste de mettre un :

codePostal = Clavier.lireString();

dans le while.

Probablement pas puisque c'est un paramètre de ta méthode : il est censé être saisi ailleurs et cette méthode ne fait que le contrôle (d'après son nom). Ou alors cette méthode serait censée gérer la saisie, et donc, le paramètre devrait être supprimé et remplacé par une variable locale, et le type de la méthode ne serait plus boolean, mais String (le retour serait le code postal). Cela change complètement la spécification de la méthode. Si la signature de la méthode t'est imposée, ce n'est certainement pas la solution.

Par ailleurs, ça ne change rien au fait que codePostal peut être vide. Que Clavier.lireString() soit appelé dans le while, dans la méthode hors du while, ou dans une autre méthode, ça ne change rien au fait que l'utilisateur peut appuyer sur la touche entrée sans avoir saisi aucun autre caractère.
15/11/2017, 00h33
BaDinh

Merci.

Je vois ce que vous voulez me dire. Mais... non, je ne vois pas comment en code .

Dans ma tete, le codage que j'ait me parait correct... mais il y a erreur !!! Va donc savoir !!!

L'erreur est-elle dans l'une des deux methodes ou dans les deux ?

Si c'est dans l'une, je me verrai plus a revoir la methode saisirEtValidercodePostale.

Si c'est les deux... c pas bien parti pour moi !!!!

Merci pour votre aide !!! Un poquito hint en code m'aiderait peut-etre :D

Bonsoir,

je me suis permis de faire quelques ameliorations dans 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
 
public static boolean positionLettresEtChiffresAlternee (String codePostal){
 
        // retourne faux si le format du code n'est pas une alternance de lettres 
        //et chiffres. Retourne vrai si le format est respecte. Ne verifie pas 
        //la longueur du code, ne fait pas de saisie ni d'affichage.
 
        boolean code;
        code = true;
 
        // au cas où il a été entré en minuscules
        codePostal=codePostal.toUpperCase();
 
        // valeurs permises
        String a= "ABCDEFGHIJKLMNOPQRSTUVWXYZ" ;        
        String c= "0123456789" ;
 
        // compare, une par une, les valeurs entrées à celles permises
        if(a.indexOf(codePostal.charAt(0))<0
        && c.indexOf(codePostal.charAt(1))<0
        && a.indexOf(codePostal.charAt(2))<0
        && c.indexOf(codePostal.charAt(3))<0
        && a.indexOf(codePostal.charAt(4))<0
        && c.indexOf(codePostal.charAt(5))<0){
 
            return false;
 
        }
        return code;
    }

Je ne vois pas d'erreurs de codage pour le moment !!!

15/11/2017, 10h56
joel.drigo

Citation:

Envoyé par BaDinh

L'erreur est-elle dans l'une des deux methodes ou dans les deux ?

Je ne sais pas de quelle erreur tu parles maintenant. D'ailleurs, je ne sais pas de quelle erreur on parlait, puisque tu ne me l'a pas dit (tu m'as juste dit : "ca crashe" et je t'ai dit que ça ne voulait rien dire, et que tu devais me dire quelle erreur tu avais. Du coup, j'ai juste commenté le code que tu montrais. Et on parlait jusqu'ici de positionLettresEtChiffresAlternee. Donc quand tu parles de while, j'en déduis que tu parles du while dans cette méthode. Et dans cette méthode, il n'y a pas à lire le code postal, puisqu'il est passé en paramètre, par appel dans saisirEtValiderCodePostal dans laquelle on fait sa lecture au clavier.

Citation:

Envoyé par BaDinh

Je ne vois pas d'erreurs de codage pour le moment !!!

Bah, moi j'en vois une de taille : le &. Le code postal est invalide si toutes les conditions sont vraies en même temps. Pourtant les codes postaux "1ab2c3" ou "aab2c3" sont invalides, non ?

Et puis je t'ai dit que si le code postal saisi ne fait pas 6 caractères exactement, par exemple "a1b2", une exception sera soulevée à l'exécution (et ton programme va s'arrêter).

Sinon, un détail, la variable code ne sert à rien, tu peux directement faire return true; à la fin de la méthode.

Bonsoir,

Merci pour votre aide.

Voici ce que je propose :

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
 
 
public static boolean positionLettresEtChiffresAlternee (String codePostal){
 
 
 
        boolean code;
        code = true;
 
        // au cas où il a été entré en minuscules
        codePostal=codePostal.toUpperCase();    
 
        // compare, une par une, les valeurs entrées à celles permises
        if( (codePostal.length() == 6)        
        && (Character.isLetter(codePostal.charAt(0)))
        && (Character.isDigit(codePostal.charAt(1)))
        && (Character.isLetter(codePostal.charAt(2)))
        && (Character.isDigit(codePostal.charAt(3)))
        && (Character.isLetter(codePostal.charAt(4)))
        && (Character.isDigit(codePostal.charAt(5))))            
        {
            return true;
        } else {
            return false;
        }
 
    }

16/11/2017, 11h42
joel.drigo
Tu enlèves déjà ça :
Code:

1 2 3 4 boolean code; code = true;
qui ne sert à rien, et c'est déjà beaucoup mieux.

Seulement, il faut juste savoir deux ou trois choses au sujet de Character.isLetter() et Character.isDigit(), qu'on peut lire dans leur javadoc.

Regarde bien, Character.isDigti(c) n'est pas exactement équivalent à (c>='0' && c<='9'). Pareil pour isLetter() (d'ailleurs, avec isLetter() le passage en haut de casse de codePostal n'est pas nécessaire.

Merci pour votre aide !!!

Selon le sujet de l'exercice, je n'ai pas le droit d'utiliser le for. Donc n'ayant pas trouve d'autres solutions ( je sais qu'il en existe plein), j'ai choisis ce code.

En le testant sur la console, ca a l'air de bien marche.

Donc pour mon niveau de debutant de 2 mois, je trouve que c deja pas mal !!!

maintenant je vous transmets le code complet a 90 %.

En utilisant la console apres la saisie d'un code postal (bon ou mauvais), on m'affiche ceci:
"Java. lang.NumberFormatException: For input string "A1a1a1"
at java.lang.NumberFormatException.forInputString(NumberFormatException.java:65)
at java.lang.Integer.parseInt(Integer.java:580)
at java.lang.Integer.parseInt(Integer.java:615)
at Clavier.lireInt(Clavier.java:201)
at test.saisirEtValiderNoCivique(test.java:23)
at test,saisirEtValiderCodePostal(test.java:61)
at.main(test.java:127)

Voici le 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
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
 
public static int saisirEtValiderNoCivique (String invite, int min, int max){
 
        int noCivique;
 
        System.out.println(invite);
 
        do  {
            noCivique = Clavier.lireInt();
            if (noCivique < min || noCivique > max){
 
                System.out.println ("ERREUR! Le numero doit etre entre 1 et" + 
                    "9999 inclusivement.");
 
            }
 
        } while (noCivique < min || noCivique > max);
        return noCivique;
    }
 
    public static boolean positionLettresEtChiffresAlternee (String codePostal){
 
        // compare, une par une, les valeurs entrées à celles permises
        if( (codePostal.length() == 6)        
        && (Character.isLetter(codePostal.charAt(0)))
        && (Character.isDigit(codePostal.charAt(1)))
        && (Character.isLetter(codePostal.charAt(2)))
        && (Character.isDigit(codePostal.charAt(3)))
        && (Character.isLetter(codePostal.charAt(4)))
        && (Character.isDigit(codePostal.charAt(5))))            
        {
            return true;
        } else {
            return false;
        }
 
    }
 
    public static String saisirEtValiderCodePostal(String invite, int noCivique)
    {
 
        // s'occupe de la saisie et de l'affichage des messages
       //d'erreur et peut appeler des méthodes de validation.
      //Le paramètre noCivique servira lors de la validation du code.
     //Retourne le code valide.
 
 
        String codePostal="";
 
        // appel de la methode public static int saisirEtValiderNoCivique 
        //(String invite, int min, int max)
        noCivique = saisirEtValiderNoCivique (invite, 1, 9999);     
        boolean valid = false;
 
        while (!valid){     
            codePostal = Clavier.lireString();
            valid = positionLettresEtChiffresAlternee(codePostal);
            codePostal = codePostal.toUpperCase();
 
            if(codePostal.length() != 6){
                // saisie code postal different de 6 caracteres: msg erreur
                System.out.println ("ERREUR! La longueur du code postal doit etre" + 
                    "egale a 6!");
                codePostal = Clavier.lireString();
 
 
            } else if (codePostal.charAt(4) %2 == 0 && noCivique %2 != 0){
                // parite 3 eme lettre alphabet pair avec num civique impair: msg erreur
                System.out.println ("ERREUR! La parite de la troisieme lettre du code"+ 
                    "postal doit correspondre a celle du numero civique!" );
                codePostal = Clavier.lireString();
 
            } else if (codePostal.charAt(4) %2 !=0 && noCivique %2 == 0){
                // parite 3 eme lettre alphabet impair avec num civique pair: msg erreur
                System.out.println ("ERREUR! La parite de la troisieme lettre du code"+ 
                    "postal doit correspondre a celle du numero civique!" );
                codePostal = Clavier.lireString();
 
            }
 
        }
 
        return codePostal;
    }
 
    public static boolean continuer (String msgInvite){
        String saisie = "";  
 
        while(true)
        {
 
            System.out.println(msgInvite);
            saisie = Clavier.lireString();
 
 
 
            if(saisie.equalsIgnoreCase("oui") || saisie.equalsIgnoreCase("o")){
                return true;
            }
 
            if(saisie.equalsIgnoreCase("non") || saisie.equalsIgnoreCase("n")){
                System.out.println("ERREUR! La reponse doit etre oui, non , o ou n" );
                return false;
            }
 
 
        }
    }
 
 
 
    // Autres m�thodes s'il y a lieu
    public static void main (String[] params) {
 
        int no1 = saisirEtValiderNoCivique("Entrez un numero" +
                "civique pour la PREMIERE adresse : ", 1, 9999);
 
        String code1 = saisirEtValiderCodePostal("Entrez un code postal SANS" + 
                " espaces pour la PREMIERE adresse", no1 );
 
        int no2 = saisirEtValiderNoCivique("Entrez un numero" +
                "civique pour la PREMIERE adresse : ", 1, 9999);
 
        String code2 = saisirEtValiderCodePostal("Entrez un code postal SANS" + 
                " espaces pour la PREMIERE adresse", no2 );
 
        boolean saisie = continuer("Voulez-vous traiter des combinaisons" + 
                "de codes postaux et de numeros civiques (oui / non)?");
 
    } // main

Merci de bien vouloir m'aider !!!

16/11/2017, 20h39
joel.drigo
Nonobstant le fait que la virgule soit bizarre, parce que ça devrait être un point (à moins que tu aies bidouillé la trace avant de la coller ici) :

Citation:

at test.saisirEtValiderNoCivique(test.java:23)
at test,saisirEtValiderCodePostal(test.java:61)

cette partie de la trace indique que le problème survient lors de l'appel de la méthode saisirEtValiderNoCivique dans la méthode saisirEtValiderCodePostal

et que

Citation:

"Java. lang.NumberFormatException: For input string "A1a1a1"

indique qu'on cherche à convertir "A1a1a1" en int, ce qui est impossible, en base 10 du moins (pour une saisie utilisateur, j'exclus à priori qu'on demande de saisir autre chose que du décimal).
1. En tout cas, la méthode saisirEtValiderNoCivique ne devrait pas être appelée dans saisirEtValideCodePostal. C'est non seulement illogique, mais également incohérent, vu qu'on passe cette information en paramètre d'appel de la méthode saisirEtValideCodePostal, parce qu'on l'a déjà saisie. La saisir une nouvelle fois ne sert à rien, est redondant et, en plus, on peut saisir quelque chose de différent de ce qu'on a saisi dans la boucle principale.
2. Je ne connais pas cette classe Clavier, puisqu'elle ne fait pas partie du JDK. Je ne connais donc pas son fonctionnement. Je ne comprends pas d'ailleurs pourquoi on continue de demander aux étudiants et élèves d’utiliser ce genre de truc alors qu'il y a une classe dans le JDK qui fonctionne très bien et pour laquelle je pourrais éventuellement avoir des idées. La seule que je peux avoir est que lorsqu'on demande la saisie du numéro civique alors qu'on devrait pas, tu saisi le code postal.
3. En dehors de la logique booléenne qui est une base indispensable pour la programmation quelque soit le langage, que la consultation de la javadoc est fortement recommandée, voir indispensable, pour programmer en Java, la compréhension d'une stack trace est également recommandée, voire fortement indispensable, et ce n'est pas vraiment très compliqué (il suffit de savoir lire, d'avoir quelques rudiments d'anglais, et un peu de logique).
16/11/2017, 23h17
BaDinh

Merci.

Mais c flou tout !!!

Pourriez vous me pointer a quels endroits je dois effectuer les modifications.

Pour le comment et le quoi mettre, je me debrouillerai !!!

Amicalement.

Voila ce que je propose pour la methode saisir et valider le code postal :

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
 
public static String saisirEtValiderCodePostal(String invite, int noCivique)
    {
        System.out.println(invite);
        // ...
        String codePostal="";
 
        // appel de la methode public static int saisirEtValiderNoCivique 
        //(String invite, int min, int max)
 
        boolean valid = false;
 
        codePostal = Clavier.lireString();
 
        while (!valid){     
 
            valid = positionLettresEtChiffresAlternee(codePostal);
            codePostal = codePostal.toUpperCase();
 
            if(codePostal.length() != 6){
                // saisie code postal different de 6 caracteres: msg erreur
                System.out.print ("\nERREUR! La longueur du code postal doit etre" + 
                    " egale a 6!");
                codePostal = Clavier.lireString();
 
            }else if (valid = true){
                System.out.print ("\nERREUR! Le format du code postal doit etre : " +
                    "\nLettre Chiffre Lettre Chiffre Lettre Chiffre.\n");
                codePostal = Clavier.lireString();
 
            }
 
 
 
        }
 
        return codePostal;
    }

ce que je voudrais faire maintenant dans le while c'est de valider la parite paire et impaire entre la 3eme lettre du code postal et le num civique. C-a-d si le num civique est pair alors la 3eme lettre du code est paire, donc true et pas de message d'erreur !!! Je sais que je dois utiliser un charAt() et du int car je ne peux pas comparer du boolean avec u int !!!

mais comment faire ... zat is ze freaking question!!!

Avant de répondre à ta question, il y a une double erreur dans ce bout de code :
Code:

1 2 3 4 5 }else if (valid = true){ System.out.print ("\nERREUR! Le format du code postal doit etre : " + "\nLettre Chiffre Lettre Chiffre Lettre Chiffre.\n"); codePostal = Clavier.lireString();
if ( valid = true ) signifie mettre true dans valid, puis après tester si valid est true, et si c'est le cas exécuter le bloc du if.
Donc la condition de ce if sera toujours vraie quoiqu'il arrive, et donc tu afficheras "ERREUR ! le foramt du code ...blah blah"... donc ça ne va pas du tout !!!

Même si tu écrivais if ( valid == true ), ça voudrait dire que si c'est valide, alors ça affiche qu'il y a une erreur !!! Donc ça ne va pas à priori.

Pour ta question, je ne vois pas pourquoi tu parles de comparer du booléen et du int. Si la parité d'un caractère doit être égale à la parité d'un autre, il suffit de tester si les parités sont égales. Cela semble être une lapalissade écrit comme ça, non ?

Regarde ce code :
Code:

1 2 3 4 5 6 7 8 int pariteDeLun = /* ici calcul de la parité */ int pariteDeLautre = /* ici calcul de la parité */ if ( pariteDeLun==pariteDeLautre ) { // les deux sont soit paires tous les deux, soit impaires tous les deux } else { // l'un est paire et l'autre impaire : leur parité sont différentes. }
Pourquoi j'utilise du int pour stocker une parité ? Parce qu'on peut facilement calculer une parité en utilisant un modulo, c'est à dire le reste de la division, par 2 en l’occurrence, puisqu'un nombre n entier est un nombre paire s'il existe un k entier tel que k*2=n et un nombre n est impaire s'il existe un k tel que k*2+1=n. Si on calcule n/k, on voit que le reste de cette division est 0 quand n est pair et 1 quand il est impaire.
Soit
Code:

1 2 3 4 5 6 int parite = nombre%2; if ( parite==0 ) { // nombre est paire } else { // nombre est impaire }
Il ne te suffit plus que de convertir un char qui contient un chiffre en int pour pouvoir calculer la parité. Et comparer celle de deux chars.

OK c regle !!! J'avance !!!

G un autre pbs !!! Comme d'hab !

J'ai un probleme avec ce code en java et je vois pas comment regler !!!
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public static boolean continuer (String msgInvite){ String saisie = ""; while(true) { System.out.println(msgInvite); saisie = Clavier.lireString(); if(saisie.equalsIgnoreCase("oui" ) || saisie.equalsIgnoreCase("o" )){ return true; } if(saisie.equalsIgnoreCase("non" ) || saisie.equalsIgnoreCase("n" )){ System.out.println("\nERREUR! La reponse doit etre oui, non , o ou n" ); return false; } } }
sur la console on m'affiche ceci :

Voulez-vous traiter des combinaisons de codes postaux et de numeros civiques (oui / non)?
non

ERREUR! La reponse doit etre oui, non , o ou n
Entrez un numerocivique pour la PREMIERE adresse :

L'idee est de repondre oui ou o ou non ou n. Si je saisis ouii ou ou noon ou naan ou autre que ce qui est emntionne debut , ca m'affiche Erreur.

Si je sais non ou n . Le programme s'arrete.

je saisis oui ou o le programme continue.

Merci !!!

Amicalement.

Lis bien ton code !
Code:

1 2 3 4 5 6 if(saisie.equalsIgnoreCase("non" ) || saisie.equalsIgnoreCase("n" )){ System.out.println("\nERREUR! La reponse doit etre oui, non , o ou n" ); return false; } }
Il me semble évident que le message d'erreur s'affiche lorsque saisie est "non" ou saisie est "n" !

C'est comme ça que ton code devrait être :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 while(true) { System.out.println(msgInvite); saisie = Clavier.lireString(); if(saisie.equalsIgnoreCase("oui" ) || saisie.equalsIgnoreCase("o" )){ return true; } if(saisie.equalsIgnoreCase("non" ) || saisie.equalsIgnoreCase("n" )){ return false; } System.out.println("\nERREUR! La reponse doit etre oui, non , o ou n" ); }
Là on fait le test pour "oui" et "o", et on quitte la méthode si saisie est l'un des deux, puis on fait le test pour "non" et "n" et on quitte la méthode si saisie est l'un des deux, et sinon on continue, et on affiche le message d'erreur. On affiche donc bien le message si les deux tests de validité échouent et juste avant de reboucler pour redemander la saisie.

Ca m'affiche la meme chose.

la methode conitnuer :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
 
public static boolean continuer (String msgInvite){
        String saisie = "";  
 
 
        while(true)
        {
            System.out.println(msgInvite);
            saisie = Clavier.lireString();
            if(saisie.equalsIgnoreCase("oui" ) || saisie.equalsIgnoreCase("o" )){
                return true;
            }
 
            if(saisie.equalsIgnoreCase("non" ) || saisie.equalsIgnoreCase("n" )){
                return false;
            }
 
            System.out.println("\nERREUR! La reponse doit etre oui, non , o ou n" );
 
        }
 
    }

et le main :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
boolean saisie = continuer("Voulez-vous traiter des combinaisons" + 
                    "de codes postaux et de numeros civiques (oui / non)?");
 
        int no1 = saisirEtValiderNoCivique("Entrez un numero" +
                "civique pour la PREMIERE adresse : ", 1, 9999);
 
        String code1 = saisirEtValiderCodePostal("Entrez un code postal SANS" + 
                " espaces pour la PREMIERE adresse", no1 );
 
        int no2 = saisirEtValiderNoCivique("Entrez un numero" +
                "civique pour la DEUXIEME adresse : ", 1, 9999);
 
        String code2 = saisirEtValiderCodePostal("Entrez un code postal SANS" + 
                " espaces pour la DEUXIEME adresse", no2 );

et ca m'affiche ceci :

Citation:

Voulez-vous traiter des combinaisonsde codes postaux et de numeros civiques (oui / non)?
n
Entrez un numerocivique pour la PREMIERE adresse :

je souhaiterais que quand je saisie non ou n ou 0: ca m'affiche un message de fin de programme.

19/11/2017, 18h20
BaDinh

Bonjour,

Je viens de regler mes problemes de boucles dans mon main.... C cool !!!

Par je souhaiterais savoir comment on fait pour compter le nombre de "oui" ou "o" saisis.

J'ai deja declarer et initialiser une variable int nbreAnalyse = 0 et int nbreTotalAnalyse = 0 ; nbreTotalAnalyse = nbreAnalyse +1

Par la suite je dois incrementer ... mais comment ? Je ne peux pas utiliser du String "oui" ou "o" avec du int.

Pourriez vous m'indiquer comment faire.

Merci pour votre aide.

Cordialement
21/11/2017, 17h10
BaDinh

Bonjour,

Merci bcp pour votre aide.

Finalement j'ai reussi a resoudre mon probleme.

Cordialement.