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

[tomlev]

tomlev a seulement omis de definir le "setter" sur la propriété Grades

C'était intentionnel
L'erreur était dans le code du constructeur (cf. mon post précédent)

[JerryOne3]

Code :

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

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

en faisant ainsi, le set ajouterait des éléments a la liste?

ensuite, lorsque je demande comment initialiser une liste lors de sa création, je parle dans le Program.cs . Comment creer une liste en y mettant directement des membres, comme par exemple :

Code :

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

var Persons = new List<Person>(("name1",{1,2,3},"name2",{2,3,4});

sauf que comme ça ça ne fonctionne pas bien évidemment

[tomlev]

Code :

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

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

en faisant ainsi, le set ajouterait des éléments a la liste?

Non, le set remplacerait la liste. C'est pour ça que je l'avais mise en lecture seule. Si tu regardes dans les classes du framework, la plupart des propriétés de type liste sont en lecture seule (mais on peut ajouter des éléments à la liste elle-même).

ensuite, lorsque je demande comment initialiser une liste lors de sa création, je parle dans le Program.cs . Comment creer une liste en y mettant directement des membres, comme par exemple :

Code :

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

var Persons = new List<Person>(("name1",{1,2,3},"name2",{2,3,4});

sauf que comme ça ça ne fonctionne pas bien évidemment

Si tu utilises C# 3 (et apparemment c'est le cas) tu peux faire comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
var Persons = new List<Person>
{
    new Person
    {
        Name = "name1",
        Grades = new List<int> {1, 2, 3}
    },
    new Person
    {
        Name = "name2",
        Grades = new List<int> {2, 3, 4}
    }};

Mais je ne vois pas trop l'intérêt, parce que en pratique les données ne seront pas codées en dur, tu les récupèreras plutôt dans un fichier ou une base de données.

[JerryOne3]

Oui, pour le set c'est bien ce que je pensais. Je comptais pas le mettre de toute façon.

sinon, pour l'initialisation de la liste, c'était juste pour savoir comment faire théoriquement.

merci beaucoup pour toutes ces réponses

Je vais tenter d'avancer dans l'exercice maintenant

faut il que je mette une balise [RESOLU] ou quelque chose comme ca maintenant que le sujet est... résolu?

[tomlev]

faut il que je mette une balise [RESOLU] ou quelque chose comme ca maintenant que le sujet est... résolu?

Oui, c'est le bouton en bas de la page