Оператор C# if с string.Contains() не работает как ожидалось

Question

Я застрял на том, что, как я думал, будет очень простой проблемой. Я пытаюсь перенаправить пользователя на другой веб-сайт, если UserAgent не содержит ряд строк. Часть, которую я не могу понять, заключается в том, что оператор if отлично работает, если я использую приведенный ниже код. Я доволен результатами с этим, но что-то говорит мне, что это не хорошая практика, чтобы иметь только инструкцию else и ничего не выполнять, если утверждение доказывает истинность.

 string strUserAgent = Request.UserAgent.ToString().ToLower(); 

     if (strUserAgent != null) 
     { 
      if (Request.Browser.IsMobileDevice == true || 
       strUserAgent.Contains("iphone") || 
       strUserAgent.Contains("blackberry") || 
       strUserAgent.Contains("mobile") || 
       strUserAgent.Contains("windows ce") || 
       strUserAgent.Contains("opera mini") || 
       strUserAgent.Contains("palm") || 
       strUserAgent.Contains("android")) 
      { 
       // Is this normal practice to only have an else block? 
      }else 
      { 
       Response.Redirect(AppState["redirectBack"].ToString()); 
      } 

Когда я пытаюсь выполнить следующий блок кода, сценарий перенаправляет пользователя независимо от того, что содержит строка UserAgent. Может кто-нибудь объяснить, почему это может произойти?

 string strUserAgent = Request.UserAgent.ToString().ToLower(); 

     if (strUserAgent != null) 
     { 
      if (Request.Browser.IsMobileDevice != true || 
       !strUserAgent.Contains("iphone") || 
       !strUserAgent.Contains("blackberry") || 
       !strUserAgent.Contains("mobile") || 
       !strUserAgent.Contains("windows ce") || 
       !strUserAgent.Contains("opera mini") || 
       !strUserAgent.Contains("palm") || 
       !strUserAgent.Contains("android")) 
      { 
       Response.Redirect(AppState["redirectBack"].ToString()); 
      } 

Answer 1

Обратить ваше заявление с помощью ! («нет»):

if(!(conditions)) { } 

Это позволит избежать необходимости использовать пустой блок кода, и вы можете просто отказаться от else.

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

Обратите внимание, что это намного проще создать коллекцию возможностей и проверить, если ваш UserAgent находится там:

if(new[] {"iphone", "somephone", "otherphone" }.Any(x => useragent.Contains(x))) {} 

Answer 2

Вы должны полностью изменить всю вещь. Помещение !A || !B - это не то же самое, что и !(A||B). В первом случае, если его A тогда это не B, то это True. Во втором - False.

if (!(Request.Browser.IsMobileDevice == true || 
      strUserAgent.Contains("iphone") || 
      strUserAgent.Contains("blackberry") || 
      strUserAgent.Contains("mobile") || 
      strUserAgent.Contains("windows ce") || 
      strUserAgent.Contains("opera mini") || 
      strUserAgent.Contains("palm") || 
      strUserAgent.Contains("android") 
    )) 
     { 
      Response.Redirect(AppState["redirectBack"].ToString()); 
     } 

Answer 3

string strUserAgent = Request.UserAgent.ToString().ToLower(); 

    if (strUserAgent != null) 
    { 
     if (!(Request.Browser.IsMobileDevice == true || 
      strUserAgent.Contains("iphone") || 
      strUserAgent.Contains("blackberry") || 
      strUserAgent.Contains("mobile") || 
      strUserAgent.Contains("windows ce") || 
      strUserAgent.Contains("opera mini") || 
      strUserAgent.Contains("palm") || 
      strUserAgent.Contains("android")))    
     { 
      Response.Redirect(AppState["redirectBack"].ToString()); 
     } 

Answer 4

Это всегда будет верно, что по крайней мере один из ваших условий не будет истинным. Например, если strUserAgent.Contains (iphone) будет false, если значение strUserAgent.Contains («blackberry») истинно.

Вам необходимо сменить оператора OR (||) логическому оператору AND (&&).

if (strUserAgent != null) 
    { 
     if (Request.Browser.IsMobileDevice != true && 
      !strUserAgent.Contains("iphone") && 
      !strUserAgent.Contains("blackberry") && 
      !strUserAgent.Contains("mobile") && 
      !strUserAgent.Contains("windows ce") && 
      !strUserAgent.Contains("opera mini") && 
      !strUserAgent.Contains("palm") && 
      !strUserAgent.Contains("android")) 
     { 
      Response.Redirect(AppState["redirectBack"].ToString()); 
     } 

Answer 5

Необходимо De Morgan's Law. Когда вы перевернули свое состояние, ваши ОР должны стать ИД

Answer 6

Не ответ, просто предложение. Вы можете сделать свой код понятнее с помощью метода расширения:

public static bool ContainsAnyOf(this string source, params string[] strings) 
{ 
    return strings.Any(x => source.Contains(x)); 
} 

А теперь написать

if (strUserAgent.ContainsAnyOf("iphone", "blackberry", "mobile", "windows ce", "opera mini", "palm", "android")) 
{ 
    // 
}