Discussion :

[Xzander]

Bonjour,

J'ai ceci:

Code :

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

1
2
3
4
5
string input;
int resultat;
 
cin >> input;
resultat = atoi(input);

Comment puis-je m'assurer que l'utilisateur entre bien une chaîne qui soit facilement convertible en int? Par exemple, si l'Utilisateur entre 1a, mon programme plante. J'ai essayé isdigit() mais il ne veut rien savoir d'un string...


Merci pour toute réponse!!

[Jean-Marc.Bourguet]

Utilise strtol plutôt qu'atoi.

[roulious]

Regarde la FAQ

[Jean-Marc.Bourguet]

Les deux vont considérer des choses différentes comme des erreurs. En particulier, 1a ne sera pas une erreur pour le code de la FAQ qui prendra le 1 et laissera le a comme étant à lire.

[roulious]

Autre solution : utiliser boost::lexical_cast. Si ta source n'est pas convertible dans le type de destination, une exception est lancée.

[Luc Hermitte]

Les deux vont considérer des choses différentes comme des erreurs. En particulier, 1a ne sera pas une erreur pour le code de la FAQ qui prendra le 1 et laissera le a comme étant à lire.

Tout à fait.
Ceci dit, je ne suis pas sûr si boost.lexical_cast teste bien que l'on a tout "lu".

[loufoque]

Code :

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

1
2
3
4
5
string input;
int resultat;
 
cin >> input;
resultat = atoi(input);

Code :

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

1
2
3
4
5
6
int resultat;
 
if(std::cin >> resultat)
{
    // c'est bon, l'utilisateur a entré un entier qui est stocké dans resultat
}

[Jean-Marc.Bourguet]

Tout à fait.
Ceci dit, je ne suis pas sûr si boost.lexical_cast teste bien que l'on a tout "lu".

J'ai teste, il le fait. Mais ca me semble une solution bien plus lourde (a utiliser parce qu'une exception n'est pas le bon moyen de signaler ce genre d'erreur qui doit etre traitee au niveau de l'appelant, et en performance egalement).

Quelque chose dont il faut tenir compte, je ne sais pas s'ils utilisent la locale globale ce qui peut être ou ne pas être un avantage.

[Xzander]

Code :

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

1
2
3
4
if(std::cin >> resultat)
{
    // c'est bon, l'utilisateur a entré un entier qui est stocké dans resultat
}

J'ai fait ceci et ça semble fonctionner, je ne savais pas que ça retournait un booléen.

Merci à tous pour vos réponses rapides!!

[Médinoc]

En fait, ça retourne cin, et la norme impose que les streams soient implicitement convertibles en un type testable.

[Laurent Gomila]

J'ai fait ceci et ça semble fonctionner, je ne savais pas que ça retournait un booléen

Donc tu n'as pas vraiment lu le lien qu'on ta donné vers la FAQ.