De início a pergunta não estava clara, agora estou destacando esta parte depois da edição porque o problema é este. A segunda parte fica como complemento para as pessoas entenderem melhor o LINQ.
Busca em banco de dados
Se o dado está em um banco de dados (a pergunta não fala nada sobre isto) a e expressão será traduzia para ser executada no banco de dados e dependendo do provedor utilizado o DateTime.Now
será traduzido para uma expressão que ele reconhece e será executado lá no banco de dados que pode estar com fuso horário diferente da aplicação. Nada disso é garantido, mas costuma acontecer. Quando pega uma variável da aplicação essa tradução não é possível e aí ele compara com o dado fixo da do momento que é mandado para o banco de dados.
Até acho que isto é um outro erro porque os horários deveriam ser todos UTC e nunca ter problema de fuso, mas não é o foco da pergunta. Provavelmente o erro é de modelagem que adotou fuso horário no armazenamento.
Outro erro comum com LINQ
O primeiro está usando uma expressão que compara um dado da coleção de dados sendo avaliada com o horário do momento. Conforme vai passando o tempo esse momento muda. Em cada verificação de um elemento da coleção o valor de Datetime.Now
muda, então neste exemplo tende a filtrar um pouco mais por ter uma valor móvel, ele vai estreitando.
O segundo pega o momento atual da execução do código e guarda em uma variável. Esse valor é fixo, ele não muda mais. Aí quando percorre a coleção de dados a comparação está sendo feita sempre com o mesmo valor, o valor não estreita, não muda durante a execução.
Mas o LINQ ainda tem uma característica que agrava mais o problema. Essa tecnologia usa uma técnica chamada lazy evaluation. Quando você executa esta linha de código nada está sendo executado de verdade. Somente quando precisa do dado ou materializa o coleção de dados é que a execução será feita. Isto pode ser feito segundos, minutos ou horas depois em alguns casos. Inclusive o tempo entre verificar um elemento e outro pode demorar bastante. Não é tão comum, mas pode acontecer.
Dá para perceber como os resultados podem ser bem diferentes? Um deles sempre usará o tempo do momento da efetiva execução de um elemento, porque você usou um código que manda pegar um tempo novo cada vez que passa por ele. O outro pega um tempo e guarda, aí toda vez que avaliar um elemento mesmo que muito tempo depois o tempo a ser usado é sempre o mesmo, é aquele que pegou logo no início da execução.
Eu gosto do LINQ, mas vejo um problema nele porque as pessoas e não entendem o que ele é, acham que há mágica dentro dele e que não precisar entender nada dessa "mágica" toda. Sem entender profundamente o LINQ eu acho que as pessoas não deveriam usá-lo.
Se você escrevesse esses códigos na mão talvez desse pra perceber melhor porque acontece isso:
foreach (var c in Lista()) {
if (c.Validade > DateTime.Now) {
//faz alguma coisa
}
//talvez chama algo aqui demorado
}
E
var agora = DateTIme.Now;
foreach (var c in Lista()) {
if (c.Validade > agora) {
//faz alguma coisa, poderia montar uma outra lista, mas seria a materialização
}
//talvez chama algo aqui demorado
}
Coloquei no GitHub para referência futura.
LINQ não é simples
E um dos problemas que mais vejo as pessoas fazerem com LINQ é usar um ToList()
para materializar a coleção de dados. Quando a pessoa faz isso provavelmente ela não deveria usar o LINQ, a vantagem desta técnica é justamente não materializar sem necessidade e se tiver necessidade sempre, então o LINQ não ajuda tanto quanto poderia. No caso você provavelmente está acessando a coleção de dados sem materializar a toa e isto é bom, mas também é o causador da diferença.
Conclusão
Qual dá o resultado esperado? Vá com ele. Eu acho que o segundo é melhor na maioria dos casos, mas vejo cenário onde o primeiro é mais útil. Ou conserte o erro de modelagem.
apenas que possuem a validade da data e hora atual + 3h
o problema é a diferença de horários entre onde o código é executado e a base de dados. Experimente usarDatetime.UtcNow
.