JPA関係マッピングを初期化するか、初期化しないか？

Question

多くのJPA関連では、空のコレクションとの関係を初期化するのがベストプラクティスと考えられますか？例えば。上記の例で

@Entity 
public class Order { 

    @Id 
    private Integer id; 

    // should the line items be initialized with an empty array list or not? 
    @OneToMany(mappedBy="order") 
    List<LineItem> lineItems = new ArrayList<>(); 

} 

は、空でArrayListかどうかのデフォルト値とlineItemsを定義することが良いですか？長所と短所は何ですか？

Answer 1

JPA自体はコレクションが初期化されているかどうかは気にしません。 JPAを使用してデータベースからOrderを取得する場合、JPAはと常に一致します。は、OrderLinesのnull以外のリストを持つOrderを返します。

理由：オーダーは0行、1行またはN行を持つことができ、空、1サイズまたはNサイズのコレクションでモデル化するのが最適です。コレクションがnullの場合は、コード内のどこでもそのコレクションをチェックする必要があります。リストがnullだった場合たとえば、この単純なループは、NullPointerExceptionが引き起こす：

for (OrderLine line : order.getLines()) { 
    ... 
} 

をだから、常にでもエンティティの新規作成されたインスタンスに対して、null以外のコレクションを持つことにより、不変ことを確認するのが最善です。これにより、生産コードはより安全で清潔な新しい注文を作成できます。これはまた、データベースから来ていないOrderインスタンスを使用して、より安全でクリーンなユニットテストを行います。私はまた、例えば、グアバの不変コレクションを使用することをお勧めします

Answer 2

、

import com.google.common.collect.ImmutableList; 
// ... 
@OneToMany(mappedBy="order") 
List<LineItem> lineItems = ImmutableList.of(); 

このイディオムは決して新しい空のリストを作成しませんが、空のリストを表す単一のインスタンスを再利用（タイプは関係ありません）。これは関数型プログラミング言語の非常に一般的な習慣であり（Scalaも同様）、ゼロにヌル値の代わりに空のオブジェクトを持つオーバーヘッドを減らし、イディオムmootに対して効率的な引数を与えます。