Tableau d'objets

Anonyme

29 mars 2008 à 16:38:26

Bonjour,

je souhaiterais créer un tableau d'objets. J'ai donc une classe Carte (qui contient un int valeur, et un string couleur), et une classe Paquet (qui contient un tableau de 52 cartes Carte cartes[52] ).
Mais je n'arrive pas a créer le constructeur de Paquet:

Voici le code de Paquet.ccp :

#include "Paquet.h"
#include "Carte.h"
Paquet::Paquet()
{
for (int i = 0; i < 13; ++i)
{
cartes[i] = new Carte(i+1,"coeur");
}
}
Paquet::~Paquet()
{
}

Lorsque je lance la compilation, il n'y a pas de problème, mais j'ai une erreur du genre 0x002314653 lorsque je lance le jeu.

Le main ne contient rien de special :

#include <iostream>
#include "Carte.h"
#include "Paquet.h"
using namespace std;
int main(void)
{
Paquet paquet;
cout << "test" << endl;
return 1;
}

Je pense que ça vient d'un problème d'allocation mémoire, mais je n'arrive pas à le résoudre.
Merci de votre aide.

Xaviou

29 mars 2008 à 16:55:34

Salut.
Si tu veux construire tes objets "Carte" avec new, il faut que le tableau soit un tableau de pointeurs, et non un tableau d'objets.

class Paquet
{
...
Carte *cartes[52];
...
};

Et dans ce cas, il faut bien penser à détruire les objets créés :

Paquet::~Paquet()
{
delete[] cartes;
}

@+
Xav'

Portail français wxWidgets : wxDev.fr | Quelques progs + libs wxWidgets compilées : mywxstuff

lmghs

29 mars 2008 à 17:00:05

Ce delete[] est seulement valable si cartes avait été un "Cartes*" alloué avec new[].

Anonyme

29 mars 2008 à 17:33:48

Merci, ca a l'air de marcher pour le constructeur.

Mais maintenant j'ai un problème pour afficher le paquet.
J'ai la fonction getPaquet() :

void Paquet::getPaquet()
{
this->cartes[1].getCarte; /*le 1 c'est juste pour tester, je vais faire une boucle*/
}

mais il me dit : request for member `getCarte' in `((Paquet*)this)->Paquet::cartes[1]', which is of non-class type `Carte*'

La fonction getCarte() renvoie void et se contente d'afficher la valeur et la couleur de la carte.
Je comprend que carte[1] n'est pas une carte mais un pointeur sur une carte, mais alors comment faire pour utiliser la fonction getCarte ?

chlablak

29 mars 2008 à 17:36:37

getCarte c'est une fonction et pas un attribut

C++ Références | Idiomes | FAQ

Nanoc

29 mars 2008 à 17:43:29

Pas besoin du this. La bonne solution est:

cartes[1]->getCarte();

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

Anonyme

29 mars 2008 à 19:09:56

Je ne comprend pas pourquoi, mais ca ne fonctionne pas.

J'ai modifier la fonction getCarte() de facon a ce qu'elle renvoie un string;
et mon paquet.cpp est :

Paquet::Paquet()
{
Carte * cartes = new Carte[52];
for (int i = 0; i < 13; i++)
{
cartes[i] = Carte(i+1,"coeur");
cout << cartes[i].getValeur() <<" de ";
cout << cartes[i].getCouleur()<< endl;
}
}
Paquet::~Paquet()
{
delete [] cartes;
}
void Paquet::getPaquet()
{
cout << cartes[1]->getCarte() << endl;
}

et j'a encore une erreur de type 0x0025346 a l'execution. Ce qui me parait bizarre, c'est que dans le constructeur j'utilise cartes[i].getValeur() et non cartes[i]->getValeur(), et ca m'affiche bien les cartes.

Nanoc

29 mars 2008 à 19:18:21

Tu es sûr que l'erreur a pas lieu avant ?

