Передавання 'цього' у прийнятій практиці виклику методу в Java

Це хороша / погана / прийнятна практика передавати поточний об'єкт у виклику методу. А саме:

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

Зокрема, чи bar.foo(this)прийнятна лінія ?

Немає причин не використовувати його, thisце поточний екземпляр, і він цілком законний для використання. Насправді часто немає чистого способу пропустити це.

Тож використовуйте його.

Оскільки важко переконатись, що це прийнятно без прикладу (негативну відповідь на таке запитання завжди легше аргументувати), я щойно відкрив один із найпоширеніших java.langкласів String, і, звичайно, знайшов випадки такого використання, наприклад

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

Шукайте (thisу великих "прийнятих" проектах, ви не зможете його знайти.

У цьому немає нічого поганого. Що НЕ є хорошою практикою, це робити те саме всередині конструкторів, оскільки ви б давали посилання на ще не повністю ініціалізований об’єкт.

Тут є подібний пост: Java просочує це в конструкторі, де вони дають пояснення, чому останній є поганою практикою.

Так , але ви повинні бути обережними щодо двох речей

1. Передача цього, коли об’єкт ще не побудований (тобто в його конструкторі)
2. Переходячи це долгоживущий об'єкт, який буде тримати посилання живий і запобігти цей об'єкт від збирача сміття.

Це цілком нормально і цілком прийнятно.

це означає поточний об'єкт. Те, що ви робите, є симетрично правильним, але я не бачу в цьому потреби, якщо ви викликаєте метод у тому ж класі.

Поганою практикою є передача поточного об'єкта у виклик методу, якщо є менш складні альтернативи для досягнення тієї ж поведінки.

За визначенням, двонаправлена ​​асоціація створюється, як тільки thisпередається від одного об'єкта до іншого.

Процитувавши рефакторинг, Мартін Фаулер:

Змінити двонаправлену асоціацію на односпрямовану (200)

Двонаправлені асоціації корисні, але вони мають свою ціну. Ціна - це додаткова складність підтримки двосторонніх посилань та забезпечення належного створення та видалення об’єктів. Двонаправлені асоціації не є природними для багатьох програмістів, тому вони часто є джерелом помилок

...

Ви повинні використовувати двонаправлені асоціації, коли це потрібно, але не тоді, коли цього не потрібно. Як тільки ви побачите, що двонаправлена ​​асоціація більше не тягне свою вагу, скиньте непотрібний кінець.

Отже, теоретично, ми повинні чути дзвіночки тривоги, коли виявляємо, що нам потрібно проїхати, thisі по-справжньому намагатись думати про інші способи вирішення сучасної проблеми. Звичайно, бувають випадки, коли, в крайньому випадку, має сенс це зробити.

Крім того, часто доводиться тимчасово пошкоджувати свій дизайн, роблячи "погані практики", під час тривалої рефакторингу коду для загального поліпшення. (Крок назад, два кроки вперед).

На практиці я знайшов мій код покращився масивно , уникаючи двонаправлені зв'язку , як від чуми.

Так. Ви можете його використовувати. Це просто поширене в програмуванні передавання. thisАле є плюси і мінуси щодо його використання. Однак це не небезпечно.

Просто додамо ще один приклад, коли передача thisправильна і відповідає хорошому дизайну: Шаблон відвідувача . У шаблоні дизайну відвідувачів метод, accept(Visitor v)як правило, реалізований таким чином, що він просто викликає v.visit(this).

Прийнятний

Фрагмент із документів Oracle JAVA:

У методі екземпляра або конструкторі це посилання на поточний об'єкт - об'єкт, метод чи конструктор якого викликається. За допомогою цього ви можете звернутися до будь-якого члена поточного об’єкта з допомогою методу екземпляра або конструктора.

Використовуючи це з полем

Найпоширеніша причина використання цього ключового слова полягає в тому, що поле затьмарене методом або параметром конструктора.

Все в Java передається за значенням. Але об'єкти НІКОЛИ не передаються методу!
Коли java передає об'єкт методу, він спочатку робить копію посилання на об'єкт, а не копію самого об'єкта. Отже, це ефективно застосовуваний метод у Java. І найчастіше використовуване використання.