probleme de NoSuchElementException

Version imprimable

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:

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);

01/12/2010, 09h47
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 ;)
01/12/2010, 10h47
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.
01/12/2010, 10h51
SucreGlace
Il y a un problème dans ton utilisation des compteurs:
Citation:
Envoyé par philippe6

Code:

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.
01/12/2010, 11h10
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
01/12/2010, 11h16
SucreGlace

Citation:

Envoyé par philippe6

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

Ah ouais :aie:
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...

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:

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:

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:

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

01/12/2010, 11h35
SucreGlace
Citation:
Envoyé par philippe6

mais si j'écris

Code:

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.

Code:

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:

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

01/12/2010, 11h40
SucreGlace

Citation:

Envoyé par rolfone

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.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
        while ((ligne = entree.readLine()) != null) {
            StringTokenizer tk = new StringTokenizer(ligne);
            int nt = tk.countTokens();
            while (tk.hasMoreTokens())//indice parcourant les lignes de notes représentés     
            //par les tokens dont le nombre est nt
            {
                int j = Integer.parseInt(tk.nextToken());
                if (j >= 0 && j <= 20) {
                    chiffre[j]++;
                }
 
            }
        }

allé un solution propre

le parcoure de ton fichier est super aussi

juste pour que tu voie un autre truc

Code:

1
2
3
4
5
6
do
{
ligne = entree.readLine();
if(ligne==null) break;
// la condition du do/while ne peux jamais etre false 
}while(ligne!=null);

aussi pourquoi des int i = 1-1; ???

01/12/2010, 11h59
philippe6

J'ai résolu dans la discussion d'à coté (je ne sais pas comment celle-ci s'est mise en route, un envoi intempestif peut-être !), vos réponses ici m'apportent d'autres solutions, le probleme est résolu.
le i=1-1 est une stupidité au départ i=1 puis au lieu de remplacer par 0, j'ai ajouté -1, puis avec des copier-coller, ça s'est peut être généralisé.

En tout cas merci, je commence java depuis peu, en fait, je vais cherché à être plus méthodique.

Philippe