Iterator, wie funkioniert das richtig?

[Anselmus]

hi hab hier meine methode:

public void executeCommandMap(String userDir){
		
		Calendar cal = new GregorianCalendar();
		String saveDate = cal.get(Calendar.YEAR) +"-"+(cal.get(Calendar.MONTH)+1)+"-"+cal.get(Calendar.DAY_OF_MONTH);
		String saveTime = cal.get(Calendar.HOUR_OF_DAY)+"-"+cal.get(Calendar.MINUTE);
		String sep = File.separator;
		String rootDirectory = userDir+sep+"stonehenge"+sep+"dct"+sep+"collection_"+saveDate+"_"+saveTime;
		
		Set commandSet = commandMap.keySet();
		
		HashMap currentMap = new HashMap();
		while(commandSet.iterator().hasNext()){
			System.out.println(commandSet.size());
			String currentMapName = (String)commandSet.iterator().next();
			
			currentMap = (HashMap)commandMap.get(currentMapName);
			for(int i = 0; i<currentMap.size(); i++){
				ArrayList commandList = (ArrayList)currentMap.get(currentMapName + i);
				for(int j = 0; j<commandList.size(); j++){
					ZDCTCommandObject commandObject = (ZDCTCommandObject)commandList.get(j);
					commandObject.executeCommand();
					commandObject.saveResult(rootDirectory, currentMapName + i);
					System.out.println(rootDirectory +" "+currentMapName);
				}
			}
		}
	}

in der while schleife sollen alle element der commandmap rausgeholt werden, aber leider führt er die while schleife unendlich aus, obwohl commandSet.size() zb nur 2 ist...

ich dachte mit iterator().next() geht das automatisch eins weiter...

kann mir jemand sagen was ich falsch macheß

gtx
stephan

 

[Beni]

"commandSet.iterator()" generiert jedesmal einen neuen Iterator. Du musst den Iterator zwischenspeichern:

Iterator iterator = commandSet.iterator();
while( iterator.hasNext()) {...}

 

[AlArenal]

for (Iterator iter = commandSet.iterator(); iter.hasNext(); ) {
  ...
  String currentMapName = (String) iter().next(); 
  ...
}

 

[Sky]

Die Methode iterator() initialisiert immer einen neuen Interator. Du mussst Den Iterator vor der while-Schleife erstellen und in der while-Schleife verwenden.

 

[supastah]

ich denke das es daran liegst das du in der while schleife einen neuen Iterator erzeugst.
mach es mal so:
vor der Schleife folgende Anweisung:
Iterator it = commandSet.iterator();
Dann als Bedingung in der whileschlife:
while(it.hasNext()) {
Und in der Whileschleife dann nicht :
String currentMapName = (String)commandSet.iterator().next();
sondern:
String currentMapName = (String)it.next();

 

[supastah]

oh da haben wohl 3 schneller getippt als ich

 

[Anselmus]

klingt einleuchtend...

vielen dank für die tips =)