web-dev-qa-db-ja.com

JPAは1対多の関係でエンティティを永続化します

構成

  • EcliplseLink 2.3.2
  • JPA 2.0
  • Entity Classes from Database ...ウィザードを使用して、エンティティはnetbeansのdbスキーマから自動作成されます。
  • コントローラークラスは、エンティティクラスからのJPAコントローラークラス...ウィザードを使用してnetbeansから自動作成されます

質問の短いバージョン

古典的なシナリオでは、1対多の関係を持つ2つのテーブル。親エンティティを作成し、次に子エンティティを作成して、子を親のコレクションに関連付けます。親エンティティをcreate（コントローラーメソッド）すると、子エンティティが作成され、親に関連付けられます。なぜそれが起こらないのですか？

ロングバージョン

親クラス

_@Entity
@XmlRootElement
public class Device implements Serializable {
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Basic(optional = false)
    private Integer id;
    @Column(unique=true)
    private String name;
    @Temporal(TemporalType.TIMESTAMP)
    private Date updated;
    @OneToMany(cascade = CascadeType.ALL, mappedBy = "deviceId")
    private Collection<NetworkInterface> networkInterfaceCollection;

    public Device() {
    }

    public Device(String name) {
        this.name = name;
        updated = new Date();
    }

    // setters and getters...

    @XmlTransient
    public Collection<NetworkInterface> getNetworkInterfaceCollection() {
        return networkInterfaceCollection;
    }

    public void setNetworkInterfaceCollection(Collection<NetworkInterface> networkInterfaceCollection) {
        this.networkInterfaceCollection = networkInterfaceCollection;
    }

    public void addNetworkInterface(NetworkInterface net) {
        this.networkInterfaceCollection.add(net);
    }

    public void removeNetworkInterface(NetworkInterface net) {
        this.networkInterfaceCollection.remove(net);
    }
    // other methods
}
_

子クラス

_@Entity
@Table(name = "NETWORK_INTERFACE")
@XmlRootElement
public class NetworkInterface implements Serializable {
    private static final long serialVersionUID = 1L;
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Basic(optional = false)
    private Integer id;
    private String name;
    @Temporal(TemporalType.TIMESTAMP)
    private Date updated;
    @JoinColumn(name = "DEVICE_ID", referencedColumnName = "ID")
    @ManyToOne(optional = false)
    private Device deviceId;

    public NetworkInterface() {
    }

    public NetworkInterface(String name) {
        this.name = name;
        this.updated = new Date();
    }

    // setter and getter methods...

    public Device getDeviceId() {
        return deviceId;
    }

    public void setDeviceId(Device deviceId) {
        this.deviceId = deviceId;
    }
}
_

メインクラス

_public class Main {
    public static void main(String[] args) {
        EntityManagerFactory emf = Persistence.createEntityManagerFactory("wifi-dbPU");
        DeviceJpaController deviceController = new DeviceJpaController(emf);
        NetworkInterfaceJpaController netController = new NetworkInterfaceJpaController(emf);

        Device device = new Device("laptop");
        NetworkInterface net = new NetworkInterface("eth0");

        device.getNetworkInterfaceCollection().add(net);
        deviceController.create(device);
    }
}
_

このクラスはNullPointerExceptionを次の行でスローします：device.getNetworkInterfaceCollection().add(net);

システムは、新しいエンティティdeviceがあり、コレクションに要素netがあることを認識しています。 dbにdeviceを書き込み、デバイスのIDを取得してnetにアタッチし、dbに書き込むことを期待していました。

これの代わりに、私はこれらが私がしなければならないステップであることがわかりました：

_deviceController.create(device);
net.setDeviceId(device);
device.getNetworkInterfaceCollection().add(net);
netController.create(net);
_

親クラスが子であることを親クラスが認識しているため、子を作成する必要があるのはなぜですか？

DeviceJpaControllerからのcreateメソッド（フィールド内の長い名前の場合は申し訳ありませんが、自動生成されます）。

_public EntityManager getEntityManager() {
    return emf.createEntityManager();
}

