Discussion :

[Inarius]

Bonjour à tous,

Alors depuis hier, j'ai codé une classe qui me permet de creer un icone avec menu dans la zone de notification. Les menus sont stockés dans un fichier XML et ils sont générés dynamiquement par une des méthodes de la classe.

Cependant, j'aimerais ajouter des EventHandler sur ces menus, comme par exemple

Code :

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

menu.Click += new EventHandler(OuvrirMenuForm);

Mais mon problême est le suivant : La méthode 'OuvrirMenuForm' est stocké dans le fichier XML sous forme de string donc. Mais comment transformer ce string en méthode ??

J'ai déjà regarder les 'delegate' mais je m'y perd un peu (beaucoup en fait...). J'ai aussi vu la méthode Control.Invoke, mais qui prend un delegate en paramètre ...

Voici le code qui pose problême :

Code :

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

menu.Add(child.Name, new TrayMenu(child.Attributes["text"].Value, new EventHandler(...)));

Donc ma question que mettre à la place des ... ?

Merci beaucoup d'avance pour votre aide.
Inarius

[meziantou]

Si j'ai bien compris dans ton fichier xml tu as quelque chose comme ca

Code :

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

<MenuItem Text = "toto" Action="void OuvrirMenuForm() {MessageBox.Show(\"blabla\");}" />

Je pense qu'il faut faire de la compilation dynamique. Pour cela il suffit de créer une methode pour ton évènement. Dans cette méthode tu compiles et exécutes le code qui est dans ta chaine de caractères.

[Inarius]

Dans mon fichier XML, je voyais plutôt quelque chose comme ça :

Code :

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

 <idMenu text="toto" action="UneClasse.UneMethode" />

Cette méthode serait en fait une méthode qui récupererait un Evennement. Ca reviendrait à faire :

Code :

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

Menu.Click += new EventHandler(UneClasse.UneMethode)

En fait dans le fichier XML, il devrait y avoir la référence d'une fonction déjà existante et compilé. Mais c'est vrai que comme le nom de la méthode n'est pas connu à la compilation, ça ressemble à de la compilation dynamique.
Je vais voir ça et je vous dit si ça fonctionne

Inarius

[meziantou]

Dans le cas que tu donnes, je pense qu'il vaut mieux utiliser la reflection pour retouver la méthode.

[Inarius]

Dsl si je parait un peu inculte mais je suis encore en école donc moi pas comprendre : réflection ? késako?

[meziantou]

Quelques articles sur la reflection
http://msdn.microsoft.com/en-us/library/ms228976.aspx
http://emerica.developpez.com/dotnet...uction/csharp/
http://stackoverflow.com/questions/1...ing-reflection

Je pense que le dernier lien t'aidera à résoudre ton problème

[Inarius]

Ok merci beaucoup pour ton aide.
Je vais potasser tout ça et je revient si jamais j'y arrive vraiment pas

Encore merci
Inarius

[Inarius]

Comme il fallait s'y attendre, j'y arrive toujours pas -.-

Voila mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
EventInfo eventInfo = this._menuItem.GetType().GetEvent( "Click" );
 
menu.Add(
	child.Name,
	new TrayMenu(
		child.Attributes["text"].Value,
		Assembly.Load( child.Attributes["assembly"].Value )
			.GetType( child.Attributes["type"].Value )
			.GetMethod( child.Attributes["method"].Value )
	)
);

Et voila la partie du constructeur qui est chargé d'associer la méthode 'method' à l'EventHandler

Code :

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

1
2
3
4
5
6
7
8
9
 
eventInfo.AddEventHandler(
	this._menuItem,
	Delegate.CreateDelegate(
		eventInfo.EventHandlerType,
		this._menuItem,
		methodOnClick
	)
);

C'est au niveau du GetType que ca plante en fait. Dans mon assembly, il y a bien une classe du nom contenue dans l'attribut 'type' avec une methode 'method'. Mais avec GetType, ca me dit

'La référence d'objet n'est pas définie à une instance d'un objet.'

Pour tester, j'ai mis la classe et la méthode en static mais ca change toujours rien.

heeelp :'(

Merci d'avance,
Inarius

[meziantou]

Pour le type essaye de préciser le type complet (Namespace + Nom)

[Inarius]

Ca marche merci

Mais (comme y a toujours un mais...) l'erreur est plus loin maintenant.
Elle est dans la méthode CreateDelegate.

Code :

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

1
2
3
4
5
6
 
Delegate edelegate = Delegate.CreateDelegate(
	eventInfo.EventHandlerType,
	this._menuItem,
	methodOnClick // MethodInfo définit juste avant
);

Erreur :

ArgumentException : Erreur lors de la liaison avec la méthode cible.

[meziantou]

Es-tu sur que le type de l'event et de la methode sont identiques ?

[Inarius]

Je vient d'avoir une autre idée

Si en fait, pour chaque menu je creer un evennement qui est lancer lors du click sur le menu. Puis je récupere cette évennement avec mon conteneur (qui contient le menu racine qui contient des menus etc..). Et enfin, je lance un evennement avec l'id du menu. Ca permettrait de n'avoir qu'un EventHandler sur la racine. Et dans les arguments de l'évennement, il y aurait l'id et le lien vers le menu.

Tu pense que c'est faisable?

Merci d'avance et désolé de changer de technique en plein milieu mais ca me parait plus facile et beaucoup plus simple

[meziantou]

En gros ce que tu veux faire c'est avoir seulement une méthode de gestion pour tous les menus. Tous les menu y sont abonnés. et pour différencier les différents menus tu utilises un id.
C'est bien ca ?

[Inarius]

En fait nan, tous les menus ont un champs event que l'on peut lancer au clique sur le menu.
C'est le conteneur de menu qui est abonnée aux événements de tous les menus.

Et un des champs de l'EventArgs personnalisé contiendra une référence sur le menu. Il sera donc possible de le changer ou de faire ce qu'on veut dessus.

[meziantou]

Et un des champs de l'EventArgs personnalisé contiendra une référence sur le menu

Je pense que tu n'as pas besoin d'un event perso contenant une référence sur ton menu. En effet le champ sender sert à ca. Il suffit de le caster dans le bon type (MenuItem dans ce cas).

[Inarius]

Alors voila j'ai persévéré et j'y suis finalement arrivé.

Voila comment ca fonctionne :

Code :

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

1
2
3
4
 
ToolStripMenuItem menu = new ToolStripMenuItem( "text", "icon" != null ? new Icon( "icon" ).ToBitmap() : null );
AddEventHandler( ref menu,  "Click", "assembly", "namespace" + "classe", "method" );
parent.DropDownItems.Add( menu );

Code :

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

1
2
3
4
5
6
7
8
9
10
 
private static void AddEventHandler ( ref ToolStripMenuItem menu, string eventType, string assembly, string type, string method ) {
	menu.GetType().GetEvent( eventType ).AddEventHandler(
		menu, 
		Delegate.CreateDelegate( 
			typeof( EventHandler ),
			Assembly.Load( assembly ).GetType( type ).GetMethod( method )
		)
	);
}

Il y a seulement une limitation, c'est que la méthode appelé doit forcement être static.

Au passage, j'ai posté le code de la classe complète ici http://www.developpez.net/forums/d90...-notification/

Encor merci meziantou pour ton aide,
Inarius