HashMap Schlüssel vergleichen

[ETK15]

Hallo zusammen,

ich lese aus zwei Textdateien Daten in zwei HasMaps ein. Anschließend sollen die Schlüssel der ersten Map mit denen der zweiten Map verglichen werden. Sobald eine übereinstimmung gefunden wurde, soll eine Methode ausgeführt werden.

Hier ist der Codeausschnitt, der die Schlüssel vergleichen soll:

Iterator<Integer> itListe1 = personalListe.keySet().iterator();
				Iterator<Integer> itListe2 = mteListe.keySet().iterator();
				FileWriter writer = new FileWriter("ausgabe.txt");
				int zaehler = 1;
				while(itListe1.hasNext()){
					int key1 = itListe1.next();
						while(itListe2.hasNext()){
							int key2 = itListe2.next();
							if(mteListe.containsKey(key1)){
								Personal p = personalListe.get(key1);
								p.setX(maListe.get(key1));
								break;
							}
						}

Mein Problem ist, sobald in Liste 1 mehrere Daten stehen, wird der Code nciht mehr korrekt ausgeführt. Steht nur 1 Element in Liste 1 funktioniert der Code tadellos.

Was mache ich falsch? Evtl eine for-Schleife?

Viele Grüße
Thorsten

 

[The_S]

Nach einem Durchlauf steht dein Iterator immernoch auf dem letzten Element. Somit wird die innere Schleife ne öffters als einmal ausgeführt.

 

[ETK15]

Nach einem Durchlauf steht dein Iterator immernoch auf dem letzten Element. Somit wird die innere Schleife ne öffters als einmal ausgeführt.

Ich habe zwei Iteratoren. Soviel ich weis, wird durch den Aufruf von .next() jeder Schlüssel durchlaufen.

Ich weis nicht, was der macht, denn die Methode, die bei übereinstimmung ausgeführt werden soll, wird nicht ausgeführt.

:-(

 

[DocRandom]

..schmeiß mal den Debugger an!
Und geh in Einzelschritten durch den Code (nur die Problemstelle).
Dannn siehst ob es so läuft, oder ob nicht doch wo ein logischer Fehler drinnen ist!

mfg
DocRandom

 

[The_S]

Nach einem Durchlauf steht dein Iterator immernoch auf dem letzten Element. Somit wird die innere Schleife ne öffters als einmal ausgeführt.

Ich habe zwei Iteratoren. Soviel ich weis, wird durch den Aufruf von .next() jeder Schlüssel durchlaufen.

Ich weis nicht, was der macht, denn die Methode, die bei übereinstimmung ausgeführt werden soll, wird nicht ausgeführt.

:-(

Ja, aber deine Innere Schleife läuft genau einmal durch, weil der Iterator nach dem 1. Durchlauf am letzten Element angekommen ist und somit bei jedem weiteren Durchlauf von der Methode hasNext() false zurückgeliefert wird.

 

[SlaterB]

Nach einem Durchlauf steht dein Iterator immernoch auf dem letzten Element. Somit wird die innere Schleife ne öffters als einmal ausgeführt.

Ich habe zwei Iteratoren. Soviel ich weis, wird durch den Aufruf von .next() jeder Schlüssel durchlaufen.

für das erste Element von Liste 1 wird in der inneren Schleife Liste 2 normal durchlaufen

beim zweiten Element von Liste 1 nicht mehr, da der Iterator itListe2 bereits alle ist, bis zum Ende durchgelaufen ist,
du musst den inneren Iterator
Iterator<Integer> itListe2 = mteListe.keySet().iterator();
jedesmal neu erzeugen!

-------
andererseits verwendest du key2 nirgendwo, wozu wird überhaupt die innere Liste durchlaufen?

reicht nicht

            Iterator<Integer> itListe1 = personalListe.keySet().iterator();
            while(itListe1.hasNext()){
                 int key1 = itListe1.next();
                 if(mteListe.containsKey(key1)){
                        Personal p = personalListe.get(key1);
                        p.setX(maListe.get(key1));
                 }
             }

?

 

[ETK15]

Ich habs nun so gelöst und benötige die Iteratoren (<- ist das Richtig? ) nicht mehr:

for(int key1 : Liste.keySet()){
					
					for(int key2 : Liste2.keySet()){
						if(Liste2.containsKey(key1)){
							Personal p = Liste.get(key1);
							p.setX(Liste2.get(key1));
							break;
						}
					}
					//weitere METHODEN				
				}

 

[Leroy42]

Ist dir nicht aufgefallen, daß du key2 (Laufvariable
der inneren Schleife) überhaupt nicht benutzt? :shock:

for (int key1 : Liste.keySet()) { 
  if (Liste2.containsKey(key1)) { 
    Personal p = Liste.get(key1); 
    p.setX(Liste2.get(key1)); 
  } 
}

macht doch genau dasselbe!

 

[ETK15]

Cool, danke.

Manchmal sieht ma den Wald vor lauter Bäumen nicht mehr.

 

[Leroy42]

Manchmal sieht ma den Wald vor lauter Bäumen nicht mehr.

Was hat denn jetzt, auf einmal, deine Mutter damit zu tun? :shock:

*SCNR*

 

[The_S]

Was hat denn jetzt, auf einmal, deine Mutter damit zu tun? :shock:

*SCNR*

Mensch, net ma Umgangssprache kann der Kerl ... was kannst du eigentlich? :lol:

 

[ETK15]

Was hat denn jetzt, auf einmal, deine Mutter damit zu tun? :shock:

*SCNR*

Mensch, net ma Umgangssprache kann der Kerl ... was kannst du eigentlich? :lol:

Kennst Du etwa nicht die Bibliothek java.util.woodstock ??? Dort gibts die methode ma(String umgangssprache). :wink: