Bien structurer mes classes ? (bis)

Bien structurer mes classes ? (bis) - C++ - Programmation

Marsh Posté le 21-06-2009 à 14:20:54    

Hello,
je suis en train de faire un petit jeu en C++ en console : un Rogue-like. Mon projet à déjà bien avancé, mais je viens de me rendre compte que j'ai fais un truc pas très jolie dans mon code. Et j'aurais besoin de vous pour savoir comment le faire bien.
 
Je vous explique la situation :  
J'ai une classe Donjon qui a comme attribut un tableau à deux dimension de Case. Cela me sert de map dans mon jeu. Les Cases ont comme attribut : char skin (qui est le char affiché à l'écran), bool is_obstacle, et bool is_visible.
 
Je n'ai que 5 types de Case dans mon jeu pour le moment :
Case vide (skin = ' '; is_obstacle = false)
Case porte (skin = '+'; is_obstacle = false)
Case escalier (skin = '>'; is_obstacle = false)
Case horizontale (skin = '-'; is_obstacle = true)
Case verticale (skin = '|'; is_obstacle = true)
Avec is_visible toujours à false à l'initialisation.
 
Pour le moment j'utilise un constructeur Case(char skin, bool is_obstacle), et pour tester le type de la case : char get_skin() { return skin; }
Bref, c'est vraiment mal foutus. Si je veux changer le skin d'un type de case ça me fait plein de modification, et puis c'est pas très jolie...
Comment organiser le code pour avoir plusieurs types de Case ? Faut il utiliser un enum ? De l'héritage ? Autre chose ?
 
Merci d'avance ! :)

Reply

Marsh Posté le 21-06-2009 à 14:20:54   

Reply

Marsh Posté le 21-06-2009 à 15:00:54    

Pattern Flyweight + Pattern Factory

Message cité 1 fois
Reply

Marsh Posté le 21-06-2009 à 17:14:57    

Joel F a écrit :

Pattern Flyweight + Pattern Factory


Merci beaucoup pour ta réponse !
N'etant pas très à l'aise avec les Design Pattern, est ce que vous pouvez me confirmer que c'est bien ce genre de chose qu'il faut faire (j'ai juste utilisé Factory) :
 

Code :
  1. public Case {
  2. private:
  3.    char skin;
  4.    bool is_visible;
  5.    bool is_osbtacle;
  7. public:
  8.    Case(char skin, bool is_osbtacle) {
  9.       this->skin = skin;
  10.       this->is_obstacle = is_obstacle;
  11.       is_visible = false;
  12.    }
  14.    char get_skin() {
  15.       return (is_visible)? skin : '.';
  16.    }
  18.    bool get_is_visible() {
  19.       return is_visible;
  20.     }
  22.    bool get_is_obstacle() {
  23.       return is_obstacle;
  24.    }
  25. };
  27. public class CaseFactory {
  28. public:
  29.    Case get_case_empty() {
  30.       return new Case(' ', false);
  31.    }
  33.    Case get_case_door() {
  34.       return new Case('+', false);
  35.    }
  37.    Case get_case_stair() {
  38.       return new Case('>', false);
  39.    }
  40.  
  41.    Case get_case_horizontal() {
  42.       return new Case('-', false);
  43.    }
  44.  
  45.    Case get_case_vertical() {
  46.       return new Case('|', false);
  47.    }
  48. };
  50. main() {
  51.    CaseFactory c;
  52.    Case c1, c2;
  54.    c1 = c.get_case_door();
  55.    c2 = c.get_case_empty();
  56. }


 
?
Et si je veux tester le type de la case, comment faire ? Car c1.get_skin() == '>' c'est moche :/


Message édité par magicpotu le 21-06-2009 à 18:37:34
Reply

Marsh Posté le 21-06-2009 à 18:45:41    

surcharge == pour Case


Message édité par Joel F le 21-06-2009 à 18:46:10
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed