problème d'entrée clavier qui ne ressort pas

**minimorphee** · 21/03/2014, 15h38

(re-)Bonjour !

Après moultes explications sur diverses problèmes et variés, et afin de me sortir un peu l'esprit du premier programme. J'ai voulu faire un exo qui me paraissait très simple au premier abord.

Voilà le speech :

Ecrire le traitement permettant de connaître, pour un jour donné de la semaine, tous les bâtiments des entreprises qui doivent faire l'objet d'une intervention.

Je me suis dit bon ça semble simple suffit de demander d'entrer un jour puis un mois et enfin une année, puis scan d'un tableau afin de voir ce qui correspond puis le faire ressortir.

Mais voilà mon soucis, le mieux est-il ici d'utiliser un tableau (avec nomEntreprise, BatimentEntreprise, jourNettoyage), ou alors une collection (je n'ai pas encore étudié le sujet juste vu le nom) ou alors une BDD (idem) ?

Merci à ceux qui pourront m'aider un peu !

**minimorphee** · 22/03/2014, 14h32

Petit up du message.

Alors je me suis dit pour le moment je vais essayer de me focaliser sur l'insertion des données par l'utilisateur.
J'ai donc voulu faire simple et faire :
rentre jour
rentre mois
rentre année

voilà le code que j'ai essayé de monter :

Code Main :

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
package exam;
 
import java.util.Scanner;
 
public class Main {
 
	public static void main(String[] args) {
char rep=' ';
Scanner sc=new Scanner(System.in);	
event E=new event();
int mois='0',jour='0',annee='0';
 
do{//boucle principale
 
	do{//boucle scanner
 
		E.jourChoix(jour);
		E.moisChoix(mois);
		E.anneeChoix(annee);
	System.out.println("jour =" +jour);
	System.out.println("mois ="+mois);
	System.out.println("annee ="+annee);
 
		do{//boucle validation date
		System.out.println("Vous désirez obtenir des informations sur la date du : " );
		System.out.println("Validez vous ce choix ? (O/N)");
		rep=sc.nextLine().charAt(0);
		if(rep!='O' && rep!='o' && rep!='N' && rep!='n')
			System.out.println("Désolé je ne comprends pas votre réponse");
		}while(rep!='O' && rep!='o' && rep!='N' && rep!='n');
	}while(rep!='O' && rep!='o');
 
}while(rep=='O' || rep=='o');
 
	}
 
}

Code Event :

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
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
package exam;
 
import java.util.Scanner;
 
public class event {
	Scanner sc=new Scanner(System.in);
 
public String[] tbleMois = {"janvier","février","mars","avril","mai","juin","juille","aout","septembre","octobre","novembre","decembre"};
 
 
 
public void jourChoix(int jour){
	do{//boucle de validation
		System.out.println("Veuillez entrer un jour s'il vous plait");
		jour=sc.nextInt();
		if(jour <=0){
			System.out.println("Ce nombre ne correspond pas à un jour valide, réssayer s'il vous plait");
		}
		else if(jour>=32){
			System.out.println("Ce nombre ne correspond pas à un jour valide, réssayer s'il vous plait");
		}
		else{
			System.out.println("vous avez choisit le jour : "+ jour);
		}
 
 
	}while(jour<=0 | jour>=32 );
}
 
 
public void moisChoix(int mois){
	do{//boucle de validation du nombre choisit
		System.out.println("Veuillez entrer un mois s'il vous plait");
		mois=sc.nextInt();
		if(mois <=0){
			System.out.println("Ce nombre ne correspond pas à un mois valide, réssayer s'il vous plait");
		}
		else if(mois>12){
			System.out.println("Ce nombre ne correspond pas à un mois valide, réssayer s'il vous plait");
		}
		else{
			System.out.println("vous avez choisit le mois : "+tbleMois[mois-1]);
		}
 
 
	}while(mois <=0 || mois>12);
}
 
 
public void anneeChoix(int annee){
		do{//boucle de validation
			System.out.println("Veuillez entrer une année s'il vous plait");
			annee=sc.nextInt();
			if(annee <2014){
				System.out.println("Cette anée est déjà passé, si vous désirez des informations sur celle-ci regarder dans les archives, merci.");
			}
			else if(annee==2014){
				System.out.println("vous avez choisit l'année : "+ annee);
			}
			else if(annee>2014){
				System.out.println("Cette année n'est pas encore visible, désolé");
			}
 
		}while(annee <2014 || annee>2014 || annee!=2014);
 
}
 
 
 
}

Seulement, quand j’exécute le code voilà ce que me ressort le programme

jour =48
mois =48
annee =48

Je continu de creuser pour savoir pourquoi, mais une fois de plus j'ai un peu de mal, alors je viens vous demander de l'aide.

