problème avec re.search

Version imprimable

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:

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 ..

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:

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))

04/04/2018, 20h52
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:

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.
04/04/2018, 22h44
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.
05/04/2018, 07h05
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.

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:

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.

Bonjour,
J'ai finalement réussi à faire se que je voulait, merci bistouille de m'avoir mis sur la voie avec les ?!,
voici mon code final:
Code:

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 not re.match(r'[a-zA-Z](-?[a-z]+){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")
Bon je trouve que le test de conditions est ni trop long ni trop incompréhensible, si jamais quelqu'un voit une amélioration à faire je suis preneur d'idée :)

05/04/2018, 21h22
bistouille

Salut,

La 1ère chose à rectifier est ton range pour vérifier la longueur qui ne sert strictement à rien puisque tu vérifies la longueur min et max.
La seconde, est dans la vérification des caractères consécutifs, il faut un break, sinon en cas de saisie de pseudo comme toooooooooooto, tu vas te retrouver avec la même erreur ajoutée x fois, perso, là j'aurais tout testé dans une expression, mais le principal est que ça fonctionne.

Aussi tant qu'à faire autant ne calculer qu'une seule fois la longueur du pseudo, bon, c'est une optimisation infinitésimale, mais c'est toujours mieux d'éviter d'appeler la même méthode à plusieurs reprises si on peut l'éviter.

Par contre, je n'ai pas compris ce qu'est censé faire [a-zA-Z](-?[a-z]+){0,2}, pourquoi 0,2 ? Cela signifie qu'il peut y avoir au maximum 2 tirets dans la chaine ?
05/04/2018, 21h54
flapili

Citation:

Envoyé par bistouille

Par contre, je n'ai pas compris ce qu'est censé faire [a-zA-Z](-?[a-z]+){0,2}, pourquoi 0,2 ? Cela signifie qu'il peut y avoir au maximum 2 tirets dans la chaine ?

oui, c'était le but enfaite à la base

Citation:

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
05/04/2018, 22h23
bistouille

Ah, ok, j'avais pas fait attention au 2 tirets max.

Dans ce cas, un pseudo comme toto-toto-toto ne passe pas ton expression doit-être modifiée :mrgreen:

En effet j'ai encore fait n'importe quoi ... x)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
 
def __init__(self):
	adress = QtWidgets.QLineEdit()
	layout.addWidget(QtWidgets.QLabel(text="adresse :"), 0, 0)
	layout.addWidget(adress, 0, 1)
 
	port = QtWidgets.QSpinBox(minimum=1, maximum=655350, objectName="port")
	port.valueChanged.connect(lambda value: port.setValue(port.maximum()/10) if value > port.maximum()/10 else None)
	layout.addWidget(QtWidgets.QLabel(text="port :"), 1, 0)
	layout.addWidget(port, 1, 1)
 
	pseudo = QtWidgets.QLineEdit()
	pseudo.setValidator(QtGui.QRegExpValidator(QtCore.QRegExp(r'[a-zA-Z](-?[a-z]+){,3}$')))
	layout.addWidget(QtWidgets.QLabel(text="pseudo :"), 2, 0)
	layout.addWidget(pseudo, 2, 1)
 
	button_connect = QtWidgets.QPushButton(text="Connexion")
	button_connect.clicked.connect(lambda: self.connect(adress.text().strip(), port.value(), str(pseudo.text().strip())))
	layout.addWidget(button_connect, 4, 0, 1, 2)
 
def connect(self, adress, port,  pseudo):
	errors = []
	if not IP:
		errors.append("erreur : IP non renseignée")
	if not pseudo:
		errors.append("erreur : pseudo non renseigné")
	else:
		if len(pseudo)<2:
			errors.append("erreur : pseudo trop court")
		elif len(pseudo)>20:
		errors.append("erreur : pseudo trop long")
		elif pseudo[:-1] == "-":
			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")

J'ai préféré me passer du module re et passer directement par un QRedExp, vaut mieux prévenir que guérir comme le dit le proverbe :)