Discussion :

[ElTchoupi]

Bonjour,

Je travaille sur un projet qui doit pouvoir loader des assembly (Workers) à la demande dans un repertoire.
La solution du serveur contient un projet "WorkerInterface" qui contient seulement la déclaration de mon Interface IWorker.
J'ai aussi un projet A dont la classe Travailleur implemente IWorker.

Mon serveur parcours un repertoire à la recherche d'assemblies, récupère leurs types et vérifie qu'ils implémentent l'interface voulue.
Si c'est le cas, on l'instancie pour le faire travailler.

Première blague (pour laquelle je cherche une explication):
Après recherche sur internet, vérifier qu'une assembly implemente une interface, c'est assez simple:

Code :

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

1
2
3
4
5
6
7
8
9
 
Assembly assembly = Assembly.LoadFrom(assemblyPath);
 
foreach (Type loadedType in assembly.GetTypes())
{
	Type interf = loadedType.GetInterface(typeof(IWorker).FullName);
	if (interf != null)
		// On en a trouvé un !
}

Et oui: 'IsAssignableFrom', 'as', 'is', '==' et 'GetGenericTypeDefinition': toutes ces solutions me renvoient un false quand je vérifie si ça match...


Seconde blague (pour laquelle je cherche une alternative):

Quand j'en ai trouvé un, je veux l'instancier:

Code :

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

1
2
3
4
5
 
object newWorker = Activator.CreateInstance(loadedType);
IWorker worker = newWorker as IWorker;
if (worker != null)
	mWorkers.Add(worker);

J'avais commencé par la version courte avec les cast direct, mais ça marche pas (pas plus que cette version).
Mais je ne comprends pas pourquoi !
Quand je passe dans ce code en pas à pas, l'object retourné par le CreateInstance présente bien mes méthodes de IWorker,
mais il refuse de le caster à la ligne du dessous.

je suis pourtant pas fou:

Code :

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

1
2
 
public class Travailleur : IWorker

Je me doute que j'ai zappé une étape, mais je vois pas où...

Toute aide ou remarque (même délirante) est la bienvenue !

Merci d'avance

[tomlev]

vérifier qu'une assembly implemente une interface

Euh... un assembly implémente rien du tout, hein, c'est le type qui implémente une interface... faute d'inattention je suppose

Première blague (pour laquelle je cherche une explication):
Après recherche sur internet, vérifier qu'une assembly implemente une interface, c'est assez simple:

Code :

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

1
2
3
4
5
6
7
8
9
 
Assembly assembly = Assembly.LoadFrom(assemblyPath);
 
foreach (Type loadedType in assembly.GetTypes())
{
	Type interf = loadedType.GetInterface(typeof(IWorker).FullName);
	if (interf != null)
		// On en a trouvé un !
}

Et oui: 'IsAssignableFrom', 'as', 'is', '==' et 'GetGenericTypeDefinition': toutes ces solutions me renvoient un false quand je vérifie si ça match...

- IsAssignableFrom : normalement ça doit marcher, mais tu l'as peut-être utilisé à l'envers, ce n'est pas super intuitif... il faut tester : typeof(IWorker).IsAssignableFrom(loadedType), et non l'inverse.
- as : je vois pas trop ce que ça vient faire là...
- is : normal que ça ne marche pas... tu as fait quoi ? loadedType is IWorker ? loadedType est de type Type, et Type n'implémente certainement pas IWorker...
- == : normal. typeof(IWorker) est le type de l'interface, et loadedType le type d'une classe réelle. Aucune raison que ce soit égal...
- GetGenericTypeDefinition : rien à voir, c'est pour les types génériques, et ton interface n'est pas générique.

La solution la plus propre, à mon sens, est celle avec IsAssignableFrom.

Seconde blague (pour laquelle je cherche une alternative):

Quand j'en ai trouvé un, je veux l'instancier:

Code :

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

1
2
3
4
5
 
object newWorker = Activator.CreateInstance(loadedType);
IWorker worker = newWorker as IWorker;
if (worker != null)
	mWorkers.Add(worker);

J'avais commencé par la version courte avec les cast direct, mais ça marche pas (pas plus que cette version).

"ça marche pas", ça veut dire que worker == null ?

Tu peux montrer un extrait de code un peu plus complet ?

[cybermaxs]

Salut,

Pour la blague n° 2 :
Tu évoques le fait que l'interface et les workitems sont dans des assemblies différentes. es tu sûr que tu charges bien ces deux assembly avant d'utiliser Activator ? Tu peux essayer d'utiliser Activator.CreateInstanceFrom(string,string) qui semble répondre à tes besoins.

Pur la blague 1 :
Pourquoi utiliser le .FullName et pas juste typeof(IWorker) ?

[tomlev]

Pour la blague n° 2 :
Tu évoques le fait que l'interface et les workitems sont dans des assemblies différentes. es tu sûr que tu charges bien ces deux assembly avant d'utiliser Activator ? Tu peux essayer d'utiliser Activator.CreateInstanceFrom(string,string) qui semble répondre à tes besoins.

S'il a déjà un objet qui représente le type, l'assembly est forcément chargé...

Pur la blague 1 :
Pourquoi utiliser le .FullName et pas juste typeof(IWorker) ?

Parce que GetInterface prend en paramètre un String, et pas un Type. Oui, c'est bête mais c'est comme ça... Je suppose que c'est pour pouvoir gérer les interfaces non managées. Pour éviter de passer par le nom, on peut utiliser IsAssignableFrom

[cybermaxs]

Citation:
Envoyé par cybermaxs
Pour la blague n° 2 :
Tu évoques le fait que l'interface et les workitems sont dans des assemblies différentes. es tu sûr que tu charges bien ces deux assembly avant d'utiliser Activator ? Tu peux essayer d'utiliser Activator.CreateInstanceFrom(string,string) qui semble répondre à tes besoins.

S'il a déjà un objet qui représente le type, l'assembly est forcément chargé

c'est vrai après reflection, que si les types étatient inconnus il aurait eu une exception.

Citation:
Envoyé par cybermaxs
Pur la blague 1 :
Pourquoi utiliser le .FullName et pas juste typeof(IWorker) ?

Parce que GetInterface prend en paramètre un String, et pas un Type. Oui, c'est bête mais c'est comme ça... Je suppose que c'est pour pouvoir gérer les interfaces non managées. Pour éviter de passer par le nom, on peut utiliser IsAssignableFrom

Après lecture de la msdn, c'est vrai. J'ai tout de même pu remarquer que les exemples de la msdn, n'utilisent pas le nom complet (FullName) mais juste le nom de l'interface (http://msdn.microsoft.com/en-us/library/tcctb9t8.aspx)

[tomlev]

J'ai tout de même pu remarquer que les exemples de la msdn, n'utilisent pas le nom complet (FullName) mais juste le nom de l'interface (http://msdn.microsoft.com/en-us/library/tcctb9t8.aspx)

C'est vrai, normalement on ne précise pas le namespace... donc il faudrait utiliser Name plutôt que FullName