**thelvin** · 22/03/2014, 16h57

Le but de tes méthodes jourChoix, moisChoix et anneeChoix, est d'obtenir des informations. Elles ne peuvent donc pas être void et doivent faire un return de l'information à obtenir.
De plus, elles n'ont pas besoin de paramètre pour obtenir ces informations. Elles ont juste besoin du Scanner qui fait partie de leur classe. Il ne faut donc pas qu'elles prennent de paramètre.

**minimorphee** · 22/03/2014, 17h21

Merci de ta réponse.

Si je comprends bien je dois donc changer mon code de la façon suivante ?

Code Event :

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
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
package exam;
 
import java.util.Scanner;
 
public class event {
	Scanner sc=new Scanner(System.in);
 
public String[] tbleMois = {"janvier","février","mars","avril","mai","juin","juille","aout","septembre","octobre","novembre","decembre"};
private int mois,jour,annee;
 
 
 
 
 
public int jourChoix(){
	do{//boucle de validation
		System.out.println("Veuillez entrer un jour s'il vous plait");
		jour=sc.nextInt();
		if(jour <=0){
			System.out.println("Ce nombre ne correspond pas à un jour valide, réssayer s'il vous plait");
		}
		else if(jour>=32){
			System.out.println("Ce nombre ne correspond pas à un jour valide, réssayer s'il vous plait");
		}
		else{
			System.out.println("vous avez choisit le jour : "+ jour);
		}
 
 
	}while(jour<=0 | jour>=32 );
	return jour;
}
 
 
public int moisChoix(){
	do{//boucle de validation du nombre choisit
		System.out.println("Veuillez entrer un mois s'il vous plait");
		mois=sc.nextInt();
		if(mois <=0){
			System.out.println("Ce nombre ne correspond pas à un mois valide, réssayer s'il vous plait");
		}
		else if(mois>12){
			System.out.println("Ce nombre ne correspond pas à un mois valide, réssayer s'il vous plait");
		}
		else{
			System.out.println("vous avez choisit le mois : "+tbleMois[mois-1]);
		}
 
 
	}while(mois <=0 || mois>12);
	return mois;
}
 
 
public int anneeChoix(){
		do{//boucle de validation
			System.out.println("Veuillez entrer une année s'il vous plait");
			annee=sc.nextInt();
			if(annee <2014){
				System.out.println("Cette anée est déjà passé, si vous désirez des informations sur celle-ci regarder dans les archives, merci.");
			}
			else if(annee==2014){
				System.out.println("vous avez choisit l'année : "+ annee);
			}
			else if(annee>2014){
				System.out.println("Cette année n'est pas encore visible, désolé");
			}
 
		}while(annee <2014 || annee>2014 || annee!=2014);
return annee;
}
 
 
 
}

**thelvin** · 22/03/2014, 19h46

Ben, ça marche déjà mieux, tu trouves pas ?

**minimorphee** · 22/03/2014, 21h01

Non je ne remarque pas de différence

J'ai toujours le même problème, désolé de t’embêter avec ça ^^

Mini

**joel.drigo** · 22/03/2014, 21h53

Mais tu as bien changé les appels aussi (sinon je ne vois pas pourquoi ça continuerait d'afficher la même chose) ?

Code :

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

jour = E.jourChoix();

En passant, si ça t'affiche 48, c'est parce que tu initialises tes variables de type int avec le char '0' dont le code est 48. Pas très logique d'initialiser une variable int décrivant un jour, un mois ou une année avec '0'.

**minimorphee** · 22/03/2014, 22h26

J'avais pas penser à faire ça, merci à vous deux problème résolu !

Avant ça m'affichait bien 48 exacte, mais maintenant tout va mieux (même en laissant l'initialisation des variables à 0).

Je sais que c'est pas le plus logique mais je fonctionne pas en terme de lundi/mardi and co, juste en numéro de jour/numéro de moi et numéro d'année. Avec mon niveau je fais comme ça pour le moment, je ferais ça mieux plus tard quand je saurais si je dois utiliser un tableau/uneliste/une bdd pour stocker les informations !

D'ailleurs question con du soir, il est possible dans un tableau d'enregistrer quelques choses comme suivant :

tbl[]={int jour, int/string mois, int année, string(ou int je pense que ça serait mieux)jour de nettoyage}

Car comme on déclare un tableau d'un type, est-il possible de multiplier les types à l'intérieur de celui là ?

Merci à vous deux en tout cas !

**joel.drigo** · 22/03/2014, 22h35

