Is it a good practice to have logger as a singleton?

Question

I had a habit to pass logger to constructor, like:

public class OrderService : IOrderService {
     public OrderService(ILogger logger) {
     }
}
         


        

Answer 1

I put a logger instance in my dependency injection container, which then injects the logger into the classes which need one.