Discussion :

[bilou_12]

Bonjour,

Maintenant, je cherche à coder la méthode d'extension qui cherche dans une liste de type T (car on sait pas encore quel type) puis renvoie le 1er élément de la liste si la liste contient qqe chose ou crée un élément sinon.

Pour le moment j'ai codé ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public static IEnumerable<T> FirstOrNew(this IEnumerable<T> Lists, Func<T, var> selector)
        {
            if (Lists == null)
            {
                return new T; 
            }
            else
            {
                return Lists.First(); 
            }
        }

J'ai encore des problèmes car je vois pas comment coder le new T. J'imagine que c'est pas bon, confirmé par du rouge dessous, mais je vois pas la syntaxe pour le remplacer étant donné qu'on ne connait pas encore son type.

Quelqu'un pourrait me filer un coup de main?

Merci d'avance!

[tomlev]

C'est presque ça, mais il faut déclarer T comme un paramètre de type générique, et lui mettre la contrainte new()

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
        public static IEnumerable<T> FirstOrNew<T>(this IEnumerable<T> list)
            where T : new()
        {
            if (list == null)
            {
                return new T(); 
            }
            else
            {
                return list.First(); 
            }
        }

(j'ai enlevé le paramètre selector vu que tu ne l'utilisais pas...)

[tomlev]

Par contre attention : avec ce code, ça va planter si la liste n'est pas nulle mais qu'elle est vide...

[bilou_12]

Merci Tomlev!!!

Par contre, liste nulle est différent de liste vide?
Quelle est la syntaxe pour dire qu'une liste est vide?

D'ailleurs sur le bout de programme que tu m'as donné, on me demande un cast sur new T()?

[StringBuilder]

list.Length == 0 ou list.Count == 0 si je ne m'abuse

[StringBuilder]

Par contre, j'ai une question...

Si j'ai une classe "Person" avec pour unique constructeur "Person(int id, string name)"

Comment ça peut marcher ?

