Avertissement lors surchage de fonction

**Neilos** · 07/01/2005, 15h21

Salut à tous,

je travail (toujours) sur un petit composant tout bête qui finallement me pose pas mal de problèmes.

Je souhaiterais maintenant effectuer une action lorsque l'utilisateur enfonce un le bouton de sa souris (évènement OnMouseDown).

Donc d'après l'aide je dois surcharger une fonction MouseDown dans la partie protected de ma classe, ce qui me donne :

Code :

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

1
2
3
4
protected:
 
   virtual void __fastcall MouseDown ( System::TObject* Sender,
         TMouseButton Button, Classes::TShiftState Shift, int X, int Y );

Cependant lors de la compilation j'obtient l'avertissement suivant :

Envoyé par C++ Builder

[C++ Avertissement] TImageButton.h(44): W8024 La classe de base '_fastcall TImageButton::MouseDown(TObject *,TMouseButton,TShiftState,int,int)' est aussi une classe de base de '_fastcall TControl::MouseDown(TMouseButton,TShiftState,int,int)'

Et j'ai bien l'impression que ma fonction MouseDown n'est jamais appelée. Je pense bien que c'est en relation avec cet avertissement mais je ne sais pas quoi faire.

Merci d'avance pour votre aide !

**10_GOTO_10** · 07/01/2005, 16h50

Exact !

Je suppose que TImageButton hérite plus ou moins directement de TControl. Or TControl a déjà une fonction MouseDown qui n'est pas virtuelle. Donc dès qu'on appelle TControl::MouseDown on passe dans la fonction de base, et non dans celle que tu as surchargé.

**Neilos** · 07/01/2005, 17h39

Pourtant dans l'aide ils disent de faire comme cela (l'exemple est avec la fonction Click).

Donc comment faire pour passer par ma fonction ? Ce doit bien être possible !

**borisd** · 07/01/2005, 18h23

Es-tu sûr d'avoir besoin de redéclarer la méthode dans ta définition de classe? Est-ce que ça n'en ajoute pas une autre au lieu de la surcharger, avec potentiellement un nombre de paramètres différents qui entrainerait la confusion du compilateur ?

**10_GOTO_10** · 07/01/2005, 18h26

Oui, je pense qu'en spécifiant explicitement la classe TImageButton::MouseDown, la bonne fonction sera appelée. Sauf que le système ne le fera pas, puisque il ne connait pas la classe TImageButton. Ce qui veut dire que les MouseDown appelés lors d'un clic réel de la souris iront dans TControl::MouseDown. Mais si toi, dans ton code, tu appelles TImageButton::MouseDown, ça appelera ta fonction.

**borisd** · 07/01/2005, 18h33

je n'ai pas bien compris ta réponse, désolé.
Ce que je voulais dire, c'est que dans l'aide, on indique simplement d'implémenter la nouvelle méthode de gestion d'événements, pas d'en déclarer une autre dans la définition de la classe. Ce qu'on veut, c'est bien que la gestion par défaut (donc la méthode de la classe "mère") soit remplacée par la nôtre (une méthode la surchargeant), non?
Si c'est le cas, pourquoi le système irait appeler la méthode MouseDown d'une classe ancêtre si la méthode est redéfinie dans la classe qui reçoit en premier l'événement ?
Enfin, j'ai peut être rien compris... j'y connais rien à la création de composants

**Neilos** · 07/01/2005, 19h43

borisd -> oui il faut le déclarer dans la classe, mais c'est pour cela qu'on met le mot clef virtual.

10_GOTO_10 -> désolé je te suis pas...pétard inside

?

Merci qd même pour votre aide...

**henderson** · 07/01/2005, 20h53

Salut !

Dans ce cas, il vaut mieux dériver à partir de TCustomControl.
En effet, les événements sont déjà publiés dans un TImage, ce qui n'est pas le
cas dans une classe de base telle que TCustomControl.

Si tu as besoin d'un bool Stretch, il suffit de faire une test, dans sa méthode Paint
pour déterminer s'il faut dessiner le Picture->Bitmap avec Canvas->Draw(...) ou avec
Canvas->StretchDraw(...).

