Probleme sur la boucle While

**Gregory.M** · 01/09/2008, 14h44

bonjour,

Voici le code que je fais ci-dessous.
Je compare un String pour voir si il existe déja dans ma collection.
Si il n'existe pas je crée un nouvel objet, sinon je ne fais rien.

en faisant ceci je pensais ne pas me tromper mais je tombe sur un Exception:
java.util.NoSuchElementException

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
 
	public static void CompareSrv(String srvname, Collection<CServer> srvs)
	{
		if(srvs.size() == 0)
			srvs.add(new CServer(srvname));
		else
		{
 
 
			Boolean test = false;
			Iterator iterator = srvs.iterator();
			while ((iterator.hasNext())||(test == false)) {
				CServer srv = (CServer) iterator.next();
 
 
				if(srvname.equals(srv.get_name()))
				{
					test=true;
				}	
			}
 
			if(test)
				srvs.add(new CServer(srvname));
		}
	}

Quelqu'un peut il m'éclairer?

**droggo** · 01/09/2008, 14h54

Hie,

Quand tu fais

Code :

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

			while ((iterator.hasNext())||(test == false))

si tu n'as pas trouvé en arrivant à la fin de ta collection, test est toujours = false, et tu vas donc boucler au-delà...

Il faut

Code :

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

			while ((iterator.hasNext()) && (test == false))

**adiGuba** · 01/09/2008, 15h04

Salut,

Autre remarques : le Boolean n'est pas très judicieux dans ce cas car à chaque itération tu auras de l'auto(un)boxing pour effectuer la comparaison avec false. Il aurait mieux value utiliser le type primitif boolean...

Mais le mieux, plutôt que d'utiliser une variable et une condition plus complexe, ce serait plus simple d'utiliser le mot-clef break...

a++

PS : Et tu devrais jeter un coup d'oeil aux conventions de nommages

PS 2 : Ta méthode fait l'inverse de ce que tu décris : elle ajoute l'objet si il est déjà présent...