Héritage et appel des constructeurs parents

Salut à tous

Je rencontre un petit souci quant à l'appel du constructeur de ma classe parent.
En fait, je voudrais un peu simplifier l'écriture, car, vous le verrez, c'est assez moche :P

En bref, j'ai une classe Button qui hérite de Rect.
Rect prend en paramètre 4 points pour former ses boundaries.
Button prend en paramètres une destination, un x et y de départ (par unités de 8 points), une chaîne et une couleur.

Code:

---

1 2 3

Button(u16 *dest, int x, int y, const char *str, unsigned short color) : Rect(x * 8 - 1, y * 8 - 1, x * 8 + strlen(str) * 8 + 2, y * 8 + 10) {

---

Affreux, pas vrai ? Et le pire, c'est que ce sont des valeurs que je dois réutiliser (notamment le strlen).

Est-il possible de le faire de manière plus propre ? Une solution à la Java serait pas mal, un peu comme ceci:

Code:

---

1 2 3 4 5 6

Button(u16 *dest, int x, int y, const char *str, unsigned short color) { int XStart = x * 8; int yStart = y * 8; int len = strlen(str);   Rect(XStart - 1, YStart - 1, XStart + len * 8 + 2, YStart+ 10);

---

Ne vous formalisez-pas pour l'emploi des char * et strlen en C++, ce n'est pas négociable avec le matériel :P

Code:

---

1 2 3 4 5 6 7 8 9

Button (u16 *dest, int x, int y, const char *str, unsigned short color) : Rect (x * 8 - 1, y * 8 - 1, x * 8 + strlen(str) * 8 + 2, y * 8 + 10) { /*... */ }

---

En mettant un peu mieux en forme :)
Pas la peine de passer par des variables intermédiaires à mon avis... Ca assombrirait le code pour rien;

Tu ne peux pas différer l'appel du constructeur de la classe mère dans la définition du constructeur de la classe fille. Sans quoi, cela voudrait dire que tu commence à exécuter du code dans la classe fille alors que la classe mère n'a pas été construite.
Donc, comme Alp te le suggère, le mieux que tu es à faire est de formater l'appel au constructeur de base :

Code:

---

1 2 3 4 5 6

Button (u16 *dest, int x, int y, const char *str, unsigned short color) : Rect ( x * 8 - 1, y * 8 - 1, x * 8 + strlen(str) * 8 + 2, y * 8 + 10)

---

Une autre solution est de passer par une classe 'helper' qui encapsulerait la transformation des paramètres :

Code:

---

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

class Button : public Rect { private : struct Button2RectCstr { static int top(int x, int y, const char *str) {return x*8-1;} // ... }; public : Button (u16 *dest, int x, int y, const char *str, unsigned short color) : Rect ( Button2RectCstr::top(x,y,str), Button2RectCstr::left(x,y,str), Button2RectCstr::bottom(x,y,str), Button2RectCstr::right(x,y,str))

---

Citation:

---

Envoyé par 3DArchi

Une autre solution est de passer par une classe 'helper' qui encapsulerait la transformation des paramètres :

---

Tant qu'à faire, autant avoir une fonction helper qui a du sens :

Code:

---

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

Rect getBoundingRect(int x, int y, char const *str, Font font, ...) {... }   class Button : public Rect { public : Button (u16 *dest, int x, int y, const char *str, unsigned short color) : Rect (getBoundingRect(x, y, str, DefaultFont,...)) // On passe là par le constructeur de copie de rect { } }

---

En outre, j'ai des gros doutes sur le fait qu'une solution d'héritage entre Rect et Button soit une bonne solution.

Merci à tous pour ces réponses.

Citation:

---

Envoyé par JolyLoic

En outre, j'ai des gros doutes sur le fait qu'une solution d'héritage entre Rect et Button soit une bonne solution.

---

C'est possible. Dans mon cas, il s'agit de trouver dans quel rectange l'utilisateur a cliqué. Je crée donc une liste de ces rectanges et fais une itération dessus pour trouver le rectangle cliqué.

C'est pour ça que je fais hériter Button de Rect. De cette manière, je peux le mettre directement dans la liste.
Mais peut-être y a-t-il une meilleure solution que je n'ai pas vue.

En passant, j'ai l'habitude de mettre le '{' sur la même ligne que les if, functions etc...

Code:

---

if(condition) {

---

Alors que vous trois l'avez mis sur la ligne suivante. C'est la convention ANSI si je ne me trompe pas, mais... vais-je me faire taper dessus un jour ou l'autre si je garde mes habitudes ?

À mon avis, un bouton n'est pas un rectangle. Un bouton possède un rectangle. En tout cas, c'est le cas dans la plupart des modèles objet représentant une interface utilisateur...

Citation:

---

Envoyé par Antoine_935

Code:

---

if(condition) {

---

Alors que vous trois l'avez mis sur la ligne suivante. C'est la convention ANSI si je ne me trompe pas, mais... vais-je me faire taper dessus un jour ou l'autre si je garde mes habitudes ?

---

Si tous tes projets suivent la même convention, alors il n'y a pas de pb. L'ennui c'est quand on retrouve de tout.

Citation:

---

Envoyé par Médinoc

À mon avis, un bouton n'est pas un rectangle. Un bouton possède un rectangle. En tout cas, c'est le cas dans la plupart des modèles objet représentant une interface utilisateur...

---

C'est plus juste, de fait. En plus, ça m'évite d'avoir ce code ignoble...

Merci pour la précision sur les conventions, 3DArchi.