此代码无效:
private void Foo(string optionalString = string.Empty)
{
// do foo.
}
但是这段代码是:
private void Foo(string optionalString = "")
{
// do foo.
}
为什么呢?因为string.Empty是一个只读字段,而不是常量,并且可选参数的默认值必须是编译时常量。
所以,关于我的问题......(好吧,关心)
这就是我必须要做的事情:
private const string emptyString = "";
private void Foo(string optionalString = emptyString)
{
// do foo.
if (!string.IsNullOrEmpty(optionalString))
// etc
}
你们如何处理可选的字符串参数?
为什么它们不能使String.Empty成为编译时常量?
答案 0 :(得分:11)
嗯......字符串optionalParm =“”又出了什么问题?为什么那么糟糕?在这种情况下,你真的认为你需要一个空字符串的符号常量吗?那怎么样呢?
const int Zero = 0;
void SomeMethod(int optional = Zero) { }
这对你来说真是愚蠢吗?
答案 1 :(得分:5)
如果您不喜欢“”值,可以使用默认值(字符串) 我玩它并且允许。
private static void foo(string param = default(string)) {
if (!string.IsNullOrEmpty(param)) // or param != default(string)
Console.WriteLine(param);
}
答案 2 :(得分:2)
处理它们的最佳方法是:
private void Foo(string optionalString = "")
{
// do foo.
}
所以你不能使用String.Empty。每个人都认可“”,但如果我发现optionalString = nullString
我不确定该怎么想。 如果没有别的,请为emptyString
命名 - 它不是空的!
答案 3 :(得分:2)
Code Analysis warning 1026表示不使用可选参数。使用重载方法的风格更好,如下所示:
private void Foo()
{
Foo(string.Empty);
}
private void Foo(string optionalString)
{
// do foo.
if (!string.IsNullOrEmpty(optionalString))
// etc
}
答案 4 :(得分:0)
我正在回答这个问题。
Why can they not make String.Empty a compile-time constant?
这是通过mscorlib.dll中的String.cs的Reflector反汇编代码
public static readonly Empty;
static String()
{
Empty = "";
WhitespaceChars = new char[] {
'\t', '\n', '\v', '\f', '\r', ' ', '\x0085', '\x00a0', ' ', ' ', ' ', ' ', ' ', ' ', ' ', ' ',
' ', ' ', ' ', ' ', '', '\u2028', '\u2029', ' ', ''
};
}
所以在windows平台上,string.Empty就是“”。但是你知道吗,Martian在他们的操作系统中对Empty和WhitespaceChars有不同的定义。
答案 5 :(得分:0)
如果您愿意玩丢失并将null,“”和空格字符视为相同,那么您可以默认为null
。当用户名和密码是可选字段时,由于可能与数据库建立可信连接,这变得非常方便。您可以更改此逻辑以将字符串重置为null
,从而修改断言和if
。重要的是要有一致的惯例。
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;
}