falscher return type ?

[sowieso]

Hi !

Ich habe ein

Set <Map.Entry <Object,Object>>

Nun will ich eine Funktion schreiben, die die Map.entry im Set returned, die einem bestimmten key der Map.Entry entspricht:

public Map.Entry<Object,Object> findMapEntry(Set <Map.Entry <Object,Object>> entrySet, Object key)
	{
		Iterator <Map.Entry <Object,Object>> it = entrySet.iterator();
		
		while (it.hasNext())
		{
			Map.Entry <Object,Object> mapEntry = (Map.Entry <Object,Object>) it.next();
			if (mapEntry.getKey()==key)
			{
				return mapEntry;
			}
         else return mapEntry;
		}
	}

Das

else return mapEntry;

hab ich da nur stehn, weil mir noch nicht eingefallen ist, wie ich eine Fehlermeldung zurückgeben kann o.ä..

Nun sagt der compiler aber

This method must return a result of type Map.Entry<Object,Object>

Wie kann das sein ? die mapEntry die returned ist doch genau von diesem Typ :shock:

Kann mir jemand helfen ??

 

[SlaterB]

es ist nicht sicher, dass die while-Schleife überhaupt begonnen wird,
was wird dann zurückgegeben?

----


wenn du den Iterator generisch machst, sollte doch
Map.Entry <Object,Object> mapEntry = it.next();
reichen?

---------


wieso überhaupt Map.Entry zurückgeben?
der Key ist schon bekannt, der Value leicht zu bekommen: map.get(key)

 

[sowieso]

Also vielen Dank schon mal so weit Ich habs jetzt so:

public Map.Entry<Object,Object> findMapEntry(Set <Map.Entry <Object,Object>> entrySet, Object key)
	{
		Iterator <Map.Entry <Object,Object>> it = entrySet.iterator();
		
		Map.Entry <Object,Object> mapEntry = it.next();
		while (it.hasNext())
		{
			if (mapEntry.getKey()==key)
			{
				return mapEntry;				
			}
       mapEntry = it.next();	
			
		}
		return null;
	}

Wenn das return in der schleife returned wird, bricht der Compiler die Funktion doch ab und "null" wird nicht mehr returned oder ??

Ich will eine Map.Entry returnen, weil ich aus einem Set bestimmte Map.Entries entfernen will, die bestimmte keys enthalten. und die remove() Funktion vom set braucht in diesem Fall die ganze Entry.Map, in der der "Übeltäter" enthalten ist (Ich hoffe das macht Sinn ^^)

 

[SlaterB]

> Wenn das return in der schleife returned wird, bricht der Compiler die Funktion doch ab und "null" wird nicht mehr returned oder ??

stimmt



> Ich will eine Map.Entry returnen, weil [..]

verschiebt die Frage nur weiter, aber ok, ist denkbar

 

[bygones]

mhm mir kommt das design aesserst komisch vor...

warum hantierst du hier mit Sets von Map.Entry ?