héritage et override methodes sous .NET

Version imprimable

salut,

j'aurais besoin d'un éclairage pour utiliser l'héritage sous .NET

J'ai un première classe parent (simplifiée pour l'occasion)
et une seconde qui en hérite et dans laquelle j'ai seulement réécrit les methodes qui ont un comportement différent en utilisant "new".
Le problème c'est que la page affichée est du type PublicationList (parent) et non DataList (fille).
En fait, la fille appelle bien le page_load du parent mais ensuite comment faire pour que ce soit les méthodes spécifiques de la classe fille qui soient appelées?

Code:

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
38
39
40
41
42
namespace web
{
    public partial class PublicationList : ListPage
    {
 
        protected void Page_Load(object sender, EventArgs e)
        {
            switch (GetMode())
            {
                case Mode.YEAR:
                    DoYear();
                    break;
 
                case Mode.TYPE:
                    DoType();
                    break;
            }
        }
 
        protected Mode GetMode()
        {
        	if (Request["mode"] == "year")
                    return Mode.YEAR;
                else 
                    return Mode.TYPE;
        }
 
        protected void DoYear()
        {
 
            title.Text = "Publications " + year.ToString();
 
        }
 
        protected void DoType()
        {
 
            title.Text = "Type";
 
        }
    }
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
namespace web
{
    public partial class DataList : PublicationList 
    {
 
        protected new void DoYear()
        {
 
            title.Text = "Data" + year.ToString();
 
        }
 
        protected new void DoType()
        {
 
            title.Text = "Data Type";
 
        }
    }
}

merci pour vos conseils

09/10/2008, 18h43
Paul Musso

Bonjour,

Pour remplacer complétement une méthode avec une classe fille, il faut que ta méthode de la classe mère soit virtuelle (virtual) et la méthode de ta classe fille doit l'overrider (override).

Sinon, les 2 méthodes existent quand même, et le fait de manipuler une classe fille avec une variable typée avec la classe mère appelera obligatoirement la méthode de la classe mère.
09/10/2008, 18h51
tomlev

En redéclarant les méthodes de la classe de base avec new, tu ne redéfinis pas la même méthode : tu la "masques" et tu en crées une nouvelle. Cette nouvelle méthode ne sera appelée que si tu l'utilise avec une référence de la classe dérivée, pas avec une référence de la classe de base. En faisant comme ça, tu "casses" le polymorphisme.
Il faut que la méthode de la classe de base soit déclarée comme virtual, et que tu la redéfinisses dans la classe dérivée avec le mot-clé override

J'ai modifié le code pour qu'il reflète un peu mieux mes vraies classes.
J'ai apporté les modifications en utilisant virtual et override.
Mais j'obtiens un message d'erreur au moment de l'appel des méthodes override.

Citation:

Object reference not set to an instance of an object.

Stack Trace:

[NullReferenceException: Object reference not set to an instance of an object.]
Web. DataList.RenderTypeList(Int32 type_id) +203
Web.PublicationList.DoType() +247
Web.PublicationList.Page_Load(Object sender, EventArgs e) +116
...

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
namespace web
{
    public partial class PublicationList : ListPage
    {
 
        protected void Page_Load(object sender, EventArgs e)
        {
            switch (GetMode())
            {
                case Mode.YEAR:
                    DoYear();
                    break;
 
                case Mode.TYPE:
                    DoType();
                    break;
            }
        }
 
        protected Mode GetMode()
        {
        	if (Request["mode"] == "year")
                    return Mode.YEAR;
                else 
                    return Mode.TYPE;
        }
 
        protected void DoYear()
        {
            title.Text = "Publications " + year.ToString();
            RenderYearList();
        }
 
        protected void DoType()
        {
 
            title.Text = "Type";
            RenderTypeList();
        }
 
        protected virtual void RenderYearList()
        {
             // render the list
        }
 
        protected virtual void RenderTypeList()
        {
             // render the list
        }
 
    }
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
namespace web
{
    public partial class DataList : PublicationList 
    {
       protected override void RenderYearList()
        {
             // render the list
        }
 
