Spring Data как правильно настроить/сохранить entity?

Question

[ScRsa]

@Entity
public class ReportPeriod {
    ...
    @OneToMany(fetch = FetchType.LAZY, mappedBy = "reportPeriod")
    private Collection<Document> documents;
}

@Entity
public class Document {
    ...
    @ManyToOne(fetch = FetchType.EAGER)
    @JoinColumn(name = "report_period_id", referencedColumnName = "id", nullable = false)
    private ReportPeriod reportPeriod;
}

...
ReportPeriod rp = new ReportPeriod();
rp.setDatePeriod('01.01.2001');
reportPeriodRepository.save(rp);
...

При создании нового периода получаю:
rp.Id = 123
rp.datePeriod = '01.01.2001'
rp.documents = null

Вопрос. Почему после сохранения периода получаю что documents == null? По идее же должна вернуться пустая коллекция?

Если сделать поиск другого, ранее сохраненного, периода: reportPeriodRepository.findOne(345). То rp.getDocuments() возвращает коллекцию а не null.

Answer 1

[Eugene Khrustalev]

Пустая коллекция будет, если объект загружается из базы. Если объект был создан в коде и сохранен в базу, то Hibernate не будет ничего инициализировать.

Я обычно использую следующий код для getter-ов внутренних коллекций (подсмотрел у JAXB):

public Collection<Document> getDocuments() {
    if (documents == null) {
        documents = new ArrayList<>();
    }
    return documents;
}

Comments

[Carburn]

Надо делать так

@OneToMany(mappedBy = "reportPeriod")
private Collection documents = new ArrayList<>();

[Eugene Khrustalev]

Carburn: Лучше инициализировать это все-таки через геттер, чтобы не отслеживать потом NullPointerException из-за setDocuments(null) да и доступ к коллекции не всегда нужен, а объект все равно будет создаваться.

Answer 2

[Александр Косарев]

rp = reportPeriodRepository.save(rp);

И будет тебе счастье. Но лучше создавать пустую коллекцию в get-методе при необходимости. А вообще, лучше отказаться от обращения к @OneToMany и @ManyToMany коллекциям.