Comment puis-je écrire ceci plus court?

Question

public static string GetUa(HttpRequest hr) 
{ 
    try 
    { 
     string visitorBrowser = hr.UserAgent.ToString(); 
     string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 

     if (!String.IsNullOrEmpty(originalBrowser)) 
     { 
      return "OPERAMINI " + originalBrowser; 
     } 
     else 
     { 
      if (!String.IsNullOrEmpty(anotherOriginalBrowser)) 
      { 
       return "NOVARRA " + anotherOriginalBrowser; 
      } 
      else 
      { 
       return visitorBrowser; 
      } 
     } 
    } 
    catch 
    { 
     return "No UA Found"; 
    } 
} 

Answer 1

Je serais plus préoccupé par la lisibilité. Cela me semble plus agréable à:

var operaAgent = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
var deviceAgent = hr.ServerVariables["X-Device-User-Agent"]; 

operaAgent = string.IsNullOrEmpty(operaAgent) ? null : "OPERAMINI" + operaAgent; 
deviceAgent = string.IsNullOrEmpty(deviceAgent) ? null : "NOVARRA" + deviceAgent; 

return operaAgent ?? deviceAgent ?? hr.UserAgent ?? "Not Found"; 

Bien sûr, si vous ne l'avez pas besoin de préfixer ces chaînes sur les UAs et n'a pas besoin de vous préoccuper avec les agents utilisateurs de chaîne vide, il serait alors simplement:

return hr.ServerVariables["X-OperaMini-Phone-UA"] ?? 
     hr.ServerVariables["X-Device-User-Agent"] ?? 
     hr.UserAgent ?? 
     "Not Found"; 

Answer 2

Je ne vois aucun moyen de raccourcir significativement cela.

Une façon d'économiser quelques lignes est de se débarrasser des accolades autour du premier autre:

if (!String.IsNullOrEmpty(originalBrowser)) 
{ 
    return "OPERAMINI " + originalBrowser; 
} 
else if (!String.IsNullOrEmpty(anotherOriginalBrowser)) 
{ 
    return "NOVARRA " + anotherOriginalBrowser; 
} 
else if (!String.IsNullOrEmpty(visitorBrowser)) 
{ 
    return visitorBrowser; 
} 
else 
{ 
    return "No User Agent Detected"; 
} 

Vous devriez faire attention à utiliser des exceptions pour le contrôle des flux, aussi. statenjason a la bonne idée.

Answer 3

À l'heure actuelle, ce que vous avez est clair et lisible. Si vous essayez d'y arriver avec moins de temps de traitement, je ne pense pas que vous allez y arriver. Si vous essayez d'y arriver avec moins de lignes de code, vous le pouvez, mais ça va être moche.

Un moyen facile de le raccourcir à l'écran (même nombre de LOC, -1) est de supprimer certains de vos accolades et non stocker visitorBrowser:

public static string GetUa(HttpRequest hr) 
{ 
    try 
    { 
     string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 

     if (!String.IsNullOrEmpty(originalBrowser)) 
      return "OPERAMINI " + originalBrowser; 
     else 
      if (!String.IsNullOrEmpty(anotherOriginalBrowser)) 
       return "NOVARRA " + anotherOriginalBrowser; 
      else 
       return hr.UserAgent.ToString(); 
    } 
    catch 
    { 
     return "No UA Found"; 
    } 
} 

Pour moi, ce qui est légèrement moins lisible, mais probablement encore habitable.

Maintenant, vous pouvez le faire vraiment court en utilisant l'opérateur conditionnel (?:), mais il va être aussi vraiment méchant pour une meilleure lisibilité. Si je voyais code comme suit dans une revue de code, je ferais le développeur réécrire pour plus de clarté:

public static string GetUa(HttpRequest hr) 
{ 
    try 
    { 
     string visitorBrowser = hr.UserAgent.ToString(); 
     string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 

     return !(string.IsNullOrEmpty(originalBrowser)) ? "OPERAMINI " + originalBrowser : 
       !(string.IsNullOrEmpty(anotherOriginalBrowser)) ? "NOVARRA " + anotherOriginalBrowser : visitorBrowser); 
    } 
    catch 
    { 
     return "No UA Found"; 
    } 
} 

Sérieusement, s'il vous plaît ne font pas le second exemple. (Je ne suis pas sûr à 100% que cela se compilera, je l'écris en haut de ma tête sur mon Mac en ce moment .Mais je suis sûr à 99,9% que ça fonctionnera, et le prochain développeur HATE vous pour elle)

Answer 4

Comme cela, par exemple:.

