Discussion :

[CryoCid]

Bonjour,

J'ai une question qui me turlupine et je ne trouve pas réponse...
Afin de passer un objet créer dans une classe vers une autre classe, j'ai du passé toutes mes classes en "public" car sinon VS me disait "Accessibilité incohérente : le type de paramètre ... est moins accessible..."

Par contre on m'a toujours dit pour la sécurité du code qu'il faut essayer de mettre le moins de code en "public", dans mon cas j'ai du passer la moitié de mes classes en public...

Je vous mets mon bout de code pour plus de clarté car j'ai surement fait une erreur de débutant ;-)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
private void BUTTON_Nouveau_Click(object sender, EventArgs e)
        {
            // Ouvrir la fenêtre de création d'un nouveau concours
            Concours concours = new Concours();
            ParametreConcours creerConcours = new ParametreConcours("Création d'un concours", ref concours);
            if (creerConcours.ShowDialog() == DialogResult.OK)
            {
                // Intégrer le nouveau concours
                listConcours.Add(concours);
                // Recharger la liste des concours
                this.Charger_Grid();
            } 
        }

Je crée ici une référence de l'objet de la classe Concours pour l'envoyer à la form "ParametreConcours" pour que celle-ci me remplisse l'objet et que je puisse l'intégrer dans ma collection par la suite.

Code :

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

1
2
3
4
5
6
7
8
public ParametreConcours(string nomForm, ref Concours concours)
        {
            // Initialisation de la fenêtre
            InitializeComponent();
            InitForm(nomForm);
 
            _concours = concours;
        }

Ici c'est le constructeur de la fenêtre "ParametreConcours", je récupère l'objet "concours" pour pouvoir le modifier dans cette fenêtre.

Le soucis est donc que j'ai du mettre la classe Concours à public car sinon j'avais l'erreur cité au début...
N'hésitez pas à me dire s'il y a une meilleur solution.

Je vous remercie d'avance.

[BenoitM]

euh les classes sont très souvent en publique a de rare exception près (classes interne a un projet/classe imbriqué)

C'est surtout les attributs qui "doivent" être en private et accessible par des propriétés ainsi que certainnes fonctions qui non pas d'utilité à être disponible de l'exterieur

Exemple un peu Bidon

Ma Classe à une fonction Save(). Dedans j'appelle la méthode CreateConnection
Save à une utilité à être public
CreateConnection pas et sera donc private.

Les constructeurs peuvent être parfois en private pour par exemple implémenter un singleton (ou gérer le nombre d'objets instanciable spossible)

[Guulh]

Pour les classes, la visibilité par defaut est 'internal' (alors que pour les membres, c'est 'private')

Si tu travailles sur une appli assez simple, avec juste un projet dans ta solution par exemple, mets toutes tes classes en public. Une classe est soit public, soit internal; si elle est internal, ça veut dire qu'elle n'est pas accessible depuis une autre assembly. Si ta solution n'a qu'un projet, et donc qu'une assembly, il n'y a aucune différence entre internal et public, autant tout mettre à public.

Par contre, à l'intérieur de tes classes, tu as effectivement intérêt à mettre le moins de choses possible en public, pour limiter le couplage entre tes différents composants.

Par ailleurs, le mot clé 'ref' ne sert à rien dans ton constructeur. Tu devrais l'enlever.

[CryoCid]

Merci bien pour vos explications claires et précises ;-)

Il est vrai que je n'ai pas besoin d'envoyer une référence de l'objet, sans fonctionne aussi, je ne pensais pas retrouver les modifications effectuées dans la form "ParametreConcours", mais en fait si.

Bonne fin de journée.