Montre nous voir le prototype de la classe.

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

chlablak

29 mars 2008 à 20:34:13

je trouve etonnant que le compilateur n'a rien dit:

Paquet::Paquet()
{
Carte * cartes = new Carte[52];
// ...
}
Paquet::~Paquet()
{
delete [] cartes; // ici tu desalloue une variable que tu as declaré dans le constructeur
}
void Paquet::getPaquet()
{
cout << cartes[1]->getCarte() << endl; // ici tu utilises une variable que tu as declaré dans le constructeur
}

C++ Références | Idiomes | FAQ

Ice_Keese

30 mars 2008 à 1:11:59

À ce que je vois, cartes est une variable locale au constructeur.

Je peux voir l'interface de la classe Paquet?

Anonyme

30 mars 2008 à 12:15:18

Voila Paquet.h :

#ifndef PAQUET_H_
#define PAQUET_H_
#include "Carte.h"
class Paquet
{
protected:
Carte *cartes[52];
public:
Paquet(void);
~Paquet(void);
void getPaquet();
};
#endif /*PAQUET_H_*/

Et Paquet.cpp :

#include "Paquet.h"
#include "Carte.h"
Paquet::Paquet()
{
Carte * cartes = new Carte[52];
for (int i = 0; i < 13; i++)
{
cartes[i] = Carte(i+1,"coeur");
cout << cartes[i].getValeur() <<" de ";
cout << cartes[i].getCouleur()<< endl;
}
}
Paquet::~Paquet()
{
delete [] cartes;
}
void Paquet::getPaquet()
{
cout << cartes[1]->getCarte() << endl;
}

Nanoc

30 mars 2008 à 12:22:28

Paquet::Paqut()
{
cartes = new Carte[52]; //PAs de Carte*
//...
}

Tu créais une variable locale au constructeur plutot que d'utiliser l'attribut de la classe.

Une meilleure solution serait d'utiliser un std::vector<> plutot que un tableau dynamique "à la C".

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

Anonyme

30 mars 2008 à 12:38:11

C'est ce que j'avais fait au debut, mais ca ne fonctionne pas.

Paquet::Paquet()
{
cartes = new Carte[52];//incompatible types in assignment of `Carte*' to `Carte[52]'
for (int i = 0; i < 13; i++)
{
cartes[i] = Carte(i+1,"coeur");
}
}

class Paquet
{
protected:
Carte cartes[52]; //encore plus d'erreurs si je met Carte *cartes[52]

Anonyme

30 mars 2008 à 12:41:40

Cartes *cartes; revois p-ê le cours.

Anonyme

30 mars 2008 à 12:52:13

Je viens de regarder les vecteurs, et ca fonctionne parfaitement.

Paquet::Paquet()
{
for (int i = 0; i < 13; i++)
{
cartes.push_back (Carte(i+1,"coeur"));
}
for (int i = 13; i < 26; i++)
{
cartes.push_back (Carte((i)%13+1,"pique"));
}
for (int i = 26; i < 39; i++)
{
cartes.push_back (Carte((i)%13+1,"carreau"));
}
for (int i = 39; i < 52; i++)
{
cartes.push_back (Carte((i)%13+1,"trèfle"));
}
}
Paquet::~Paquet()
{
}
void Paquet::getPaquet()
{
for (int i = 0; i < 52; ++i) {
cout << cartes[i].getCarte();
}
}

Merci de m'avoir suggéré cette solution, je ne connaissais pas.
Merci a tout le monde de votre aide.

Tableau d'objets

× Après avoir cliqué sur "Répondre" vous serez invité à vous connecter pour que votre message soit publié.

× Attention, ce sujet est très ancien. Le déterrer n'est pas forcément approprié. Nous te conseillons de créer un nouveau sujet pour poser ta question.

Tableau d'objets

Tableau d'objets

Supprimer

Modérer

Signaler le message aux modérateurs