SingleOrDefault () renvoie une exception sur plus d'un élément

J'obtiens une exception chaque fois que je chercher comme ça

Feature f = o.Features.SingleOrDefault(e => e.LinkName == PageLink);

parce que cela peut renvoyer un ou plus d'un élément. Qu'est-ce que l'approche alternative que je peux utiliser pour résoudre ce problème?

source d'informationauteur Fraz Sundal

  1. 26

    Unique et SingleOrDefault sont conçus pour le lancer si plus d'une correspondance existe dans la séquence. Une conséquence de cela est que l'ensemble de la séquence doit être réitéré avant la fin. Il n'a pas l'impression que c'est ce que vous voulez. Essayez FirstOrDefault à la place:

    Feature f = o.Features
    .FirstOrDefault(e => e.vcr_LinkName == PageLink && e.bit_Activate == true);

    Ce sera (généralement) plus performants, car il se termine dès qu'une correspondance est trouvée.

    Bien sûr, si vous voulez les conserver plus d'un élément, une clause where serait plus approprié:

    IEnumerable<Feature> fs = o.Features
    .Where(e => e.vcr_LinkName == PageLink && e.bit_Activate == true);
  2. 16

    Alternativement, si vous voulez seulement le point quand il y a exactement un match et ne veux pas jeter quand il y a plus d'un, cela peut être facilement accompli. J'ai créé une méthode d'extension dans mon projet:

    public static class QueryableExtensions
{
    public static TSource SingleWhenOnly<TSource>(this IQueryable<TSource> source)
    {
        if (source == null)
            throw new ArgumentNullException("source");

        var results = source.Take(2).ToArray();

        return results.Length == 1 ? results[0] : default(TSource);
    }

    public static TSource SingleWhenOnly<TSource>(this IQueryable<TSource> source, Expression<Func<TSource, bool>> predicate)
    {
        if (source == null)
            throw new ArgumentNullException("source");
        if (predicate == null)
            throw new ArgumentNullException("predicate");

        var results = source.Where(predicate).Take(2).ToArray();

        return results.Length == 1 ? results[0] : default(TSource);
    }
}
  3. 12

    Si vous voulez seulement le première élément, utilisez FirstOrDefault à la place.

    En gros, voici les options en termes de résultats valides (c'est à dire là où vous ne voulez pas pour les jeter) et ce à utiliser:

    • Exactement: Single
    • Un ou zéro: SingleOrDefault
    • Un ou plusieurs: First
    • Zéro ou plus: FirstOrDefault

    (ElementAt et ElementAtOrDefaultLast et LastOrDefault sont également disponibles.)

  4. 4

    SingleOrDefault suggère que vous vous attendez à 0 ou 1 les résultats de votre requête. Si vous avez plus de 1 alors il ya quelque chose de mal avec votre données ou d'une requête.

    Si vous vous attendez à plus de 1 suite et que vous souhaitez uniquement le premier, puis FirstOrDefault doit être utilisé.

  5. 3

    J'ai trouvé, j'ai besoin de le comportement de retourner une valeur par défaut s'il n'est pas exactement un seul élément (c'est à dire zéro, deux, ou plus) plus souvent que j'ai besoin de la normale SingleOrDefault comportement, alors voici ma version adaptée de Pieter van Ginkel réponse:

    public static class LinqExtensions
{
    ///<summary>
    ///Returns the only element of a sequence, or a default value if the sequence is empty or contains more than one element.
    ///</summary>
    public static TSource SingleOrDefaultIfMultiple<TSource>(this IEnumerable<TSource> source)
    {
        var elements = source.Take(2).ToArray();

        return (elements.Length == 1) ? elements[0] : default(TSource);
    }

    ///<summary>
    ///Returns the only element of a sequence, or a default value if the sequence is empty or contains more than one element.
    ///</summary>
    public static TSource SingleOrDefaultIfMultiple<TSource>(this IEnumerable<TSource> source, Func<TSource, bool> predicate)
    {
        return source.Where(predicate).SingleOrDefaultIfMultiple();
    }

    ///<summary>
    ///Returns the only element of a sequence, or a default value if the sequence is empty or contains more than one element.
    ///</summary>
    public static TSource SingleOrDefaultIfMultiple<TSource>(this IQueryable<TSource> source)
    {
        var elements = source.Take(2).ToArray();

        return (elements.Length == 1) ? elements[0] : default(TSource);
    }

    ///<summary>
    ///Returns the only element of a sequence, or a default value if the sequence is empty or contains more than one element.
    ///</summary>
    public static TSource SingleOrDefaultIfMultiple<TSource>(this IQueryable<TSource> source, Expression<Func<TSource, bool>> predicate)
    {
        return source.Where(predicate).SingleOrDefaultIfMultiple();
    }
}

    J'ai omis de l'argument null vérifie parce que je suis OK avec en s'appuyant sur la Take et Where appels à lancer des exceptions lorsque les arguments sont nuls, mais vous pourriez vous sentir autrement.

  6. 2

    Single signifie que vous vous attendez à être un élément dans la séquence.
    SingleOrDefault signifie que vous vous attendez à être zéro ou un des éléments dans la séquence.
    Cette fonction doit être utilisée lorsque vous voulez savoir il y a un (ou zéro) et que vous souhaitez bloquer lorsque plus d'un, elle revint.

    Si vous êtes après une, utilisation First (ou FirstOrDefault) comme suggéré ci-dessus, mais assurez-vous de commander correctement les données.

  7. 1

    Si vous utilisez SingleOrDefault si la condition de satisfaire plus de résultat qu'il va jeter erreur.

    vous pouvez atteindre votre résultat à l'aide de FirstOrDefault