Constructor injection is good when you want to force the caller to pass something and not default.
In practice, I tend to use a mix of the two. I use setter injection mainly when:
- Injecting more than one thing
- I don't control the constructor
- I'm only injecting for testing and the "real code" doesn't need to inject.