Discussion :

[agparchitecture]

Bonsoir à tous, je rencontre un problème que je ne vois pas comment résoudre.

donc j'utilise un class avec la propriété suivante :

Code :

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

1
2
3
4
5
6
7
8
 
[XmlElement("Lien"), DisplayName("Lien")]
public string Href
{
get { return _href; }
set { _href = value; }
}
//autre propriété de la class

dans un datagrid view j'affiche une liste de cette classe de la manière suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
dtGridMarks.AutoGenerateColumns = false;
...			
int index;		
index = dtGridMarks.Columns.Add("Href", "Href");
dtGridMarks.Columns[index].DataPropertyName = "Titre";
dtGridMarks.DefaultCellStyle.ForeColor = Color.Black;
...//autre propriété de la class
dtGridMarks.DataSource = lst;
...

afin de trier la liste sur base de l'entête cliqué du datagrid view, je fait appel à la méthode de cette manière

Code :

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

this.DataSource = _ctrlrOrder.OrderMarks((List<Mark>)this.DataSource, this.Columns[e.ColumnIndex].HeaderText);

Ma méthode de tri étant la suivante:

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 List<Mark> OrderMarks(List<Mark> lst, string orderOn){
 
switch(orderOn){
case "Titre" :
lst = (_orderDescending == true) ? 
lst.OrderByDescending(x => x.Titre).ToList() :
lst.OrderBy(x => x.Titre).ToList();
break;
 
case "Href" :
lst = (_orderDescending == true) ? 
lst.OrderByDescending(x => x.Href).ToList() :
lst.OrderBy(x => x.Href).ToList();
break;
 
default:
break;
}
 
_orderDescending = (_orderDescending == true) ? false : true;
 
return lst;
}

la question est donc comment puis-je faire pour généraliser mon switch en remplacant le x=>x.Href de manière générique et ainsi ne pas faire un cas pour chaques propriétes de la classe.

Je pensais éventuellement quelque chose comme :

Code :

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

1
2
3
4
5
6
 
public List<Mark> OrderMarks(List<Mark> lst, string orderOn){
var prop = typeof(Mark).GetProperty(orderOn);
lst =  lst.OrderByDescending(x => x.prop).ToList();
return lst;
}

evdement ca ne marche pas. :-(

[François DORIN]

Bonjour,

Tu es sur la bonne piste. Le soucis est que tu ne vas pas jusqu'au bout du raisonnement.

GetProperty te retourne des infos sur une propriété, et non la valeur de la propriété elle-même. Pour cela, il faut utiliser la méthode PropertyInfo.GetValue.

Code c# :

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

1
2
3
4
5
public List<Mark> OrderMarks(List<Mark> lst, string orderOn){
var prop = typeof(Mark).GetProperty(orderOn);
lst =  lst.OrderByDescending(x => prop.GetValue(x)).ToList();
return lst;
}

[agparchitecture]

Impecable ca marche comme je veux. Merci

Est-il possible de faire de même avec le displayname de la propriété?

[François DORIN]

Avec la propriété Name peut-être ?

Sinon, il faut que tu me dises ce que tu entends par "displayName"

[agparchitecture]

Ce que j'entend par le displayname c'est l'attribut de la propriété qui est utilisé lors de création automatique des entêtes des datagridview

exemple :

Code :

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

1
2
3
4
5
6
7
 
[XmlElement("Lien"), DisplayName("Lien")]
public string Href
{
get { return _href; }
set { _href = value; }
}

[François DORIN]

Ok. Donc oui tu peux. A partir de ton instance de PropertyInfo, tu peux récupérer les attributs qui sont dessus. Il faut que tu t'intéresses à l'attribut qui a le type "DisplayNameAttribute"

[katkiller]

Bonjour,

Voici un petit exemple :
Classe Test:

Code :

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

1
2
3
4
5
6
7
8
    public class Test
    {
        [DisplayName("Test id")]
        public int Id { get; set; }
 
        [DisplayName("Test name")]
        public string Name { get; set; }
    }

Lister les propriétés et leur "displayName" :

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
        static void Main(string[] args)
        {
            var properties = typeof(Test).GetProperties()
                .Select(p => new
                {
                    PropertyName = p.Name,
                    DisplayName = p.GetCustomAttributes(typeof(DisplayNameAttribute), false)
                    .Cast<DisplayNameAttribute>().Single().DisplayName
                });
 
            foreach (var property in properties)
            {
                Console.WriteLine("PropertyName : {0} - DisplayName : {1}",
                    property.PropertyName, property.DisplayName);
            }
 
            Console.ReadLine();
        }

Par contre j'ai l'impression que tu es entrain de fabriquer une usine à gaz... (Utiliser la réflexion juste pour ordonner un liste avec un méthode non générique)...