c#: Anweisung in if-Abfrage safe?

[J. W. (nuernberger)]

string sopt=null;
if (str.StartsWith(sopt="+UUSORD:")||
   str.StartsWith(sopt="+USORD:"))
{
  // sopt ist entweder "+UUSORD:" oder "+USORD:"
}

Ich möchte eine If-Abfrage mit zwei or-conditions vereinfachen.
Damit ich innerhalb des if-branches noch unterscheiden kann, welcher der 
beiden werte zutrifft, halte ich diesen Wert in einer Variablen fest 
(sopt).
In C habe ich das schon immer so gemacht.
Mit Mono-Projekt in C# funktioniert es.
Frage: Funktioniert das generell (d.h. auch in Visual Studio)?
Ist sowas sauber programmiert? Oder sollte man doch 2 If-Branches 
nehmen?

[Felix U. (ubfx)]

J. W. schrieb:
> Ich möchte eine If-Abfrage mit zwei or-conditions vereinfachen.
> Damit ich innerhalb des if-branches noch unterscheiden kann, welcher der
> beiden werte zutrifft, halte ich diesen Wert in einer Variablen fest
> (sopt).
> In C habe ich das schon immer so gemacht.
Da ist das zwar auch nicht schön, aber noch vertretbar weil man da dann 
innerhalb des if-blocks nur einen Pointer und kein ganzes Objekt nochmal 
vergleichen muss.

> Mit Mono-Projekt in C# funktioniert es.
> Frage: Funktioniert das generell (d.h. auch in Visual Studio)?
Ja.
> Ist sowas sauber programmiert?
Nein. Einen string zu benutzen ist mit Sicherheit nicht sauber 
programmiert. Konkret sieht das dann nämlich so aus:

//string test = "123abc123";
IL_000c: ldstr "123abc123"
IL_0011: stloc.0

//string prefix = null;
IL_0012: ldnull
IL_0013: stloc.1

//if (test.StartsWith(prefix = "abc") ||
//  test.StartsWith(prefix = "123"))
IL_0014: ldloc.0
IL_0015: ldstr "abc"
IL_001a: dup
IL_001b: stloc.1
IL_001c: callvirt instance bool [mscorlib]System.String::StartsWith(string)
IL_0021: brtrue.s IL_0032

IL_0023: ldloc.0
IL_0024: ldstr "123"
IL_0029: dup
IL_002a: stloc.1
IL_002b: callvirt instance bool [mscorlib]System.String::StartsWith(string)
IL_0030: brfalse.s IL_006d

// {
// if (prefix == "abc")
IL_003d: ldloc.1
IL_003e: ldstr "abc"
IL_0043: call bool [mscorlib]System.String::op_Equality(string, string)
IL_0048: brfalse.s IL_0055

// if (prefix == "123")
IL_0055: ldloc.1
IL_0056: ldstr "123"
IL_005b: call bool [mscorlib]System.String::op_Equality(string, string)
IL_0060: brfalse.s IL_006d

Visual Studio mit Standardeinstellungen optimiert das also schon mal 
nicht. Innerhalb des if-blocks werden die Strings nochmal neu auf den 
Stack gepusht und dann wird eine lib-Funktion für den Vergleich 
aufgerufen.

Schöner und effizienter geht es so:

bool b_pre_abc = test.StartsWith("abc"),
b_pre_123 = test.StartsWith("123");

if (b_pre_123 || b_pre_abc)
{
    MessageBox.Show("Match");

    if (b_pre_abc)
        ...
    if (b_pre_123)
        ...
}

Das sieht dann nämlich innerhalb des if-blocks schon wesentlich 
schlanker aus:

//if (b_pre_abc)
IL_0095: ldloc.2
IL_0096: brfalse.s IL_00a3

//if (b_pre_123)
IL_00a3: ldloc.3
IL_00a4: brfalse.s IL_00b1

Man spart sich also zwei library calls, viele Stringoperationen und 
gewinnt auch noch Lesbarkeit.