C # linq возможны рекомендации по множественному перечислению

Question

C # linq возможны рекомендации по множественному перечислению

1

Иногда я использую конструкции LINQ в своем источнике С#. Я использую VS 2010 с ReSharper. Теперь я получаю предупреждение "Возможное множественное перечисление IEnumerable" от ReSharper.

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

IEnumerable<String> codesMatching = from c in codes where conditions select c;
String theCode = null;
if (codesMatching.Any())
{
  theCode = codesMatching.First();
}
if ((theCode == null) || (codesMatching.Count() != 1))
{
  throw new Exception("Matching code either not found or is not unique.");
}
// OK - do something with theCode.

Вопрос. Должен ли я сначала сохранить результат выражения LINQ в списке? (Я уверен, что он не вернется больше, чем несколько строк - скажем, максимум 10).

Любые намеки оценили.

Спасибо, Павел

Pavel Foltyn 20 май 2014, в 11:42

Источник

Теги:

c#

linq

3 ответа

2

Поскольку вы хотите проверить, является ли ваше условие уникальным, вы можете попробовать это (и да, вы должны сохранить результат):

var codes = (from c in codes where conditions select c).Take(2).ToArray();
if (codes.Length != 1)
{
  throw new Exception("Matching code either not found or is not unique.");
}

var code = codes[0];

João Simões 20 май 2014, в 08:10

1

Зачем Take(2) ?
Magnus 20 май 2014, в 09:51
2

Я предполагаю, чтобы соответствовать, что есть больше чем один элемент, удовлетворяющий условиям
Sergey Litvinov 20 май 2014, в 09:52
0

Потому что вы предотвращаете использование try-catch, что более эффективно, если он ожидает, что условие будет возвращать более одного значения часто.
João Simões 20 май 2014, в 09:54
0

Спасибо, Жуан. Ваш код намного проще и понятнее.
Pavel Foltyn 20 май 2014, в 12:58

Показать ещё 2 комментария

1

Финализация, перечислимая с помощью .ToList()/.ToArray(), избавится от предупреждения, но для того, чтобы понять, лучше ли это, чем несколько перечислений или нет, будет зависеть от реализации codes и conditions. .Any() и .First() являются ленивыми примитивами и не будут выполняться за первым элементом, а .Count() может вообще не пострадать, поэтому преобразование в список может оказаться более расточительным, чем получение нового счетчика.

Ilya Kozhevnikov 20 май 2014, в 08:40

Ещё вопросы

Я предполагаю, чтобы соответствовать, что есть больше чем один элемент, удовлетворяющий условиям
Потому что вы предотвращаете использование try-catch, что более эффективно, если он ожидает, что условие будет возвращать более одного значения часто.
Спасибо, Жуан. Ваш код намного проще и понятнее.

Sergey Litvinov · Accepted Answer · 2014-05-20T08-13-00.000Z

Да, вам нужно сохранить результат как List\Array, а затем использовать его. В этом случае он не будет перечислять его пару раз.

В вашем случае, если вам нужно быть уверенным, что есть только один элемент, удовлетворяющий условию, вы можете использовать Single - если будет более одного элемента, удовлетворяющего условиям, он будет генерировать исключение. Если вообще не будет предметов, это тоже исключение.

И ваш код будет проще:

string theCode = (from c in codes where conditions select c).Single();

Но в этом случае вы не можете изменить текст исключения, или вам нужно обернуть его в собственный блок try\catch и перестроить его с помощью настраиваемого текста \exception

Но если есть ровно один элемент, и этот элемент равен null , Single не сгенерирует исключение, но код OP сработает. Обычно это не проблема, но стоит отметить, что это не обязательно замена.
Вы также можете прочитать ReadOnlyCollection <T>, если хотите только прочитать результаты вашего запроса.
@hvd, да, согласен. Но странно, если в исходных conditions есть null предметы. Но да, это может быть.
Я не знал о Single (). Хорошее решение. Я не ожидаю, что нулевые значения будут в результате. Так что использование Single () хорошо для меня. Благодарю.