Discussion :

[benthebest]

Hello à tous

Je chercher à récupérer une string issue d'un pattern regex (un mail notamment) et je m'aperçois que ma méthode ne fonctionne pas !

Voila la fonction qui est sensé récupérer mon pattern mail, s étant la string cencée contenir éventuellement un mail:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public static String getPatternMailIfExist(String s)
    {
        String pattern = "^[A-Z0-9._%+-]+@[A-Z0-9.-]+\\.[A-Z]{2,4}$";
        Pattern p = Pattern.compile(pattern);
        Matcher m = p.matcher(s);
        while(m.find())
        {
            return (m.group(1));
        }
        return null;
    }

Et bien lorsque je teste cette fonction sur la string :

Code :

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

1
2
 
String s = "(PIC)(PIC)\n\nCharles\n\n\nMART\n\n\nN\n\n\ncharles@explisites.com\nTél\n\n\n:\nOl\n38\n45\n23\n63";

Ça ne fonctionne pas (retourne null et mois je voudrais que ca me retourne "charles@explisites.com")

Quelqu'un a une idée ?

Merci

[gailuris]

Salut,
de ce que je vois ton expression reguliere n'accepte que les majuscule pour que les minuscule passe il faudrait la changer en quelque chose du style (j'ai enlevé les symboles de début de chaine et fin de chaine : en les laissant il ne peut y avoir que l'adresse dans la chaine que tu testes, rien avant ni après):

Code :

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

"[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\\.[a-zA-Z]{2,4}"

[benthebest]

Gailuris merci de ta réponse !

Ca à l'air de marcher mais là j'ai une autre erreur, c'est au niveau de ma fonction:

Code :

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

1
2
3
 
Error Type: java.lang.IndexOutOfBoundsException
No group 1

m.group(1) n'exsite pas apparement...

Il faut mettre quoi ici alors ? (0 ?)

[thelvin]

Il faut mettre quoi ici alors ? (0 ?)

Oui, comme indiqué par la JavaDoc de Pattern

(À noter au passage que reconnaître une adresse email avec un pattern, c'est pas à la portée du premier venu et que toute tentative de le faire va forcément créer plein de faux négatifs. Bref, à part pour vérifier s'il y a bien un @ avec quelque chose de pas transparent devant et derrière, c'est stupide.
C'est une stupidité souvent demandée par les clients, je le sais, mais je conseille de bien s'assurer qu'ils y tiennent, aux bugs qu'ils te demandent d'insérer.)

[benthebest]

Ok super les amis merci à vous c'est ok !!