Przekazuje 'this' w wywołaniu metody akceptowanej praktyki w Javie

Czy przekazywanie bieżącego obiektu w wywołaniu metody jest dobrą / złą / akceptowalną praktyką. Jak w:

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);
    }
}

W szczególności, czy linia jest bar.foo(this)akceptowalna?

Nie ma powodu, aby go nie używać, thisjest to bieżąca instancja i jest całkowicie uzasadnione. W rzeczywistości często nie ma prostego sposobu na pominięcie tego.

Więc użyj go.

Ponieważ ciężko jest przekonać, że jest do przyjęcia bez przykładu (negatywna odpowiedź na takie pytanie zawsze jest łatwiejsza do argumentacji), właśnie otworzyłem jedną z najczęściej spotykanych java.langklas, tę Stringi oczywiście znalazłem przykłady takiego użycia np.

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

Szukaj (thisw dużych „zaakceptowanych” projektach, na pewno go znajdziesz.

Nie ma w tym nic złego. To, co NIE jest dobrą praktyką, to robienie tego samego wewnątrz konstruktorów, ponieważ podajesz odniesienie do jeszcze nie w pełni zainicjowanego obiektu.

Jest tu podobny post: Java wycieka to w konstruktorze, gdzie wyjaśniają, dlaczego ta ostatnia jest złą praktyką.

Tak , ale należy uważać na dwie rzeczy

1. Przekazywanie tego gdy obiekt nie został jeszcze skonstruowany (tj. W jego konstruktorze)
2. Przekazanie tego do długo żyjącego obiektu, który utrzyma odniesienie przy życiu i zapobiegnie wyrzucaniu tego obiektu do śmieci.

To zupełnie normalne i całkowicie do przyjęcia.

oznacza to aktualny obiekt. To, co robisz, jest poprawne pod względem systemowym, ale nie widzę takiej potrzeby, jeśli wywołujesz metodę w tej samej klasie.

Przekazywanie bieżącego obiektu w wywołaniu metody jest złą praktyką, jeśli istnieją mniej złożone alternatywy umożliwiające osiągnięcie tego samego zachowania.

Z definicji, powiązanie dwukierunkowe jest tworzone zaraz po thisprzekazaniu z jednego obiektu do drugiego.

Cytując Refaktoryzację Martina Fowlera:

Zmień skojarzenie dwukierunkowe na jednokierunkowe (200)

Dwukierunkowe skojarzenia są przydatne, ale mają swoją cenę. Ceną jest dodatkowa złożoność utrzymania dwukierunkowych połączeń i zapewnienie prawidłowego tworzenia i usuwania obiektów. Skojarzenia dwukierunkowe nie są naturalne dla wielu programistów, dlatego często są źródłem błędów

...

Powinieneś używać skojarzeń dwukierunkowych, kiedy tego potrzebujesz, ale nie wtedy, gdy tego nie robisz. Gdy tylko zobaczysz, że dwukierunkowa asocjacja już nie pociąga za sobą, upuść niepotrzebny koniec.

Tak więc teoretycznie powinniśmy słyszeć dzwonki alarmowe, kiedy stwierdzimy, że musimy spasować thisi naprawdę bardzo starać się wymyślić inne sposoby rozwiązania problemu. Oczywiście są chwile, kiedy w ostateczności warto to zrobić.

Często też konieczne jest tymczasowe uszkodzenie projektu, robiąc „złe praktyki” podczas dłuższej refaktoryzacji kodu w celu ogólnej poprawy. (Jeden krok do tyłu, dwa kroki do przodu).

W praktyce znalazłem mój kod poprawiła się znacznie poprzez unikanie dwukierunkowych linków jak ognia.

Tak. możesz go użyć. jest to po prostu powszechne w programowaniu, aby przejść, thisale są zalety i wady korzystania z tego, ale nie jest to niebezpieczne.

Aby dodać jeszcze jeden przykład, gdzie podanie thisjest poprawne i zgodne z dobrym projektem: wzorzec odwiedzających . We wzorcu projektowym gościa, metodaaccept(Visitor v) jest zwykle implementowana w sposób, który po prostu wywołuje v.visit(this).

Do przyjęcia

Fragment z dokumentów Oracle JAVA:

W ramach metody instancji lub konstruktora jest to odniesienie do bieżącego obiektu - obiektu, którego metoda lub konstruktor jest wywoływana. Możesz odwołać się do dowolnego elementu członkowskiego bieżącego obiektu z poziomu metody wystąpienia lub konstruktora, używając this.

Używanie tego z polem

Najczęstszym powodem używania słowa kluczowego this jest to, że pole jest przesłonięte przez metodę lub parametr konstruktora.

Wszystko w Javie jest przekazywane według wartości. Ale obiekty NIGDY nie są przekazywane do metody!
Kiedy java przekazuje obiekt do metody, najpierw tworzy kopię referencji do obiektu, a nie kopię samego obiektu. Stąd jest to doskonale stosowana metoda w Javie. I najczęściej obserwowane użycie.