Discussion :

[olibara]

Bonjour

Sans etre vraiment bloqué, j'aimerais connaitre la methode la plus astucieuse pour faire une comparaison parametrée

Je m'explique

L'utilisateur dispose d'un ComBo : <, >, <=, >=, =
Et d'un Numeric textbox

J'aimerais dans l'application faire une comparaison entre une valeur, le mode de comparaison defini dans le combo et la valeur de reference utilisateur

Bien sur je peux faire un switch case mais je pense qu'il y a moyen de faire mieux
Sans doute un delegate mais je n'ai jamais defini moi meme de delegate et je me sens un peu tout con devant l'obstacle

[ZaaN]

De toute manière. à une place tu sera bien obliger de faire la relation entre tes signes de comparaison "user" et les operateurs de comparaison C#.

A mon avis, tu peux mettre tout les statagèmes en place pour encapsuler cette comparaison (ce qui est propre, SDEP), qque part, on retrouvera un switch.

SDEP = Soit dit en passant.

[olibara]

Moi je pensais definir x fonction de comparaison

Et un delegate qui prends la fonction demandée qui sera la meme pour toute la boucle de traitement !!

[tomlev]

Moi je pensais definir x fonction de comparaison

Et un delegate qui prends la fonction demandée qui sera la meme pour toute la boucle de traitement !!

Oui, mais à un moment ou un autre tu seras bien obligé de choisir la fonction à utiliser, et pour ça tu feras un switch...

[SaumonAgile]

Il y a aussi la solution de générer dynamiquement une méthode en utilisant un generateur IL, mais c'est peut-être un peu compliqué pour une comparaison

[suvuk]

Pas forcément (le switch), si tu range tes delegates de fonctions de comparaison dans un tableau et que tu utilise l'index sélectionné de la combo pour récupérer le delegate à cet index du tableau...
Mais je suis pas persuadé du gain de performance, surtout que tu n'as pas beaucoup de fonctions de comparaison.

(c'était plus clair dans mon petit cerveau avant que je l'écrive )

[Guulh]

On peut imaginer ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
delegate bool ComparerCustom<T>(T item1, T item2);
class Machin
{
public string Name;
public ComparerCustom<int> Compareur;
// et un constructeur qui va bien, j'ai la flemme de le mettre ici :)
}
List<Machin> liste = new List<Machin>();
liste.Add(new Machin("=", delegate(int référence, int valeurUser) { return référence == valeurUser; }));
liste.Add(new Machin("<", delegate(int référence, int valeurUser) { return référence < valeurUser; }));
...

Puis binder la combo sur la liste de machins, mettre le DisplayMember qui va bien, et le résultat attendu sera égal à

Code :

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

(laCombo.SelectedItem as Machin).Compareur(valeurRéférence, valeurUser);

Un bon gros switch est pas plus mal, je trouve.

[tomlev]

ou avec un Dictionary<string, ComparerCustom<T>>
la clé est l'opérateur, et la valeur le delegate à utiliser...

[olibara]

Vous etes tous géniaux !

Il y a plein de bonnes idées ici (a garder a chaud)

Oui, mais à un moment ou un autre tu seras bien obligé de choisir la fonction à utiliser, et pour ça tu feras un switch...

Mais il n'y a rien de dérangeant de l'avoir dans la validation UI


Dans le cas qui m'occupe je voulais eviter le case dans la boucle de select qui parcours des milliers de records, d'une part et puis j'aime pas avoir plein de code dans une boucle donc tant qu'a appeller une fonction pour rendre une boucle plus lisible autant rendre la fonction la plus efficace possible !

Donc je pense définir mes x fonctions de comparaisons

Ensuite, lors de la validation du select je prends la valeur du combo et je cree mon delegate

MyCompDelegate MyComp=new MyCompDelegate(fonction_adequate);


Et dans la boucle de select j'utilise MyComp