JPA многоманое однонаправленное картографирование

Question

У меня есть два класса: ExternalTask ​​и ExternalSource. ExternalTask ​​имеет список ExternalSource и множество однонаправленных отношений от ExternalTask ​​к ExternalSource.

Когда я хочу удалить ExternalSource, я проверяю, есть ли на него ссылка ExternalSask. Если это так, я проверяю, есть ли в этой ExternalTask ​​только этот внешний источник в списке. Если я полностью удаляю ExternalTask; в противном случае я удаляю этот externalsource из списка и объединяю externaltask. Затем я удаляю externalsource. Тем не менее, это дает нарушение ограничения. Я попытался использовать jointable без cascade, cascadetype.update и cascadetype.refresh, но он все еще не работает. Любая помощь?

Это метод удаления:

public class Foo{
      public boolean deleteExternalDataStorage(Long sid) {
            EntityManager em = getEntityManager();
            EntityTransaction et = em.getTransaction();
            try {
                et.begin();
                ExternalDataStorage s = em.find(ExternalDataStorage.class, sid);
                List<ExternalTask> tasks=(List<ExternalTask>) em.createQuery("SELECT t FROM ExternalTask t ").getResultList();
               for(ExternalTask t:tasks) {
                  if(t.getExternalSources().contains(s)){
                   t.getExternalSources().remove(s);
                   if(t.getExternalSources().isEmpty()){
                         em.remove(t);
                   }else{
                       em.merge(t);
                   }
                   }
               }


                em.remove(s);
                et.commit();
                return true;
            } catch (Exception e) {
                e.printStackTrace();

            } finally {
                if (et.isActive()) {
                    et.rollback();
                }
            }
            return false;
        }


    }


    @Entity
    public class ExternalTask {

        @ManyToMany
        @JoinTable(name = "ExternalTask_ExternalSource", joinColumns = @JoinColumn(name = "TID"), inverseJoinColumns = @JoinColumn(name = "EXID"))
        private  List<ExternalDataStorage> externalSources=new ArrayList<ExternalDataStorage>();


        @ManyToMany
        @JoinTable(name = "ExternalTask_Archive", joinColumns = @JoinColumn(name = "TID"), inverseJoinColumns = @JoinColumn(name = "AID"))
        protected  List<Archive> archives=new ArrayList<Archive>();


        @Id
        @GeneratedValue(strategy = GenerationType.IDENTITY)
        @Column(name = "TID")
        protected Long id;


        @NotNull
        @Column(name = "name")
        protected String name;

        @Column(name = "description")
        protected String description;

        @Column(name="timeinterval" )
        protected String interval;



        @Column(name="startdate")
        @Temporal(javax.persistence.TemporalType.TIMESTAMP)
        protected Date startDate;

        ...
    }

    @Entity
    @Table(name = "ExternalSource")
    public class ExternalDataStorage implements Serializable {
        private static final long serialVersionUID = 3926424342696569894L;        

        @Id
        @GeneratedValue(strategy = GenerationType.IDENTITY)
        @Column(name = "EXID")
        private Long id;

        @NotNull
        @Column(name = "NAME")
        private String name;

        @Column(name = "DESCRIPTION")
        private String description;

        @NotNull
        @Column(name = "USERNAME")
        private String username;

        @Column(name = "PATH")
        private String path;


        @Column(name = "PORT")
        private int port = 22;

        @NotNull
        @Column(name = "ISSECURE")
        private boolean isSecure=true;

        @NotNull
        @Column(name = "ISINDEXINGENABLED")
        private boolean indexingEnabled;

        @Column(name = "INDEXINGREGEXP")
        private String indexingRegExp = "({time}\\d{8}-\\d{6})";

        @NotNull
        @Column(name = "IP")
        private String ip;

        @Column
        private String password;

        @Column
        private String protocol;

        @Transient
        private String publicKey;

        @Column(name = "RSA_PRIV_KEY", length = 4096)
        private String privateKey;

        @Transient
        private String regExpTestStr="";

        @Transient
        private boolean testSucced;

        @Transient
        private InputAddress inputAddr;

        @Column
        private boolean authenticationType;

        @Column
        private boolean timeStampingEnable=true;

        @Column
        private String sshPath;

        @Column
        private  String filename="(.*\\.log)";

        public ExternalDataStorage() {
              inputAddr=new InputAddress();
        }

        ...
    }

Answer 1

Хорошо, у вас должны быть двунаправленные отношения «многие ко многим» между вашей задачей и источником. И, как видно из ваших требований, источник должен быть владельцем отношения и не использовать CascadeType = All при определении отношения на стороне источника, но использовать его на стороне задачи. Если у вас есть двунаправленные отношения, вам не нужно проверять полные списки всех заданий. Эта часть кода является квадратичной, ее можно легко оптимизировать с помощью b-направленного отображения. Кроме того, если вы сделаете источник владельцем, не используя каскадные параметры, зависимость от источника будет связана с задачей. Поэтому все, что вам нужно сделать, это:

if(source.getTaskList().size() == 1) {
    //remove the task source.getTaskList().get(0), this will remove the source also
}
else {
   //remove the source. The task is unaffected, as the source is the owning side
}