[Debutant] Programme avec les fonctions

Salut a tous!

Je suis nouveau et j'ai pris la décision de m'inscrire sur le forum car je le trouve tres bien, de tout les points de vue. De même pour le site, tres bien fait et complet. :)

Donc voila je suis assez débutant en C++ et j'ai en meme temps pas mal de difficultés auquel je pense faire de mon mieux pour y remédier.

J'en viens maintenant au sujet, je dois faire un programme qui porte sur les fonctions et qui simule un tirage au sort à l'aide d'une pièce de monnaie: Pile ou Face (seule contrainte: la valeur Pile correspond soit à Vrai soit à 1 et la valeur Face correspond soit à Faux soit à 0).

Voici mon code:
PileOuFace.h

Code:

---

1 2 3 4

//fonction qui simule un tirage au sort à l'aide d'une pièce de monnaie   //retourne donc deux valeurs: "PILE" ou "FACE" int PileOuFace();

---

Main.cpp

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

#include <iostream> using namespace std;   #include "PileOuFace.h"   void main() { if(PileOuFace()==true); cout<<"Pile"<<endl; else cout<<"Face"<<endl; }

---

PileOuFace.cpp

Code:

---

1 2 3 4 5 6 7 8 9

#include <iomanip>   bool PileOuFace() { bool Pile; Pile=rand()%2;   return(Pile); }

---

Voila j'ai surement et j'en suis sur des erreurs qui sautent aux yeux mais comme je vous l'ai déjà dis je suis nul donc j'ai beau chercher mais je ne trouve rien:(

Je vous remerci pour votre aide!

Citation:

---

Envoyé par Tilk*sg1

Voila j'ai surement et j'en suis sur des erreurs qui sautent aux yeux mais comme je vous l'ai déjà dis je suis nul donc j'ai beau chercher mais je ne trouve rien:(

---

Tu as essayé? Si oui, qu'est-ce qui ne va pas. Si non, pourquoi ne pas le faire? En regardant simplement je n'ai rien vu de très grave.

Voici néanmoins quelques commentaires...

Citation:

---

Code:

---

1 2 3 4

#include <iostream> using namespace std;   #include "PileOuFace.h"

---

---

Ne jamais utiliser using namespace std; dans un en-tête, ni avant d'avoir inclu tous les en-têtes. Sur des programmes important, c'est même mieux de ne pas l'utiliser du tout.

Citation:

---

Code:

---

void main()

---

---

main retourne int.

Citation:

---

Code:

---

1 2	{ if(PileOuFace()==true);

---

---

Pourquoi pas (PileOuFace() == true == true == true == true) ? PileOuFace retourne un bool, inutile de le comparer à true.

Citation:

---

Code:

---

1 2 3 4 5

cout<<"Pile"<<endl; else cout<<"Face"<<endl; }

---

---

Essaye de surveiller ton indentation.

Citation:

---

Code:

---

1 2 3 4 5 6 7 8 9

#include <iomanip>   bool PileOuFace() { bool Pile; Pile=rand()%2;   return(Pile); }

---

---

Au dessus, tu fais une comparaison explicite à true qui n'a guère de sens, ici, tu convertis implicitement un int en un bool. C'est ici que j'aurais mis une comparaison.

Autre chose, quand je lis que PileOuFace retourne un bool, je ne sais pas a priori ce que signifie true. Soit tu changes le nom de la fonction de manière à ce que la signification soit claire (estPile() par exemple), soit tu retournes un enum { face, pile };

Il y a une incohérence entre la déclaration et la définition de ta fonction PileOuFace(). Dans sa déclaration (dans ton .h), tu lui fait retourner un int, et dans sa définition (dans le cpp), tu lui fais retourner un bool...

Citation:

---

Envoyé par Jean-Marc.Bourguet

Citation:

---

Tu as essayé? Si oui, qu'est-ce qui ne va pas. Si non, pourquoi ne pas le faire? En regardant simplement je n'ai rien vu de très grave.

---

Tout d'abord je te remerci pour ta réponse tres complete ! Et pour te répondre oui j'ai essayé de compiler et j'obtient 2 warning dont 1 erreur:
"-warning C4805: '==' : mélange risqué de type 'int' et de type 'bool' dans l'opération
-warning C4390: ';' : une instruction contrôlée a été trouvée vide ; est-ce ce que vous souhaitiez ?
-error C2181: instruction else sans if correspondant non conforme"
Pour les warning, je ne les comprends pas tres bien mais l'erreur encore moins.

Voici néanmoins quelques commentaires...

Citation:

---

Ne jamais utiliser using namespace std; dans un en-tête, ni avant d'avoir inclu tous les en-têtes. Sur des programmes important, c'est même mieux de ne pas l'utiliser du tout.

---

Pourquoi ne pas l'utiliser ? Cela évite a chaque fois de remarquer "std::" avant les cin/cout c'est quand meme pour fait pour gagner du temps non ?

Citation:

---

main retourne int.

---

La, j'avoue je n'arrive pas a comprendre. On m'a dit que void ou int dans ce cas la c'était pareil!

Citation:

---

Pourquoi pas (PileOuFace() == true == true == true == true) ? PileOuFace retourne un bool, inutile de le comparer à true.

---

Ok, mais comment le comparer alors ? :?

Citation:

---

Essaye de surveiller ton indentation.

---

Oui j'ai rajouté des espaces quand je previsualisait le message en pensant que tout était aligné, désolé je ferai attention la prochaine fois ;)

Citation:

---

Au dessus, tu fais une comparaison explicite à true qui n'a guère de sens, ici, tu convertis implicitement un int en un bool. C'est ici que j'aurais mis une comparaison.

---

Code:

---

int PileOuFace()==true

---

comme ceci ??

Citation:

---

Autre chose, quand je lis que PileOuFace retourne un bool, je ne sais pas a priori ce que signifie true. Soit tu changes le nom de la fonction de manière à ce que la signification soit claire (estPile() par exemple), soit tu retournes un enum { face, pile };

---

Ahh oui je vois, c'est vrai :)

---

Citation:

---

Envoyé par Eusebe

Il y a une incohérence entre la déclaration et la définition de ta fonction PileOuFace(). Dans sa déclaration (dans ton .h), tu lui fait retourner un int, et dans sa définition (dans le cpp), tu lui fais retourner un bool...

---

Exact, même si Jean-Marc.Bourguet me l'a fait remarqué, je te remerci:mrgreen:

Le premier warning est dû à la comparaison d'un bool et d'un int dans l'expression "PileOuFace()==true" (puisque tu as déclaré PileOuFace() comme retournant un int).

Si PileOuFace() retourne finalement un bool, il suffit de faire

Code:

---

if(PileOuFace())

---

Si PileOuFace() retourne un int, il faut faire

Code:

---

if(PileOuFace()==1)

---

Le deuxième Warning est dû au point-virgule mis après le if... Par là, tu indiques au compilateur que le if se termine, sans rien faire même si l'expression qu'il teste est true.

Enfin, l'erreur est liée à ce point virgule en trop. Puisque le compilateur "sort" du if, quand il arrive au else, il ne sait plus à quoi le rattacher...