Il va automatiquement créer un new Person(0, string.Empty) ? (Valeurs par défaut de ces deux types de valeur lorsqu'ils ne sont pas initialisés si je ne m'abuse.

[tomlev]

Par contre, liste nulle est différent de liste vide?

une "liste nulle" (c'est un abus de langage), c'est une absence de liste (l'objet liste n'existe pas)

Une liste vide, c'est un objet liste qui existe, mais qui contient 0 élément.

Quelle est la syntaxe pour dire qu'une liste est vide?

cf. la réponse de StringBuilder, mais je vais y revenir...

D'ailleurs sur le bout de programme que tu m'as donné, on me demande un cast sur new T()?

En fait c'est le type de retour qui n'est pas bon : si tu veux récupérer un élément de la liste, le type de retour devrait être T et non pas IEnumerable<T>

list.Length == 0 ou list.Count == 0 si je ne m'abuse

.Count en effet pour une liste, sauf qu'en l'occurrence, c'est un IEnumerable<T>, pas une IList<T>...

La solution pour tester si un IEnumerable<T> est vide est d'utiliser la méthode d'extension Any() :

Code :

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

1
2
3
4
if (!list.Any())
{
    // liste vide
}

Le problème, c'est qu'on veut ensuite récupérer le premier élément si la liste n'est pas vide. Et si on appelle Any(), puis First(), on déclenche 2 fois l'énumération de la collection. Si c'est par exemple une liste en mémoire, ce n'est pas gênant, mais si c'est par exemple une requête Linq to Entities, c'est plus gênant car ça va réexécuter la requête SQL correspondante.

On ne peut pas non plus utiliser FirstOrDefault() pour vérifier que la liste est vide : si T est un type valeur, FirstOrDefault() ne renverra pas null, donc on ne pourra pas savoir si la liste était vide ou si la valeur renvoyée fait vraiment partie de la liste.

La solution est de manipuler directement l'énumérateur :

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
        public static T FirstOrNew<T>(this IEnumerable<T> list)
            where T : new()
        {
            if (list != null)
            {
                using (var enumerator = list.GetEnumerator())
                {
                    if (enumerator.MoveNext())
                    {
                        return enumerator.Current;
                    }
                }
            }
            return new T(); 
        }

Si j'ai une classe "Person" avec pour unique constructeur "Person(int id, string name)"

Comment ça peut marcher ?

Il va automatiquement créer un new Person(0, string.Empty) ? (Valeurs par défaut de ces deux types de valeur lorsqu'ils ne sont pas initialisés si je ne m'abuse.

Non, la contrainte new() impose l'existence d'un constructeur public sans paramètre. Il n'est malheureusement pas possible de spécifier les paramètres dans la contrainte.

cf. cette discussion pour plus de détails

(et string est un type référence, donc sa valeur par défaut n'est pas String.Empty mais null...)

[Nathanael Marchand]

Connaissez vous DefaultIfEmpty() ?

Code :

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

1
2
3
4
5
6
7
8
9
 
public static T FirstOrNew<T>(this IEnumerable<T> list, T defaultValue)
{
    return list.DefaultIfEmpty(defaultValue).First();
}
public static T FirstOrNew<T>(this IEnumerable<T> list, Func<T,bool> predicate, T defaultValue)
{
    return list.Where(predicate).DefaultIfEmpty(defaultValue).First();
}

ou

Code :

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

1
2
3
4
5
6
7
8
9
 
public static T FirstOrNew<T>(this IEnumerable<T> list) where T:new()
{
    return list.DefaultIfEmpty(new T()).First();
}
public static T FirstOrNew<T>(this IEnumerable<T> list, Func<T,bool> predicate) where T:new()
{
    return list.Where(predicate).DefaultIfEmpty(new T()).First();
}

ou encore:

Code :

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

1
2
3
4
5
6
7
8
9
 
public static T FirstOrNew<T>(this IEnumerable<T> list, Func<T> factory)
{
    return list.DefaultIfEmpty(factory()).First();
}
public static T FirstOrNew<T>(this IEnumerable<T> list, Func<T,bool> predicate, Func<T> factory)
{
    return list.Where(predicate).DefaultIfEmpty(factory()).First();
}

Les possibiltés sont infinies

[tomlev]

Connaissez vous DefaultIfEmpty() ?

Oui mais DefaultIfEmpty ça colle pas par rapport à ce qu'il veut faire :
- ça n'accepte pas une séquence nulle
- ça renvoie la valeur par défaut (null pour un type référence), alors que le PO veut créer une instance de T

[Nathanael Marchand]

Oui mais DefaultIfEmpty ça colle pas par rapport à ce qu'il veut faire :
- ça n'accepte pas une séquence nulle
- ça renvoie la valeur par défaut (null pour un type référence), alors que le PO veut créer une instance de T

Ben si ca colle, ca renvoit null si tu lui dis de renvoyer null, si tu lui précises quoi renvoyer ca marche. Dans mon deuxieme exemple ca le fait exactement, dans le troisième tu peux même passer une factory de T
Et pour la séquence nulle, aucune méthode linq ne marche si l'enumerable en question est nul. Ce qui est logique!
Ca serait assez perturbant de pouvoir faire ceci (même si méthode d'extension=sucre syntaxique):

Code :

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

((MonType)null).MaMethode();

[tomlev]

Et pour la séquence nulle, aucune méthode linq ne marche si l'enumerable en question est nul. Ce qui est logique!

Bah ça semble logique dans la mesure où les méthodes d'extension s'utilisent comme des méthodes d'instance, mais justement ce n'est pas des méthodes d'instance... je profite souvent de ça pour pouvoir justement les appeler sur des valeurs qui peuvent être nulles. Par exemple j'ai une méthode EmptyIfNull(), qui fait précisément ce que son nom indique...

Ca serait assez perturbant de pouvoir faire ceci (même si méthode d'extension=sucre syntaxique):

Code :

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

((MonType)null).MaMethode();

Apparemment ça perturbe un peu le compilateur en tous cas, il génère un warning sur ce code...
https://connect.microsoft.com/Visual...ension-methods
(enfin pas exactement sur ce code, mais si tu fais default(MonType).MaMethode())