public static string GetUa(HttpRequest hr) { 
    try { 
    string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
    string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; 
    return 
     !String.IsNullOrEmpty(originalBrowser) ? "OPERAMINI " + originalBrowser : 
     !String.IsNullOrEmpty(anotherOriginalBrowser) ? "NOVARRA " + anotherOriginalBrowser : 
     hr.UserAgent; 
    } catch { 
    return "No UA Found"; 
    } 
} 

Answer 5

public static string GetUa(HttpRequest hr) 
{ 
    try 
    { 
     string visitorBrowser = hr.UserAgent.ToString(); 
     string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     if (!String.IsNullOrEmpty(originalBrowser)) return "OPERAMINI"+originalBrowser; 
     string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 
     if (!String.IsNullOrEmpty(anotherOriginalBrowser)) return "NOVARRA" + anotherOriginalBrowser; 
     return visitorBrowser; 
    } 
    catch 
    { 
     return "No UA Found"; 
    } 
} 

Answer 6

Comme cela (tout le reste est juste du code supplémentaire ne rien faire):

public static string GetUa(HttpRequest hr) 
{ 
    if (!String.IsNullOrEmpty(hr.ServerVariables["X-OperaMini-Phone-UA"])) 
     return "OPERAMINI " + hr.ServerVariables["X-OperaMini-Phone-UA"])) ; 
    if (!String.IsNullOrEmpty(hr.ServerVariables["X-Device-User-Agent"])) 
     return "NOVARRA " + hr.ServerVariables["X-Device-User-Agent"])) ; 
    return hr.UserAgent ?? "Not Found"; 
} 

Et vous ne devriez jamais utiliser des exceptions dans votre application normale chemin d'écoulement.

Answer 7

Voici une version condensée. Puisque vous êtes return ing à l'intérieur de chaque instruction if, vous pouvez éliminer vos else s. En outre, j'ai éliminé le besoin d'utiliser des exceptions pour le flux.

public static string GetUa(HttpRequest hr) 
    { 
     string visitorBrowser = hr.UserAgent; 
     string originalBrowser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     string anotherOriginalBrowser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 
     if (string.IsNullOrEmpty(visitorBrowser)) 
      return "No UA Found"; 
     if (!String.IsNullOrEmpty(originalBrowser)) 
      return "OPERAMINI " + originalBrowser; 
     if (!String.IsNullOrEmpty(anotherOriginalBrowser)) 
      return "NOVARRA " + anotherOriginalBrowser; 
     return visitorBrowser; 
    } 

Answer 8

seconde si-statment:

return (!String.IsNullOrEmpty(anotherOriginalBrowser) ? ("NOVARRA " + anotherOriginalBrowser) : visitorBrowser); 

Vous pouvez probablement utiliser ce combiné avec le premier si-statment trop

Answer 9

Je n'aime pas quand il est en train de faire le travail qu'il n » Je dois faire. Donc, je l'écrire comme suit:

public static string GetUa(HttpRequest hr) 
    { 
     string browser = hr.ServerVariables["X-OperaMini-Phone-UA"]; 
     if (!String.IsNullOrEmpty(browser)) 
      return "OPERAMINI " + browser; 

     browser = hr.ServerVariables["X-Device-User-Agent"]; //novarra 
     if (!String.IsNullOrEmpty(browser)) 
      return "NOVARRA " + browser; 

     if (!String.IsNullOrEmpty(hr.UserAgent)) 
      return hr.UserAgent; 

     return "No UA Found"; 
    } 

Answer 10

A .NET 4 un peu plus dynamique approche:

private static readonly Tuple<string, string>[] SpecialUas = 
     { 
      Tuple.Create("X-OperaMini-Phone-UA", "NOVARRA"), 
      Tuple.Create("X-Device-User-Agent", "OPERAMINI") 
     }; 

    public static string GetUa(HttpRequest r) 
    { 
     return (
        from specialUa in SpecialUas 
        let serverVariable = r.ServerVariables[specialUa.Item1] 
        where !string.IsNullOrEmpty(serverVariable) 
        select string.Concat(specialUa.Item2, " ", serverVariable) 
       ).FirstOrDefault() ?? (
        string.IsNullOrEmpty(r.UserAgent) 
        ? "No UA Found" 
        : r.UserAgent 
       ); 
    } 

Cela peut être personnalisé avec UAs spéciaux supplémentaires très facilement en ajoutant d'autres tuples.

Il ne serait pas difficile de remplacer les tuples avec quelque chose d'autre si vous n'êtes pas en cours d'exécution sur .NET 4.