Problème d'une énumération

Bonjour à tous,
je copie ici ce que j'ai posté dans une mauvaise section du forum (j'espère ne pas m'être trompé cette fois):


Alors voici, je voudrais créer un jeu de carte en C++ et pour ce faire, j'ai utilisé l'énumération pour créer mes cartes:

Code:

---

enum TValeur{un=1,deux=2,...,roi=13};

---

(de même pour les quatre couleurs que j'ai appelé "TCouleur")

Cela dis, étant débutant en C++ (comme en programmation en général) je ne connais pas très bien l'énumération.

mon problème est que je dois faire une boucle for dans ma classe paquetdecarte comme suit:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

Code :   PaquetDeCarte::PaquetDeCarte() { int i=0;   for(enum TValeur vl=un; vl<=roi; vl++) { for(enum TCouleur cl=coeur; cl<=pique; cl++) { T[i]=new Carte(vl, cl); // T étant un pointeur sur Carte. i++; } } }

---

Mais voila, le compilateur m'affiche une erreur concernant les vl++ et cl++
voici l'erreur:

Citation:

---

no `operator++(int)' declared for postfix `++', trying prefix operator instead
no match for 'operator++' in '++((Carte*)this)->Carte::Val'

---

Voila donc je souhaiterais un coup de pouce de votre part (sachant que je compile sous devC++ 4.9.9.2 BETA)

Si vous avez besoin de tout mon projet, je peux vous le passer.

A ma connaissance, le mot clef enum ne peut être utilisé que pour définir ton enum (je crois qu'il n'est même pas possible de faire une déclaration anticipée).

Tu vas avoir des problemes si tu essaye de faire ++ sur une enum.

La solution est de convertir ton enum en type de base comme int ou size_t

Code:

---

1 2 3

for( int vl=un; vl<=trois; ++vl) { }

---

Ok j'essaierai ça dès ce soir,
la conversion d'une enum vers un int ce fait bien automatiquement sans ligne de programme?

Je n'avais pas compris que tu n'avais pas vu dans mon message qu'on pouvait utiliser un entier pour désigner un enum. Oui, la conversion est automatique.

Autre solution, surcharger l'opérateur ++ mais tu n'en as pas nécessairement besoin mais je l'ai déjà expliqué ici http://www.developpez.net/forums/sho...d.php?t=456045

Oui, j'avais essayé pour la surcharge de l'operateur++ mais celle ci me posait des erreurs de compilation aussi et étant donné que tu m'avais dis qu'il n'y avait normalement pas de problème tant qu'il n'y avait pas de trou dans mon enum.

Je vais essayer le code maintenant...

Citation:

---

Envoyé par Somato

tu m'avais dis qu'il n'y avait normalement pas de problème tant qu'il n'y avait pas de trou dans mon enum

---

Non, ce n'est pas ce que j'avais voulu dire mais j'avoue que c'était ambigu. Si tu veux pouvoir utiliser un entier pour désigner un enum, il faut absolument qu'il n'y ait pas de trous.

La surcharge de l'opérateur ++ telle que je te l'ai donné ne devrait pas poser de problèmes de compilation mais dans ton cas n'est pas forcément nécessaire.

Ok, j'ai essayé la solution à l'instant, sa a l'air de marcher pour la conversion des enum en int,
mais la suite de mon programme coince:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

PaquetDeCarte::PaquetDeCarte() { enum TValeur vl; enum TCouleur cl; int i=0;   for(int vl=un; vl<=roi; vl++) { for(int cl=coeur; cl<=pique; cl++) { T[i]=new Carte(vl, cl); //c'est ici que sa coince i++; } } }

---

je récolte le type de message d'erreur suivant:

Citation:

---

invalid conversion from `int' to `TValeur'
initializing argument 1 of `Carte::Carte(TValeur, TCouleur)'

---

La conversion d'enum vers int se passe donc apparement bien mais la suite un peu moins.

Oui je ne connais pas de solution très propre pour résoudre ce problème... En général les enums en C++ sont à utiliser avec parcimonie (c'est ma politique en tout cas).

un static_cast devrait résoudre ton problème mais ce n'est pas très beau ni pratique...

Code:

---

1 2 3 4 5 6

enum TValeur { un = 1, deux = 2 };   int main() { TValeur test = static_cast<TValeur>(1); }

---

Merci pour ta proposition mais finalement on m'a indiqué le chemin à suivre, et il semble que se soit propre!

Mais cette fois, j'ai un autre (petit) problème.
Je vous mets le code nouveau:

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

PaquetDeCarte::PaquetDeCarte() {   int i=0;   for(TValeur vl=un; vl<=roi; vl= (TValeur) ((int) vl + 1)) { for(TCouleur cl=coeur; cl<=pique; cl= (TCouleur) ((int) cl + 1)) { T[i]=new Carte(vl, cl); i++; } } }     PaquetDeCarte::~PaquetDeCarte() {   int i=0;   for(TValeur vl=un; vl<=roi; vl= (TValeur) ((int) vl + 1)) { for(TCouleur cl=coeur; cl<=pique; cl= (TCouleur) ((int) cl + 1)) { delete[i]; i++; } } }

---

Mon nouveau problème est mon delete de tableau dans le destructeur.
je voudrais donc détruire tout les pointeurs vers Carte, et je sais que la syntaxe et :

Code:

---

delete[]

---

j'ai pourtant essayé

Code:

---

delete T[i];

---

et le compilateur ne m'affiche plus d'erreur, mais comment vérifier qu'il fait bien le travail demandé? surtout étant donné qu'on ne nous à pas enseigné sa en cours (ou j'ai mal compris)

que dois-je donc mettre?

A savoir aussi que dans mon main je fais :

Code:

---

1 2	PaquetDeCarte P1(); cout << P1;

---

et que ceci ne m'affiche que "1"
Alors que j'arrive très bien à afficher une carte seule de cette manière:

Code:

---

1 2	Carte C1(un, trefle); cout << C1;

---

je ne pense pas que ceci est due à un problème de pointeur (auquel cas j'aurais une liste d'adresses hexadecimales en affichage)

La syntaxe du delete[] est simplement : delete[] NomDuTableau;

Code:

---

1 2	delete [] T;

---

mais c'est pour si tu as utilisé new [] (c'est a dire crée un tableau de valeurs dynamiques).

Comme tu as alloué un tableau statique, mais chaque contenu du tableau avec un pointeur donné par un simple new à chaque fois, il faut faire delete pour chaque pointeur que tu as crée en faisant une boucle (comme à la création).

Citation:

---

Envoyé par Somato

j'ai pourtant essayé

Code:

---

delete T[i];

---

et le compilateur ne m'affiche plus d'erreur, mais comment vérifier qu'il fait bien le travail demandé? surtout étant donné qu'on ne nous à pas enseigné sa en cours (ou j'ai mal compris)

que dois-je donc mettre?

---

c'est donc il me semble la bonne a chose à faire, pour autant que ce soit dans une boucle sur tous les éléments que tu as crée avec new.

PS : dans ce que t uas posté de complet,

Code:

---

delete[i];

---

n'as pas de sens, tu n'indiques pas le tableau, seulement l'indice.

D'accord je comprends mieux avec ces histoires de tableau dynamique et statique, mais je dois pourtant avoir un problème dans mon constructeur car j'ai un paquet vide en faisant ce code dans mon main.cpp:

Code:

---

1 2 3

PaquetDeCarte P1(); cout << P1;

---

sachant que mon opérateur << doit être correctement surchargé étant donné que j'arrive à afficher

Citation:

---

Deux de cœur

---

lorsque je crée une carte seule.

Qu'en pensez vous d'après mon code? moi je n'arrive pas à identifier ce qui cloche, tout en sachant que lorsque je compile le code dans mon main, j'ai un warning sur:

Code:

---

PaquetDeCarte p1();

---

me disant en gros que ceci ne peut prendre comme valeur que:

Citation:

---

true

---

(d'où l'affichage d'un "1" à l'exécution).

Code:

---

PaquetDeCarte P1();

---

Ceci est une déclaration de fonction (nommée P1, ne prenant aucun paramètre et renvoyant un PaquetDeCarte).

Je pense que tu veux plutôt ceci :

Code:

---

PaquetDeCarte P1;

---

J'ai déjà essayé ta solution, elle bourre mon main.cpp d'erreurs

je veux créer un paquet de 52 pointeurs vers 52 différentes cartes dans le constructeur par defaut de ma classe paquetdecarte. (d'où les boucle for et les new, par ailleur dans les membres privés de mon paquetdecarte.h j'ai ceci:

Code:

---

1 2 3 4 5

const int maxi=52;   class PaquetDeCarte{ private: Carte* T[maxi]

---

en déclenchant le code:

Code:

---

PaquetDeCarte P1();

---

cela devrait créer P1 en passant par le constructeur PaquetDeCarte() de mon .cpp que j'ai présenté plus haut,
je me trompe?

J'essaierai ce soit de faire:

Code:

---

cout << T[1];

---

Oui tu te trompes (désolé :p).

Comme t'a expliqué Laurent, ce code :

Code:

---

PaquetDeCarte P1();

---

est une déclaration anticipée d'une fonction exactement comme :

Code:

---

int fonction();

---

Si tu veux un objet PaquetDeCarte, il faut enlever les parenthéses.
Le fait que tu ais des erreurs indique que ta classe a un soucis dans sa définition.

Peut-être pourrais-tu faire un copier coller des erreurs ?

Donc il faut que je fasse?:

Code:

---

PaquetDeCarte P1 = PaquetDeCarte (); //???

---

ou alors:

Code:

---

PaquetDeCarte P1;

---

utilise t'il le constructeur par défaut??

Dans mon main.cpp j'ai aussi ceci:

Code:

---

1 2	Carte C1 (Deux, pique); Cout << C1;

---

et sa marche très bien... je ne vois pas la différence, veuillez m'en excuser ^^"

(pour les erreurs de compilation, je ne pourrais vous les poster que ce soir)

Les deux marchent.

Code:

---

PaquetDeCarte P1;

---

Cette solution est la mieux car seul le constructeur par defaut est appellé.

Code:

---

PaquetDeCarte P1 = PaquetDeCarte (); //???

---

Dans cette solution, si ton compilateur n'est pas très malin il peut créer un objet avec le consteur par défaut et appeller le constructeur par copie.

Code:

---

Carte C1 (Deux, pique);

---

Citation:

---

je ne vois pas la différence

---

Et bien dans ce code tu passe des arguments, autrement dit des valeurs concretes, il ne peut donc en aucun cas s'agir d'une définition de fonction :

Code:

---

int fonction( 2, 3 );

---

Celà ne compile pas non plus, j'espère que tu saisis bien la subtilité.

En effet je pense avoir maintenant compris,

et d'ailleurs c'est pas ce code qui pose des erreurs, c'est mon affichage,
et en fait il n'y a qu'une seule erreur, le reste sont des "note" et je n'avais encore jamais vu ça...(même si je suis débutant en programmation)
pour plus de clarté je vous poste un imprim ecran (j'espere que se sera suffisant car les copier collé sous devC++ sont long...)

Voici l'image en question:
http://img221.imageshack.us/img221/4...ilationks3.jpg
(j'espere que ce n'est pas trop lourd)

Quel est le type de P1 ?

P1 est un PaquetDeCarte, c'est donc un tableau de 52 pointeurs vers Carte.
C'est ce que tu voulais savoir?

Et l'erreur dit que tu n'as pas défini de surcharge de l'opérateur << pour ostream et PaquetDeCarte.

Le prototype d'une telle fonction doit être du genre:

Code:

---

1 2 3 4 5 6 7

std::ostream & operator<< (std::ostream &o, PaquetDeCarte const &p) { //Faire ce qu'il faut pour l'affichage du contenu de p //...   return o; }

---

J'ai justement définit mon opérateur << pour qu'il m'affiche une carte dans le genre "deux de coeur".
quand je fais le code:

cout << P1;

je veux juste qu'il m'affiche les 52 cartes.

si je dois quand même surcharger l'opérateur <<, est-ce que je peux le faire dans ma classe PaquetDeCarte tout en sachant que je l'ai déjà surchargé dans ma classe Carte ou est-ce que cela va poser problème?

La surcharge se fait à l'extérieur de toute classe, pour le <<.
Mais il n'y a aucun problème pour faire une surcharge par type, tu sais...

qu'entends tu par "surcharge par type" est ce que je pourrais avoir un exemple, même pas concret?

C'est pourtant simple:

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

//Deux types:   class UneClasse { //... };   class UneAutreClasse { //... };   //Deux surcharges std::ostream & operator<< (std::ostream &o, UneClasse const &uc) { //...   return o; }   std::ostream & operator<< (std::ostream &o, UneAutreClasse const &uac) { //...   return o; }

---

Deux types, deux surcharges de l'opérateur <<. Une par type.

Ok merci, j'avais un doute mais c'est bon j'ai compris!
Je vais m'atteler à ça

Bon ça va j'ai bien cerné le problème maintenant,
Mais j'ai une autre question (désolé de vous ennuyer comme ça...)

J'ai fait une méthode pour echanger des valeurs de pointeur (pour mélanger mes cartes)
voila ce que ça donne:

Code:

---

void melanger ((Carte*)& P1, (Carte*)& P2);

---

Code:

---

1 2 3 4 5 6 7

void PaquetDeCarte::melanger ((Carte*)& P1, (Carte*)& P2) { Carte* ptr; ptr=P2; P2=P1; P1=ptr; }

---

Dois-je surcharger l'operateur = pour que cela fonctionne?

Non pas besoin, par contre, tu dois peut-etre avoir un probleme pour compiler car les parentheses qui entourent les (Carte*) sont interprétées comme des cast C-style. Donc vire les.

Et je te conseillerai d'utiliser des références (plus pratiques à utiliser à mon avis) .