vérifier Pythagore en C++, des erreurs dans mon code

Version imprimable

je vous explique mon problème: je veux créer un programme reprenant le théorème de Pythagore et permettant, par conséquent, de prouver qu'un triangle est rectangle et, pour cela; j'ai fais le code suivant:

Code:

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
27
28
29
30
31
32
33
 
#include <cstdlib>
#include <iostream>
 
using namespace std;
 
bool VerifTriRect (double hypo,double cote1,double cote2) 
{
    double hypocarre=hypo*hypo;
    double cotecarre=(cote1*cote1)+(cote2*cote2);
    if ( hypocarre==cotecarre);
 {
    return true;
 }
    "else";
 {
    return false;
}
int main(bool condition=VerifTriRect)
 
    bool VerifTriRect (double,double,double);
 
    if (condition==true)
 {
    cout << "le triangle est rectangle: le carré de l'hypoténuse est bien égal au carré de la somme des deux autres longueurs";
 }
    "else";
 {
    cout << "le triangle n'est pas rectangle veuillez recommencer l'opération avec d'autres mesures";
 }
 
    return EXIT_SUCCESS;
}

mais dans ce code il y a quelques erreurs e je ne sait pas comment les résoudre
j'espère que quelqu'un pourra m'aider !

14/01/2007, 20h50
fullben

simple indication: j'utilise Dev-C++ comme compilateur
14/01/2007, 21h06
atnnn

Il y a beaucoup d'erreurs dans ce code, ca ne ressemble que legerement a du c++. Essaye de commencer par des programmes plus simples et d'apprendre la bonne syntaxe.

Une fonction comme VerifTriRect doit etre appelle avec des arguments, le compilateur ne peut pas les deviner. La fonction main peut etre declare avec aucun arguments ou deux arguments de type int et char**. La clause else s'ecrit simplement else et non "else";. Il ne doit pas y avoir de point-virgule apres le if(). Les blocks de plus d'une ligne doivent etre a l'interieur de { et }.

Il faut aussi faire attention quand on compare des valeures a virgule flottante, car celles-ci sont souvent approximes en base 2 et ne sont pas egales.

Il y a de la bonne documentation sur la FAQ et dans les livres et tutoriels:

http://c.developpez.com/cours/#cours-cpp
14/01/2007, 23h09
thewho
Citation:

Envoyé par atnnn

Il y a beaucoup d'erreurs dans ce code, ca ne ressemble que legerement a du c++. Essaye de commencer par des programmes plus simples et d'apprendre la bonne syntaxe.

Une fonction comme VerifTriRect doit etre appelle avec des arguments, le compilateur ne peut pas les deviner. La fonction main peut etre declare avec aucun arguments ou deux arguments de type int et char**. La clause else s'ecrit simplement else et non "else";. Il ne doit pas y avoir de point-virgule apres le if(). Les blocks de plus d'une ligne doivent etre a l'interieur de { et }.

Il faut aussi faire attention quand on compare des valeures a virgule flottante, car celles-ci sont souvent approximes en base 2 et ne sont pas egales.

Il y a de la bonne documentation sur la FAQ et dans les livres et tutoriels:

http://c.developpez.com/cours/#cours-cpp

+1

Où as-tu lu qu'on déclare des prototypes dans une fonctions, comme tu le fais ici :
Code:

1 2 3 int main(bool condition=VerifTriRect) bool VerifTriRect (double,double,double);
et j'ajouterais : prends soin de l'indentation.

Elle doit être régulière et cohérente.

Entre autres, chaque bloc doit être ndenté par rapport au bloc qui le contient,
et tout ce qui est dans le même bloc doit avoir la même indentation.

Exemple, ton code
Code:

