Discussion :

[steliseb]

Bonjour,

Sur des cours trouvé sur votre site, il est expliqué comment redéfinir des opérateur pour une classe.

J'ai essayé le code mais le résultat obtenu n'est pas celui attendu.

Voici ma classe ListeDePersonne qui est sensé redéfinir l'opérateur pour pouvoir concaténé une personne en plus dans la liste.

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
public static ListeDePersonne operator +( ListeDePersonne l,personne p)
    {
        l.Add(p);
        return l;
    }
 
    public static void Main()
    {
        ListeDePersonne l=new ListeDePersonne();
 
        l = l + new personne("michel", "DUPONT", 29) + new personne("david", "PIGNON", 32); 
 
        Console.Out.WriteLine("liste =" + l );
    }
 
    public override string ToString()
    {
 
        String liste = "(";
        int i;
 
        for (i = 0; (i < Count - 1); i++)
        {
            liste += "[" + base[i].ToString() + "]" + ",";
        }
        if (Count != 0) liste += "[" + base[i].ToString() + "]";
        liste += ")";
 
        return liste;
    }

En plus de celle-ci, je définie une classe personne qui me créé une personne (Prénom, Nom,age).

Résulat attendu :
liste = [Michel dupont 29],[David pignon 32]

Résultat obtenu :
liste = [personne],[personne]

Quelqu'un peut-il me dire pourquoi

MERCI

[SirJulio]

As tu bien overridé le .ToString() de Personne ? J'en doute vu les resultats obtenus.

Quelque petit conseil cependant, ton string construit le avec un stringbuilder pas une concatenation de string (à 1000 instance de personne, tu vas rendre fou ton GC pour rien). Enfin deuxieme chose, faire un ToString de collection qui renvoit une chaine aggrégé de ce qu'il contient, c'est pas une bonne idée (AMHA, bien sur).

[steliseb]

Merci pour tes conseils.
Je n maitrise pas du tout donc j'essaye de suivre ce qui est dit en essayant de comprendre donc tout ce qui est amélioration de code pour l'instant je n'y pas pas de temps.
Je me suis rendu compte qu'il manquait du code en effet ma classe ListeDePersonne n'est pas un héritage d personne mais une ArrayList comme le montre le code suivant.

public class ListeDePersonne:ArrayList
{

public static ListeDePersonne operator +( ListeDePersonne l,personne p)
{
l.Add(p);
return l;
}

public static void Main()
{
ListeDePersonne l=new ListeDePersonne();

l = l + new personne("michel", "DUPONT", 29) + new personne("david", "PIGNON", 32);

Console.Out.WriteLine("liste =" + l );
l=l+new personne("Bassou","METRAL",34);
Console.Out.WriteLine("liste = " + l);

}

public override string ToString()
{

String liste = "(";
int i;

for (i = 0; (i < Count - 1); i++)
{
liste += "[" + base[i] + "]" + ",";
}
if (Count != 0) liste += "[" + base[i] + "]";
liste += ")";

return liste;
}
}

[SirJulio]

Il faut surtout redefinir le .ToString de la classe Personne pour faire ce que tu veux faire. =)

[steliseb]

MERCI.

En effet il fallait overrider mon ToString dans ma classe personne.
Donc si j'ai bien compris, il faut overrider TOUTES les classe (Filles et mères) lors de l'utilisation d'une instruction comme ToSring ?

[SirJulio]

Non, pas du tout.

Normalement (dans les plupart des cas), un .ToString() te renvoie une description de l'objet (comprendre principalement son type, le cas ici), si tu veux qu'un ToString te renvoie autre chose tu le redefinies.

Ici, tu voulais que ta collection ait une description particuliere ET que ton objet Personne ait aussi une description particuliere, donc tu dois redefinir Personne et ListeDePersonne. C'est pas plus compliqué que ca. =)

[SaumonAgile]

Pourquoi tu utilises un ArrayList plutot qu'une collection typée ?

[steliseb]

Je connais pas la collection typée.
Je lis un cours sur C# et dans ce chapitre ils expliquent le changement d'opérateur et pour cette exemple ils sont passé par une arraylist.

[Chubyone]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class Personne
{}
 
psvm()
{
   List<Personne> kList = new List<Personne>();
   kList.Add( new Personne("BadBoy") ); 
foreach( Personne kPersonne in kList )
{
  Console.Write( kpersonne ); // necessite redef .ToString de Personne ; plutot que de Console.Write, un StringBuilder c'est plus cool 
}
}

Définir un opérateur "+" sur une Collection pour pas utiliser le "add" c'est quand même aimer la surcharge de travail

[Bluedeep]

Définir un opérateur "+" sur une Collection pour pas utiliser le "add" c'est quand même aimer la surcharge de travail

Oui, ça a un coté un peu marteau pilon pour écraser un cafard.