Is circular reference between objects a bad practice?
circular-reference, javascript
Solution
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.
Problem
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 with is the following ``` var Validator = function(model) { this.model = model; }; var Model = function() { this._attributes = {}; this.validator = new Validator(this); }; var model = new Model(); ``` This code creates a circular reference between those 2 objects. Is this a bad practice that will cause memory leaks? Any other ideas on how to implement it? P.S. I have seen such circular references between objects in Angular.js scopes.