До выхода .NET 4 мы делали что-то подобное для проверки нулевой/пустой строки:
String s;
if ((s + "").Trim().Length == 0)
{
//do something
}
Хотя приведенный выше метод работает довольно хорошо для нас, я знаю, что в .NET 4 у нас есть метод IsNullOrWhiteSpace, чтобы делать то же самое.
У меня вопрос, какой из них лучше? Должны ли мы вместо этого переключиться на использование встроенной функции?
В .NET 4 я бы определенно использовал встроенный метод. В конце концов, в нем точно указано, что вы пытаетесь сделать.
Если вы застряли до .NET 4, почему бы не использовать:
if (s == null || s.Trim() == "")
Or
if (s == null || s.Trim().Length == 0)
? Оба они говорят, чего вы хотите достичь.
Я определенно не стал бы использовать здесь конкатенацию строк. Помимо производительности, вас не интересует конкатенация строк. Всякий раз, когда вы обнаружите, что ваш код делает что-то, что на самом деле не является частью того, чего вы пытаетесь достичь, вы должны попытаться улучшить его ради удобочитаемости.
Trimmed (и превратить их в пустую строку, если это null). Является ли конкатенация, чтобы избежать null, все еще плохой идеей для этого случая?
- person andri; 18.03.2013
Лично я использую IsNullOrWhiteSpace, в основном потому, что с его помощью код становится более понятным для чтения и он обрабатывает больше случаев (часть WhiteSpace). Это зависит от ваших предпочтений, потому что оба метода делают одно и то же.
Почему бы не написать вспомогательный метод для реализации того, что делает IsNullOrWhiteSpace до .NET 4? Как
public static boolean IsNullOrWhiteSpace(string input)
{
return string.IsNullOrEmpty(input) || input.Trim() == string.Empty;
}
Не используйте здесь конкатенацию, как сказал Джон. Не рекомендуется проверять значение null/пусто.
:().
- person andri; 18.03.2013
