Vérifier si une chaîne ne contient que des lettres, des chiffres et des caractères de soulignement

J'ai pour vérifier si une chaîne ne contient que des lettres, des chiffres et des caractères de soulignement.
C'est de cette façon que j'ai essayé mais ça ne fonctionne pas:

for(int i = 0; i<=snameA.Length-1; i++)
{
    validA = validA && (char.IsLetterOrDigit(snameA[i])||snameA[i].Equals("_"));
}
  • essayez d'utiliser des Regex pour ce qui vérifie les plus de ce modèle: ([A-Za-z0-9\-_]+)
  • Je ne sais pas. validA est une valeur booléenne qui doit être définie sur true uniquement si les conditions sont réalisées, mais il est défini sur true, même si la chaîne contient des caractères spéciaux
  • Que signifie "ne fonctionne pas", impliquent? les erreurs? des résultats non valides?
  • Remarque: le code a été changé dans le temps.
  • J'ai ajouté "validA &&" mais ça ne fonctionne toujours pas fonctionné.
InformationsquelleAutor | 2015-12-14
  1. 10

    J'aime Linq pour ce genre de question:

    bool validA = sname.All(c => Char.IsLetterOrDigit(c) || c.Equals('_'));
    InformationsquelleAutor Thomas Ayoub
  2. 10

    De l'affectation d' validA chaque fois à nouveau, sans vérifier sa valeur précédente. Maintenant, vous obtenez toujours la valeur de la dernière vérification exécutée.

    Vous pourriez 'et' le résultat:

    validA &= (char.IsLetterOrDigit(snameA[i]) || snameA[i] == '_');

    Cela signifie que vous courez toujours tous les personnages, ce qui peut être inutile si le premier échec de la vérification. Il est donc préférable de simplement sortir si elle échoue:

    for(int i = 0; i<=snameA.Length-1; i++)
{
    validA = (char.IsLetterOrDigit(snameA[i]) || snameA[i] == '_');

    if (!validA)
    { break; } //<-- see here
}

    Ou avec LINQ:

    validA = snameA.All(c => char.IsLetterOrDigit(c) || c == '_');
    • J'ai rétracté mon downvote. À l'aide de regexp est une question de choix et je pense que votre réponse est tout à fait correct aussi.
    • Merci. Je ne dis pas que l'OP ne faut pas utiliser les regex, mais maintenant, il sait ce qui ne va pas avec son code. 🙂 Merci pour votre honnêteté.
    InformationsquelleAutor Patrick Hofman
  3. 4

    vous pouvez utiliser des regex

    Regex regex1 = new Regex(@"^[a-zA-Z0-9_]+$");

if(regex1.IsMatch(snameA))
{

}
    • j'en oublie un énoncé..merci
    • Regex en prenant son propre exemple en tant que paramètre!
    • btw, a-zA-Z0-9_ est le même que \w
    • non, il y a une différence subtile: a-zA-Z0-9_ accepte anglais l'alphabet seulement, quand \w accepte aucun lettres, par exemple Переменная (Variable en russe). Vous avez tout à fait raison: l'actuel inplementation dans la question (via char.IsLetterOrDigit) correspond \w* dans l'expression régulière
    InformationsquelleAutor Sachu
  4. 3

    Vous pouvez essayer correspondant à une expression régulière. Il est construit dans le type de "des lettres, des chiffres et des caractères de soulignement", qui est "\w".

    Regex rgx = new Regex(@"\w*");
rgs.IsMatch(yourString);

    Si vous avez besoin de 1 ou plus, puis utilisez "\w+".

    Plus d'informations ici: Regex.IsMatch

    • si une chaîne contient des uniquement des lettres, des chiffres et des caractères de soulignement => à toi de vérifier si la chaîne contient des aucun des lettres, des chiffres et des caractères de soulignement
    InformationsquelleAutor Dinos
  5. 2

    Je voudrais utiliser une Regex

    string pattern = @"^[a-zA-Z0-9\_]+$";
Regex regex = new Regex(pattern);

//Compare a string against the regular expression
return regex.IsMatch(stringToTest);
    InformationsquelleAutor Lucas Rodriguez
  6. 1

    D'abord, lettre est un peu vague: voulez-vous dire a..z et A..Z caractères ou lettre pourrait appartenir à tout l'alphabet, par exemple а..я et А..Я (russe, l'alphabet Cyrillique). En fonction de vos actuel de la mise en œuvre, vous voulez le deuxième option.

    Solution typique avec boucle pour vérifier jusqu'à la première contre-exemple:

      Boolean validA = true; //true - no counter examples so far

  //Why for? foreach is much readble here
  foreach(Char ch in sname) 
    //"!= '_'" is more readable than "Equals"; and wants no boxing
    if (!char.IsLetterOrDigit(ch) && ! (ch != '_')) { 
      Boolean validA = false; //counter example (i.e. non-letter/digit symbol found)

      break; //<- do not forget this: there's no use to check other characters
    }

    Cependant, vous pouvez simplifier le code en Linq:

      validA = sname.All(ch => Char.IsLetterOrDigit(ch) || ch == '_');

    Ou expression régulière:

      validA = Regex.IsMatch(sname, @"^\w*$");
    InformationsquelleAutor Dmitry Bychenko