Discussion :

[flapili]

Bonjour,
je galère avec les regex .. et je ne comprend pas où est l'erreur

Les conditions :

• taille comprise entre 2 et 20 caractères
• majuscule possible uniquement en début de pseudo
• pas de caractères spéciaux sauf "-" à ne pas placer en fin de pseudo, 2max
• 2 lettres identiques successives max

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if not pseudo:
errors.append("erreur : pseudo non renseigné")
else:
if not len(pseudo) in range(2, 20):
	if len(pseudo)<2:
		errors.append("erreur : pseudo trop court")
	elif len(pseudo)>20:
		errors.append("erreur : pseudo trop long")
if re.match("^[\a]?(-?[:lower:]+){0,2}$", pseudo):
	errors.append("erreur : pseudo invalide")
else:
	for i in range(len(pseudo)-2):
		if pseudo[i]==pseudo[i+1]==pseudo[i+2]:
			errors.append("erreur : pseudo invalide")

mais par exemple 'A-' est valide et 'a--a-a-a-a-a-a' l'est aussi ..

[bistouille]

Salut,

Je n'ai pas compris pourquoi 'a--a-a-a-a-a-a' ne serait pas valide, selon tes conditions, cela devrait l'être, ou alors j'ai mal saisi quelque chose.

Sans re, j'aurais fait ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
import itertools
 
def test_pseudo(pseudo) :
    mes = None
    if not 2 <= len(pseudo) <= 20 :
        mes = 'longueur du pseudo'
    elif not pseudo[1:].islower() :
        mes = 'majuscule dans le pseudo'
    elif [c for c in pseudo.lower() if ord(c) not in list(range(97, 123)) + [45]] :
        mes = 'Caractères non autorisés'
    elif pseudo[-1] == '-' :
        mes = '- en fin de pseudo'
    elif [n[0] for n in itertools.groupby(pseudo) if len(tuple(n[1])) > 2] :
        mes = 'répétition de caractères'
    return mes
 
for p in ('bob'*7, 'bOb', 'Bob-', 'bo_b', 'Booob', 'Bob-bob', 'bo') :
    print(p, test_pseudo(p))

[tyrtamos]

Bonjour,

Je ne suis pas un spécialiste des regex, mais voilà ce que j'ai trouvé, en fonction de ce que j'ai compris de tes conditions:

Code :

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

1
2
3
4
5
6
7
motif1 = r"^[a-zA-Z][a-z\-]{0,18}[a-z]$"
motif2 = r"([a-z\-])\1{2,}"
 
ch = r"A--a-a-a-a-a-a"
ok = re.match(motif1, ch)!=None and re.search(motif2, ch)==None
print(ok)
True

Le search avec le motif2 doit échouer pour qu'on sache qu'il n'y a jamais plus de 2 caractères identiques consécutifs. Par exemple, ch = r"A--a-aaa-a-a-a-a" échouera.

Vérifie que ça colle avec toutes tes conditions.

[flapili]

Merci de vos réponses,
J'ai ça pour l'instant comme regex r"^[a-zA-Z][a-z]*-?[a-z]+-?[a-z]+$" mais il faut au moins 3 caractères, le seul moyen est de faire r"^[a-zA-Z][a-z]*-?[a-z]+-?[a-z]*$" et séparément vérifier que pseudo[-1:] != "-".

Existe t'il un moyen en regex de faire des conditions ?
Du style si y'as "-" il doit obligatoirement y avoir un caractère après ? plutôt que de faire il y a 0 ou 1 "-" et après 1 ou plus caractères.

[tyrtamos]

Bonjour,

Essaie ma solution, puisqu'elle répond à ta dernière question (pas de '-' à la fin).

Par contre, s'il doit y avoir 3 caractères mini (tu avais dit 2 dans tes conditions), dans mon motif1, il faut corriger le {0,18} => {1,18}.

Et dans ton motif, tu as oublié le '-', mais même en le corrigeant de ça, il ne marche pas (il devrait refuser r"A--a-a-aaa-a-a-a"). Pour ma part, je n'ai pas trouvé de solution pour tester "pas plus de 2 caractères identiques consécutifs" dans un seul motif.

[bistouille]

Faire tout avec une expression complexe, je ne sais pas si c'est vraiment une bonne solution à adopter, car c'est difficilement lisible, et encore plus si on revient beaucoup plus tard sur notre code.

Quelque chose qui pourrait tendre vers ce que tu veux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
reg = re.compile(r'^(?!-)([A-Z])?(([a-z-])(?!\3{2,})){2,19}(?(1)|[a-z])(?<!-)$')
 
for p in (
    'Bo',
    'bo',
    'bob',
    'Booo',
    'bbb',
    'bo'*9 + 'b',
    'bo'*10,
    'bo'*9 + 'b-',
    'bob'*7,
    'Bob-',
    '-bob',
    'bob-bob',
    'Bob',
    '-Bob',
    'bob---bob',
) :
    print(p, len(p), bool(reg.match(p)))

Pas sûr que ça concorde bien avec tous tes critères.

Mais ça reste tout de même imbitable.