Discussion :

[StringBuilder]

Désolé pour le titre, je deviens chèvre.

Habitué au SQL, j'aime utiliser l'opérateur "x between y and z".

J'ai donc développé la méthode d'extension suivante :

Code csharp :

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

1
2
3
4
5
 
public static bool IsBetween<T>(this T item, T start, T end)
{
        return Comparer<T>.Default.Compare(item, start) >= 0 && Comparer<T>.Default.Compare(item, end) <= 0;
}

C'est magique, maintenant je peux faire des int.IsBetween(int, int) et des long.IsBetween(long, long)

Mais pas des short.IsBetween(short, short) !

Voici ce qu'il me sort :

Code :

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

Erreur	CS1929	'short' ne contient pas de définition pour 'IsBetween' et la meilleure surcharge de méthode d'extension 'Extensions.IsBetween<int>(int, int, int)' nécessite un récepteur de type 'int'	CRMStationsManager	E:\CRMStationsManager\CRMStationsManager\CRMStationCollection.cs	65	Actif

Pourtant, si je passe la souris sur mla ligne en question, voici ce que j'ai comme aide contextuelle :


C'est encore mon Visual Studio qui déconne ou quoi ?

[StringBuilder]

Bon ben .NET n'est pas buggé qu'un peu...

Voilà, ça ça marche :

Code csharp :

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

1
2
3
4
5
6
 
        public static bool IsBetween(this short item, short start, short end)
        {
            return Comparer<short>.Default.Compare(item, start) >= 0
                && Comparer<short>.Default.Compare(item, end) <= 0;
        }

Ça m'apprendra à vouloir faire du code générique !

[Pol63]

en effet pour faire ca on utilise pas T
et dans le framework aussi on trouve des méthodes identiques 10x si nécessaire avec des types différents, mais ce n'est pas bien long à écrire ...

[ebastien]

Bonjour,

Bon ben .NET n'est pas buggé qu'un peu...

Pas vraiment. Maintenant, il pourrait faire un warning plutôt qu'une erreur. Un int rentre dans un long mais pas dans un short.
En convertissant explicitement 6000 et 6999 en short, ça fonctionne.

Code :

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

numStation.IsBetween((short)6000,(short)6999)

ou bien

Code :

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

1
2
3
4
short num1 = 4;
short num2 = 3;
short num3 = 7;
num1.IsBetween(num2, num3)

Là, pas d'erreur !

[StringBuilder]

Bonjour,



Pas vraiment. Maintenant, il pourrait faire un warning plutôt qu'une erreur. Un int rentre dans un long mais pas dans un short.
En convertissant explicitement 6000 et 6999 en short, ça fonctionne.

Code :

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

numStation.IsBetween((short)6000,(short)6999)

ou bien

Code :

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

1
2
3
4
short num1 = 4;
short num2 = 3;
short num3 = 7;
num1.IsBetween(num2, num3)

Là, pas d'erreur !

Argh ! Bien vu !
Du coup je vais réutiliser la fonction générique et passer par des constantes typées ! Ça m'apprendra à laisser des Magic Numbers dans mon code

[Noxen]

Sur ce cas je crois que j'aurais testé en forçant le type pour voir ce que ça aurait donné :

Code :

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

numStation.IsBetween<short>(6000, 6999)

D'ailleurs, pour avoir testé :

Code :

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

1
2
3
4
5
6
short s = 10;
s.IsBetween<short>(2, 10000); // 10000 est accepté comme un short
s.IsBetween<short>(2, 100000); // 100000 est un int => impossible de convertir int en short
var big = 100000;
s.IsBetween<short>(2, big); // big est un int => impossible de convertir int en short
s.IsBetween<short>(2, (short)big); // ça passe, mais à vos risque et périls

Et pour éviter un cast incertain de short en int, on peut faire l'inverse :

Code :

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

((int)s).IsBetween(2, big);

[François DORIN]

A noter qu'il n'existe pas de suffixe pour typer directement un litéral en short. On est donc obligé de passer soit par une conversion, soit par une variable pour avoir un short...