Discussion :

[Erty2100]

Bonjour, aprés avoir cherché dans la faq et dans des livres et cours de c++, j'arrive pas à trouver quelque chose qui doit être finalement assez bateau...

Mon programme invite l'utilisateur à entrer une valeur numerique de type double, et je cherche à vérifier si cette valeur est bien de type double ou pas char afin de le réinviter à saisir la valeur si elle est pas de type double.
Sinon ça me donne des résultats bizarre par la suite.

Je suis parti du principe que l'on pouvait comparer un double et un char, mais ça n'a pas l'air de fonctionner, donc si quelqu'un pouvait m'orienter, soit en me disant comment effectuer la convertion soit quel autre chemin adopter, je lui en serait reconnaissant!

if (devises=='e')
{ euro:
cout << "Veuillez entrer le montant en euros : ";
double euro;
cin >> euro;
if ((double)euro!=euro)
{ cout << "Votre montant en dollars : ";
double repe = euro*coursED;
cout << repe <<endl;
}
else
{ goto euro;
}

Voila mon code, le deuxieme if n'est pas valide, c'est là que se pose mon probléme...

merci d'avance

[bigboomshakala]

salut

oulah attends, le cout est capable de détecter lui-même lel type que tu lui passes

Code :

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

1
2
3
4
5
6
7
8
9
10
	double d;
 
	if (cin >> d)
	{
		cout << "double" << endl;
	}
	else
	{
		cout << "pas double" << endl;
	}

de plus, ta condition est bizarre, tu compares euro avec lui-même.

NOTE : n'utilise pas de goto

[Laurent Gomila]

http://c.developpez.com/faq/cpp/?pag...#SL_cin_verify

NOTE : n'utilise pas de goto

Et surtout n'utilise pas le même nom pour une étiquette et une variable.

[Herode]

Le deuxième if ne peut pas fonctionner en effet :
- La variable "euro" est un double.
- Par conséquent : (double) euro == euro dans tous les cas.

Pour le traitement qui t'intéresse, le principe est le suivant : la ligne

Code :

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

cin >> euro

envoie dans euro (un double) le contenu du buffer du flot d'entrée cin.
Si le contenu est interprétable comme un double, tout se passe bien. Sinon, le flot cin sera en erreur.
Il faut donc tester l'état de cin :

Code :

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

1
2
3
4
5
6
7
8
double euro;
cin >> euro;
if ( cin.good() ) {
  // la saisie est valable
}
else {
  // erreur de saisie
}

NB : pour plus d'infos, voir la documentation sur basic_ios (la classe mère de basic_istream, dont cin est une instance).

[Herode]

Oups, superposition de réponses...
NB : L'écriture

Code :

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

 if ( cin >> euro )

conformément à ce qui est proposé dans le lien donné par Loulou24, est d'ailleurs plus classique (et plus claire aussi...) que celle de mon post ci-dessus.

[Erty2100]

j'ai essayé ce que vous m'avezconseillé.
voila ce que ça donne pour le test :

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
bool choice(double valeur)
 
  {    while(!(cin >> valeur) || valeur<=0)
         {   if(cin.eof())
            { return false;
            }
            else if (cin.fail())
            { cout<<"Saisie incorrecte, veuillez recommencer : ";
            cin.clear();
            cin.ignore( numeric_limits<streamsize>::max(), '\n');
            }
            else
            { cout<<" La valeur est toujours incorrecte, recommencez : ";
            }
         } 
         return true;
  }

Le probléme c'est que si je le fais au cas par cas ça marche mais ça fait se repeter le meme code, alors que si je tente de le mettre avant la fonction main, là ça marche plus. Je peu compiler, mais le programme lorsqu'il arrive à l'appel du test ci dessus s'arrete...

pour être plus précis, le programme rejette les valeurs qui ne sont pas des entiers ou des doubles donc ça c'est bon.
mais pour l'operation qui suit il s'arrete et le deboggueur m'aide pas

Code :

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

1
2
3
4
5
6
7
  cout << "Veuillez entrer le montant en euros : ";
         double euro;
         if (choice(euro))
         { cout << "Votre montant en dollars : ";
           double repe = euro*coursED;
           cout << repe <<endl;
         }

coursED etant un double définit précedement dans mon programme