C# 4.0 - Làm thế nào để xử lý chuỗi tùy chọn thông số

Question

Mã này là không hợp lệ:

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

Nhưng mã này là:

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

Tại sao? Vì string.Empty là một trường chỉ đọc, không phải là một hằng số và các giá trị mặc định cho các tham số tùy chọn phải là một hằng số biên dịch-thời gian.

Vì vậy, vào câu hỏi của tôi ... (tốt, mối quan tâm)

Đây là những gì tôi đã làm:

private const string emptyString = ""; 

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

Làm thế nào để các bạn xử lý các thông số chuỗi bắt buộc?

Tại sao họ không thể tạo String.Empty hằng số biên dịch?

Answer 1

Ummm ... có gì sai với chuỗi tùy chọnParm = "" một lần nữa? Tại sao điều đó xấu? Bạn có thực sự nghĩ rằng bạn cần một hằng số tượng trưng cho một chuỗi rỗng trong trường hợp này? Thế thì sao?

const int Zero = 0; 

void SomeMethod(int optional = Zero) { } 

Điều đó có vẻ ngớ ngẩn với bạn không?

Answer 2

Cách tốt nhất để xử lý chúng là với:

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

Vì vậy, bạn không thể sử dụng String.Empty. Mọi người đều nhận ra "", nhưng nếu tôi tìm thấy optionalString = nullString Tôi sẽ không chắc chắn nên nghĩ gì. Nếu không có gì khác, hãy đặt tên cho điều emptyString - nó không phải là rỗng!

Answer 3

Tôi đang trả lời câu hỏi này.

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

Đây là mã tháo rời qua Reflector của String.cs trong mscorlib.dll

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

Vì vậy, trong nền tảng cửa sổ, string.Empty là chính xác "". Nhưng bạn có biết, sao Hỏa có một định nghĩa khác cho Empty và WhitespaceChars trong hệ điều hành của họ.

Answer 4

nếu bạn không thích giá trị "", bạn có thể sử dụng giá trị mặc định (chuỗi).
Tôi đã chơi với nó và nó được cho phép.

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

Answer 5

Nếu bạn sẵn sàng để chơi mất và điều trị null, "", và ký tự khoảng trắng để được như vậy, thì bạn có thể mặc định null. Điều này trở nên rất hữu ích khi tên người dùng và mật khẩu là các trường tùy chọn do khả năng kết nối đáng tin cậy với một db. Bạn có thể thay đổi logic này để đặt lại các chuỗi thành null và do đó sửa đổi xác nhận và if. Phần quan trọng là có một quy ước nhất quán.

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 không nên sử dụng tham số tùy chọn.Đó là cách tốt hơn để sử dụng các phương pháp quá tải, như sau:

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