Discussion :

[doditch77]

Bonjour,
J'ai crée une classe et volière :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class Voliere
{
      private:
              int nbOiseau;
              int capacite;
              Oiseau ** tab;
 
      public:
             Voliere(int c=0);
             void insere(Oiseau* adresse);
             void afficher();
 
      };

J'ai un probleme lors de l'éxécution de la méthode insere:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void Voliere :: insere(Oiseau* adresse)
{
     if(nbOiseau==capacite)
     {cout<<"Il n'y a plus de place dans la volière"<<endl;
                           }
     else
     {tab[nbOiseau]=adresse;
     nbOiseau=nbOiseau+1;
     }
 
 }

Dans ma fonction main, apres avoir crée un oiseau O, et une volière je fais :
V.insere(&O);
pour insérer cette oiseau à la volière. Ca compile bien mais au moment de l'éxecution ça se stoppe et j'ai une fenêtre qui apparait avec le message suivant : " Projet1.exe a rencontré un problème et doit fermer. Nous vous prions de nous excuser pour le désagrément encouru."

J'ai besoin d'aide. Merci d'avance



balises [code] ajoutées par r0d. Merci d'y penser dorénavant.

[ram-0000]

C'est normal que cela ne marche pas.

Ta classe Voliere contient un membre "Oiseau ** tab" qui est un tableau de pointeur sur Oiseau.

Lors de l'insertion d'un oiseau, ce tableau n'est pas réalloué (ni même probablement alloué) quant tu fais "tab[nbOiseau]=adresse"

Il manque donc une allocation. De plus, tu stockes des pointeurs sur Oiseau (cela est peut être voulu) mais cela peut se révéler dangeureux si tes oiseaux sont détruits sans que la volière ne soit au courant.

Donc pour supprimer le mal de tête:
* utilisation d'un vector (ou list) pour stocker tes oiseaux
* instanciation des constructeur de copie, destructeur et opérateur d'affectaion de tes oiseaux pour avoir une copie "safe"

Et enfin, ne pas oublier le tag [code] pour rendre le code plus lisible sur ce forum

[r0d]

Bonjour et bienvenue sur developpez.com

Quelques questions:
-> Comment est créé ton oiseau O?
-> Quel est ton environnement de developpement? Peux-tu débugguer et voir quelle ligne exactement pose problème?

ps: pense aux balises

[doditch77]

La liste des oiseaux présents dans la voliere doit être stockée sous forme
d'un tableau (alloué dynamiquement) de pointeurs contenant l'adresse des oiseaux
de la liste. Je n'ai peut etre pas bien construit ce tableau. ()

Mon oiseau est crée par le constructeur suivant:

Code :

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

1
2
3
4
5
6
7
8
 
 
Oiseau :: Oiseau(string id, double k, double vit)
{
       identifiant=id;
       nbKm=k;
       vitesse=vit; 
               }

[yan]

C'est normal que cela ne marche pas.

Ta classe Voliere contient un membre "Oiseau ** tab" qui est un tableau de pointeur sur Oiseau.

Lors de l'insertion d'un oiseau, ce tableau n'est pas réalloué (ni même probablement alloué) quant tu fais "tab[nbOiseau]=adresse"

Il manque donc une allocation. De plus, tu stockes des pointeurs sur Oiseau (cela est peut être voulu) mais cela peut se révéler dangeureux si tes oiseaux sont détruits sans que la volière ne soit au courant.

Donc pour supprimer le mal de tête:
* utilisation d'un vector (ou list) pour stocker tes oiseaux
* instanciation des constructeur de copie, destructeur et opérateur d'affectaion de tes oiseaux pour avoir une copie "safe"

Et enfin, ne pas oublier le tag [code] pour rendre le code plus lisible sur ce forum

+1
sinon

V.insere(&O);

ca me semble trés louche.... si O est une variable temporaire, tu va avoir plein d'erreur mémoire car tu va pointer sur des zone mémoire que ne seront plus utilisé