Discussion :

[bilou_12]

Bonjour,

Je cherche à créer un constructeur. Je suis les tutoriels mis à la disposition des débutants mais j'ai encore un pb...

Pouvez-vous me dire qu'est ce qu'il ne va pas dans le code ci-dessous sachant que this.ExpirationDate = expirationDate; est souligné en rouge...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class Web:Account
    {
        //Propriété
        public DateTime ExpirationdDate { get; set; }
 
 
        //Constructeur
        public Web(DateTime expirationdDate);
        this.ExpirationDate = expirationDate; 
 
    }

Merci d'avance!

[Aeronia]

Bonjour,

je te conseilles de lire un tutoriel sur les bases de la programmation orienté objet (POO). Tu en trouveras sur ce site.

Aussi non pour répondre à ta question :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Web:Account
    {
        //Propriété
        public DateTime ExpirationdDate { get; set; }
 
 
        //Constructeur
        public Web(DateTime expirationdDate)
        {
             ExpirationDate = expirationDate; 
         }
 
    }

et en plus "propre" :

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
 
class Web : Account
{
            private DateTime _expirationDate;
 
            public DateTime ExpirationdDate 
            { 
                get { return _expirationDate; }
                set { _expirationDate = value; }
            }
 
            public Web(DateTime expirationDate)
            {
                ExpirationdDate = expirationDate;
            }
 
}

[bilou_12]

Merci!

J'y suis en plein dans les tuto

C'est normal que ça me souligne encore en rouge?

[Aeronia]

Autant pour moi pour la première version, je n'avais pas fait attention

tu passes une variable nommée expirationdDate dans ton constructeur, et tu utilise expirationDate dans ton constructeur, tu as tout simplement un d en trop dans le passage de paramètre.


Voici le code correct :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
class Web:Account
    {
        //Propriété
        public DateTime ExpirationdDate { get; set; }
 
 
        //Constructeur
        public Web(DateTime expirationDate)
        {
             ExpirationDate = expirationDate; 
         }
 
    }

[sisqo60]

Bonjour,

c'est quoi ta classe Account?
ça reste souligné en rouge, ça veut rien dire...
C'est quoi l'erreur affichée à la compilation?

PS : Aeronia les codes suivants sont équivalents:

Code :

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

public DateTime ExpirationdDate { get; set; }

Code :

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

1
2
3
4
5
6
7
private DateTime _expirationDate;
 
public DateTime ExpirationdDate 
{ 
    get { return _expirationDate; }
    set { _expirationDate = value; }
}

[bilou_12]

Super! Merci de votre patience!

[Aeronia]

Bonjour,

c'est quoi ta classe Account?
ça reste souligné en rouge, ça veut rien dire...
C'est quoi l'erreur affichée à la compilation?

PS : Aeronia les codes suivants sont équivalents:

Code :

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

public DateTime ExpirationdDate { get; set; }

Code :

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

1
2
3
4
5
6
7
private DateTime _expirationDate;
 
public DateTime ExpirationdDate 
{ 
    get { return _expirationDate; }
    set { _expirationDate = value; }
}

Ca revient à la même chose oui sauf que je trouve que c'est un meilleur réflexe à avoir dans le cas ou tu as des accesseurs particuliers.

Autant utiliser la même philosophie dans toutes les classes et pour toutes les propriétés.

[StringBuilder]

+1, je ne vois pas l'intérêt des get/set sans corps.
=> Si un jour il doit y avoir un corps, on a autant de boulot que si on n'a pas utilisé de propriété, donc l'intérêt me semble très compromis...