C# 4.0 - Wie Griff Optional String-Parameter

Question

Dieser Code ist nicht gültig:

private void Foo(string optionalString = string.Empty) 
{ 
    // do foo. 
} 

Aber dieser Code:

private void Foo(string optionalString = "") 
{ 
    // do foo. 
} 

Warum? Da string.Empty ein schreibgeschütztes Feld und keine Konstante ist und die Standardwerte für optionale Parameter eine Kompilierungszeitkonstante sein müssen.

Also, auf meine Frage ... (na ja, Sorge)

Das ist, was ich zu tun haben musste:

private const string emptyString = ""; 

private void Foo(string optionalString = emptyString) 
{ 
    // do foo. 
    if (!string.IsNullOrEmpty(optionalString)) 
     // etc 
} 

How do you Jungs behandeln optionale String-Parameter?

Warum können sie nicht String.Empty eine Kompilierzeitkonstante machen?

Answer 1

Ummm ... was ist falsch mit String optionalParm = "" wieder? Warum ist das so schlimm? Denken Sie wirklich, dass Sie in diesem Fall eine symbolische Konstante für eine leere Zeichenfolge benötigen? Wie wäre es damit?

const int Zero = 0; 

void SomeMethod(int optional = Zero) { } 

Das scheint überhaupt albern zu sein?

Answer 2

Der beste Weg, sie zu behandeln ist mit:

private void Foo(string optionalString = "") 
{ 
    // do foo. 
} 

So können Sie nicht String.Empty verwenden. Jeder erkennt "", aber wenn ich optionalString = nullString finden würde, wäre ich mir nicht sicher, was ich denken soll. Wenn nichts anderes, benennen Sie das Ding emptyString - es ist nicht null!

Answer 3

Ich beantworte diese Frage.

Why can they not make String.Empty a compile-time constant?

Hier ist der disassemble Code über Reflektor aus String.cs in mscorlib.dll

public static readonly Empty; 
static String() 
{ 
    Empty = ""; 
    WhitespaceChars = new char[] { 
     '\t', '\n', '\v', '\f', '\r', ' ', '\x0085', '\x00a0', ' ', ' ', ' ', ' ', ' ', ' ', ' ', ' ', 
     ' ', ' ', ' ', ' ', '​', '\u2028', '\u2029', '　', '' 
    }; 
} 

So in Windows-Plattform, ist string.Empty genau "". Aber wissen Sie, Martian haben eine andere Definition für Empty und WhitespaceChars in ihrem Betriebssystem.

Answer 4

Wenn Sie "" Wert nicht mögen, können Sie Standard (Zeichenfolge) verwenden.
Ich spielte damit und es ist erlaubt.

private static void foo(string param = default(string)) { 
    if (!string.IsNullOrEmpty(param)) // or param != default(string) 
     Console.WriteLine(param); 
} 

Answer 5

Wenn Sie bereit sind zu spielen zu verlieren und behandeln null, „“ und Leerzeichen gleich sein, dann können Sie auf null Standard. Dies ist sehr praktisch, wenn Benutzername und Kennwort optionale Felder sind, da eine vertrauenswürdige Verbindung zu einer Datenbank möglich ist. Sie könnten diese Logik ändern, um Strings auf null zurückzusetzen und somit den Assert und den if zu modifizieren. Der wichtige Teil ist eine konsistente Konvention.

private void RunSql(string serverName, string databaseName, string userName = null, string password = null) 
{ 
    userName = Strip(userName); 
    password = Strip(password); 

    // The `MsTest` assert - works in both `Debug` and `Release` modes. 
    Assert.AreEqual<bool>(
     userName == String.Empty, 
     password == String.Empty, 
     "User name and password should be either both empty or both non-empty!"); 
    Assert.IsFalse(String.IsNullOrWhiteSpace(serverName)); 
    Assert.IsFalse(String.IsNullOrWhiteSpace(databaseName)); 

    var cmdBuilder = new StringBuilder(); 
    cmdBuilder.AppendFormat("sqlcmd -E -S {0} -d {1} ", serverName, databaseName); 
    if (userName.Length > 0) 
    { 
     cmdBuilder.AppendFormat("-U {0} -P {1} ", userName, password); 
    } 

    // Complete the command string. 
    // Run the executable. 
} 

// Cannot think of a good name. Emptify? MakeNullIfEmpty? 
private string Strip(string source) 
{ 
    if (String.IsNullOrWhiteSpace(source)) 
    { 
     return String.Empty; 
    } 

    return source; 
} 

Answer 6

Code Analysis warning 1026 besagt, keine optionalen Parameter zu verwenden.Es ist besser, Stil Überlastung Methoden zu verwenden, wie folgt aus:

private void Foo() 
{ 
    Foo(string.Empty); 
} 
private void Foo(string optionalString) 
{ 
    // do foo. 
    if (!string.IsNullOrEmpty(optionalString)) 
     // etc 
}