Discussion :

[leninelenine]

Bonjour

Pour m'exercer en c++, j'ai essayé d'écrire un petit programme pour dessiner des rectangles.

La fonction «main» sert à me demander si je veux dessiner un rectangle.
La fonction «demandeLongHaut» sert à me demander la longueur, et la hauteur du rectangle.
La fonction «dessin» sert à dessiner le rectangle.
La fonction «continuer» sert à me demander si je veux dessiner un nouveau rectangle.

Le programme est buggé mais je ne sais pas d’où ça vient.

Voici le 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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
#include <iostream>
 
using namespace std;
 
void dessin(int l,int h)
{
    for(int ligne(0);ligne<h;ligne++)
        {
        for (int d(0);d<l;d++)
        {
        cout <<"*";
        }
    cout << endl;
    continuer();
}
 
void demandeLongHaut()
{
      cout<<"largeur?"<<endl;
      int larg;
      cin>>larg;
      cout<<"hauteur?"<<endl;
      int haut;
      cin>>haut;
      dessin(larg,haut);
}
 
void continuer()
{
cout<< "voulez vous redessiner un rectangle? oui = o non = n";
          string r;
          cin >>r;
          switch(r)
          {
              case o:
                demandeLongHaut();
                break;
              case n:
                return;
                break;
              default:
                cout <<"cette option n'est pas reconue";
                continuer();
                break;
          }
}
 
int main()
{
cout<< "voulez vous dessiner un rectangle? oui = o non = n"<<endl;
          char r;
          cin >>r;
          switch(r)
          {
              case o:
                demandeLongHaut();
                break;
              case n:
                return;
                break;
              default:
                cout <<"cette option n'est pas reconue"<<endl;
                main();
                break;
          }
}

Voici le raport de gcc:

Code :

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

1
2
3
4
5
6
7
||=== Build: Debug in exercice (compiler: GNU GCC Compiler) ===|
C:\Users\moi\Documents\c++\exercice\main.cpp||In function 'void dessin(int, int)':|
C:\Users\moi\Documents\c++\exercice\main.cpp|14|error: 'continuer' was not declared in this scope|
C:\Users\moi\Documents\c++\exercice\main.cpp|18|error: a function-definition is not allowed here before '{' token|
C:\Users\moi\Documents\c++\exercice\main.cpp|29|error: a function-definition is not allowed here before '{' token|
C:\Users\moi\Documents\c++\exercice\main.cpp|66|error: expected '}' at end of input|
||=== Build failed: 4 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

Pouvez vous m'aider s'il vous plaît?

Merci

[jo_link_noir]

Revois l'indentation car il manque une accolade fermante dans la boucle.

Concernant l'erreur sur la fonction continuer non déclarée, il faut déclarer la fonction avant de l'utiliser. S'il n'y a pas de déclaration de fonction avant son utilisation, le compilateur ne peut pas la connaître et émet une erreur.

Si tu ne veux qu'un caractère ('o' ou 'n'), alors n'utilise pas std::string mais un char dans la demande de continuation.

C'est case 'o': ... et pas case o: ....

int pour des longueurs n'est pas un type approprié, comme une longueur ne pas être négative, il vaut mieux utiliser un non signé (unsigned).

dessin fait beaucoup plus que dessiner un rectangle, elle appelle continuer alors que ce n'est pas son rôle. Idem pour demanderLongHaut.

Tu remarques aussi que main et continuer font la même chose à une chaîne de caractère près. Chaîne qui peut être passée en paramètre de fonction.

continuer pour être fait sous forme d'une boucle.

On ne doit jamais appeler la fonction main directement.

[Astraya]

Salut,

Il vaut mieux ajouter une forward déclaration. Tu ne peux pas mettre la fonction void continuer() avant les deux autres tout simplement parce que tu as une dépendance cyclique : dessin() -> continuer() -> demanderHautLong() -> dessin() -> contin..... etc
Pour briser cette dépendance on utilise la forward déclaration.

Ajoute juste void continuer(); avant ta fonction void dessin() ceci permet de dire au compilateur que tu as une fonction void continuer() qui existe. Le compilateur acquiescera et compilera.
Par contre si tu n'ajoute pas ta fonction void continuer() après, avec son corps{...}, de mémoire tu aura une erreur au moment de l'édition de lien ( fait le test de compiler en commentant la fonction void continuer() mais en laissant la forward déclaration pour bien comprendre de quoi je parle)

[jo_link_noir]

Une forward déclaration concerne les classes, pas les fonctions. Les fonctions ont des prototypes (déclaration) et des implémentations (définition).

[ternel]

C'est un choix de vocabulaire.
Il s'agit bien de déclarer sans définir. Ici, une fonction.
Et de même, pour une variable, on utiliserait la syntaxe extern int i;.

[Médinoc]

Attention au passage, cette dépendance cyclique est mauvaise: Tu emploies de la récursivité là où une simple boucle serait plus appropriée. C'est censé être un programme d'exercice, pas une pièce de l'International Obfuscated C Code Contest.