O motivo é que, ao fazer menção a range
ou a qualquer método de extensão dele, você estará chamando GetEnumerator()
e iterando de alguma forma sobre ele.
Para listas pequenas como esta, não há muita diferença de performance. A coisa muda com uma enumeração de milhares de registros.
Vamos fundo na dor. Imagine o seguinte:
var range = Enumerable.Range(0, 100000);
Ao atribuir a variável, você já criou uma enumeração iterável por 100000 registros. Ao chamar:
var primeiro = range.First();
Você está chamando GetEnumerator()
de novo, pois uma enumeração não é uma lista concreta. Tanto que isto dá um erro de compilação:
public List<MeuObjeto>()
{
for (int i = 0; i < 10; i++) {
yield return MeuObjeto();
}
}
E isto funciona:
public IEnumerable<MeuObjeto>()
{
for (int i = 0; i < 10; i++) {
yield return MeuObjeto();
}
}
Imagine agora o custo desta última função sobre 100000 registros. E que no seu caso, ela é executada 2 vezes.
É por isso que o ReSharper sugere que você concretize a lista. Ele não sabe o que tem por trás, mas supõe o pior caso sempre.
IEnumerable<T>
.