Do ponto de vista geral isso é errado, mas preste atenção porque agora falarei do que é o comum de se fazer, depois explico que o DbContext
é especial.
Toda classe que implementa IDisposable
deve chamar o Dispose()
de alguma forma. O using
é a forma mais óbvia dentro de métodos. Quando ele faz parte de um objeto, independente de como ele foi colocado lá (Injeção de Dependência não resolve o problema por si só), esse objeto precisa implementar IDisposable
e algo precisa chamar Dispose()
dessa em algum lugar (alguns frameworks poderiam fazer isso por você). No Dispose()
da sua classe precisa chamar o Dispose()
do DbContext
ou outras classes que implementem IDdisposable
, ou seja, ele é viral.
Quando você não não chama o Dispose()
o objeto fica largado ali até que o garbage collector chame o método para você. Isso é o fim do mundo? Para certos objetos não, e até onde eu sei o DbContext
não é trágico. Mas não é a forma ideal, fica consumindo recursos sem necessidade. Não sei dizer se daria algum problema em algum código concorrente, mas acredito que não, pelo que eu vi por aí.
Se o que você usa chama esse código e morre pouco depois, ou seja, é uma execução efêmera, típica de script, então isso realmente não causará problema, ainda que não seja o ideal. Mas se fora algo que fica rodando por muito tempo, pode ser que em algum momento possa ter algum problema de memória.
Como na maioria das vezes funciona as pessoas acham que está certo.
A grosso modo deveria criar algo assim:
protected IDbContext GameDbContext {get;}
protected virtual void Dispose(bool disposing) {
if (!disposedValue) {
if (disposing) GameDbContext.Dispose();
disposedValue = true;
}
}
public void Dispose => Dispose(true);
Coloquei no GitHub para referência futura.
A recomendação específica para o DbContext
é que ele seja usado só dentro de métodos, então se você o fizer não precisa se preocupar com isso. Se não fizer deveria saber explicar porque está evitando fazer o que é recomendado. Se não sabe, não faça.
O DbContext
gerencia seu recurso (a conexão com o banco de dados) por conta própria, então o que é mais importante de fazer tão logo termine seu uso ele já resolve, por isso deixar o objeto solto não é um grande problema. Então todo acesso que for fazer ele abre e fecha a conexão por você. Se ele tem esse comportamento, por que acha que é tão importante manter o objeto na classe em vez de criá-lo só no momento que vai usar?
Na verdade há quem recomende nem usar o using
com DbContext
. E se não deveria em algo tão específico, na classe também não deveria descartar o objeto. Particularmente não gosto muito de tratar certos objetos com certas características como exceções, mas faz algum sentido. Talvez o DbContext
nem deveria ter implementado IDisposable
(entendo porque foi).
De fato quando usa com Injeção de Dependência é comum que o objeto nem seja descartado (depende de como usa a DI). E não é o fim do mundo.
Eu consigo entender a motivação de usar assim e não me parece que terá problemas. Mas não é errado fazer o certo :) Eu só não saberia dizer oque é mais eficiente (ainda que isso não mude muito).
Também já vi quem fala o oposto de tudo isto, eu não vou confirmar, nem negar, tome suas próprias conclusões lendo o artigo. Lembre-se que essa é uma posição diferente até do que a equipe do Entity Framework recomenda. Mas faz algum sentido.