Un tableau est par définition une liste d'éléments de même type. On peut contourner ça en utilisant un Object[], dans lequel on peut mettre des valeurs de type quelconques, mais c'est non seulement se compliquer la vie, mais aussi faire un code qui n'est ni facile à maintenir, ni simple pour garantir un fonctionnement sans erreur. Enfin bref, c'est de la bidouille. Java est un langage Objet : pourquoi vouloir toujours utiliser des trucs non objets quand l'utilisation d'un objet simplifie les choses tout en assurant d'avoir un code qui sera plus facile à faire, à faire évoluer et à maintenir.

Si tu as une structure décrite par plusieurs informations, fait une classe, qui a les variables correspondantes.

**minimorphee** · 23/03/2014, 08h19

Je suis désolé mais pourrais tu me donner un exemple de comment faire dans mon exercice pour remplacer mon "tableau" qui n'est pas valide.

Car j'ai un peu du mal à voir comment faire.

**joel.drigo** · 23/03/2014, 11h04

Salut,

Tout d'abord pour en revenir à :

Envoyé par minimorphee

Je sais que c'est pas le plus logique mais je fonctionne pas en terme de lundi/mardi and co, juste en numéro de jour/numéro de moi et numéro d'année. Avec mon niveau je fais comme ça pour le moment,

ma remarque n'avait rien à voir avec le nom des jours, ni même avec le fait que tu stockes des numéros de jour. Pour initialiser un int à 0, on écrit int jour=0; et pas int jour='0';.

Sinon, pour répondre à ta question, pour faire une classe qui représente une structure de données, tu peux commencer comme ça, tout simplement :

Code :

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

1
2
3
4
5
6
public class MaStructureDeDonnees /*ou le nom qui te convient*/{
   public int jour;
   public String mois;
   public int annee;
   public int jourNettoyage; // ou String jourNettoyage
}

Dans ton code, pour s'en servir :

Code :

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

1
2
3
MaStructureDeDonnees data = new MaStructureDeDonnes();
data.jour = choixJour();
...

Bon, c'est une solution de base, qui présente, par exemple, le défaut qu'on peut modifier les variables comme on veut sans respecter les contraintes qui s'appliquent à elles, comme mettre -1 dans jour, par exemple. Pour débuter ça peut passer, mais ça peut favoriser les bugs. Le stockage du mois en String n'est pas la meilleure façon de faire non plus, mais pour débuter ça peut passer.

La seconde étape pour cette classe est d'avoir des variables privées et des méthodes pour modifier et lire les valeurs, ce qui te permettra de gérer les contraintes de manière interne à la classe 'c'est la classe elle-même qui sait comment se gérer :

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
public class MaStructureDeDonnees /*ou le nom qui te convient*/{
   private int jour;
   private int mois;
   private int annee;
   private int jourNettoyage; 
   private String[] tbleMois = {"janvier","février","mars","avril","mai","juin","juille","aout","septembre","octobre","novembre","decembre"};
 
   public void setJour(int jour) {
 
    if ( jour<=0 && jour>=32 ) throw new IllegalArgumentException("numéro de jour incorrect"); 
       this.jour=jour;
   }
 
   public int getJour() {
       return this.jour;
   }
 
   public void setMois(int mois) {
       if ( mois<=0 && mois>12 ) throw new IllegalArgumentException("numéro de mois incorrect"); 
       this.mois=mois;
   }
 
   public int getMois() {
       return this.mois;
   }
 
   public String getNomDuMois() {
       return tbleMois[this.mois-1];
   }
 
   ...
 
}

Dans ton code, pour s'en servir :

Code :

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

1
2
3
MaStructureDeDonnees data = new MaStructureDeDonnes();
data.setJour(choixJour());
...

Tu peux même modifier ta méthode de saisie, par exemple :

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
public void moisChoix(MaStructureDeDonnees data){
        int mois;
        boolean moisValide;
	do{//boucle de validation du nombre choisit
		System.out.println("Veuillez entrer un mois s'il vous plait");
		mois=sc.nextInt();
                try {
                    data.setMois(mois);
                    moisValide=true;
                }
                catch(IllegalArgumentException e) {
			System.out.println("Ce nombre ne correspond pas à un mois valide, réssayer s'il vous plait");
                    moisValide=false;
                }
	}while(!moisValide);
	System.out.println("vous avez choisit le mois : "+data.getNomDuMois());

Avec l'appel :

Code :

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

1
2
3
4
MaStructureDeDonnees data = new MaStructureDeDonnes();
jourChoix(data);
moisChoix(data);
...

**minimorphee** · 23/03/2014, 18h19

Merci de ta réponse, je ne connais pas encore bien les exceptions, je vais étudier ça, mais seulement dans une semaine (absence).

Je reviens vite vers vous pour continuer tout ça!

Merci encore

problème d'entrée clavier qui ne ressort pas

avec Java

Discussions similaires

Partager

Partager