Discussion :

[new_wave]

Bonjour,

soit le code suivant de la méthode GetValidEmailAddressesvalidation de validation d'adresses email

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
private static List<string> GetValidEmailAddresses(string input, string pattern){
var regex = regex.Matches(input);
var validEmailAddresses = new List<string>();
foreach(Match match in matches)[
 
if(match.Success){
 
validEmailAddresses.Add(match.Value);}
}
 
return validEmailAddresses;
 
}

Normalement, la variable regex contient toutes les occurrences de la chaine input qui correspondent à l'expression régulière.
Pourquoi retester dans la boucle foreach, if(match.Success)alors qu'il est clair que tous les éléments match de type Match correspondent bien à l'expression régulière.


Je vous remercie de votre réponse sur ce point.

Cordialement.
new_wave

[CosmoKnacki]

Il n'y a effectivement aucune raison, c'est tout à fait inutile.

Je pense que l'auteur a dû s'emmêler les pinceaux avec la méthode Regex.Match(String, String) qui serait plus appropriée ici vue qu'elle implique directement le bouclage pour obtenir les résultats successifs:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
private static List<string> GetValidEmailAddresses(string input, string pattern) {
 
    var validEmailAddresses = new List<string>();
    Match match = Regex.Match(input, pattern);
 
    while (match.Success) {
        validEmailAddresses.Add(match.Value);
        match = match.NextMatch();
    }
 
    return validEmailAddresses;
}

On peut aussi écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
private static List<string> GetValidEmailAddresses(string input, string pattern) {
 
    var validEmailAddresses = new List<string>();
 
    foreach (Match match in Regex.Matches(input, pattern)) {
        validEmailAddresses.Add(match.Value);
    }
 
    return validEmailAddresses;
}

[new_wave]

Merci à toi de ta réponse.

Les choses sont claires à présent.

Bien cordialement

new_wave