Discussion :

[Antoine_935]

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

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 :

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

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 :

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

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

[Alp]

Code :

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

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;

[3DArchi]

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 :

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

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 :

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

[JolyLoic]

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 :

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

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.

[Antoine_935]

Merci à tous pour ces réponses.

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 :

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

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 ?

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

[3DArchi]

Code :

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

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.

[Antoine_935]

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