Discussion :

[philippe6]

Bonjour,

je veux compter l'occurence des differentes notes qu'un groupe de 31 enseignants attribue à leurs élèves, pour cela j'ai un listing de note et j'écris le code suivant :

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
 
 
do
	{
	ligne = entree.readLine();
	if(ligne==null) break;
StringTokenizer tk = new StringTokenizer(ligne);
				int nt = tk.countTokens();
				int j=1-1;
				int i=1-1;
				while(j<68+2)
					{
					while(i<nt)
						{
						if(j==Integer.parseInt(tk.nextToken()))
						chiffre[j]++;
						i++;
						}
					j++;
					}
				nblu++;
				}
				while(nblu<nbtirage2);

[michel.di]

Salut,

Petits conseils :
- Détaille ton code
- Donne des noms de variables compréhensibles

Pourquoi?
C'est illisible et ça ne donne pas envie de lire

[Jimmy_]

L'exception ne peux être lancé que par tk.nextToken(), cela signifie qu'il n'y a plus d'element tout simplement.

Il suffit de catcher ça et de sortir proprement.

[SucreGlace]

Il y a un problème dans ton utilisation des compteurs:

Code :

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

1
2
3
4
5
6
7
8
9
10
StringTokenizer tk = new StringTokenizer(ligne);
int nt = tk.countTokens();
int i=0;
while(i<nt)
{
	if (j==Integer.parseInt(tk.nextToken()))
	{
		i++;
	}
}

Le compteur i n'est incrémenté que si la condition est remplie, il ne représente donc pas le nombre d'éléments parcourus dans ton StringTokenizer, or tu l'utilises comme condition d'arrêt du parcours...

Tu devrais arrêter de t'embêter avec ce compteur inutile, et utiliser tout simplement le test st.hasMoreTokens() pour déterminer s'il faut continuer le parcours ou non.

[philippe6]

Je pense que si mon code est illisible, ce n'est pas de ma faute, k'avais envoyé 2 programmes complets, avec de nombreuses explications et ce n'est pas du tout ce qui apparait dans mon message,
pour répondre à sucreglce qui a eu cependant le courage de se débrouller avec ça, le i++ ne dépend pas de la condition
l'algo est
si
faire
i++
et pas
si
{
faire
i++
}
merci cependant je vais reécrire ma demande que j'avais pourtant vérifier après l'envoi

merci

Philippe

[SucreGlace]

pour répondre à sucreglce le i++ ne dépend pas de la condition

Ah ouais
Je vais blâmer le manque d'accolades et le manque d'indentation pour expliquer mon erreur.

Cependant, j'insiste, créer des compteurs au lieu d'utiliser les méthodes prévues par la classe StringTokenizer, ça rend le code moins clair et ça augmente les risques d'erreurs (et l'obtention d'une NoSuchElementException montre bien que tu as du faire une erreur quelque part).


EDIT: pourquoi 2 discussions avec le même problème ? diviser les réponses, ça n'aide pas à avancer...

[philippe6]

voici ce que je pensais vous avoir adressé :
Il s'agit de compter les notes
de 0 à 20 d'un groupe d'enseignants
après avoir ouvert le fichier note voici le code que j'écris :

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
 
do
{
ligne = entree.readLine();
if(ligne==null) break;
StringTokenizer tk = new StringTokenizer(ligne);
int nt = tk.countTokens();
int j=1-1;
int i=1-1;
while(j<20)//les notes
	{
	while(i<nt)//indice parcourant les lignes de notes représentés     
                           //par les tokens dont le nombre est nt
	             {
		if(j==Integer.parseInt(tk.nextToken()))
		chiffre[j]++;
		i++;
	             }
	j++;
	}
}while(ligne!=null);

donc ce que je pensais vous avoir envoyé est que le partie suivante de ce code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
while(j<20)
	{
	while(i<nt)     
	             {
		if(j==Integer.parseInt(tk.nextToken()))
		chiffre[j]++;
		i++;
	             }
	j++;
	}

est fausse car i continu de grandir, car il n'est pas reinitialisé et donc un seul j est traité
mais si j'écris

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while(j<20)
	{
	while(i<nt)     
	             {
		if(j==Integer.parseInt(tk.nextToken()))
		chiffre[j]++;
		i++;
	             }
             i=0;	
             j++;
	}

alors, j'obtient l'exception NoSuchElementException

Qu'en pensez vous ?
merci

Philippe

[SucreGlace]

mais si j'écris

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
while(j<20)
	{
	while(i<nt)     
	             {
		if(j==Integer.parseInt(tk.nextToken()))
		chiffre[j]++;
		i++;
	             }
             i=0;	
             j++;
	}

alors, j'obtient l'exception NoSuchElementException

Comme MigouW l'a déjà fait remarquer ici: le StringTokenizer a déjà été parcouru en entier, remettre le compteur à zéro ne suffit pas, il faudrait réinitialiser le StringTokenizer.

Franchement, avec une seule discussion, ça serait plus facile à suivre.

[rolfone]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
while(j<20)
	{
	while(i<nt)     
	             {
		if(j==Integer.parseInt(tk.nextToken()))
		chiffre[j]++;
		i++;
	             }
             i=0;	
             j++;
	}

non mes an plus c'est ridicule de parcoure 20 x la meme ligne.

Code :

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

1
2
3
4
5
6
7
8
 
	while(i<nt)     
	             {
		int j = Integer.parseInt(tk.nextToken());
		chiffre[j]++;
		i++;
	             }
             i=0;

en plus d'apres ton code j<20 cela veux dir que les prof ne peuvent pas mettre la note 20

[SucreGlace]

non mes an plus c'est ridicule de parcoure 20 x la meme ligne.

Ça aussi J'en étais encore à "expliquer l'erreur" sans chercher plus loin.

Dans ton code, il faudrait tout de même rajouter un test sur la valeur lue par Integer.parseInt(tk.nextToken()) afin de s'assurer qu'il s'agit d'un indice valable pour le tableau chiffre.