Discussion :

[narfight]

Bonjour tout le monde,

Je tentes de compiler un projet que je viens de convertir de VS2003 à VS2008. Je ne sais pas si le projet fonctionnait sur VS2003 mais je bloque sur une classe statique que voici

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
    public class GlobalHelper
    {
        public static Servers dbServers = new Servers();
        public static Groups dbGroups = new Groups();
        public static ApplicationSettings appSettings = new ApplicationSettings();
 
        public static ControlHideShow controlHideShow = new ControlHideShow();
        public static Form[] MDIChildrens;
 
 
        public static LiveInformationBox.InfoWindow infoWin = new LiveInformationBox.InfoWindow(
            System.IO.Path.Combine(Application.StartupPath, "XMLInfoFile.xml"),
            LiveInformationBox.InfoWindow.WindowPositions.BOTTOM_RIGHT
        );
 
        public static void PopulateGroupsDropDown(ComboBox cb, string selected_text)
        {
            (...)
        }
    }

Le programme léve une exception sur

Code :

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

GlobalHelper.controlHideShow.Enable = false;

L'erreur est:

Une exception a été levée par l'initialiseur de type pour 'MultiRemoteDesktopClient.GlobalHelper'.

En débug, la variable GlobalHelper.controlHideShow est à Null et le constructeur de GlobalHelper.controlHideShow n'est jamais appelé. J'ai tenté de faire un GlobalHelper.controlHideShow = new ControlHideShow(); mais j'ai la même erreur sur cette ligne.

Quelqu'un peut me dire comment raisoudre le problème ?

Merci beaucoup d'avance

[tomlev]

C'est une erreur lors de l'initialisation de la classe. Regarde l'exception interne (InnerException) pour avoir plus de détails, là on peut pas savoir ce qui ne va pas (mis à part le fait que cette classe est un condensé de mauvaises pratiques...)

[Bluedeep]

Accessoirement la classe n'est pas une classe statique.

[Bluedeep]

(mis à part le fait que cette classe est un condensé de mauvaises pratiques...)

Oui, à ce stade, ça relève presque de la caricature. Une sorte de bétisier du C# à elle toute seule

[narfight]

Merci pour vos informations.

Selon les erreurs plus détaillées, c'est une Dll de SQLite qui me manque.

Par contre, j'avoue que je ne vois pas le problème de cette class.
C'est le fait de faire un new dans une var static qui elle-même n'est pas dans une class static ?

[tomlev]

Par contre, j'avoue que je ne vois pas le problème de cette class.
C'est le fait de faire un new dans une var static qui elle-même n'est pas dans une class static ?

Non, ça ce n'est pas un problème...

C'est plutôt des problèmes conceptuels :
- C'est une classe fourre-tout qui sert à tout et n'importe quoi. Il faut séparer les responsabilités dans différentes classes
- Mettre des bouts d'interface graphique accessibles comme ça de façon globale, c'est pas très propre
- C'est quoi ce "MDIChildren" ? Tu vas le mettre à jour à la main à chaque fois qu'un enfant MDI sera ajouté ou supprimé ?
- les champs publics, c'est mal ; et dans les rares cas où ça se justifie, il faut qu'ils soient en lecture seule.

Bref, on est assez loin des bonnes pratiques... mais bon, c'est malheureusement très classique