        protected override void RenderTypeList()
        {
             // render the list
        }
    }
}

10/10/2008, 09h48
Pongten

En fait, j'ai l'impression que c'est ta "structure" qui ne va pas...

Qu'essayes-tu de faire ?
10/10/2008, 10h10
julien.63

En fait, je crée une nouvelle classe (la fille) qui affiche les mêmes données qu'une autre déjà existante (la mère). Sauf que la fille filtre les résultats pour n'en afficher qu'une partie en fonction de certains critères.

Du coup, la plupart des méthodes sont scrupuleusement identiques. Sauf trois qui font le tri en question et que j'essaie "overrider".

Ca fonctionne si je fais un copier coller des méthodes dans la classe fille (qui n'est plus vraiment une fille puisqu'elle n'a plus besoin d'héritage dans ce cas). Mais, je pense que c'est une situation où je devrais profiter de l'héritage pour réutiliser le code et faciliter sa maintenance par la suite.
10/10/2008, 10h53
Pongten

Je pense que tu aurais encore plus simple en utilisant une architecture N-tiers ou ta page ne ferait qu'afficher les données qui lui serait présentées par un objet métier les filtrant selon les besoins.

Ce mis à part, pourquoi ne donnes-tu pas directement le type "DataList" à ta page ?

J'ai du mal à cerner ton architecture en fait, désolé si je suis à côté de la plaque...
10/10/2008, 11h12
julien.63

oui ce que tu évoques parait séduisant mais je reprends une solution en cours de route. Je n'ai pas envie de refaire ce qui a été fait avant moi.

Puis je trouve que c'est pas mal d'avoir des objets spécialisés plutôt que des méthodes avec une série de if qui gère tous les cas possibles.

Enfin, je ne suis pas très familier de c# et je voyais là une bonne occasion de voir comment mettre en oeuvre l'héritage.

A propos du message d'erreur, une idée ? ;)
10/10/2008, 11h19
Pongten

Comment initialises-tu tes objets ? quel est leur type ?
10/10/2008, 12h51
julien.63

Citation:

Comment initialises-tu tes objets ?

euuh à vrai dire, je ne sais pas quoi répondre à cette question :oops:

Citation:

quel est leur type ?

il s'agit de webform aspx

Finalement,
j'ai recopié la méthode pageLoad dans la classe fille.
ça fonctionne mais je ne sais pas si c'est la meilleure pratique?

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
namespace web
{
    public partial class PublicationList : ListPage
    {
 
        protected void Page_Load(object sender, EventArgs e)
        {
            switch (GetMode())
            {
                case Mode.YEAR:
                    DoYear();
                    break;
 
                case Mode.TYPE:
                    DoType();
                    break;
            }
        }
 
        protected Mode GetMode()
        {
        	if (Request["mode"] == "year")
                    return Mode.YEAR;
                else 
                    return Mode.TYPE;
        }
 
        protected void DoYear()
        {
            title.Text = "Publications " + year.ToString();
            RenderYearList();
        }
 
        protected void DoType()
        {
 
            title.Text = "Type";
            RenderTypeList();
        }
 
        protected virtual void RenderYearList()
        {
             // render the list
        }
 
        protected virtual void RenderTypeList()
        {
             // render the list
        }
 
    }
 
 
namespace web
{
    public partial class DataList : PublicationList 
    {
 
      protected void Page_Load(object sender, EventArgs e)
        {
            switch (GetMode())
            {
                case Mode.YEAR:
                    DoYear();
                    break;
 
                case Mode.TYPE:
                    DoType();
                    break;
            }
        }
 
       protected override void RenderYearList()
        {
             // render the list
        }
 
        protected override void RenderTypeList()
        {
             // render the list
        }
    }
}

Salut

Tu essayes d'overrider des méthodes qui ne sont pas prévues pour...

Ne peux-tu pas simplement déclarer les méthodes overridable dans la classe PublicationList

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
    public partial class PublicationList : ListPage
    {
 
