utilisation du conteneur pair

Version imprimable

Bonjour,
je réalise un programme capable de jouer au jeu Agora.
le jeu se constitue d'un plateau rectangulaire.
pour jouer un coup, on déplace d'une case à l'autre puis on se met soit au dessus soit en dessous des pions existant sur la case.
j'ai une classe plateau et une classe coup pour l'instant.
Je voudrais savoir si ma structure de données pour un coup est juste.
j'utilise le conteneur pair de la stl.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 class coup { private: pair<int,int> position; //position d'un pion sur un plateau pair<position,position> positions; // couple position de départ et position d'arrivée pair<positions,bool> c;// coup est représenté par une position de départ et une position d'arrivée plus un bool dessus/dessous vector<c> listedescoups;// vecteur liste des coups //constructeurs et selecteurs coup() {} coup(int lignedepart, int coldepart, int lignearrivee, int colarrivee, bool dessus) : c(positions(position(lignedepart, coldepart), position(lignearrivee, colarrivee)), dessus) {}
En fait je voudrais savoir si c'est suffisant de déclarer mes types 'postion' et 'positions' comme je l'ai fait ou est ce qu'il faut que je créer des classes qui leur correspondent.

merci

25/01/2008, 15h39
r0d

Bonjour,

Citation:

Envoyé par zakalova

En fait je voudrais savoir si c'est suffisant de déclarer mes types 'postion' et 'positions' comme je l'ai fait ou est ce qu'il faut que je créer des classes qui leur correspondent.

Ca dépend de ce que tu entend par "c'est suffisant".
Si tu veux juste que ça compile et que ça marche, alors oui, c'est suffisant.
Si tu souhaites aller plus loin (notamment utiliser des 'position' et des 'positions' comme des types), alors non, ce n'est pas suffisant.

Dans ton code, position et positions sont des variables membres de ta classe Coup, donc uniquement utilisables en tant que tel.

Hope it helps.
25/01/2008, 16h44
yan

Citation:

Envoyé par r0d

Bonjour,

Ca dépend de ce que tu entend par "c'est suffisant".
Si tu veux juste que ça compile et que ça marche, alors oui, c'est suffisant.

?? manque pas des typedef???

Citation:

Envoyé par r0d

Si tu souhaites aller plus loin (notamment utiliser des 'position' et des 'positions' comme des types), alors non, ce n'est pas suffisant.

Dans ton code, position et positions sont des variables membres de ta classe Coup, donc uniquement utilisables en tant que tel.

Hope it helps.

+1
25/01/2008, 17h16
liverbird

ok,

donc si je mes des typedef, est ce que je pourrai les utiliser comme des types?
merci
25/01/2008, 17h19
r0d

Citation:

Envoyé par Mongaulois

?? manque pas des typedef???

Hum... tu me fais douter... mais je crois bien que oui. Mais je peux pas tester de suite, j'ai cassé gcc :aie:
26/01/2008, 10h50
Alp

Il me semble bien que ce que voudrait faire le PO c'est bien mettre des typedef devant 'positions' etc...

salut
j'ai mis des typesdef mais je ne peux pas encore tester :(
donc voilà à quoi ressemble mon fichier coup.h

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
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <vector>
#include <iostream>
#includ <utility>
#include <deque>
using namespace std;
 
class Coup {
 
private:
 
typedef pair<int,int> Position; //position d'un pion sur un plateau
 
typedef pair<Position,Position> Positions; // couple position de départ et position d'arrivée
 
pair<Positions,bool> c;// coup est représenté par une position de départ et une position d'arrivée plus un bool dessus/dessous
 
typedef vector<c> Listedescoups;//  vecteur liste des coups
 
bool dessus;
 
public:
 
//constructeurs 
Coup() {}
Coup(int lignedepart, int coldepart, int lignearrivee, int colarrivee, bool dessus) ; 
c(Positions(Position(lignedepart, coldepart), Position(lignearrivee, colarrivee)), dessus) {}
 
//selcteurs et accesseurs
int getlignedepart() const {
    return c.first.first.first;
  }
int getcoldepart() const {
    return c.first.first.second;
  }
int getlignearrivee() const {
    return c.first.second.first;
  }
int getcolarrivee() const {
    return c.first.second.second;
  }
 
friend ostream& operator<<(ostream& o, const Coup& c);
friend istream& operator>>(istream& i, Coup& c);
 
};

