Discussion :

[sleeg]

Bonjour,

J'ai une classe personne et une classe adresse.
Lorsque j'essaie d'affecter une valeur (personne.adresse.adresse1 = "mon adresse" à mon champ adresse1 de la classe Adresse depuis la classe Personne. J'ai une erreur : 'La référence d'objet n'est pas définie à une instance d'un objet.'
Comment faire ? Merci d'avance

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
 
public class Adresse
    {
        public string adresse1 { get; set; }
        public string adresse2 { get; set; }
        public string cp { get; set; }
        public string ville { get; set; }
        public string pays { get; set; }
    }
 
    public class Personne
    {
        public string nom { get; set; }
        public string age { get; set; }
        public Adresse adresse { get; set; }
    }

et

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
 
static void Main(string[] args)
        {
            Personne personne = new Personne();
 
            personne.age = 20;
            personne.adresse.adresse1 = "mon adresse";
 
            List<Personne> annonces = new List<Annonce>();
            annonces.Add(annonce);
 
            foreach (Annonce ann in annonces)
            {
                Console.WriteLine(ann.adresse.adresse1);
                Console.WriteLine(ann.nbre_pieces);
 
 
            }
            Console.ReadKey();
        }

[Arsiezis]

Ta classe Adresse a aussi besoin d'être initialisée via un new Adresse adr = new Adresse();

Une des solutions est de faire l'init ainsi ;

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Personne personne = new Personne
{
      nom = "",
      age = 0,
      adresse = new Adresse
      {
            adresse1 ="",
            adresse2=""
      }
}

[ebastien]

Bonjour,

Je mettrais plutôt l'instanciation de la classe Adresse dans le constructeur de Personne :

Code :

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

1
2
3
4
5
6
7
8
9
10
public class Personne
{
    public Personne()
    {
        adresse = new Adresse();
    }
    public string nom { get; set; }
    public string age { get; set; }
    public Adresse adresse { get; set; }
}

Le code qui instancie Personne ne devrait pas avoir à s'en préoccuper.

[sleeg]

Merci pour vos réponses.

C'est dans le constructeur, qu'il me semble que ce soit le plus "propre", au niveau du code.

Est-ce bien toujours comme cela, que je devrais construire mes classes avec les sous-classes ? (en terme de bonne pratiques)