Discussion :

[Erty2100]

Bonjour, je débute en programmation c++ et j'ai rencontré un probleme avant de compiler mon programme. Habituellemen j'arrive à trouver les erreurs mais là je séche. Mon objectif est que apres une invite, l'utilisateur tape une lettre qui oriente entre 2 chemins différents. J'ai essaye for, if, mais au final j'ai pense que strncmp était plus adapté. Si vous avez des conseils n'hésitez pas, je suis ouvert à tout!

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
char devises;
     cout << "Veuillez specifier si vous voulez convertir en euro ou en dollars, tapez e pour euro, d pour dollar : ";
     cin >> devises;
   char valable = "e";
   char ok = "d";
   //comparaison des chaines de char
   int strncmpeuro(devises, valable,1);
   int strncmpdollar(devises, ok,1);
 
   if (strncmpeuro==1)
   {   cout << "Veuillez entrer le montant en euros : ";
         double euro;
         cin >> euro;
         cout << "Votre montant en dollars : ";
         double repe = euro*(1/coursED);
         cout << repe <<endl;

voila ma portion de code qui pose probléme, et voila les messages d'erreurs que j'obtiens :
invalid conversion from `const char*' to `char' (pour l'attribution de "e" à valable et de "d" à ok)
initializer expression list treated as compound expression (pour les 2 int strncmp)

merci d'avance de m'aider!

[Invité(e)]

Bonjour
que sont strncmpdollar et strncmpeuro ? des objets, des fonctions ? des entiers ?

à mon avis, s'il sagit de fonctions (ou de méthodes) il faudrai plutôt faire

Code :

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

1
2
3
4
5
6
7
   if (1==strncmpeuro(devises, valable,1))
   {   cout << "Veuillez entrer le montant en euros : ";
         double euro;
         cin >> euro;
         cout << "Votre montant en dollars : ";
         double repe = euro*(1/coursED);
         ...

Pour t'aider plus efficacement, il nous faut connaitre strncmpdollar et strncmpeuro.
Merci
mabu

[Pragmateek]

Salut!

Toute ces variables et paramètres de fonction sont ils vraiment utiles?
Car ils alourdissent vraiment le code.

[Pragmateek]

que sont strncmpdollar et strncmpeuro ? des objets, des fonctions ? des entiers ?

Des entiers? Impossible! Non?

[Pragmateek]

Code :

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

"e"

signifie chaine de 1 caractere; elle est vue par l'ordi ainsi:

Code :

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

e\0

\0 est le caractere de terminaison de chaine.
Pour un simple char il faut mettre des simples quotes '':

Code :

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

char valide='e';

devrait marcher.

[Erty2100]

d'abord, merci de vos réponses.
J'ai essayé ce que vous m'avez dit mais ça ne change rien voire ça augmente le nombre d'erreur.

pour les 2 fonctions, je les ai trouvés dans C++ pour les nuls :

Code :

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

 int strncmp(source1, source2, n)

compare les n premiers caractères de deux chaînes. Retrourne 0 si les 2 chaînes sont identiques.

Donc déja faut que je mette 0 et pas 1 dans la condition du if, mais ça ne change pas mon probléme.

Je précise que j'ai éssayé aussi en gardant strncmp sans en modifier le nom, mais cela crée un conflit entre celui pour les euros et celui pour les dollars...

Je sais pas si ça peut être utile, mais j'utilise Dev-C++

ps : merci seriousme de me rappeller cela

[niglo]

Code :

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

1
2
3
4
5
6
//comparaison des chaines de char
   int strncmpeuro(devises, valable,1);
   int strncmpdollar(devises, ok,1);
 
   if (strncmpeuro==1)
   ...

Ca ne ressemble a rien ce que tu essais de faire ici ! C'est syntaxiquement incorrect donc ca ne peut pas fonctionner !

[Erty2100]

peut tu être plus précis stp

[niglo]

Pourquoi ne pas faire tout simplement :

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
 
char devises;
cout << "Euro (e) ou dollars (d) : ";
cin >> devises;
 
if (strcmp(devises,'e'))
{  
     //traitement pour les euros
}
 
else if (strcmp(devises,'d'))
      {
           //traitement pour les dollars
      }
      else
            //message d'erreur et retour au début

[Erty2100]

comment la condition if sait que c'est 0 qui doit être renvoyé, si on ne precise pas ==0?

et ça ne semble pas marcher

[Erty2100]

en fait j'aimerais surtout savoir ce a quoi fait reférence ce message d'erreur svp : 15 C:\Dev-C++\essaies\correction2.cpp initializer expression list treated as compound expression

merci

[niglo]

Pourquoi ne pas faire tout simplement :

Code :

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

if (strcmp(devises,'e'))

Essai plutot ca :

Code :

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

if ( ! strcmp(devises,'e'))

Sinon, strcmp renvoi 0 si les deux chaines sont identiques. Documente toi sur une fonction avant de l'utiliser c'est plus simple.
http://www.cplusplus.com/ref/cstring/strcmp.html

[Erty2100]

je l'ai modiié comme ça, c'est ce qui produit le moins d'erreur (1seule) :

Code :

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

 if ( ! strncmp(lettre,char win='e',1))

l'erreur est : expected primary-expression before "char"

je pensais m'être documente, entre C++ pour les nuls et le cours de c++ sur developpez.com, j'ai bien regarde la fonction, mais le probléme c'est que j'ai aussi eut des erreurs qui me disais que je pouvais pas utiliser strncmp en tant que fonction...

[Erty2100]

voila j'ai réussi à regler mon probléme, votre aide a été très apprécié, merci à tous!

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
 char devises;
     cout << "Veuillez specifier si vous voulez convertir des euro ou des dollars, tapez e pour euro, d pour dollar : ";
     cin >> devises;
 
 
   if (devises=='e' || devises=='d')
   {   if (devises=='e')
       {  cout << "Veuillez entrer le montant en euros : ";
         double euro;
         cin >> euro;
         cout << "Votre montant en dollars : ";
         double repe = euro*coursED;
         cout << repe <<endl;
       }
        else
       {   cout << "Veuillez entrer le montant en dollars : ";
         double dollar;
         cin >> dollar;
         cout << "Votre montant en euros : ";
         double repd = dollar*(1/coursED);
         cout << repd <<endl;        
        }
   }
   else
   {   cout<< "Vous n'avez pas tape e ou d! ";
   }

merci!