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.

[flapili]

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 :

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

[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 ?

[flapili]

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

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

[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

[flapili]

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

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