[POO][SDL_gfx] Initialisation de variable bizarre ....

Version imprimable

Bonjour a tous !

Me voila arriver sur ce forum qui m'a l'air bien sympathique avec une question sur les initialisations de classes en C++.
J'ai en fait dans ma classe des variables de type Sint16 (pour la librairie SDL_gfx) declaré private.
Je les initialise avec une fonction memebre comme cela:
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 void Circle::circle(Sint16 xB, Sint16 yB, Uint8 rB, Uint8 gB, Uint8 bB, Uint8 alphaB, Sint16 loudB, Sint16 radB, SDL_Surface *screenB) { cout << "Initialisation .... " << endl; if(xB != NULL) xB = x; cout << "x = " << xB << endl; if(yB != NULL) yB = y; cout << "y = " << yB << endl; if(rB != NULL) rB = r; cout << "r = " << rB << endl; if(gB != NULL) gB = g; cout << "g = " << gB << endl; //etc ... pour toute les valeurs transmise en parametre ... cout << "[fait] !" << endl << endl; }
Voila, cec fait quand je cré mon objet puis que je fait appelle a cette fonction, jobtien des resultats tels que -9216 pour x ou encore ù pour g !

Voila, mais la je ne vois vraiment pas pourquoi j'obtiens des valeurs aussi bizzarre ....

merci d"avance

06/04/2005, 13h31
Selenite

On initialise de préférence avec le constructeur.
Je te conseillerais de plus de créer des types spécifiques pour les points et les couleurs.

Ensuite, l'expression a = b; affecte b à a, et non l'inverse.Donc remplace les xB = x par des x = xB.

NULL n'a plus de raison d'être en C++, et (même en C) ne doit pas être utilisé avec autre chose que des pointeurs. Surtout que si l'utilisatuer veut que r vaille zéro, avec ton code il ne peut pas. Tu peux supprimer les if ( xxx != NULL ). Apparament, tu voulais tester si le paramètre est défini. Un paramètre est TOUJOURS défini.
06/04/2005, 13h47
no login

:oops: :oops: :oops: :oops:

BOURDE monumental.

Merci beauoup !

(surtout pour l'histoire des NULL, je ne savais pas du tout, au moins cela m'aura appris quelque chose :roll: Merci encore)

Merci.
06/04/2005, 13h50
no login

Hmm je m'explique quand meme pourquoi je n'est pas utilisé de constructeur: car cette fonction est aussi utilisé pour redefinir les variables ....

Et merci encore.
06/04/2005, 13h58
Selenite

Citation:

Envoyé par no login

Hmm je m'explique quand meme pourquoi je n'est pas utilisé de constructeur: car cette fonction est aussi utilisé pour redefinir les variables ....

Plait-il ?

Une mauvaise conception n'a pas d'excuse. :twisted:
06/04/2005, 14h00
no login

Ops double post. (je crois que je vais aller dormir ..... :oops: :oops: :oops:
Voir dans 2 messages.
06/04/2005, 14h00
Selenite

Citation:

Envoyé par no login

Hmm je m'explique quand meme pourquoi je n'est pas utilisé de constructeur: car cette fonction est aussi utilisé pour redefinir les variables ....

Plait-il ?

Une mauvaise conception n'a pas d'excuse. :twisted:
06/04/2005, 14h00
no login

Mais est-il raisonnable de faire une autre methode pour modifier ces données ?! :?:
06/04/2005, 14h03
Selenite

Poste toute ta classe pour voir.

Tout ce que je peux te dire, c'est que l'apparition d'une fonction init ou équivalente, ce n'est pas terrible (Que se passe-t-il si cette fonction n'est pas appelée ? Si elle est appelée plusieurs fois ? Bref, à réserver au C). Les champs de couleurs, je les mettrait en public. Les champs de position, il faut voir.

Voici ma classe:

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
 
class Circle //classe gerant les ronds (attention contient le(s) animations)
{
private: // ... clarifions .....
  Sint16 x;Sint16 y;                    //position X et Y
  Uint8 r;Uint8 g;Uint8 b;Uint8 alpha; //RVBA
  Sint16 loud;Sint16 rad;              //loud = épaissseur; rad = rayon
  SDL_Surface *ScreenC;                 //Surface sur laquel on doit copier
 
  bool SpecialColorB; //couleur melangé ou pas ?
 
  void hide(void); //RAZ de l'écran
 
public:
  int RVB;
 
  void SetColor(int r, int v, int b); //regler une couleur uniforme pour le cercle
  int GetColor(void);//avoir la couleur si SpecialColor = false
  void SpecialColor(int type); //mettre un fond "space" => type = type de fond.
 
  void anim1(void);  //dessiner un cecrcle => animation
  void draw(void); //dessiner un cercle => pas d'animation ...
 
 
  void circle(Sint16 xB, Sint16 yB, Uint8 rB, Uint8 gB, Uint8 bB, Uint8 alphaB, Sint16 loudB, Sint16 radB, SDL_Surface *screenB); //pseudo-constructeur pouvant etre appelé comme une methode
  ~Circle(); //destructeur
};

Voila donc ma classe.

[quote = "Selenite"]
Les champs de couleurs, je les mettrait en public
[/quote]

Question tout de meme sombre dans mon esprit:Le but de la poo n'est-il pas de mettre le moins de chose possible en public ?
:?:

06/04/2005, 15h01
Selenite

Citation:

Envoyé par no login

Citation:

Envoyé par Selenite

Les champs de couleurs, je les mettrait en public

Question tout de meme sombre dans mon esprit:Le but de la poo n'est-il pas de mettre le moins de chose possible en public ?
:?:

Non.
Se contenter de tout encapsuler dans des classes, c'est pas de la POO, c'est juste du C amélioré. Tout comme écrire des fonction get et set à tout va n'est pas une fin en soit.
La POO c'est l'abstraction des données: héritage, notion d'interface, polymorphisme, etc. (Si tu en connais pas, Google est ton ami.)

Mettre un membre en public, ça veut simplement dire que ce membre fait partie de l'interface. C'est-à-dire qu'il fait intrinsèquement partie de l'objet, et non pas qu'il est juste présent pour les besoins du programmeur. Quand un membre devrait être devrait être publique (ex : le pointeur vers la surface cible) mais qu'un controle doit être exercé dessus (pour éviter un pointeur invalide par ex.), on écrit une paire de get/set.

J'aimerais t'en dire plus, mais je ne suis pas très bon pédagogue... Lis donc ceci: Le Langage C++, de Bjarne Stroustrup.
07/04/2005, 10h20
no login

Ok.

Merci beaucoup pour le bouquin :D . (je crois en effet que j'en ai besoin !)

Merci encore . ( 8) )