        protected void Page_Load(object sender, EventArgs e)
        {
            switch (GetMode())
            {
                case Mode.YEAR:
                    DoYear();
                    break;
 
                case Mode.TYPE:
                    DoType();
                    break;
            }
        }
 
        protected Mode GetMode()
        {
        if (Request["mode"] == "year")
                    return Mode.YEAR;
                else 
                    return Mode.TYPE;
        }
 
        protected virtual void DoYear()
        {
            title.Text = "Publications " + year.ToString();
            RenderYearList();
        }
 
        protected virtual void DoType()
        {
 
            title.Text = "Type";
            RenderTypeList();
        }
 
        protected virtual void RenderYearList()
        {
             // render the list
        }
 
        protected virtual void RenderTypeList()
        {
             // render the list
        }
 
    }

Code:

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
   public partial class DataList : PublicationList 
    {
 
       protected override void DoYear()
        {
 
            title.Text = "Data" + year.ToString();
 
        }
 
        protected override void DoType()
        {
 
            title.Text = "Data Type";
 
        }
 
       protected override void RenderYearList()
        {
             // render the list
        }
 
        protected override void RenderTypeList()
        {
             // render the list
        }
    }

13/10/2008, 13h18
julien.63

je n'ai pas besoin "d'overrider" les méthodes DoYear et DoType. Au contraire je veux les hériter de la classe mère.
J' "override" seulement RenderYearList, RenderTypeList et PageLoad.

Cette dernière est identique dans la classe et la classe fille. Mais j'ai l'impression qu'il n'y a pas moyen de faire autrement si je veux effectivement appeler les méthodes "overridées".
13/10/2008, 13h35
knowl

Tu peux overrider la méthode OnLoad..

C'est là que je comprends pas ton problème, normalement tu n'as pas besoin de page_Load.

Les méthodes que tu as overridées sont automatiquement appelées à partir du moment que l'instance de la classe que tu as créée est bien une instance de la classe DataList.

Code:

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
38
39
40
 
public class Class1 
{    
    public void Test() 
    { 
        Console.WriteLine(this.Libelle); 
        Console.WriteLine(this.Libelle2); 
    } 
 
    public string Libelle() 
    { 
        return "Libellé classe 1"; 
    } 
 
    public virtual string Libelle2() 
    { 
        return "Libellé 2 classe 1"; 
    } 
 
} 
 
public class Class2 : Class1 
{ 
    public new string Libelle() 
    {         
        return "Libellé classe 2";  
    } 
 
    public override string Libelle2() 
    { 
        return "Libellé 2 classe 2";  
    }     
} 
 
 
Resultats :
// Méthode de la classe 1 même si je fais un public new dans ma Class2
> Libellé classe 1 
//Méthode de la classe 2 avec override
> Libellé 2 classe 2

oui effectivement ça marche.
Je ne sais pas ce que j'ai fait de travers, la première fois, ça n'avait pas marché... :?

Dans la continuité de ceci, est-il possible qu'une page aspx hérite d'une autre page.
Pour poursuivre avec l'exemple utilisé auparavant, PublicationList.aspx et DataList.aspx sont identiques sauf la première ligne au niveau des paramètres CodeBehind et Inherits.
Code:

1 2 3 4 <%@ Page Language="C#" EnableViewState="false" AutoEventWireup="true" MasterPageFile="~/pd/MasterPages/Default.master" CodeBehind="PublicationList.aspx.cs" Inherits="Web.PublicationList" %> <%@ Page Language="C#" EnableViewState="false" AutoEventWireup="true" MasterPageFile="~/pd/MasterPages/Default.master" CodeBehind="DataList.aspx.cs" Inherits="Web.DataList" %>
J'ai essayé comme ça, en faisant hériter le webform (via Inherits) mais ça ne fonctionne pas.

Code:

<%@ Page Language="C#" EnableViewState="false" AutoEventWireup="true" MasterPageFile="~/pd/MasterPages/Default.master" CodeBehind="DataList.aspx.cs" Inherits="Web.PublicationList" %>