I personally consider the example I gave as being a nicer way of doing dependency injection than the more verbose constructor based injection. But in general y

I personally consider the example I gave as being a nicer way of doing dependency injection than the more verbose constructor based injection. But in general yes; cyclic dependencies are a bad idea.