A noter que la couleur de fond d'un TCustomControl est donnée par Brush->Color.

J'ai développé pour tester un objet style TButton.. et ça fonctionne bien.

A plus !

**Neilos** · 07/01/2005, 21h14

Et bien j'ai suivi tes conseils...je suis parti d'un TCustomControl, mais j'ai toujours ces avertissements...

**henderson** · 07/01/2005, 23h00

Salut !

Je les déclare en public :

Code :

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

1
2
void __fastcall MouseDown(TMouseButton Button, TShiftState Shift,
                            int X, int Y);

Code :

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

1
2
void __fastcall MouseUp(TMouseButton Button, TShiftState Shift,
                            int X, int Y);

A plus !

**Neilos** · 08/01/2005, 00h06

Salut, je dois avoir la poisse c'est pas possible.

Je les ais déclarer en public, et j'ai changé leur entête pour avoir la même chose que toi (ce que j'avais avant venait de l'aide mais je l'avais pas pris au bon endroit).

Résultat :

[C++ Erreur] ImageButton.h(37): E2113 Conflits de fonction virtuelle '_fastcall TImageButton::MouseDown(TMouseButton,TShiftState,int,int)' avec la classe de base 'TControl'

idem pour MouseUp....

Qu'ais-je fait encore

:

Merci pour ton aide henderson !

**henderson** · 08/01/2005, 00h43

Salut !

Voici la class de mon composant pour tester tout ça :

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
class PACKAGE TCustomControl2 : public TCustomControl
{
private:
TPicture *FPictureUp;
TPicture *FPictureDown;
TMouseEvent FOnMouseDown;
TMouseEvent FOnMouseUp;
bool FStatus;
bool FStretch;
bool FAutoSize;
protected:
public:
    __fastcall TCustomControl2(TComponent* Owner);
    __fastcall ~TCustomControl2();
    void __fastcall Paint();
    void __fastcall SetPictureUp(TPicture *Value);
    void __fastcall SetPictureDown(TPicture *Value);
    void __fastcall SetStatus(bool Value);
    void __fastcall SetStretch(bool Value);
    void __fastcall SetAutoSize(bool Value);
    void __fastcall MouseDown(TMouseButton Button, TShiftState Shift,
                            int X, int Y);
    void __fastcall MouseUp(TMouseButton Button, TShiftState Shift,
                            int X, int Y);
__published:
__property bool AutoSize = {read=FAutoSize, write=SetAutoSize};
__property bool Status = {read=FStatus, write=SetStatus};
__property bool Stretch = {read=FStretch, write = SetStretch};
__property TPicture *PictureUp = {read=FPictureUp, write=SetPictureUp};
__property TPicture *PictureDown = {read=FPictureDown, write=SetPictureDown};
__property TMouseEvent OnMouseDown = {read=FOnMouseDown, write=FOnMouseDown};
__property TMouseEvent OnMouseUp = {read=FOnMouseUp, write=FOnMouseUp};
 
};

Par contre je me demande s'il ne faut pas déclarer les deux OnMouse... en protected ?

Autrement ça fonctionne très bien ! Le seul gag que j'ai, est qu'à l'édition, mes
propriétes du type Picture ne sont pas correctement initialisées : l'une indique (vide)
ce qui est normal mais par contre l'autre inque (Bitmap) comme si... alors qu'à ce moment
là elle devrait être vide. Dans le constructeur, je ne fait rien d'autre que de leur affecter
un new TPicture !

Pour info je travaille toujours avec BCB3 Pro !

A plus (A demain) !

**Neilos** · 08/01/2005, 12h02

Salut,

je n'avais pas déclarer les évènement FOnMouseDown, mais cela ne me rapporte que plus d'erreur. Déjà sous la version 6 pour la souris ce sont des TMouseEvent, ensuite à la ligne

Code :

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

1
2
 
__property OnMouseDown = { read = FOnMouseDown, write = FOnMouseDown };

J'obtient l'erreur suivante :

[C++ Erreur] ImageButton.h(38): E2347 Mauvaise correspondance de paramètre dans le spécificateur d'accès read de la propriété OnMouseDown

