string.IsNullOrEmpty użyciu trim()

Question

Chcę usunąć pierwszej linii:

    !string.IsNullOrEmpty(cell.Text) 

spowoduje to żadnego problemu?

natknąłem to w niektórych kodu:

   if ((id % 2 == 0) 
        && !string.IsNullOrEmpty(cell.Text) 
        && !string.IsNullOrEmpty(cell.Text.Trim()) 
        ) 

Myślę, że pierwszy string.IsNullOrEmpty będzie return false na sznurku ze spacjami
i zgodnie z trim() dba o to, więc pierwszy IsNullOrEmpty jest bezużyteczny

Ale zanim usunę linię bez wykończenia myślałem, że będę go prowadził przez grupę.

Answer 1

Pierwszy IsNullOrEmpty połowy wartości null przed rzucają NullReferenceException z tapicerką() .

Jednakże, istnieje lepszy sposób:

if ((id % 2 == 0) && !string.IsNullOrWhiteSpace(cell.Text)) 

Answer 2

jeśli komórka.Tekst jest pusty, istnieje wyjątek bez tego pierwszego sprawdzenia.

Answer 3

W .NET 4.0:

if (id % 2 == 0 && !string.IsNullOrWhiteSpace(cell.Text)) 
{ 
    ... 
} 

W starszych wersjach należy zachować dwa testy, ponieważ po usunięciu pierwszego i cell.Text jest null, dostaniesz NRE na drugi podczas próby wywołania .Trim na instancji zerowej.

Albo można też to zrobić:

if (id % 2 == 0 && string.IsNullOrWhiteSpace((cell.Text ?? string.Empty).Trim())) 
{ 
    ... 
} 

lub nawet lepiej, można napisać extension method dla typu string, że zrobi to tak, że można po prostu:

if (id % 2 == 0 && !cell.Text.IsNullOrWhiteSpace()) 
{ 
    ... 
} 

które mogłyby wygląda tak:

public static class StringExtensions 
{ 
    public static bool IsNullOrWhiteSpace(this string value) 
    { 
     return string.IsNullOrEmpty((value ?? string.Empty).Trim()); 
    } 
} 

Answer 4

Uważam, że test polega na upewnieniu się, że komórka.tekst nie jest pusta po pierwsze ... jeśli tak, próba ominięcia go i uzyskania samego cell.text.trim() mógłby się dusić, ponieważ nie można wykonać przycięcia na pustym ciągu.

Answer 5

Dlaczego nie użyć !string.IsNullOrWhitespace(call.Text) i usunąć dwa poprzednie kontrole?

Answer 6

Nie można usunąć tylko pierwszej IsNullOrEmpty jako komórki.Tekst może mieć wartość zerową, a tym samym wywoływanie przycinania na nim spowoduje odrzucenie i wyjątek. Użyj IsNullOrWhiteSpace, jeśli korzystasz z .Net 4.0 lub pozostaw obie kontrole.

if ((id % 2 == 0) && !string.IsNullOrWhiteSpace(cell.Text)) 

Answer 7

Jeśli cell.Text jest null, string.IsNullOrEmpty wyrażenie (cell.Text.Trim()) wygeneruje wyjątek, ponieważ stara się uruchomić metody trim() na komórkę.

Znacznie więcej readble jeśli warunek będzie:! Cell.Text = null & & cell.Text.Trim() = ""

Answer 8

Można użyć metody rozszerzenie takiego:

/// <summary> 
/// Indicates whether the specified string is null or empty. 
/// This methods internally uses string.IsNullOrEmpty by trimming the string first which string.IsNullOrEmpty doesn't. 
/// .NET's default string.IsNullOrEmpty method return false if a string is just having one blank space. 
/// For such cases this custom IsNullOrEmptyWithTrim method is useful. 
/// </summary> 
/// <returns><c>true</c> if the string is null or empty or just having blank spaces;<c>false</c> otherwise.</returns> 
public static bool IsNullOrEmptyWithTrim(this string value) 
{ 
    bool isEmpty = string.IsNullOrEmpty(value); 
    if (isEmpty) 
    { 
     return true; 
    } 
    return value.Trim().Length == 0; 
}