C # - цикл foreach - хорошая практика

Question

Хорошо это или плохо:

foreach (Match match in serverNameRegex.Matches(loginPage))
{
 ....
}

или я должен использовать это для лучшей скорости:

MatchCollection matches = serverNameRegex.Matches(loginPage);
foreach (Match match in matches)
{
  ...            
} 

Answer 1

Пока MatchCollection не может быть нулевым, я бы сказал, что это ваш выбор.

Но если он окажется нулевым, и вы не проверите его заранее, вы столкнетесь с NullReferenceException.

Answer 2

Я предпочитаю последнее, хотя между ними нет никакой разницы. Потому что второй кажется более понятным, и вы можете легко добавить больше в будущем.

MatchCollection matches = serverNameRegex.Matches(loginPage);
foreach (Match match in matches)
{

}
//do something to matches in the future

Answer 3

Второй может быть предпочтительнее, так как:

• Вы можете легко послать точку останова после нее, чтобы отследить ее (отладка)
• Если вам нужно внести изменения, вы также не изменяете цикл (который не должен быть затронут).
• Легче читать
• Обязательно, если вы хотите добавить проверки перед циклом

Answer 4

Нет никакой разницы в скорости вообще. Это только вопрос стиля.

В обоих случаях IEnumerator будет создан методом GetEnumerator(). Это происходит только один раз. В каждой последующей записи цикла будет вызываться только метод MoveNext().

Answer 5

Зависит от того, понадобится ли вам matches в других местах кода.Я думаю, что может быть более чистым использовать

foreach (Match match in serverNameRegex.Matches(loginPage))
{
}

, потому что он очистит переменную после завершения цикла

Answer 6

в этом нет различий, но я говорю: Второй - более понятный, понятный и легко читаемый, чем первый.

Для подробной информации используйте деаасемблер или рефлятор.

и для второго полезно, если вы хотите использовать matches collection после цикла foreach.

Answer 7

Существует разница в скорости между двумя и первым:

 foreach (Match match in serverNameRegex.Matches(loginPage))
 {
      ....
 }

быстрее.

Причина в первом - serverNameRegex.Matches (loginPage) оценивается один раз, и цикл for уже знает о каждом значении совпадения, тогда как в:

 MatchCollection matches = serverNameRegex.Matches(loginPage);
 foreach (Match match in matches)
 {
    ...            
 } 

цикл должен оценивать коллекцию совпадений на каждой итерации.