Analyse d'une chaîne en C#; Y a-t-il une manière plus propre?

Question

C#, .NET 3.5

Cela me sent juste moche, mais je ne peux pas penser à une autre façon.

Étant donné une chaîne au format "Joe Smith (jsmith)" (sans guillemets), je voudrais analyser uniquement la chaîne "jsmith" entre parenthèses. Je suis venu avec ceci:

private static string DecipherUserName(string user) 
{ 
    if(!user.Contains("(")) 
     return user; 

    int start = user.IndexOf("("); 

    return user.Substring(start).Replace("(", string.Empty).Replace(")", string.Empty); 
} 

Autre que mon (non) saine aversion pour RegEx, est-il un moyen plus simple d'analyser le sous-chaîne?

Edit: Pour clarifier les choses, la chaîne à analyser sera toujours de: "Joe Smith (jsmith)" (sans guillemets).

Answer 1

Vous ne devriez pas avoir le premier remplacer puisque vous pouvez simplement ajouter 1 à la "(position".

private static string DecipherUserName (string user) {   
    int start = user.IndexOf("("); 
    if (start == -1) 
     return user; 
    return user.Substring (start+1).Replace(")", string.Empty); 
} 

Answer 2

Depuis la fonction IndexOf renvoie -1 lorsque la valeur n'existe pas, vous pouvez faire des choses un peu différentes ...

private static string DecipherUserName(string user) 
{   
    int start = user.IndexOf("("); 

    if (start > -1) 
    { 
     return user.Substring(start).Replace("(", string.Empty).Replace(")", string.Empty); 
    } 
    else 
    { 
     return user; 
    } 
} 

Answer 3

Regexes sont si utiles que vous vous sauverez une tonne de chagrin d'amour mordre la balle et les apprendre. Pas tout le shebang, juste les bases.

Une regex qui fonctionnera est "\ w + \ ((. *) \)" - jsmith serait dans Match.Groups [1].

Un moyen facile de ramasser regexes est de trouver un site qui va vous permettre de taper dans un regex et un texte puis cracher les allumettes ...

Answer 4

Type d'un hack ... ^^

return user.Substring(user.IndexOf('(') + 1).TrimEnd(')'); 

Si user ne contient pas entre parenthèses d'ouverture, IndexOf() renvoie -1, nous en ajoutons un, obtiens zéro, et SubString() renvoie la chaîne entière. TrimEnd() n'aura aucun effet à moins que le nom de l'utilisateur se termine par une parenthèse fermante.

Si user contient une parenthèse d'ouverture, IndexOf() retourne son index, nous Skipp la parenthèse d'ouverture en ajoutant un, et extraire le reste de la chaîne avec Substring(). Enfin, nous supprimons la parenthèse fermante avec TrimEnd().

Answer 5

J'utilise

int start=user.IndexOf('('); 
if (start != -1) { 
    end = user.IndexOf(')', start); 
    return user.Substring(start+1,end-start-1); 
} else 
    return user; 

Mais c'est un simple changement cosmétique: en utilisant des caractères à IndexOf est un peu plus rapide, et en utilisant la méthode Substring semble exprimer plus exactement ce qu'il faut faire (et méthode est plus robuste si vous avez plusieurs paires de parenthèses ...)

Cela dit, méthode de Daniel L (en utilisant String.Split) pourrait être plus simple (mais ne traite pas très bien avec les chaînes malformées et doit construire un tableau de chaînes).

Dans l'ensemble, je vous suggère de surmonter votre aversion pour les expressions régulières, puisque cette situation est exactement ce à quoi elles sont destinées :-) ...

Answer 6

Si la chaîne utilisateur est toujours au format "Joe Smith (jsmith)", cela devrait fonctionner.

private static string DecipherUserName(string user) 
{ 
    string[] names = user.Split(new char[] {'(', ')'}); 
    return names.Length > 2 ? names[1] : user; 
} 

Et si la chaîne utilisateur est toujours "Joe Smith (jsmith)", cela fonctionnera toujours.

private static string DecipherUserName(string user) 
{ 
    return "jsmith"; 
} 

La deuxième entrée à des fins d'humour uniquement.