Acredito que você esteja confundindo as 2 formas e fazer injeção de dependências.
Parâmetro no construtor
Como você mesmo citou, colocando um parâmetro no construtor é o de prata receber uma instância do repositório que você espera. Para tornar essa instância disponível para um método da classe normalmente criamos um campo na classe para guardar esse parâmetro recebido no construtor.
Essa forma de DI é possível a mais comum em aplicações ASP.net:
public class MyController : Controller // ou ApiController
{
// campo para guardar o repositório injetado
// usando "readonly" somente prata deixar explícito
// que ele só será setado uma vez no construtor
private readonly IMyRepository myRepo;
public MyController(IMyRepository repo)
{
// o construtor sempre será executado então podermos considerar
// que myRepo sempre terá o valor esperado quando qualquer um
// dos métodos é executado.
this.myRepo = repo;
}
public List<string> GetRecords()
{
return this.myRepo.GetRecords().ToList();
}
}
Parâmetro na action (método)
Actions, que são métodos públicos de um controller, também podem receber instâncias por DI diretamente com o atributo [FromService]
. É importante notar que actions não são normalmente invocadas via código, mas sim pelo framework quando uma requisição é feita para uma URl específica. Ou seja, a responsabilidade de invocar actions é do framework e ele sabe como lidar com as dependências via atributos.
Injeção de dependências fora do controller
Outra coisa que você comentou é isolar a regra de negócio do controller. A grande maioria das bibliotecas de injeção de dependência pra .Net vai automaticamente injetar as dependências (que tiverem sido registradas) para a "árvore de objetos" que forem criados.
Um exemplo simples, imagina que você quer executar algumas validações antes de adicionar um novo objeto no seu banco de dados. Você poderia fazer algo como:
public class PedidosController : ApiController
{
private readonly IPedidoService pedidoService;
public PedidosController(IPedidoService pedidoService)
{
// Se pedidoService for null, gera exceção
this.pedidoService = pedidoService ?? throw new ArgumentNullException(nameof(pedidoService));
}
[HttpPost]
public ActionResult Finalizar(PedidoDto pedidoDto)
{
var pedido = new Pedido { // extrai propriedades de "pedidoDto" };
this.pedidoService.FinalizarPedido(pedido);
}
}
public class PedidoService : IPedidoService
{
private readonly IEstoqueRepository estoqueRepository;
private readonly IDescontoService descontoService;
private readonly IPedidoRepository pedidoRepository;
// Essa classe será instanciado pelo sistema de DI com as dependências necessárias
public PedidoService(IPedidoRepository pedidoRepository, IEstoqueRepository estoqueRepository, IDescontoService descontoService)
{
this.pedidoRepository = pedidoRepository ?? throw new ArgumentNullException(nameof(pedidoRepository));
this.estoqueRepository = estoqueRepository ?? throw new ArgumentNullException(nameof(estoqueRepository));
this.descontoService = descontoService ?? throw new ArgumentNullException(nameof(descontoService));
}
public void FinalizarPedido(Pedido pedido)
{
// Aqui você pode usar uma regra de negócio mais complexa,
// e o melhor de duto é que essa código pode ser facilmente executado de qualquer lugar, não somente de um controller.
this.descontoService.ValidarDescontos(pedido);
if (!this.estoqueRepositor.IsProdutoEmEstoque(pedido.ProdutoId))
{
// Colocar em espera ou mandar e-mail?
}
else
{
var pedidoCriado = this.pedidoRepository.CriarPedido(pedido);
// enviar e-mail com numero do pedido
}
}
}
Update para o seu caso específico
Baseado na discussão nos comentários, o WsEstoque
é um serviço que deveria ser injetado no seu controller diretamente pelo sistema de injeção de dependências. Para isso você precisa registrar essa dependência junto com as outras:
// Normalmente interfaces são geradas para desacoplar consumidores da implementação de um dependência
// e também para facilitar na criação de testes unitários.
services.AddScoped<IWsEstoque, WsEstoque>();
É comum ter dezenas de serviços sendo registrados dessa forma. Sistemas mais complexos utilizam bibliotecas mais robustas de injeção de dependências para facilitar no processo de registrar dependências e também para gerar "módulos" separando diferente áreas do sistema de forma (exemplo: Autofac, Simple Injector).
Existe também uma biblioteca para auxiliar a registrar várias dependências de uma só vez com o sistema de DI padrão do .net core, o NetCoreAutoRegisterDi. Com ele você pode registrar todas as classes no seu assembly/projeto com um único comando:
// Registra todas as classes públicas onde o nome começa com
// a string "Ws" para a sua respectiva interface.
service.RegisterAssemblyPublicNonGenericClasses()
.Where(c => c.Name.StartsWith("Ws"))
.AsPublicImplementedInterfaces();