L'analyse d'une chaîne de caractères en C#; est-il le moyen le plus propre?

C#, .NET 3.5

Ça sent juste affreux pour moi, mais je ne peux pas penser à une autre façon.

Donné une chaîne de caractères au format "Joe Smith (jsmith)" (sans les guillemets), j'aimerais analyser seulement les "jdupont" chaîne à l'intérieur de la parenthèse. J'ai trouvé 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 (onu)saine aversion pour la RegEx, est-il un moyen plus simple d'analyser la sous-chaîne?

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

InformationsquelleAutor Metro Smurf | 2009-04-07
  1. 9

    Vous ne devriez pas besoin de la première remplacer puisque vous pouvez simplement ajouter 1 à l' "(" position.

    private static string DecipherUserName (string user) {           
    int start = user.IndexOf( "(" );
    if (start == -1)
        return user;
    return user.Substring (start+1).Replace( ")", string.Empty );
}
    • À noter: j'ai accepté le refactor par Pax sur les RegEx solution à condition que je me sens sur le long terme, ce sera plus facile à maintenir jusqu'à ce que je devienne plus versé dans la RegEx. Que, et un test rapide de 1000 noms avaient la RegEx solution à .1165195 ms vs Chaîne de solution à .0077423 mme.
    • Pour un texte simple match comme cela, REs sont généralement plus lent (même s'il peut être établi une fois pour plus de vitesse dans une boucle). Leur avantage réel est livré avec leur expressivité pour les cas plus complexes (par exemple, en autorisant et en supprimant des espaces à l'intérieur des parenthèses). Très bien vaut d'apprentissage, afin de le garder.
    InformationsquelleAutor paxdiablo
  2. 20

    Regexes sont donc utiles que vous économiserez de vous-même une tonne de chagrin mordre la balle et de leur apprentissage. Pas tout le toutim, juste l'essentiel.

    Une regex qui va travailler est "\w+\((.*)\)" - jsmith serait dans le Match.Les groupes[1].

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

    • +1 pour les regex, puisque c'est le bon outil
    • Il n'y a aucune exigence explicite que la parenthèse-joint le nom d'utilisateur doit suivre un nom sans espace. "(.*)" devrait être suffisant.
    • Bon outil, mais la mauvaise RE - vous besoin d'un qui va gérer le cas d'un utilisateur sans les parenthèses. Le faire et je vais vous donner un vote 🙂
    • Mon expérience avec C# et les regex, c'est puissant aussi longtemps que votre chaîne est inférieur à 10 000 caractères environ. Il devient très lente après que, au point où à l'aide de remplacer et indexof sont mille fois plus rapide. Mais c'est toujours le bon outil pour ce que vous faites en dessous de ce chiffre.
    • Une expression régulière est vraiment le bon outil pour le travail dans ce cas.
    • Le regroupement devrait être échappé à l'intérieur de la parenthèse, de sorte que vous n'avez pas à dépouiller les supports par la suite; c'est à dire "((.*))"
    • doh, vous avez raison, le groupement doit être dans l'échappée parens
    • Certes, je n'ai pas vraiment poursuivi RegEx est principalement en raison de ne pas en avoir besoin très souvent et finissent par oublier ce que j'ai appris. @James, et al - la première RegEx de "(.*)" cherche à exprimer tout à fait bien (et simplement) mes intentions. Je vais vous donnez ce un aller quand je suis de retour sur ma machine de dev. Cheers!
    • Croyez-moi, prenez le temps d'apprendre les Expressions Régulières. Ils vous permettent de gagner de la journée (et vous donner ajouté geek cred) lorsque vous y attendez le moins. Je sais que j'ai donné une réponse différente, mais c'était pour l'amour de nettoyage de code 8^D
    • À noter: j'ai accepté le refactor par Pax sur les RegEx solution à condition que je me sens sur le long terme, ce sera plus facile à maintenir jusqu'à ce que je devienne plus versé dans la RegEx. Que, et un test rapide de 1000 noms avaient la RegEx solution à .1165195 ms vs Chaîne de solution à .0077423 mme.
    • "Certaines personnes, lorsqu'ils sont confrontés à un problème, pense que "je sais, je vais utiliser des expressions régulières." Maintenant, ils ont deux problèmes." - jwz

    InformationsquelleAutor Arnshea
  3. 5

    Genre de bidouille ... ^^

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

    Si user contient pas de parenthèse ouvrante, IndexOf() retourne -1, nous ajoutez-en un, obtenez zéro, et SubString() retourne l'ensemble de la chaîne. 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 ouvrante, IndexOf() retourne son index, nous skipp l'ouverture de la parenthèse par l'ajout de l'un, et d'en extraire le reste de la chaîne avec Substring(). Enfin, nous enlever la parenthèse fermante avec TrimEnd().

    • Belle utilisation de la chaîne de l'API.
    • Cela ne reflète pas l'intention très bien.
    • Première ligne: une Sorte de hack ... ^^ 😉
    • J'aime vraiment les expressions régulières et moi-même préférons une regex solution. Mais c'est une belle oneliner et pas plus de confusion que "\w+((.*))" pour le profane.
    InformationsquelleAutor Daniel Brückner
  4. 5

    Si l'utilisateur de la chaîne est toujours sous la forme "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 l'utilisateur de la chaîne est toujours "Joe Smith (jsmith)", ce le sera toujours. 

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

    La deuxième entrée de l'humour seulement.

    InformationsquelleAutor Matthew Sposato
  5. 2

    Depuis le IndexOf fonction retourne -1 si la valeur n'existe pas, vous pourriez faire des choses un peu différente...

    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;
   }
}
    • Oublié que IndexOf sera de retour -1 si la valeur n'existe pas. Merci!
    InformationsquelleAutor Dillie-O
  6. 1

    J'utiliserais

    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 juste un changement cosmétique: à l'aide de caractères dans IndexOf est un peu plus rapide, et à l'aide de la méthode sous-Chaîne semble exprimer plus exactement ce qui doit être fait (et la méthode est plus robuste si vous avez plusieurs paires de parenthèses...)

    Cela dit, Daniel L's de la méthode (à l'aide de String.Split) qui pourrait être plus simple (mais ne traitent pas très bien avec la malformation de chaînes de caractères et doit construire un tableau de chaîne).

    Dans l'ensemble, je vous suggère de surmonter votre aversion pour les expressions régulières, depuis que la situation est exactement ce qu'ils sont destinés 🙂 ...

    • Eh bien, c'est un (des nations unies)en santé de l'aversion (-_^) Votre commentaire à propos de la sous-chaîne à être plus expressif est certainement vrai.
    InformationsquelleAutor MartinStettner