Il riferimento circolare tra gli oggetti è una cattiva pratica?

Question

Ho un modello che "porterà" (Model.validator) un'istanza di validatore con esso, e ho bisogno che il validatore abbia accesso agli attributi del modello. Quindi, quello che ho trovato è il seguente

var Validator = function(model) { 
    this.model = model; 
}; 

var Model = function() { 
    this._attributes = {}; 
    this.validator = new Validator(this); 
}; 

var model = new Model(); 

Questo codice crea un riferimento circolare tra questi 2 oggetti. È una cattiva pratica che causerà perdite di memoria? Qualche altra idea su come implementarla?

P.S. Ho visto riferimenti circolari simili tra oggetti negli ambiti Angular.js.

Answer 1

Questo tipo di codice non causerà perdite di memoria con i browser di oggi; come mentioned on MDN tutti i principali browser sono stati spediti con GC mark-and-sweep (che possono gestire i cicli semplicemente bene) per un po 'di tempo (ad esempio, Firefox stesso ha avuto un collettore di cicli dalla versione 3).

Da un punto di vista architettonico, questo tipo di codice introduce un accoppiamento moderatamente stretto tra i due oggetti (se uno cambia anche in modo minore, l'altro deve essere rivisto per determinare se deve essere modificato) e di conseguenza essere evitato se possibile Ma non c'è nulla di intrinsecamente sbagliato in questo.

Answer 2

Non ci saranno problemi ne sono sicuro. I parser JS della maggior parte dei browser possono funzionare con dipendenze ciclo durante la raccolta dei dati inutili. Non ci sono più problemi potenziali qui.

Answer 3

Non sarà un problema per la garbage collection: qualsiasi nuovo Garbage Collector (> IE6) gestirà i riferimenti circolari bene!

Potrebbe essere un problema se si eseguono funzioni ricorsive o si stampa l'oggetto.

Quindi la risposta è: non è un problema a meno che non si fallisce voi :-)