Discussion :

[Anthares]

Bonjour à tous,

j'ai un petit problème dans mon application .

j'ai un classe Pair, qui est une classe générique dont voici le code

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
public class Pair<T>
    {
        public T First { get; set; }
 
        public object Second { get; set; }
 
        public Pair() { }
 
        public Pair(T first, object second)
        {
            First = first;
            Second = second;
        }
 
        public Pair(T first)
            : this(first, null)
        { }
    }

et j'ai une méthode qui me retourne un string en fonction du type du paramètre

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
public static string ValueToStringFormat(object someValue)
        {
            string FormattedValue = "";
 
            if (someValue == null)
            {
                FormattedValue = "NULL";
            }
            else
            {
                switch (someValue.GetType().Name)
                {
                    case "String": FormattedValue = "'" + ((string)someValue).Replace("'", "''") + "'"; break;
                    case "DateTime": FormattedValue = "'" + ((DateTime)someValue).ToString("yyyy/MM/dd hh:mm:ss") + "'"; break;
                    case "DBNull": FormattedValue = "NULL"; break;
                    case "Boolean": FormattedValue = (bool)someValue ? "1" : "0"; break;
                    case "Pair`1":
                        //?????????
                        break;
                    default: FormattedValue = someValue.ToString(); break;
                }
            }
            return FormattedValue;
        }

Mon but est de remplacer les ????? par un truc du genre

Code :

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

ValueToStringFormat(someValue.First) + (someValue.Second != null) ? " And " + ValueToStringFormat(someValue) : String.Empty;

Le problème, c'est que pour ça, il faudrait que je puisse caster someValue en Pair<>, vu que je ne suis pas censé connaitre le type, mais ça ne marche pas et je ne sais pas du tout si c'est possible.

Si quelqu'un a une idée, je l'en remercie

[mermich]

Tout d'abord une petite remarque, ton code de ValueToString n'est pas générique, car ton switch-case est en dur:

Code :

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

 case "Pair`1":

Ensuite pour ce que tu veux faire

• soit tu castes ton objet en "Pair", vu que ton switch est en dur

• soit via la réflexion tu récupères toutes les propriétés, sur lesquelles tu récupères le type pour pouvoir ensuite récupérer la valeur suivant le format voulu.

[theMonz31]

salut

plutot que de faire comme celà, il serait plus logique que ton objet générique fournisse une méthode "ToString" ou équivalente. Ainsi, la personne implémentant ton type générique pourra redéfinir la méthode "ToString()" pour lui faire dire ce qu'elle veut...

Je ferais à l'équivalent du Framework avec sa méthode ToString() de chaque objet qui est "overridable"

[mermich]

Par contre le coup du ToString() je suis plutot contre, en effet sur le coup c'est bien pratique, mais sur le long terme si tu dois mettre en place un certain format c'est plus très maintenanble.

Ou mieux, faire que ta classe hérite d'une interface

[mathrb]

Personnellement je rejoins l'idée de theMonz31, car dans le cas où tu auras des Pair<Pair<String>>, tu devras encore rajouter dans ton switch un nouveau cas, et on en sort plus vraiment.