Programmier-Richtlinien

[MarcoGomez]

Hallo,

wie löse ich folgende Anweisungen elegant:

if (connectionName == "GameServerDatabase")
{
return connectionString = "blabla";

}
else
{
return "";
oder throw irgendwas?
}

LG

 

[maki]

Elegant? Oder meinst du richtig?
Wäre gut zu Wissen was du überhaupt vorhast...

Jedenfalls werden Strings mit equals verglichen.

 

[ARadauer]

du willst wissen ob du einen leerstring zurück geben sollst oder eine exception werfen sollst?

Im Normalfall sollte eine Verbindung vorhandensein, wenn das nicht der Fall ist, wird ist das wahrscheinlich eine grobe ausnahme verletzung.... ich würd eine exception werfen..

 

[SnooP]

und das ist auch nicht elegant -bzw. grob fahrlässig:
connectionName == "GameServerDatabase"

benutze .equals bei String-Vergleichen und zwar so rum: "GameServerDatabase".equals(connectionName).
Und ich würde im Fehlerfall tatsächlich eine IllegalArgumentException werfen.

 

[Guest]

und zwar so rum: "GameServerDatabase".equals(connectionName).

Wenn ich das grad mal lese, warum so rum und nicht connectionName.equals("GameServerDatabase") ?

Bei nem == ist es doch auch egal was vorne steht?!?

 

[maki]

und zwar so rum: "GameServerDatabase".equals(connectionName).

Wenn ich das grad mal lese, warum so rum und nicht connectionName.equals("GameServerDatabase") ?

Bei nem == ist es doch auch egal was vorne steht?!?

Es ist nur wichtig, dass das, was "vorne steht", nicht null ist.
Da "GameServerDatabase" garantiert ein String ist und nicht null, bietet sich das an.

 

[Gast]

cool, verstanden danke sehr