Question

When setting a property via Dependency Injection (in this case a logging dependancy) is it “normal” to make the getter protected so that objects outside of the dependant object cannot call that property?

i.e.

public ILogger Logger { protected get; set; }

vs

public ILogger Logger { get; set; }

(I’m actually using null object pattern in these but that’s beside the point).

I dont want to use constructor injection and yet via public property injection, it’s not “right” for any other object to call this object’s logger.

So above is what im going with but it does seem weird.

Answer 1

Why do you need a getter for the logger at all?

Objects that you inject should be implementation details of the thing that they are injected into. So in general you should just have a setter for such items.