Solucionado
É "certo" a classe DataService ter métodos de acesso a banco de dados relacionados a "todos" os modelos da aplicação? Não seria melhor termos uma folder DAO(ou qualquer outro nome) com uma classe para cada model e essas classes fazendo as requisições ao banco através do EF? Sei que a pergunta é mais relacionada a design patterns do que MVC core, mas gostaria de ter outras opiniões.
Olá, Diogo
Não seria melhor termos uma folder DAO(ou qualquer outro nome) com uma classe para cada model e essas classes fazendo as requisições ao banco através do EF?
Sem dúvida, o uso do padrão DAO é melhor para organização do código e aumentar a escalabilidade da aplicação.
Porém, não estamos pecando (muito) quanto à separação de responsabilidades, já que temos uma classe DataServices dedicada a fornecer serviços de dados. Acho que não é tão trabalhoso implementar um padrão DAO a partir do código do curso. Se fizer isso, melhor ainda.
Você pode ver uma discussão interessante sobre isso no Stack Overflow:
Obrigado.