erst containsKey oder gleich get

[stev.glasow]

Wie macht ihr das? so:

if(myMap.containsKey(key) ){
   aa = (GuluGulu)myMao.get(key);
}
else{
  aa = new GuluGulu();
  myMap.put(key, aa);
}

oder so:

aa = (GuluGulu)myMao.get(key);
if(aa == null){
     aa = new GuluGulu();
     myMap.put(key, aa);   
}

Der erste ist ja etwas sauberer, da aa = (GuluGulu)myMao.get(key); ja auch null liefern kann, aber der zweite müsste doch eigentlich etwas schneller laufen, oder? Wenn aa nicht null sein darf oder kann, nehm ich eigentlch immer die zweite Variante.
Ich glaub das hab ich schon mal gefragt ???:L

 

[meez]

Ich machs mit contains...
Wahrscheinlich läuft der zweite schon schneller, aber eine Ueberlagerung von return Werten sollte wenn möglich vermieden werden (Das ist so'n C mist)...

 

[Beni]

Das erste hat 3 Suchen, das zweite nur 2. Also läuft die zweite Variante um 50% schneller als die erste.

Deine Entscheidung... ich würde die performatere Lösung vorziehen.

 

[meez]

Das erste hat 3 Suchen, das zweite nur 2. Also läuft die zweite Variante um 50% schneller als die erste.

Da man den put "weglassen" kann, ist es also 2 zu 1 => 100% schneller :bae:

 

[stev.glasow]

Wahrscheinlich läuft der zweite schon schneller, aber eine Ueberlagerung von return Werten sollte wenn möglich vermieden werden (Das ist so'n C mist)...

Warum?

 

[meez]

Weil in Java nicht die Performance unbedingt im Vordergrund steht, sondern die Wiederverwertbarkeit und die Lesbarkeit des Codes. Wenn du nun eine Funktion hast, die einen int zurückgibt, und jede der 4 Milliarden Zustände eine andere Funktion/Meinung hat (überlagert ist), ist das nicht gerade förderlich für die "entkomplexisierung" des Codes.

 

[Beni]

Eine extra Operation, die nicht gerade billig ist, nur um den Code ein bisschen schöner zu halten?

Da würd ich doch eher eine zusätzliche Variable vorschlagen, "perhapsNullReturnValueReadyToOverwrite" oder so...

 

[meez]

Übrigens:

http://java.sun.com/j2se/1.3/docs/api/java/util/HashMap.html#get(java.lang.Object)

Returns the value to which this map maps the specified key. Returns null if the map contains no mapping for this key. A return value of null does not necessarily indicate that the map contains no mapping for the key; it's also possible that the map explicitly maps the key to null. The containsKey operation may be used to distinguish these two cases.

Genau das Problem mit dem überlagern...

 

[stev.glasow]

Der erste ist ja etwas sauberer, da aa = (GuluGulu)myMao.get(key); ja auch null liefern kann,...

[edit] ah, auch wenn der key matcht, meinte ich

Wenn aa nicht null sein darf oder kann, nehm ich eigentlch immer die zweite Variante.

Das ist doch gemeint oder nicht?

 

[Beni]

Ok, dann sind die Codeschnippsel von stevg semantisch unterschiedlich... dann kommt es drauf an, was stevg mit einem "contains" aber ist trotzdem "null" machen will.