Discussion :

[fullben]

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 :

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
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 !

[fullben]

simple indication: j'utilise Dev-C++ comme compilateur

[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

[thewho]

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 :

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

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 :

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

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 :

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

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 :

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

 "else";

[warubi]

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

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
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;
}

[fullben]

merci warubi pour le code je vais voir ce que ça donne

[warubi]

N'oublis pas de mettre si la reponse te conviens

[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

[warubi]

Pourquoi ? Ce morceau de code ne te conviens pas ?

[bigboomshakala]

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
#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 :

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

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

[Matthieu Brucher]

Sauf que ça ne marchera pas, sans doute, avec des float ou des double, on ne teste jamais l'égalité.

[fullben]

ok merci tout le monde je vais mettre Résolu

[fullben]

Pourquoi ? Ce morceau de code ne te conviens pas ?

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!

[fullben]

Désolé warubi mais ton code comporte une erreur à la ligne

Code :

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

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

[Matthieu Brucher]

Tu regardes parfois les warnings et ce que tu nous donnes comme code ?

[Aszarsha]

Tu regardes parfois les warnings

Et les errors ?

[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.

[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!

[Matthieu Brucher]

En tout cas, le code que tu dis recopier ne correspond pas au code que tu as cité, donc corrige tes erreurs.

[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.