Discussion :

[olibara]

Bonjour

Je cherche le meilleur moyen de parcourir l'ensemble des form d'une application afin d'extraire une liste des controls et de leur texte

Je sais bien entendu parcourir tous les controls d'une form
Mais est il possible de parcourir toutes les forms de l'application ?

Merci de votre aide

[olibara]

Bonjour

Il y a bien

Code :

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

     Application.OpenForms

Mais ca ne donne que les forms instancieée

Est il possible a votre avis de parcourir Toutes les form au runtime
Ou bien dois-je envisager de faire un parseur des les sources du projet ?

Une alternative serait evidemment d'utiliser un outil existant qui fait ca, mais je n'en ai pas trouvé ?

[ctxnop]

Salut,
Si tu recherches toutes les forms d'une application, sans se soucier du fait qu'elles soient ou non instanciées et en combien d'exemplaire, c'est que tu recherche toutes les classes qui héritent de System.Windows.Forms.Form.
Donc utilise la réflexion. Tu parts de l'AppDomain et, pour chaque assembly qui t'intéresse, tu énumères les types (publiques ou non, selon ce que tu cherche), puis tu vérifie si un type hérite de Form.

[olibara]

Merci ctxnop

Yes !

Voici donc ce que ca donne
Apres je vais broder autour !!

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
      Assembly[] assemblies = AppDomain.CurrentDomain.GetAssemblies();
      Form f = null;
      foreach (Assembly A in assemblies)
      {
        Type[] types = A.GetTypes();
        foreach (Type t in types)
        {
          if (t.IsPublic && t.BaseType == typeof(Form))
          {
            if (t.FullName.Contains("frm"))
            {
              f = (Form)A.CreateInstance(t.FullName);
            }
          }
        }
      }

[Bluedeep]

Tu peux utiliser un truc du genre :

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
          static List<Type> getForms()
        {
            List<Type> list = new List<Type>();
            AppDomain.CurrentDomain.GetAssemblies()
                   .Where(a => a.GlobalAssemblyCache == false)
                   .Where(a => 
                        ((AssemblyProductAttribute)a.GetCustomAttributes(typeof(AssemblyProductAttribute), true).First())
                          .Product == "MyProduct")
                   .ToList()
                   .ForEach(a =>  a.GetTypes()
                        .Where(t => t.IsSubclassOf(typeof(Form)))
                        .ToList()
                        .ForEach( t=> list.Add(t))  
                        );
            return list;
        }

EDIT : version plus clean.

[olibara]

Merci bluedeep

on s'est croisé

[Bluedeep]

Merci bluedeep

on s'est croisé

Oui, mais comme le dit CTXNOP, je ne vois pas pourquoi tu tentes d'instancier chaque form.

Par ailleurs, les tests fondés sur un pattern spécifique du nom ne me semblent pas une bonne idée.

[ctxnop]

Faudrait voir aussi à ne pas prendre tous les assembly, ce n'est à mon avis pas ce que tu recherches.
Le fait est qu'en faisant ainsi, tu charges et parcours une très grande partie du framework .net et donc des milliers de types, ce sera très lent.

Autres détails :
- Si tu ne veux que les types publiques, utilise assembly.GetExportedTypes(), tu n'auras que les types publiques donc beaucoup moins d'itérations dans ta boucle et tu n'auras pas à tester le IsPublic.
- Mieux vaut utiliser IsSubclassOf ou IsAssignableFrom plutôt que de te baser sur BaseType, BaseType te donne le type directement hérité, donc si A hérite de Form, et B de A, B est bien une form, mais ne correspond pas à ton critère.
- Rien ne garanti que le nom de ton formulaire débute bien par "frm". Si c'est une règle de codage tu trouvera tout ceux qui respectent la règle, mais si certains ont échappé à la règle pour une raison X ou Y, ils ne seront pas trouvés. C'est d'ailleurs une bonne façon de trouver tout ceux qui ne respectent pas la règle.
- C'est assez dangereux de faire un CreateInstance de chaque formulaire, tu ne sais pas ce que le formulaire provoque à son instanciation. S'il accède à des choses statiques, qu'il initialise des choses dans une base de données, qu'il écrit sur disque, etc... Tu me dira que tout ça sont de mauvaises pratiques, mais ne veut pas dire que tous les formulaires respectent les bonnes pratiques.
- Si tu es pour faire un CreateInstance, fait un using autour pour disposer le formulaire, et utilise directement Activator.CreateInstance(Type t), c'est plus rapide est c'est de toute façon que ce fini par faire le Assembly.CreateInstance(string typename)

[Bluedeep]

Faudrait voir aussi à ne pas prendre tous les assembly, ce n'est à mon avis pas ce que tu recherches.
Le fait est qu'en faisant ainsi, tu charges et parcours une très grande partie du framework .net et donc des milliers de types, ce sera très lent.

Il peut bypasser cela en testant la propriété GlobalAssemblyCache de l'assembly (et en partant du principe que, à ce stade, ses assemblies ne sont pas dans le GAC).

[olibara]

Merci pour toutes ces tres judicieuces remarques ctxnop

Pour le createInstance je ne suis pas certain que ce soit nécessaire
mais comme l'intention in fine est de browser tous les controls de la form il me semblait que l'intanciation est nécessaire

Et pour le using tu a tout a fait raison !