Выполняется ли проверка того, является ли строка пустой или пустой?

Question

Для моего экзамена по программированию мне пришлось защищать код, который я написал. Одна из линий:

if(app.Logourl == "" || app.Logourl == null) 

Он спросил меня, существует ли разница между нулем и пустой строкой. Я сказал ему, что разница в том, что null означает, что он не указывает ни на что, поэтому он не создается, но пустая строка.

После экзамена я подошел к нему и спросил его, правильно ли я, так как я увидел смешной взгляд на его лице. Он сказал мне, что это правда, что они разные, но порядок, в котором я проверил значения, был неправильным.

Теперь через несколько дней я верю, что нет ничего плохого в заказе. Я прав?

TL; DR

является

if(app.Logourl == "" || app.Logourl == null) 

эквивалентно

if(app.Logourl == null || app.Logourl == "") 

Answer 1

Это нормально, как вы сделали это, потому что перегрузка == для System.String вызовов String.Equals, что позволяет null с.

Это не является универсальным, однако: если вы хотите, чтобы проверить длину строки вместо использования == "", ваш первый фрагмент кода будет в беде:

if(app.Logourl.Length == 0 || app.Logourl == null) // <<== Wrong! 

, а второй один будет в порядке:

if(app.Logourl == null || app.Logourl.Length == 0) // <<== Correct 

причиной этого является короткое замыкание в оценке || и && операторов: как только они знают, что результат (true для ||, false для &&), они прекращают оценку. Во втором фрагменте выше, если app.Logourl - null, вторая половина выражения будет проигнорирована, поэтому app.Logourl.Length не будет выбрасывать исключение с нулевой ссылкой.

Примечание: В знак признания проверок, как это происходит повсюду, библиотека C# класс offers a convenience method for doing this check:

if (string.IsNullOrEmpty(app.Logourl)) { 
    ... 
} 

Answer 2

Нет, это не имеет значения (в вашем случае).

Одна вещь, чтобы знать, однако, является то, что логические операторы && и || короткого замыкания, что означает, если у вас есть a || b и a является true, то b не оценивается.

Например,

app.Logourl == null || app.Logourl == "" 

Если app.Logourl является null, то app.Logourl == "" никогда даже не оценивали.

В вашем случае нет реальной разницы, проверяете ли вы один или другой. первый или второй. Если бы проверки были разными, это могло бы иметь значение.

Например,

app.Logourl == null || app.Logourl.Equals("") 

Если вы сделали другой заказ, вы получите исключение, если app.Logourl является null, так как вы не можете вызывать функции-члены нулевых ссылок.

---

Хотя я бы использовал String.IsNullOrEmpty(app.Logourl), который является стандартной LIB.

Answer 3

, когда я уверен, что мой объект является string я всегда предпочитаю следующее:

if (string.IsNullOrEmpty(yourString)) 
    { 
     // this string is null or empty 
    } 

или это:

if (string.IsNullOrWhiteSpace(yourString)) 
    { 
     // this string is null or empty (or got only a space) 
    } 

Answer 4

Это всего лишь комментарий.

В обычных ситуациях это не имеет значения. Но могут быть побочные эффекты. Вот простой пример, чтобы узнать из:

static class Program 
{ 
    static string Logourl 
    { 
     get 
     { 
      Console.WriteLine("getter runs"); 
      return null; 
     } 
    } 

    static void Main() 
    { 
     if (Logourl == "" || Logourl == null) 
     { 
     } 
    } 
} 

Эта программа будет написать:

getter runs 
getter runs

Если поменять порядок проверок, getter runs будет печатать только один раз. Если вы измените свойство на return "";, это будет наоборот.

Использование string.IsNullOrEmpry(Logurl) будет всегда получать собственность ровно один раз, конечно.

Answer 5

private static bool IsNullOrEmpty(string s) 
    { 
     return s == null || s == ""; 

     /* 
     Lets look behind the scenes here: 
     ================================= 
     IL_0000: ldarg.0 => load s on the evaluation stack 
     IL_0001: brfalse.s IL_000f => GoTo label 'IL_000f' if loaded argument is null 

     IL_0003: ldarg.0 => load s on the evaluation stack 
     IL_0004: ldstr "" => load constant string "" to the evaluation stack 
     IL_0009: call bool [mscorlib]System.String::op_Equality(string, string) 
           => Call String.Equality(string,string) with s and "" 
           loaded to the evalutation stack 
           that will pop the two values compare them for equality and load the result. 
           to the evaluation stack.          

     IL_000e: ret => Return to the caller with equlity result on the evauation stack. 

     IL_000f: ldc.i4.1 => Load constant value 1(4 byte which will represent "True") to the evaluation stack 
          and return to the caller.In our flow it's the case when s is null. 
     IL_0010: ret 

     In Summary: 
     =========== 
     1.) IL instructions total code size 17 bytes. 
     2.) Best case scenario execution path => 2 IL instructions. 
     3.) Worst case scenario execution pat => 8 IL instructions. 

     */ 
    } 

    private static bool IsEmptyOrNull(string s) 
    { 
     return s == "" || s == null; 

     /* 
     Lets look behind the scenes here: 
     ================================= 
     IL_0000: ldarg.0 => load s on the evaluation stack 
     IL_0001: ldstr "" => load constant string "" to the evaluation stack 
     IL_0006: call bool [mscorlib]System.String::op_Equality(string, string) 
     IL_000b: brtrue.s IL_0012 

     IL_000d: ldarg.0 => load s on the evaluation stack 
     IL_000e: ldnull => load constant null on the evaluation stack 
     IL_000f: ceq => Pop two loaded values compare and push the result back on the evaluation stack 
     IL_0011: ret 

     IL_0012: ldc.i4.1 => Load constant value 1(4 byte which will represent "True") to the evaluation stack 
          and return to the caller.In our flow it's the case when s is null. 
     IL_0013: ret 

      In Summary: 
     =========== 
     1.) IL instructions total code size 20 bytes. 
     2.) Best case scenario execution path => 6 IL instructions. 
     3.) Worst case scenario execution path => 10 IL instructions. 
    */ 

    } 

Вывод:

Судя только IL излучаемого код "если (app.Logourl == "" || app.Logourl == NULL)" является "microptimization" лучше производительность мудрым:)