Discussion :

[johnny3]

Bonjour,

J'ai un tableau d'objets avec le nom d'élèves et 3 de leurs notes. Je voudrais, quand je rentre les noms des élèves, avoir un message d'erreur du type "nom déjà donné" et être invité à retaper le nom de l'élève.

Je me suis basé sur un autre programme que j'avais écrit et qui fonctionnait, mais là, j'obtiens le message d'erreur en boucle.

Je vous mets ici mon programme, qui fonctionne sauf pour ce message d'erreur:

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
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
/*
 * a) Définir une classe « ELEVE » ayant pour données le nom d’un élève , sa note de français , sa
 * note de mathématiques et sa note d’informatique . Les trois notes comptent à parts égales . La
 * classe comportera une méthode pour établir la moyenne de ces trois notes . b)Définir une
 * application « GROUPE » permettant d’enregistrer les données pour cinq élèves . L’ application
 * devra indiquer le nom et la moyenne de chaque élève . Elle devra aussi calculer et afficher la
 * moyenne générale du groupe .
 */
class eleve {
	private String nom;
	private double noteFrancais = 0;
	private double noteMath = 0;
	private double noteInfo = 0;
	private double moyenneEleve = 0;
 
	public double calculMoyenneEleve()
	{
		this.moyenneEleve = (this.noteFrancais + this.noteMath + this.noteInfo) / 3;
		return moyenneEleve;
	}
 
	public void setMoyenneEleve(double me)
	{
		this.moyenneEleve = me;
	}
 
	public double getMoyenneEleve()
	{
		return this.moyenneEleve;
	}
 
	public void setNomEleve(String nom)
	{
		this.nom = nom;
	}
 
	public String getNomEleve()
	{
		return this.nom;
	}
 
	public void setNoteMath(double nm)
	{
		this.noteMath = nm;
	}
 
	public double getNoteMath()
	{
		return this.noteMath;
	}
 
	public void setNoteInfo(double ni)
	{
		this.noteInfo = ni;
	}
 
	public double getNoteInfo()
	{
		return this.noteInfo;
	}
 
	public void setNoteFrancais(double nf)
	{
		this.noteFrancais = nf;
	}
 
	public double getNoteFrancais()
	{
		return this.noteFrancais;
	}
 
	static eleve lireDonneesEleves()
	{
		eleve res = new eleve();
		Terminal.ecrireStringln("Entrez le nom de l'élève: ");
		res.nom = Terminal.lireString();
		Terminal.sautDeLigne();
		do
		{
			try
			{
				Terminal.ecrireStringln("Entrez la note de Français de l'élève: ");
				res.noteFrancais = Terminal.lireDouble();
				if (res.noteFrancais < 0 || res.noteFrancais > 20)
				{
					Terminal.ecrireStringln("La note ne peut être négative ou dépasser 20");
					Terminal.sautDeLigne();
				}
			} catch (Exception e)
			{
				Terminal.ecrireStringln("mauvaise saisie");
				res.noteFrancais = -1;
			}
		} while (res.noteFrancais < 0 || res.noteFrancais > 20);
		Terminal.sautDeLigne();
 
		do
		{
			try
			{
				Terminal.ecrireStringln("Entrez la note de Mathématiques de l'élève: ");
				res.noteMath = Terminal.lireDouble();
				if (res.noteMath < 0 || res.noteMath > 20)
				{
					Terminal.ecrireStringln("La note ne peut être négative ou dépasser 20");
					Terminal.sautDeLigne();
				}
			} catch (Exception e)
			{
				Terminal.ecrireStringln("mauvaise saisie");
				res.noteMath = -1;
			}
		} while (res.noteMath < 0 || res.noteMath > 20);
		Terminal.sautDeLigne();
 
		do
		{
			try
			{
				Terminal.ecrireStringln("Entrez la note d'Informatique de l'élève: ");
				res.noteInfo = Terminal.lireDouble();
				if (res.noteInfo < 0 || res.noteInfo > 20)
				{
					Terminal.ecrireStringln("La note ne peut être négative ou dépasser 20");
					Terminal.sautDeLigne();
				}
			} catch (Exception e)
			{
				Terminal.ecrireStringln("mauvaise saisie");
				res.noteInfo = -1;
			}
		} while (res.noteInfo < 0 || res.noteInfo > 20);
 
		res.moyenneEleve = res.calculMoyenneEleve();
		Terminal.sautDeLigne();
		return res;
	}
 
	void affichageDonneesEleves()
	{
		Terminal.ecrireStringln("Nom de l'élève:" + this.getNomEleve());
		Terminal.ecrireStringln("Note de Français:" + this.getNoteFrancais());
		Terminal.ecrireStringln("Note de Mathématiques:" + this.getNoteMath());
		Terminal.ecrireStringln("Note d'Informatique:" + this.getNoteInfo());
		System.out.printf("Moyenne: " + "%.2f \n", this.getMoyenneEleve());
		Terminal.sautDeLigne();
	}
}
class groupe {
	private final int NB_ELEVES = 5;
	public eleve [] tabEleves = new eleve[NB_ELEVES];
	int nbEleves = 0;
	private double moyenneGenerale = 0;
 
	public int getNbEleves()
	{
		return this.NB_ELEVES;
	}
 
	public void calculMoyenneGenerale()
	{
		for (int i = 0; i < NB_ELEVES; i++)
		{
			this.moyenneGenerale += this.tabEleves[i].getMoyenneEleve();
		}
		this.moyenneGenerale /= NB_ELEVES;
	}
 
