Discussion :

[rvzip64]

Hello tout le monde,

Voici une petite question pour savoir qu'elle méthode vous utiliseriez.
Donc j ai une appli avec un Tmenu , et un sous menu "Enregister" qui doit enegistrer avec un format spécifique des traitements effectuées.

Sauf que dans mon application le traitement peut etre différent selon la frame que j affiche dans ma form. (j'espere que tout le monde arrive a me suivre)

Donc chaque frame a sa procédure pour générer le fichier à enregistrer, et il faut que j appele cette méthode a chaque fois que je clique sur enregistrer.

Comment feriez vous ?

[Flode]

Je que j'ai fait dans le même cas :
- Créer une frame "Modele" contenant :
- des méthodes virtuelles
- des méthodes réelles (pour les traitement commun)
- Créer les frames dérivées du modèle contenant
- les méthodes réelles (pour les virtuelles)
- le complément des méthodes (avec inhérited)

Voilou

[macumba]

Première idée à chaud :

Je commencerais par lier le menu à une action Enregistrer.

Ensuite sur chacune des fiches je déclarerais une action spécifique d'enregistrement.
Je lierais cette action à l'action Enregistrer dans l'évènement OnActivate de chacune des formes.
Reste à voir comment désactiver l'action quand il n'y pas de fiches actives(donc FMDi). Mais avec quelques essais ca doit se trouver assez facilement.

Si tu démarres un nouveau projet, je commencerais par coder une fiche qui sera l'ancêtre pour toutes les fiches de ton application dans laquelle tu désactiverais l'action dans le Onactivate. Ainsi pour toutes les fiches qui n'ont pas d'actions Energistrer le menu enregistrer serait automatqiuement grisé.

Enfin voilà grosso modo comment je procéderais.

En tout cas l'action me semble nécessaire, elle te permettra aisément si par la suite tu ne veux plus utiliser un menu, mais un bouton par exemple de refaire un lien il n'y aura que la propriété action du bouton à lier pour que cela marche.

[rvzip64]

Flode ton idée pas mal vu que le structure de mon appli est comme tu le dis avec une frame model.

Et aprés j ai plus qu'a faire du polymorphisme lors de la création de mes frames comme ça le code a l'intérieur de mon button "enregistrer" reste identique.

J'aurais du y penser.

Excellente idée.

Je suis quand meme preneur d 'autre idées on ne sait jamais un petit brainstorming
peut faire ressortir des idées intéressantes.

[Clorish]

Deja, moi je separe l'evenement du traitement.
C'est a dire que mes events en general appellent une methode qui s'occupe d'effectuer le traitement.
Cela permet de pouvoir faire appel au traitmeent sans passer par les evenements et de maniere plus propre que d'appeler directement le gestionnaire d'evenement .. qui peut etre amené a changer lors d'u nremaniement du design. Je pense au passage d'un menu a une toolbar ou pour justement gerer en parallele le menu et la toolbar.

Ensuite, pourquoi ne pas externaliserle traitement en faisant appel a une procedure externe (a la maniere des evenements).
Si vide, alors on appelle la fonction par defaut definie dans la form, sinon celle specifié .... chaques frames s'occupant d'affecter sa propre fonction a la form quand elle est activée.

Apres on peut aussi jouer sur les interfaces ....

[Fabrice ROUXEL 1]

J'utiliserais peut être un patron de méthode.

L'idée étant de définir la trame de l'algorithme au niveau de la classe de base abstraite, puis compléter cette trame dans les classes dérivées.

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
TEnregistrementAbstrait = class
private
protected
  procedure Traitement1(); virtual; abstract;
  procedure Traitement2(); virtual; abstract;
public
  procedure Enregistrer();    // <--- le patron de méthode
end;
 
procedure TEnregistrementAbstrait.Enregistrer();
begin
   Traitement1();
   Traitement2(); 
end;
 
---------
TEnregistrementFrame1 = class(TEnregistrementAbstrait)
protected
  procedure Traitement1(); override;
  procedure Traitement2(); override;
end;
 
TEnregistrementFrame2 = class(TEnregistrementAbstrait)
protected
  procedure Traitement1(); override;
  procedure Traitement2(); override;
end;

Puis dans le gestionnaire d'évenement du sous-menu, faire un appel à Enregistrer() via une variable de type TEnregistrementAbstrait en tirant profit de l'appel polymorphique des procedures de traitement.



ps: Cette conception est très voisine de celle proposée par Flode.

[sjrd]

Si par contre il n'y a pas de frame modèle, et que l'application est déjà à un stade avancé où il est difficile de tout remettre sur base d'une frame modèle, on peut remplacer les méthodes virtuelles par des méthodes messages.

D'abord il faut déclarer le message que l'on va utiliser. Puisque les frames sont descendantes de TWinControl, il faut éviter tout message en-dessous de WM_USER. Donc :

Code :

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

1
2
const
  MsgSave = WM_USER + 1;

Ensuite, on définit le record contenant les informations d'enregistrement. On pourrait supposer que l'information contient le nom du fichier où enregistrer :

Code :

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

1
2
3
4
5
type
  TMsgSave = record
    MsgID : Word; // obligatoire, vaudra MsgSave
    FileName : TFileName;
  end;

Il faut encore intercepter le message dans chaque frame, avec une méthode déclarée comme suit :

Code :

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

procedure Save(var Msg : TMsgSave); message MsgSave;

Et on l'implémente comme il faut.

Finalement, pour demander à la frame active de s'enregistrer :

Code :

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

1
2
3
4
5
6
var Msg : TMsgSave;
 
if not SaveDialog.Execute then exit;
Msg.MsgID := MsgSave;
Msg.FileName := SaveDialog.FileName;
ActiveFrame.Dispatch(Msg);

Et valà.

(combien on parie que Clorish va me dire que je cherche midi à 14:03 ? )

[Clorish]

(combien on parie que Clorish va me dire que je cherche midi à 14:03 ? )

lol pas tant que ca
En fait si quand meme un peu ....
tu peux pas directement implementer des gestionnaires d'evenements dans les frames et les affecter a l'evenement OnClick du menuItem lorsqu'elle est active ?

[sjrd]

lol pas tant que ca
En fait si quand meme un peu ....

J'l'avais dit

tu peux pas directement implementer des gestionnaires d'evenements dans les frames et les affecter a l'evenement OnClick du menuItem lorsqu'elle est active ?

Si, mais ce n'est pas propre : les frames ne devraient pas être au courant de l'existence de ce menu, qui est situé à un niveau hiérarchique plus élevé (la fiche contenante).

Déjà tu vas être presque obligé de faire des références circulaires dans les implementation, sauf si tu passes une fonction de callback pour l'affectation du OnClick à la création des frames, ou alors de passer directement l'élément de menu. Mais comme je le disais plus haut, niveau conception c'est horrible.

En plus, il est fort possible que la fiche principale aie un code commun à effectuer pour chaque frame (comme l'affichage du SaveDialog par exemple). Ca forcerait encore chaque frame à appeler une méthode avec ce code commun, ce qui ferait encore des références "moches".

Bref je préfère les message

[Clorish]

Bref je préfère les message

C'est pour ca que j'ai dit pas tant que ca
C'est clair que c'est plus propre
J'takin' ....

[Fabrice ROUXEL 1]

Si, mais ce n'est pas propre : les frames ne devraient pas être au courant de l'existence de ce menu, qui est situé à un niveau hiérarchique plus élevé (la fiche contenante).

Tout à fait juste.
Et il est préférable de dissocier la vue(Menu, frame) du modèle.(TEnregistrementAbstrait).

[rvzip64]

Merci a tous pour vos idées et vos réponses ...
Je vais étudie tout ça .