public void create(Device device) {
    if (device.getNetworkInterfaceCollection() == null) {
        device.setNetworkInterfaceCollection(new ArrayList<NetworkInterface>());
    }
    EntityManager em = null;
    try {
        em = getEntityManager();
        em.getTransaction().begin();
        Collection<NetworkInterface> attachedNetworkInterfaceCollection = new ArrayList<NetworkInterface>();
        for (NetworkInterface networkInterfaceCollectionNetworkInterfaceToAttach : device.getNetworkInterfaceCollection()) {
            networkInterfaceCollectionNetworkInterfaceToAttach = em.getReference(networkInterfaceCollectionNetworkInterfaceToAttach.getClass(), networkInterfaceCollectionNetworkInterfaceToAttach.getId());
            attachedNetworkInterfaceCollection.add(networkInterfaceCollectionNetworkInterfaceToAttach);
        }
        device.setNetworkInterfaceCollection(attachedNetworkInterfaceCollection);
        em.persist(device);
        for (NetworkInterface networkInterfaceCollectionNetworkInterface : device.getNetworkInterfaceCollection()) {
            Device oldDeviceIdOfNetworkInterfaceCollectionNetworkInterface = networkInterfaceCollectionNetworkInterface.getDeviceId();
            networkInterfaceCollectionNetworkInterface.setDeviceId(device);
            networkInterfaceCollectionNetworkInterface = em.merge(networkInterfaceCollectionNetworkInterface);
            if (oldDeviceIdOfNetworkInterfaceCollectionNetworkInterface != null) {
                oldDeviceIdOfNetworkInterfaceCollectionNetworkInterface.getNetworkInterfaceCollection().remove(networkInterfaceCollectionNetworkInterface);
                oldDeviceIdOfNetworkInterfaceCollectionNetworkInterface = em.merge(oldDeviceIdOfNetworkInterfaceCollectionNetworkInterface);
            }
        }
        em.getTransaction().commit();
    } finally {
        if (em != null) {
            em.close();
        }
    }
}
_
javajpanetbeanspersistence
12
Aris F.

ようやく1対多のエンティティを永続化する背後にある論理を理解しました。プロセスは次のとおりです。

  1. 親クラスを作成する
  2. 持続する
  3. 子クラスを作成する
  4. 子供を親と関連付ける
  5. 永続的な子（親コレクションが更新されます）

コード付き：

public class Main {
    public static void main(String[] args) {
        EntityManagerFactory emf = Persistence.createEntityManagerFactory("wifi-dbPU");
        DeviceJpaController deviceController = new DeviceJpaController(emf);
        NetworkInterfaceJpaController netController = new NetworkInterfaceJpaController(emf);

        Device device = new Device("laptop");                 // 1
        deviceController.create(device);                      // 2

        NetworkInterface net = new NetworkInterface("eth0");  // 3
        net.setDeviceId(device.getId());                      // 4
        netController.create(net);                            // 5 
        // The parent collection is updated by the above create     
    }
}

これで、デバイス（IDなど）を見つけることができ、そのすべての子を使用して取得できます

Collection<NetworkInterface> netCollection = device.getNetworkInterfaceCollection()

質問で投稿したデバイスエンティティクラスでは、メソッドaddNetworkInterfaceおよびremoveNetwokrInterfaceは必要ありません。

22
Aris F.

@Dima Kは彼らの言うことで正しいです。これを行うと：

_    Device device = new Device("laptop");
    NetworkInterface net = new NetworkInterface("eth0");

    device.getNetworkInterfaceCollection().add(net);
    deviceController.create(device);
_

デバイスのコレクションは初期化されていないため、コレクションに追加しようとするとNPEを取得します。 Deviceクラスでは、Collectionを宣言するときに、初期化することもできます。

_private Collection<NetworkInterface> networkInterfaceCollection = new CollectionType<>();
_

持続に関しては、あなたの仮定は正しいですが、実行は間違っていると思います。デバイスを作成したら、すぐにJPAで永続化します（必要に応じてトランザクション管理を行います）。

_Device device = new Device("laptop");
getEntityManager().persist(device);
_

NetworkInterfaceについても同じようにします。

_NetworkInterface net = new NetworkInterface("eth0");
getEntityManager().persist(net);
_

これで、両方のエンティティが保持されるため、一方を他方に追加できます。

device.getNetworkInterfaceCollection().add(net);

JPAは、他の永続化を呼び出さなくても、残りの部分を処理する必要があります。

4
Sotirios Delimanolis

これは、コレクションデータメンバーの既知の動作です。最も簡単な解決策は、コレクションを遅延作成するようにコレクションゲッターを変更することです。

@XmlTransient
public Collection<NetworkInterface> getNetworkInterfaceCollection() {
    if (networkInterfaceCollection == null) {
        networkInterfaceCollection = new Some_Collection_Type<NetworkInterface>();
    }
    return networkInterfaceCollection;
}

また、このデータメンバーは、getterメソッドを介してのみ参照するようにしてください。

2
Dima K

この例外は、まだ永続化されていないエンティティを（おそらくem.getReference（）によって）検索しようとしていることを意味します。まだPKがないエンティティに対してem.getReference（）またはem.find（）を実行することはできません。

0
Dima K