C# String.IsNullOrEmpty: хорошо или плохо?

Question

После инцидента на работе, где я неправильно использовал String.IsNullOrEmpty с переменной Session, мой коллега теперь отказывается принять мое использование String.IsNullOrEmpty. После некоторых исследований, по-видимому, есть ошибка в списке для IsNullOrEmpty на MSDN (link) (прочитать примечание внизу):

По состоянию на 4 апреля 2006 года, есть ошибка (возможно в JIT), что делает этот метод не работает при оптимизации включен. Как известно, это влияет как на C#, так и на VB.

Дополнительную информацию можно найти здесь (link). Microsoft - ошибка «якобы» фиксированных пост-орков, но, к сожалению, мой работодатель все еще использует VS2005. Но если проблема будет исправлена ​​в 2008 году, пусть будет так. Все в порядке со мной.

Хотя отказ моего коллеги из моего кода с IsNullOrEmpty мне слепо невежество (IMO), он, конечно, не может сказать мне, почему не использовать его, кроме неправильного использования с переменным сеансом. Я использовал IsNullOrEmpty по всему нашему коду без каких-либо проблем. Лично я считаю его более читаемым, помимо того, что он делает две вещи в одном утверждении.

После того, как вы нашли мнение по этому вопросу, я нашел сайты, которые занимают профи. Вот некоторые из сайтов, которые я читал об этом:

https://blog.rthand.com/post/2006/06/22/1063.aspx

http://www.omegacoder.com/?p=105

Один сайт (http://dotnetperls.com/isnullorempty) суммирует метод (ИМХО) довольно хорошо:

Здесь мы посмотрел, что метод IsNullOrEmpty на тип строки, который предоставляет нам хороший и относительно эффективный метод проверки, Строка в порядке, чтобы сохранить или использовать. Тем не менее, для исполнения, может быть лучше использовать ручные проверки. Пустые строки также могут быть протестированы другими способами, и мое исследование показывает, что проверка является самой быстрой.

Предполагая, что ошибка исправления на месте (и работает правильно) в VS2008/2010/и др., Есть ли причина не использовать String.IsNullOrEmpty с VS2005 и за его пределами? Я понимаю, что это может показаться немного переполненным таким глупым методом, но я хотел бы знать, есть ли что-то за кулисами, и если у кого есть альтернативные объяснения.

Answer 1

Эта проблема была устранена в .NET 2.0 sp1. Теперь нет причин, чтобы избежать его использования.

Если вы используете .NET 2, в любом случае у вас должен быть sp1 по многим другим причинам - я не вижу причин, чтобы избежать этого для ошибки, которая больше не существует.

Answer 2

Если она сломана в вашей версии, то это тривиально просто иметь статический метод, который будет делать чек, так что просто сделать:

public static bool isNull(String s) { 
    return s == null || s.trim().length == 0; 
} 

Нет смысла в том, чтобы в большой вопрос о чем-то, что следует относительно легко исправить.

Вам не нужно менять его везде, хотя вы можете сделать глобальную замену одного статического метода другим.

Answer 3

можно написать модульный тест, который проходит в пустую строку и один, который проходит в пустую строку, чтобы проверить этот материал, и запустить его в VS2005 и после того, как в 2008 году, и посмотреть, что случилось

Answer 4

В этом сообщении об ошибке в ссылка, в которую вы указываете:

Эта ошибка исправлена ​​в Microsoft .NET Framework 2.0 с пакетом обновления 1 (SP1).

Поскольку это не имеет значения, если вы используете VS 2005, если у вас установлен SP1 для .NET 2.

Что касается использования или нет, ознакомьтесь с этим post by CodingHorror.

Answer 5

Я уверен, что она была установлена ​​на SP1, но в любом случае вы можете создать свой собственный нуль или пустой метод :)

Answer 6

Как и с любым языком или его части, это все о зная, плюсы/минусы и сделав основанное на этой информации. ИМХО.

Answer 7

Мы используем метод расширения для string.IsNullOrEmpty:

public static bool IsNullOrEmpty(this string target) 
{ 
    return string.IsNullOrEmpty(target); 
} 

Используя этот подход, даже если бы он был разорен в какой-то предыдущей версии, исправлены ошибки только одна строка кода.

И добавил полезность возможность использовать метод на строки, например, что может быть пустым:

string myString = null; 
if (myString.IsNullOrEmpty()) 
{ 
    // Still works 
} 

Answer 8

Я слышал о том, что ошибка раньше, и от того, что я могу собрать это никогда не происходит в любой реальный код, только в коде, подобном примеру, который на самом деле ничего не делает. Кроме того, ошибка не связана с самим методом IsNullOrEmpty, поэтому это произойдет независимо от того, как вы проверяете строку.

Если метод делает именно то, что вы хотите сделать, вы должны его использовать. Однако вы не должны использовать его в каждой ситуации, чтобы проверить пустую строку. Иногда вам нужно только проверить, является ли строка пустой, а не нулевой.

Если переменная строка имеет нулевое значение, это будет просто пропустить блок кода:

if (!String.IsNullOrEmpty(str)) { ... } 

Если переменная строка имеет нулевое значение, это вызовет исключение:

if (str.Length > 0) { ... } 

Если переменная не должен быть нулевым, вы, вероятно, хотите исключение вместо кода, обрабатывающего пустое значение как пустую строку.Если что-то не так, вы хотите поймать его как можно раньше, потому что будет сложнее отследить проблему до источника, чем дольше исключение связано с причиной.

Answer 9

Интересно, почему люди используют string.Empty, это нехорошо, потому что это инициализированная строка &, эта концепция существует только в кадре .Net, где это допустимая строка с len 0 (серверы db очень четко различают между это и будет жаловаться, если у вас есть логика, проверяющая значение null, но вы получаете и пустую строку). Я думаю, что string.IsNullOrEmpty является одним из пяти наихудших методов/функций, которые я когда-либо видел, потому что он каким-то образом поощряет/заставляет его выглядеть нормально, чтобы начать их строки, а можно рассматривать как null. Эта функция никогда не была добавлена, и я думаю, что ребята .Net должны попытаться поэтапно ее отключить :) Кому нужна и пустая строка? Я никогда не использовал его, если бы у меня не было из-за существующих проектов.

Answer 10

При реализации проверки аргументов в API я обычно проверяю каждое условие отдельно и выдаю разные исключения: ArgumentNullException для нулевой ссылки или, в зависимости от API спецификации, ArgumentException для пустой строки. В этом случае использование String.IsNullOrEmpty не позволяет различать эти два отдельных условия ошибки.

if (str == null) 
{ 
    throw new ArgumentNullException("str"); 
} 
if (str == string.Empty) 
{ 
    throw new ArgumentException("The string cannot be empty.", "str"); 
}