Discussion :

[liverbird]

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 :

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

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

[r0d]

Bonjour,

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.

[yan]

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???

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

[liverbird]

ok,

donc si je mes des typedef, est ce que je pourrai les utiliser comme des types?
merci

[r0d]

?? 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

[Alp]

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

[liverbird]

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

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
#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

[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 :

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

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.

[liverbird]

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

Code :

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

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 :

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

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 :

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

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

[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)

[poukill]

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

Code :

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

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 !

[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

[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...

[liverbird]

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