Spring boot-多对多关联未删除联接表数据

Question

我在Spring Boot中遇到了一个多对多关系问题。代码如下：

public class Task {

  @Id
  @GeneratedValue
  private Long id;

  @ManyToMany(cascade = {PERSIST, MERGE}, fetch = EAGER)
  @JoinTable(
      name = "task_tag",
      joinColumns = {@JoinColumn(name = "task_id", referencedColumnName = "id")},
      inverseJoinColumns = {@JoinColumn(name = "tag_id", referencedColumnName = "id")}
  )
  @Builder.Default
  private Set<Tag> tags = new HashSet<>();

  public void addTags(Collection<Tag> tags) {
    tags.forEach(this::addTag);
  }

  public void addTag(Tag tag) {
    this.tags.add(tag);
    tag.getTasks().add(this);
  }

  public void removeTag(Tag tag) {
    tags.remove(tag);
    tag.getTasks().remove(this);
  }

  public void removeTags() {
    for (Iterator<Tag> iterator = this.tags.iterator(); iterator.hasNext(); ) {
      Tag tag = iterator.next();
      tag.getTasks().remove(this);
      iterator.remove();
    }
  }

  @Override
  public boolean equals(Object o) {
    if (this == o) return true;
    if (!(o instanceof Task)) return false;
    return id != null && id.equals(((Task) o).getId());
  }

  @Override
  public int hashCode() {
    return id.intVal();
  }
}

和

public class Tag {

  @Id
  @GeneratedValue
  private Long id;

  @NotNull
  @Column(unique = true)
  private String name;

  @ManyToMany(cascade = {PERSIST, MERGE}, mappedBy = "tags", fetch = EAGER)
  @Builder.Default
  private final Set<Task> tasks = new HashSet<>();

  @Override
  public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;
    Tag tag = (Tag) o;
    return Objects.equals(name, tag.name);
  }

  @Override
  public int hashCode() {
    return id.intVal();
  }

}

当然，我有task_tag表，在其中，在任务中插入标签并保存该任务后，会出现一个条目。但是，当我删除标签（或清除它们）时，条目不会从连接表中删除。这是测试：

@Test
  void entityIntegration() {
    Task task = taskRepo.save(...);

    Tag tag1 = Tag.builder().name(randomString()).build();
    Tag tag2 = Tag.builder().name(randomString()).build();
    Tag tag3 = Tag.builder().name(randomString()).build();
    Tag tag4 = Tag.builder().name(randomString()).build();
    final List<Tag> allTags = Arrays.asList(tag1, tag2, tag3, tag4);
    tagRepo.saveAll(allTags);

    task.addTag(tag1);
    taskRepo.save(task);
    final Long task1Id = task.getId();
    assertTrue(tag1.getTasks().stream().map(Task::getId).collect(Collectors.toList()).contains(task1Id));

    task.clearTags();
    task = taskRepo.save(task);
    tag1 = tagRepo.save(tag1);
    assertTrue(task.getTags().isEmpty());
    assertTrue(tag1.getTasks().isEmpty());

    task.addTags(allTags);
    task = taskRepo.save(task); // FAILS, duplicate key ...

  }

task\u tag表确实在这两列（并且仅在这两列）上形成了一个复合索引。

我做错了什么？我遵循了每一条建议和建议——使用集合代替列表，使用助手方法，清理等等。。。我找不到窃听器。

非常感谢。

Answer 1

请将DELETE关键字添加到多对多注释的cascade属性中。我认为Tag类的task属性的注释应该更改如下。

你可以试试下面的映射

public class Task {

  @Id
  @GeneratedValue
  private Long id;

  @ManyToMany(cascade = {PERSIST, MERGE,DELETE}, fetch = EAGER)
  @JoinTable(
      name = "task_tag",
      joinColumns = {@JoinColumn(name = "task_id", referencedColumnName = "id")},
      inverseJoinColumns = {@JoinColumn(name = "tag_id", referencedColumnName = "id")}
  )
  @Builder.Default
  private Set<Tag> tags = new HashSet<>();

  public void addTags(Collection<Tag> tags) {
    tags.forEach(this::addTag);
  }

  public void addTag(Tag tag) {
    this.tags.add(tag);
    tag.getTasks().add(this);
  }

  public void removeTag(Tag tag) {
    tags.remove(tag);
    tag.getTasks().remove(this);
  }

  public void removeTags() {
    for (Iterator<Tag> iterator = this.tags.iterator(); iterator.hasNext(); ) {
      Tag tag = iterator.next();
      tag.getTasks().remove(this);
      iterator.remove();
    }
  }

  @Override
  public boolean equals(Object o) {
    if (this == o) return true;
    if (!(o instanceof Task)) return false;
    return id != null && id.equals(((Task) o).getId());
  }

  @Override
  public int hashCode() {
    return id.intVal();
  }
}

public class Tag {

  @Id
  @GeneratedValue
  private Long id;

  @NotNull
  @Column(unique = true)
  private String name;

  @ManyToMany(cascade = {PERSIST, MERGE,DELETE}, mappedBy = "tags", fetch = EAGER)
@JoinTable(
      name = "task_tag",
      joinColumns = {@JoinColumn(name = "tag_id", referencedColumnName = "id")},
      inverseJoinColumns = {@JoinColumn(name = "task_id", referencedColumnName = "id")}
  )
  @Builder.Default
  private final Set<Task> tasks = new HashSet<>();

  @Override
  public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;
    Tag tag = (Tag) o;
    return Objects.equals(name, tag.name);
  }

  @Override
  public int hashCode() {
    return id.intVal();
  }

}

Answer 2

让我印象最深刻的是，你的标签是相等的，哈希码是不匹配的。

您的“equals”基于对象的名称相同来驱动相等，这对“标签就是名称”的咒语有逻辑意义。但是基于“id”的哈希码驱动是等价的，根本不使用名称。

暂时忘记JPA/Hibernate，只是当这两个不同步时，普通的旧集合本身变得非常不可预测。

您可以在这里阅读更多关于这一点的信息，特别是为什么不匹配相等的哈希代码最终会哈希到错误的bucket，并导致在hashset中保持其完整性的混乱：为什么我需要重写Java中的equals和hashCode方法？

有很多方法可以使它们重新同步（使用像Lombok这样的库和IDE中的代码生成工具），但我不想指定一种，而是简单地指向这个web资源，它方便地为他的示例创建了一个具有完全相同概念的标记，所以我怀疑您可以自己使用完全相同的模式。

https://vladmihalcea.com/the-best-way-to-use-the-manytomany-annotation-with-jpa-and-hibernate/

这里有另一个有用的SO线程，我发现它讨论了关系和身份/相等/哈希代码，因为它影响JPA：JPA哈希代码（）/相等（）困境