Discussion :

[Mattk]

Bonjour,

Bien qu'ayant certaines bases en C#, certains concepts m'échappent encore un peu. J'ai notamment essayé de faire une classe C# contenant des propriétés.

Mon problème est le suivant : à chaque fois que j'initialise ma classe (voir code ci-dessous) avec le constructeur prenant en paramétre un string, ma propriété Name provoque une exception de type StackOverflow, et en effet en regardant la pile d'exécution, je constate qu'il y'a un appel récursif sur le "set" de ma propriété. J'ai le même problème sur le "get". Comment puis-je résoudre ce petit problème ?

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
class VideoFile
    {
        public string Name { get { return this.Name; } set { this.Name = value; } }
        public string Title { get { return this.Title; } set { this.Title = value; } }
        public string Author { get { return this.Author; } set { this.Author = value; } }
 
 
        public VideoFile()
        { }
 
        public VideoFile(string file)
        {
            this.Name = file;
            this.extractInfos();
        }
 
        public void extractInfos()
        {
            this.Author = "";
            this.Title = new FileInfo(this.Name).Name;
        }
    }

Je vous remercie d'avance pour votre réponse,
Mattk

[Thomas Lebrun]

C'est normal

Regarde ton code:

Code :

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

1
2
 
public string Name { get { return this.Name; } set { this.Name = value; } }

La propriété Name appelle, via le get, le this.Name donc à chaque fois, tu te réappelles....

Il faut utiliser un champ privé pour déclarer des propriétés:

Code :

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

1
2
3
 
private string _Name;
public string Name { get { return this._Name; } set { this._Name = value; } }

Pareil pour les autres

[smikar]

Hello,

si tu regardes ton code, le bloc get de ton accesseur renvoie la variable accesseur elle-même, qui redéclenche le bloc get (et ainsi de suite). Le bloc set affecte la variable accesseur elle-même également, et tu obtiens donc à nouveau ce comportement erroné qui crée inévitablement une récursion infinie et donc un stack overflow.

On crée généralement pour chaque accesseur une variable privée de même type que tu utilises pour stocker l'information. L'accesseur l'utilise ensuite pour la mettre à jour ou la lire. Exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
class VideoFile
{
    private string _Name;
    public string Name
    { 
        get { return _Name; }
        set { _Name = value; }
    }
// ...
}

Bonne fêtes !

[Lefauve]

Sinon si t'a pas envie de te faire chier avec tes Get Set
Tu cré ta variable :

private string _var;

puis tu utilise les touche de raccourci ctrl+r+e
cela encapsule ton champs dans des Get Set et évite les erreur

[Thomas Lebrun]

Sinon si t'a pas envie de te faire chier avec tes Get Set
Tu cré ta variable :

private string _var;

puis tu utilise les touche de raccourci ctrl+r+e
cela encapsule ton champs dans des Get Set et évite les erreur

Ou tu tapes prop et tu appuyes sur Tab, dans Visual Studio, pour qu'il te génère tout ce qu'il te faut

[Mourad]

ou tout simplement tu commences sérieusement à revoire tes bases en C#