1 2 3 4 5 6 7 8 double cotecarre=(cote1*cote1)+(cote2*cote2); if ( hypocarre==cotecarre); { return true; } "else"; { ...
devrait plutôt ressembler à ceci:
Code:

1 2 3 4 5 6 7 8 double cotecarre=(cote1*cote1)+(cote2*cote2); if ( hypocarre==cotecarre); { return true; } "else"; { ...
Et pour enfoncer le clou, où as-tu pu prendre l'idée d'écrire ?

Code:

"else";

Bonjour ! Ca ne ressemble pas du tout a du c++, ni meme a aucun langaga mais voila l'amelioration de ton programme :

Code:

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
27
28
29
30
31
32
33
34
35
36
37
 
 
#include <cstdlib>
#include <iostream>
#include <casser>
 
using namespace std;
 
bool VerifTriRect (double hypo, double cote1, double cote2) 
{
 /*Tu peux meme ajouter un test pour savoir si les mesures sont correct 
C'est a dire si l'hypothenuse est plus grand que les autres cotes */   
assert(hypo < cote1 || hypo < cote2);
double hypocarre = (hypo*hypo);
double cotecarre= (cote1*cote1) + (cote2*cote2);
   if ( hypocarre == cotecarre)
   {
    return true;
   }
    else
   {
    return false;
   }
}
int main(bool condition)
{
condition = VerifTriRect(12, 4, 5); /*Mesures au hasard */
   if (condition == true) /*Si la condition est vraie, le progamme affiche  le message suivant */
   {
   cout <<"Le triangle est rectangle" << endl;
   }
   else /*Sinon, ca affiche le message suivant */
   {
   cout <<"Le triangle n'est pas rectangle !" << endl;
   }
return 0;
}

21/01/2007, 18h19
fullben

merci warubi pour le code je vais voir ce que ça donne
21/01/2007, 18h38
warubi

N'oublis pas de mettre :resolu: si la reponse te conviens
22/01/2007, 20h17
fullben

oui je sais mais je vais attendre encore un peu avant de mettre Résolu pour voir si d'autre personnes me répondent
24/01/2007, 07h59
warubi

Pourquoi ? Ce morceau de code ne te conviens pas ? :koi:

:koi:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include <iostream>
#include <casser>
 
using namespace std;
 
bool VerifTriRect (double hypo, double cote1, double cote2) 
{
  assert(hypo < cote1 || hypo < cote2);
  return hypo*hypo == cote1*cote1+cote2*cote2;
}
int main()
{
   if (VerifTriRect(12, 4, 5))
      cout <<"Le triangle est rectangle" << endl;
   else
      cout <<"Le triangle n'est pas rectangle !" << endl;
 
   return EXIT_SUCCESS;
}

ou meme

Code:

1
2
3
4
5
6
7
8
9
10
11
#include <iostream>
#include <casser>
 
int main()
{
   double hypo = 12;
   double cote1 = 4;
   double cote2 = 5
   std::cout << ((hypo*hypo == cote1*cote1+cote2*cote2) ? "Le triangle est rectangle" : "Le triangle n'est pas rectangle !") << std::endl;
   return EXIT_SUCCESS;
}

ou un mix des deux :aie:

24/01/2007, 20h49
Matthieu Brucher

Sauf que ça ne marchera pas, sans doute, avec des float ou des double, on ne teste jamais l'égalité.
25/01/2007, 20h47
fullben

ok merci tout le monde je vais mettre Résolu
25/01/2007, 20h50
fullben

Citation:

Envoyé par warubi

Pourquoi ? Ce morceau de code ne te conviens pas ? :koi:

ce n'est pas qu'il ne "me conviens pas" c'est que j'attends de voir les autres réponses! mais merci quand même!
25/01/2007, 20h56
fullben
Désolé warubi mais ton code comporte une erreur à la ligne
Code:

1 2 3 4 { else }
en tout cas comme j'ai vu, dans d'autres discussions, que tu utilises Dev-C++ et que je l'utilise également, et comme tu disais que sa marchait j'ai essayé mais sa bloque...
comment faire ?? Je técoute 8O
25/01/2007, 21h14
Matthieu Brucher

Tu regardes parfois les warnings et ce que tu nous donnes comme code ?
26/01/2007, 02h07
Aszarsha

Citation:

Envoyé par Miles

Tu regardes parfois les warnings

Et les errors ? :aie:
26/01/2007, 07h50
warubi

Excuse moi fullben mais il n'y a aucune erreur (J'ai recompile ce matin !).
Et meme si il y a une erreur, ca ressemble un peu plus a du c++ que toi.
26/01/2007, 15h38
fullben

peut-être mais avec le même compilateur que toi et sans toucher au code sa ne fonctionne pas: il y a un problème!
26/01/2007, 15h45
Matthieu Brucher

En tout cas, le code que tu dis recopier ne correspond pas au code que tu as cité, donc corrige tes erreurs.
26/01/2007, 17h05
warubi

Merci Miles ;) car depuis le debut, je m'entete a lui dire que mon code ne contient pas d'erreurs ni de warnings. Je crois fullben que ton compilateur a un leger souci. Dis-nous les messages d'erreurs qui apparaissent lors de la compilation de mon programme pour que l'on regarde ce qui ne va pas.
27/01/2007, 21h40
fullben

mon compilateur me met qu'à la ligne 19 je dois inspecter la première expression avant "else"

et sur le code la ligne de "else" est en rouge avec une croix rouge devant si vous voulez tous savoir

mais je ne sais pas du tout pourquoi sa bloque mais je n'ai pas du tout changer le code
28/01/2007, 07h44
warubi

Excuse moi mais je crois que ton compilateur a un gros probleme car en effet
j'utilise Dev c++ mais lorsque je compile le code ci dessus donne, tout marche tres bien ( 0 warnings et 0 erreurs). Je crois que le code que tu compile n'est pas l'original.
28/01/2007, 21h00
Matthieu Brucher

Non, c'est parce qu'il a mal recopié le code, mais il ne veut pas l'avouer, c'est tout.