Et mes deux erreurs qui sont toujours là :

[C++ Erreur] ImageButton.h(29): E2113 Conflits de fonction virtuelle '_fastcall TImageButton::MouseDown(TMouseButton,TShiftState,int,int)' avec la classe de base 'TControl'

[C++ Erreur] ImageButton.h(30): E2113 Conflits de fonction virtuelle '_fastcall TImageButton::MouseUp(TMouseButton,TShiftState,int,int)' avec la classe de base 'TControl'

Merci henderson mais ça marche toujours pas

**henderson** · 08/01/2005, 13h49

Salut !

Chez moi aussi puisque j'ai bien donné :

Code :

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

1
2
3
private :
TMouseEvent FOnMouseDown;
TMouseEvent FOnMouseUp;

Code :

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

1
2
3
4
public :
//Sans doute à mettre dans protected
__property TMouseEvent OnMouseDown = {read=FOnMouseDown, write=FOnMouseDown};
__property TMouseEvent OnMouseUp = {read=FOnMouseUp, write=FOnMouseUp};

Dans le .cpp, voici ce que donne la MouseDown :

Code :

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

1
2
3
4
5
6
7
8
9
void __fastcall TCustomControl2::MouseDown(TMouseButton Button, TShiftState Shift,
                                    int X, int Y)
{
SetStatus(true);
if(FOnMouseDown != NULL)
    {
    FOnMouseDown(this,Button, Shift, X, Y);
    }
}

Tandis que la MouseUp donne ceci :

Code :

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

1
2
3
4
5
6
7
8
9
void __fastcall TCustomControl2::MouseUp(TMouseButton Button, TShiftState Shift,
                                    int X, int Y)
{
SetStatus(false);
if(FOnMouseUp != NULL)
    {
    FOnMouseUp(this, Button, Shift, X, Y);
    }
}

Pour comprendre SetStatus :

Code :

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

1
2
3
4
5
void __fastcall TCustomControl2::SetStatus(bool Value)
{
FStatus = Value;
Paint();
}

Pour comprendre Paint :

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
void __fastcall TCustomControl2::Paint()
{
 
TPicture *Picture;
if(FStatus)
    {
    Picture = FPictureDown;
    }
else
    {
    Picture = FPictureUp;
    }
Canvas->Pen->Color = Brush->Color;
Canvas->Brush->Color = Brush->Color;
Canvas->Rectangle(0,0,Width, Height);
if(Stretch)
    {
    TRect D = Rect(0, 0, Width, Height);
    Canvas->StretchDraw(D, Picture->Bitmap);
    }
else
    {
    //L'image disparait à l'exécution si je ne rajoute pas les deux tests avec le test
    //du AutoSize pour un resize automatique des dimensions du contrôle ! (???...?)
    if((AutoSize) && (Picture->Bitmap->Width != 0) && (Picture->Bitmap->Height != 0))
        {
        Width = Picture->Bitmap->Width;
        Height = Picture->Bitmap->Height;
        }
    Canvas->Draw(0,0,Picture->Bitmap);
    }
}

A plus !

**Neilos** · 08/01/2005, 14h39

Oui j'ai en gros le même code que toi dans les fonctions...mais bon les erreurs sont toujours là !

**Neilos** · 09/01/2005, 15h51

Voila j'ai trouvé la solution...merci developpez.net et surtout CGi -> ICI.

Donc eh bien il suffisait de rajouter le mot DYNAMIC devant la déclaration des fonctions. Moi je les ais mis en protected (ils disaient de le mettre ici dans l'aide), j'ai donc :

Code :

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

1
2
 DYNAMIC void __fastcall MouseDown( TMouseButton Button, Classes::TShiftState Shift, int X, int Y);
   DYNAMIC void __fastcall MouseUp( TMouseButton Button, Classes::TShiftState Shift, int X, int Y);

Et ça fonctionne

Merci beaucoup à henderson, borisd et 10_GOT0_10 pour leur aide...et CGi pour son article

Avertissement lors surchage de fonction

C++Builder

Discussions similaires

Partager

Partager