Is circular reference between objects a bad practice?

Question

I have a Model that will \"carry\" (Model.validator) a validator instance with it, and I need the Validator to have access to the Model\'s attributes. So, what I have come up wi

Answer 1

This kind of code will not cause memory leaks with today's browsers; as mentioned on MDN all major browsers have been shipping with mark-and-sweep GCs (that can handle cycles just fine) for some time now (e.g. Firefox itself has had a cycle collector since version 3).

From an architectural standpoint, this kind of code introduces moderately tight coupling between the two objects (if one changes in even a minor way, the other needs to be reviewed to determine if it needs to change as well) and should consequently be avoided if possible. But there is nothing inherently wrong with it.