Discussion :

[lostsoul]

Bonjour,

Je recupere le texte d'un lineedit, pour l'ecrire ensuite dans un fichier. Mais avant, je voudrais verifier qu'il ne contient que des chiffres.

Code :

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

1
2
3
4
5
 
QString tel;
tel = ui->lineEdit_3->text();
if (tel.contains(QRegExp("[a-zA-Z]*")))
QMessageBox::information(this,("Info"), ("erreur"));

Mais il m'affiche "erreur" quoi que j'inscrive...
Je fais quoi de mal ?

Merci d'avance,
Tres cordialement,

[Troudhyl]

Là tu ne vérifies pas qu'"il ne contienne que des chiffres" mais qu'"il ne contienne pas de lettres". Je ne suis pas très calé en QRegExp mais en tout cas faut faire ce qu'on dit sinon on laisse des failles.
Je ne réponds donc pas à ta question "qu'est-ce que j'ai fait de mal" parce que je ne vois pas.

Par contre pour faire ce que tu dis donc, j'essaierais :

Code :

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

1
2
if (!tel.contains(QRegExp("^\\d+$")))
    QMessageBox::information(this,("Info"), ("erreur"));

Je laisse quelqu'un d'autre répondre à ta question A priori, je dirais que c'est à cause de l'étoile, qui veut dire "0 ou +". A remplacer par le + (1 ou +). A cause de l'étoile ça me parait logique que cette expression soit toujours vraie.

[lostsoul]

ça semble marcher en ajoutant un ! au début.
Merci beaucoup ;-)

[Troudhyl]

"Rajouter un !" ça ne veut pas dire grand chose (à part complètement inverser le test), je suis sceptique sur le fait que "ça marche", peux-tu partager la solution complète et la traduction en français ? (si blabla ne contient pas truc alors erreur). Merci

[lostsoul]

Avec ce code, pas de de message d'erreur uniquement si tel (QString) ne contient que des chiffres :

Code :

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

1
2
3
 
if (!tel.contains(QRegExp("^\\d+$")))
            QMessageBox::information(this,("Téléchargement"), ("erreur"));

[Troudhyl]

Ah ok, donc t'as repris ce que j'avais dit, j'ai cru que t'avais juste rajouté un "!" à ce que t'avais fait. Là je suis d'accord.

Une autre façon, peut-être moins lourde que la vérif par regexp :

Code :

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

1
2
3
4
bool isInt;
tel.toInt(&isInt); // conversion en entier qu'on ne sauvegarde pas, seule la réussite ou non nous intéresse
if (!isInt)
    QMessageBox::information(this,("Téléchargement"), ("erreur"));

La conversion foire même s'il y a une virgule ou un point (ça convertit par les décimaux en entier quoi) par contre ça accepte les espaces autour.