Discussion :

[JerryOne3]

bonsoir a tous

étudiant en informatique débutant en C# sur visual studio 2008, j'ai un problème sur le début d'un exercice, et apres de longues heures après lesquelles il me reste peu de cheveux sur la tete, je vous soumet ma requête :

je cherche a créer une liste de type Person, la classe Person intégrant un string étant le nom d'un élève, et une liste représentant sa liste de notes.

je cherche a ce que, lors de la creation de la Person, il y ait des notes differentes. pour ce faire j'ai procédé comme suit :

la classe Person :

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
21
22
23
24
25
26
27
28
29
30
31
 
using System;
using System.Collections.Generic;
using System.Text;
 
namespace targil_6
{
    class Person
    {
        private string _FirstName;
        public List<int> Grades;
 
        public string FirstName
        {
            get { return _FirstName; }
            set { _FirstName = value; }
        }
 
        Person()
        {
            FirstName = " ";
            Grades = null;
        }
 
        public Person(string name, List<int> grades)
        {
            FirstName = name;
            Grades = grades;
        }
    }
}

la classe Program :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace targil_6
{
    class Program
    {
        static void Main(string[] args)
        {
            var Persons = new List<Person>();
            var grades = new List<int>();
 
            for (int i = 70; i < 80; i++)
            {
                grades.Add(i);
            }
 
            for (int j = 0; j < 10; j++)
            {
                for (int k = 0; k < 10; k++)
                {
                    grades[k] = grades[k] + 1;
                }
                Persons.Add(new Person("Firstname" + j, grades));
            }
 
            foreach (Person item in Persons)
            {
                Console.WriteLine(item.ToString());
            }
 
            Console.ReadKey();
        }
    }
}

J'ai mis dans le ToString l'impression de la première note de chaque Person, afin de vérifier que tout va bien, et lors du writeline, ca donne 80 pour tous les élèves. En debug, on se rend compte que la note des Person change dans ceux qui sont déjà dans la liste de Persons en même temps qu'évolue la liste de notes sensées être celles du suivant a entrer dans la liste de Persons...

voila, une idée de ce qui cloche ?

Sinon, petite question en plus, comment initialiser une liste comme celle la lors de sa creation en y integrant immediatement des membres ?

merci d'avance

[lysiandad]

bonsoir a tous
[...]voila, une idée de ce qui cloche ?

Posez vous ces questions :

1) que signifie (c-à-d : quel est l'effet), à l'exécution du programme, cette ligne en gras souligné ? (essayer de "visualiser mentalement" ce que vous avez en memoire de la machine, à chaque fois que cette ligne est executée) :

Code :

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

1
2
3
4
5
6
        public Person(string name, List<int> grades)
        {
            FirstName = name;
            Grades = grades;
        }

2) idem (1) pour :

Code :

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

1
2
3
4
5
6
            for (int j = 0; j < 10; j++)
            {
// [...][...]
                Persons.Add(new Person("Firstname" + j, grades));
            }

3) quelle est la corrélation entre mes questions (1) et (2) ci dessus, vis à vis de votre problème ?

Enfin (tuyau ), relisez la partie de votre cours qui fait la distinction entre passage par valeur et passage par référence dans un environnement d'exécution (que ce soit la pile ou "le tas"..)

'HTH

[JerryOne3]

bonjour,

De ce que je comprends, j'aurais créé une sorte de pointeur vers la liste grades, ce qui fait que celle ci n'existe qu'une fois et que les Grades de tous les membres de la liste Persons pointent vers elle?

Est il possible de remplir ainsi la liste de chacun des membres de Persons différemment, ou faut il que je rentre manuellement les valeurs ?

merci de votre aide

[tomlev]

De ce que je comprends, j'aurais créé une sorte de pointeur vers la liste grades, ce qui fait que celle ci n'existe qu'une fois et que les Grades de tous les membres de la liste Persons pointent vers elle?

C'est ça. List<int> est une classe, donc un type référence. En faisant Grades = grades, on ne copie qu'une référence vers le même objet. Il faut donc créer une nouvelle instance de List<int> pour chaque élève.

En fait, il vaudrait mieux déclarer Grades comme une propriété en lecture seule :

Code :

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

1
2
3
4
5
6
 
private List<int> _grades;
public List<int> Grades
{
    get { return _grades; }
}

Et l'initialiser dans le constructeur pour que ça ne soit jamais null :

Code :

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

1
2
3
4
5
Person()
{
    FirstName = " ";
    Grades = new List<int>();
}

[JerryOne3]

je comprends.

par contre, lorsque je modifie le code comme vous me l'avez suggéré en mettant Grades en lecture seule et et l'initialisant dans le constructeur, j'obtiens une erreur dans les constructeurs aux lignes d'initialisation de grades :

Code :

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

Property or indexer 'targil_6.Person.Grades' cannot be assigned to -- it is read only

comment faire ?

sinon j'ai réussi a régler le probleme d'initialisation des grades en modifiant le Program.cs comme suit :

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace targil_6
{
    class Program
    {
        static void Main(string[] args)
        {
            var Persons = new List<Person>();

            for (int j = 0; j < 10; j++)
            {
                var grades = new List<int>();

                for (int i = 70; i < 80; i++)
                {
                    grades.Add(i);
                }
                for (int k = 0; k < 10; k++)
                {
                    grades[k] = grades[k] + j;
                }
                Persons.Add(new Person("Firstname" + j, grades));
            }

            foreach (Person item in Persons)
            {
                Console.WriteLine(item.ToString());
            }

            Console.ReadKey();
        }
    }
}

Ça fonctionne parfaitement, ça me donne exactement ce que je cherchais, c'est a dire décalé de 1 pour chaque Person.

est ce une méthode valable ?

enfin, je repose ma question du début :

comment initialiser une liste comme celle la lors de sa creation en y integrant immediatement des membres ?

merci encore

[tomlev]

par contre, lorsque je modifie le code comme vous me l'avez suggéré en mettant Grades en lecture seule et et l'initialisant dans le constructeur, j'obtiens une erreur dans les constructeurs aux lignes d'initialisation de grades

Erreur d'étourderie de ma part... mets "_grades" à la place de "Grades" dans le constructeur

est ce une méthode valable ?

Ca me semble correct

comment initialiser une liste comme celle la lors de sa creation en y integrant immediatement des membres ?

Ben tu peux toujours initialiser le contenu de Grades dans le constructeur de Person. Mais encore faut-il savoir quelles données tu y mets...

[lysiandad]

Code :

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

Property or indexer 'targil_6.Person.Grades' cannot be assigned to -- it is read only

comment faire ?

tomlev a seulement omis de definir le "setter" sur la propriété Grades, si vous la voulez en lecture / ecriture :

Code :

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

1
2
3
4
5
6
 
public List<int> Grades
{
    get { return _grades; }
    set { _grades = value; }
}

'HTH