Sta passando 'this' in una chiamata al metodo accettata come pratica in java

È buona / cattiva / accettabile passare l'oggetto corrente in una chiamata al metodo. Come in:

public class Bar{
    public Bar(){}

    public void foo(Baz baz){
        //  modify some values of baz
    }
}

public class Baz{
    //constructor omitted

    public void method(){
        Bar bar = new Bar();
        bar.foo(this);
    }
}

Nello specifico, la linea è bar.foo(this)accettabile?

Non c'è motivo per non usarlo, thisè l'istanza corrente ed è perfettamente legittimo da usare. In effetti, spesso non esiste un modo pulito per ometterlo.

Quindi usalo.

Poiché è difficile convincere che sia accettabile senza esempio (una risposta negativa a una domanda del genere è sempre più facile da discutere), ho appena aperto una delle java.langclassi più comuni , Stringquella, e ovviamente ho trovato esempi di questo utilizzo, ad esempio

1084        // Argument is a String
1085        if (cs.equals(this))
1086            return true;

Cerca (thisnei grandi progetti "accettati", non mancherai di trovarlo.

Non c'è niente di sbagliato in questo. Ciò che NON è una buona pratica è fare lo stesso all'interno dei costruttori, perché daresti un riferimento a un oggetto non ancora completamente inizializzato.

C'è una sorta di post simile qui: Java che trapela questo nel costruttore dove danno una spiegazione del perché quest'ultima è una cattiva pratica.

Sì , ma dovresti stare attento a due cose

1. Passando questo quando l'oggetto non è stato ancora costruito (cioè nel suo costruttore)
2. Passando questo a un oggetto di lunga durata, ciò manterrà vivo il riferimento e impedirà a questo oggetto di essere spazzato via.

È perfettamente normale e perfettamente accettabile.

questo sta per l'oggetto corrente. Quello che stai facendo è sistematicamente corretto, ma non vedo la necessità di questo se stai chiamando il metodo nella stessa classe.

È una cattiva pratica passare l'oggetto corrente in una chiamata al metodo se esistono alternative meno complesse per ottenere lo stesso comportamento.

Per definizione, un'associazione bidirezionale viene creata non appena thissi passa da un oggetto all'altro.

Per citare Refactoring, di Martin Fowler:

Cambia associazione bidirezionale in unidirezionale (200)

Le associazioni bidirezionali sono utili, ma hanno un prezzo. Il prezzo è l'ulteriore complessità del mantenimento dei collegamenti a due vie e la garanzia che gli oggetti vengano creati e rimossi correttamente. Le associazioni bidirezionali non sono naturali per molti programmatori, quindi spesso sono fonte di errori

...

Dovresti usare associazioni bidirezionali quando ne hai bisogno, ma non quando non lo fai. Non appena vedi che un'associazione bidirezionale non sta più tirando il suo peso, lascia cadere l'estremità non necessaria.

Quindi, in teoria, dovremmo sentire un campanello d'allarme quando scopriamo di dover passare thise sforzarci di pensare ad altri modi per risolvere il problema in questione. Ci sono, naturalmente, momenti in cui, in ultima istanza, ha senso farlo.

Inoltre è spesso necessario corrompere temporaneamente il progetto, facendo "cattive pratiche", durante un refactoring a lungo termine del codice per un miglioramento generale. (Un passo indietro, due passi avanti).

In pratica ho trovato il mio codice è migliorata in maniera massiccia , evitando i collegamenti bidirezionali come la peste.

Sì. Puoi usarlo È comune nella programmazione passare, ma thisci sono pro e contro nell'usarlo, ma non è pericoloso farlo.

Solo per aggiungere un altro esempio in cui il passaggio thisè corretto e segue un buon design: il modello del visitatore . In Visitor design pattern, il metodo accept(Visitor v)viene tipicamente implementato in un modo che chiama semplicemente v.visit(this).

Accettabile

Snippet dai documenti Oracle JAVA:

All'interno di un metodo di istanza o di un costruttore, questo è un riferimento all'oggetto corrente, l'oggetto il cui metodo o costruttore viene chiamato. Puoi fare riferimento a qualsiasi membro dell'oggetto corrente dall'interno di un metodo di istanza o di un costruttore usando this.

Usandolo con un campo

Il motivo più comune per utilizzare la parola chiave this è perché un campo è ombreggiato da un metodo o da un parametro del costruttore.

Tutto in java viene passato per valore. Ma gli oggetti non vengono MAI passati al metodo!
Quando java passa un oggetto a un metodo, prima fa una copia di un riferimento all'oggetto, non una copia dell'oggetto stesso. Quindi questo è un metodo perfettamente utilizzato in java. E l'uso più comunemente seguito.