est ce que ça à l'air potable tout ça? merci :D

26/01/2008, 12h15
r0d

Bonjour,

je n'ai pas regardé dans le détail, mais quelques remarques:

-> Je ne sais pas si c'est un problème de copier/coller, mais ton code n'est pas indenté. C'est trés important d'indenter son code. Voir ce post

-> Tu déclare tes typedef à l'intérieur de ta classe. Cela signifie que ces types ne sont accessibles que dans la portée de cette classe. Techniquement, ça ne fais pas de grande différence, mais sémantiquement, cela signifie qu'ils sont fortement liés à cette classe. Est-ce vraiment ce que tu souhaites?

-> La ligne:

Code:

c(Positions(Position(lignedepart, coldepart), Position(lignearrivee, colarrivee)), dessus) {}

N'a aucun sens. Je ne sais pas trop ce que tu veux faire, mais là tu confond ta variable membre c, qui est une paire, et ton vecteur Listedescoups.

Bonjour,
merci pour le conseil je vais intendé le code.
pour ce qui est de la ligne

Code:

c(Positions(Position(lignedepart, coldepart), Position(lignearrivee, colarrivee)), dessus) {}

il manquait les deux points ':' juste avant la lignedans le code, en fait c'est comme ça
Code:

1 2 Coup(int lignedepart, int coldepart, int lignearrivee, int colarrivee, bool dessus) : c(Positions(Position(lignedepart, coldepart), Position(lignearrivee, colarrivee)), dessus) {}
En fait mon projet consiste à réaliser d'un programme qui joue au jeu Agora. Donc j'ai deux classes principales, une classe Plateau qui représente le plateau du jeu et une classe Coup.
je ne suis pas sur de la syntaxe mais c'est pour construire mon Coup c.
je veux dire que un coup c est une paire il est constitué d'une ligne et colonne de départ que je stock dans une Position puis d'une ligne et colonne d'arrivée que je stock aussi dans une Position. je stock les deux Position dans une paire Positions et tout ça constitue le premier element de la paire c. le deuxième élément est le booléen qui dit si le pion et au dessus ou au dessous du pion ennemi.
la manière dont j'ai définie mes accesseurs va avec mais je ne sais pas si c'est juste
Code:

1 2 3 4 5 6 7 8 9 10 11 12 int getlignedepart() const { return c.first.first.first; } int getcoldepart() const { return c.first.first.second; } int getlignearrivee() const { return c.first.second.first; } int getcolarrivee() const { return c.first.second.second; }
Merci

29/01/2008, 08h13
Invité

Est-ce que je sens une influence lispienne? :-)

Plutôt que d'utiliser une "paire" pour représenter tous les couples qui interviennent dans ton problème, tu gagnerais à définir des classes spécialisées. Tu vas forcément finir par t'emmêler les pinceaux avec des "first.second.second.first.second.first" comme ça...

(car l)
29/01/2008, 10h09
poukill

Je dirais même pire, ça sent un couplage fort. Avoir des

Code:

ObjetA.getObjetB().setTemperature(getTempFromReceptor() );

c'est pas bon du tout ! ;)
Une bonne règle c'est qu'une classe ne doit connaitre que le minimum d'une autre classe... Or là avec des first.second.first, on commence à engranger de la connaissance ! :mrgreen:
29/01/2008, 13h11
liverbird

Merci pour les conseils,
une question, est ce que le code comme j'ai fait marchequand même ou pas?
car je n'utilise les "c.first....." que pour avoir les positions de depart et d'arrivée. je ne les utilise pas beaucoup en fait donc pour moi il n'y a pas confusion entre les diffirents accesseurs
29/01/2008, 14h28
poukill

Oui ça va marcher, ce que je veux te dire ici (comme 5hdumatin), c'est qu'une paire d'une paire d'une paire, ça commence à devenir compliqué.
Tu gagnerais à écrire ta propre classe qui aurait le bon comportement...
30/01/2008, 10h37
liverbird

ok je comprends
c'est clair que des classes c'est plus simple
je tiens au courant si j'ai des soucies :p
à+