SingleOrDefault () вызывает исключение более чем для одного элемента

Single и SingleOrDefault предназначены для выдачи, если в последовательности существует более одного совпадения. Следствием этого является то, что вся последовательность должна быть повторена до завершения. Не похоже, что это то, что вы хотите. Попробуйте вместо этого FirstOrDefault:

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

Это (как правило) будет работать лучше, потому что будет выполнено, как только будет найдено совпадение.

Конечно, если вы действительно хотите сохранить более одного элемента, предложение Where было бы более подходящим:

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

В качестве альтернативы, если вы хотите получить предмет только при наличии ровно одного совпадения и не хотите бросать, когда их больше одного, это можно легко сделать. Я создал для этого в своем проекте метод расширения:

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);
    }
}

Если вам нужен только элемент first, используйте вместо него FirstOrDefault.

По сути, вот варианты с точки зрения действительных результатов (т.е. куда вы не хотите бросать) и что использовать:

• Ровно один: Single
• Один или ноль: SingleOrDefault
• Один или несколько: First
• Ноль или больше: FirstOrDefault

(Также доступны ElementAt и ElementAtOrDefault, Last и LastOrDefault.)

Я обнаружил, что мне нужно поведение возврата значения по умолчанию, если нет ровно одного элемента (т. Е. Ноль, два или более) чаще, чем мне нужно нормальное поведение SingleOrDefault, поэтому вот моя адаптированная версия Ответ Питера ван Гинкеля:

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();
    }
}

Я пропустил проверку нулевого аргумента, потому что я не против полагаться на вызовы Take и Where для выдачи исключений, когда аргументы равны нулю, но вы можете подумать иначе.

SingleOrDefault предполагает, что вы ожидаете от вашего запроса 0 или 1 результат. Если у вас их больше 1, то с вашими данными или запросом что-то не так.

Если вы ожидаете более одного результата и хотите получить только первый результат, следует использовать FirstOrDefault.

Single означает, что вы ожидаете, что в последовательности будет один элемент. SingleOrDefault означает, что вы ожидаете, что в последовательности будет один или ноль элементов. Это следует использовать, когда вы хотите знать, что есть один (или ноль), и вы хотите, чтобы он вылетал, когда возвращалось более одного.

Если вам нужен только один, используйте First (или FirstOrDefault), как предложено выше, но убедитесь, что вы правильно упорядочили данные.

Если вы используете SingleOrDefault, если условие удовлетворить больше, чем результат, это вызовет ошибку.

вы можете добиться своего результата, используя FirstOrDefault