Блог

Допустим, у нас есть следующие две сущности.

Библиотека

 public class Library
{
    private Collection<Book> books = new ArrayList<Book>();

    public boolean add(Book book)
    {
        return books.add(book);
    }
    public boolean remove(Book book)
    {
        return books.remove(book);
    }
}
  

Книга

 public class Book
{
    private Library library;
    public setLibrary(Library library)
    {
        this.library = library;
    }
}
  

Это двунаправленная связь, которая, скорее всего, довольно быстро разорвется. Даже если вы добавляете книгу в библиотеку, библиотека книги не обновляется. И даже если вы обновляете библиотеку книги, библиотека не обновляется.

Итак, как бы вы правильно поддерживали эти двунаправленные отношения? Я мог бы, конечно, просто поставить add remove в setLibrary и наоборот, но это, насколько я вижу, привело бы к циклу, который, вероятно, закончился бы на StackOverflowError .

Определите «владельца» для отношения. Библиотека «владеет» книгой или книга владеет библиотекой?

Ответ на этот вопрос говорит вам, кто обновляет объект «slave».

Я чувствую, что библиотека владеет книгой, поэтому код «добавить книгу в библиотеку» должен перейти в библиотеку, которая затем обновит обратные ссылки.

Хотя я бы рекомендовал метод определения владельца отношений Аарона, альтернативой, которая позволяет set / add / remove методам оставаться там, где они есть, является также определение соответствующих has методов ( hasBook в Library и hasLibrary в Book ). Перед вызовом метода add or remove на другом конце отношения сначала запрашивайте, существует ли он уже, и если он существует, не вызывайте его.

Я что, тупой?

 public boolean add(Book book)
{
   book.setLibrary(this);
   return books.add(book);
}
public boolean remove(Book book)
{
   book.setLibrary(null);
   return books.remove(book);
}
  

В предлагаемом вами решении не могли бы вы проверить ссылки на равенство, чтобы избежать цикла?

 public boolean add(Book book)
{
   if(book.getLibrary != this) book.setLibrary(this);
   return books.add(book);
}