	public double getMoyenneGenerale()
	{
		return this.moyenneGenerale;
	}
 
	void ecrireTabEleves()
	{
		for (int i = 0; i < NB_ELEVES; i++)
		{
			(this.tabEleves[i]).affichageDonneesEleves();
		}
		System.out.println();
		this.calculMoyenneGenerale();
		System.out.printf("Moyenne générale de la classe: " + "%.2f \n", this.getMoyenneGenerale());
	}
 
	void ajoutEleves(eleve el)
	{
		this.tabEleves[this.nbEleves] = el;
		this.nbEleves++;
	}
 
	public void getDonnees(eleve [] tabEleves)
	{
		for (int i = 0; i < this.NB_ELEVES; i++)
		{
			(this.tabEleves[i]).affichageDonneesEleves();
		}
	}
 
	public boolean trouve(String nom, eleve [] tabEleves)
	{
		for (int i = 0; i < this.NB_ELEVES; i++)
		{
			if (this.tabEleves[i].getNomEleve() == nom) return true;
		}
		return false;
	}
}
public class MoyenneGeneraleClasse {
	public static void main(String [] args)
	{
		groupe g = new groupe();
		for (int i = 0; i < g.getNbEleves(); i++)
		{
			g.ajoutEleves(eleve.lireDonneesEleves());
 
			 while (g.trouve(g.tabEleves[i].getNomEleve(), g.tabEleves)) { System.out.println();
			  System.out.println("Nom déjà donné."); }
 
		}
		g.ecrireTabEleves();
	}
}

Mon souci est donc là:

Code :

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

1
2
 while (g.trouve(g.tabEleves[i].getNomEleve(), g.tabEleves)) { System.out.println();
			  System.out.println("Nom déjà donné."); }

J'avais écrit un code similaire qui fonctionnait bien avec un programme de moto:

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
public double tempsRealise()
	{
		do
		{
			try
			{
				Terminal.ecrireStringln("Donnez le temps réalisé en secondes.");
				this.temps = Terminal.lireDouble();
				if (this.temps <= 0)
				{
					Terminal.ecrireStringln("Le temps ne peut être négatif ou nul");
					Terminal.sautDeLigne();
				}
			} catch (Exception e)
			{
				Terminal.ecrireStringln("mauvaise saisie");
				this.temps = -1;
			}
		} while (this.temps <= 0);
		return this.temps;
	}

Code :

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

1
2
3
4
5
6
7
8
public boolean trouve(double temps, double [] tabTemps)
	{
		for (double element : tabTemps)
		{
			if (element == temps) return true;
		}
		return false;
	}

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
public double [] tableauTemps()
	{
		for (int i = 0; i < this.getNbEssais(); i++)
		{
			Terminal.ecrireStringln("Essai " + (i + 1) + ".");
			while (this.trouve(this.tempsRealise(), tabTemps))
			{
				Terminal.sautDeLigne();
				Terminal.ecrireStringln("Valeur déjà donnée.");
			}
			tabTemps[i] = this.temps;
			Terminal.sautDeLigne();
		}
		return tabTemps;
	}

Mais finalement, je ne parviens pas à transposer ce que j'avais déjà réussi à faire auparavant. Je vais chercher ce soir, mais si quelqu'un a une idée, je suis preneur!

Merci par avance,
Johnny

[bleporini]

Hello,

Sans avoir débuggé ton programme, j'ai quand même un conseil:
Pourquoi ne pas utiliser une Map (http://java.sun.com/j2se/1.5.0/docs/.../util/Map.html)pour stocker tes élèves et lever une exception dans la méthode ajoutEleves si le nom fait déjà partie des clefs de la Map. Cela permettrait que la règle de gestion d'ajout des élèves soit au bon endroit dans le code.
De plus si je ne m'abuse, lorsque tu détectes que l'élève saisi l'a déjà été, tu ne le retires pas du tableau.

[mhamedbj]

Bonjour,

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 class MoyenneGeneraleClasse {
	public static void main(String [] args)
	{
		groupe g = new groupe();
		for (int i = 0; i < g.getNbEleves(); i++)
		{
			g.ajoutEleves(eleve.lireDonneesEleves());
 
			 while (g.trouve(g.tabEleves[i].getNomEleve(), g.tabEleves)) { System.out.println();
			  System.out.println("Nom déjà donné."); }
 
		}
		g.ecrireTabEleves();
	}
}

Tu rentre dans la while sans incrémenter le "i" !!!!!!

dans ta boucle while ton "i" ne va pas s'incrémenter tout seul il est fixe et tu boucle en infini

évite le while et fait un procédure qui s'arrête et renvois un boollean dès qu'elle trouve un élément !


Bon courage ! et n'oublie pas le !


cdt
Mhamed

[CheryBen]

Bonjour, ta fonction trouve n'est pas bonne.

Code :

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

1
2
3
4
5
6
7
8
public boolean trouve(String nom, eleve [] tabEleves)
{
    for (int i = 0; i < this.NB_ELEVES; i++)
    {
	if (this.tabEleves[i].getNomEleve() == nom) return true;
    }
    return false;
}

FAQ : Comment comparer des chaînes de caractères ?

[johnny3]

hello,

merci pour vos conseils. Effectivement, une hashmap est une bonne idée.

Pour la fonction trouve, j'ai compris: c'est un problème entre un String et une adresse! Je ne comparais pas le nom de l'élève mais l'adresse où se trouve ce nom!

Et effectivement, j'ai vu que j'avais aussi un souci avec ma boucle while.

Merci pour vos conseils!

Johnny