java - 在Spring JPA中，有没有更干净的方法来构建MappedSuperclass Tree类？

我目前有几个实体，它们像一棵树，需要将它们保存到数据库中。

因此，为了没有重复的代码，我建立了这个类：

@MappedSuperclass
public abstract class TreeStructure<T extends TreeStructure>
{
    @ManyToOne(cascade = CascadeType.PERSIST)
    private T  parent;

    @OneToMany(mappedBy = "parent", fetch = FetchType.LAZY, cascade = CascadeType.PERSIST)
    protected Set<T> children = new HashSet<>();

    /**
     * Function that is used before deleting this entity. It joins this.children to this.parent and viceversa.
     */
    @Transactional
    @PreRemove
    public void preDelete()
    {
        unregisterInParentsChildren();

        while (!children.isEmpty())
        {
            children.iterator().next().setParent(parent);
        }

    }

    public abstract long getId();

    protected void setParent(T pParent)
    {
        unregisterInParentsChildren();
        parent = pParent;
        registerInParentsChildren();
    }

    /**
     * Register this TreeStructure in the child list of its parent if it's not null.
     */
    private void registerInParentsChildren()
    {
        getParent().ifPresent((pParent) -> pParent.children.add(this));
    }

    /**
     * Unregister this TreeStructure in the child list of its parent if it's not null.
     */
    private void unregisterInParentsChildren()
    {
        getParent().ifPresent((pParent) -> pParent.children.remove(this));
    }

    /**
     * Move this TreeStructure to an new parent TreeStructure.
     *
     * @param pNewParent the new parent
     */
    public void move(final T pNewParent)
    {
        if (pNewParent == null)
        {
            throw new IllegalArgumentException("New Parent required");
        }

        if (!isProperMoveTarget(pNewParent) /* detect circles... */)
        {
            throw new IllegalArgumentException(String.format("Unable to move Object %1$s to new Object Parent %2$s", getId(), pNewParent.getId()));
        }

        setParent(pNewParent);
    }

    private boolean isProperMoveTarget(TreeStructure pParent)
    {
        if (pParent == null)
        {
            return true;
        }
        if (pParent == this)
        {
            return false;
        }

        return isProperMoveTarget(pParent.parent);
    }

    public int getLevel()
    {
        return getParent().map(pParent -> pParent.getLevel() + 1).orElse(1);
    }

    /**
     * Return the <strong>unmodifiable</strong> children of this TreeStructure.
     *
     * @return the child nodes.
     */
    public Set<T> getChildren()
    {
        return Collections.unmodifiableSet(this.children);
    }

    public Optional<T> getParent()
    {
        return Optional.ofNullable(parent);
    }

    public Optional<Long> getParentCategoryId()
    {
        return parent == null ? Optional.empty() : Optional.of(parent.getId());
    }
}

然后，要真正实现它，我只需执行以下操作：

@Entity(name = "CATEGORY")
public class Category extends TreeStructure<Category>
{
    @Id
    @GeneratedValue(strategy = GenerationType.AUTO)
    @JsonProperty("category_id")
    private long id;

// etc...

据我所知，一切都像万灵药一样工作，但是每次我进入TreeStructure类Intellij时都会看到一些错误：


  appedBy =“ parent”->无法解析父属性。
  
  children.iterator（）。next（）。setParent（parent）->未选中对setParent（T）的调用，将其作为原始类型TreeStructure的成员
  
  pParent.children.add（this）->未经选中的对add（E）的调用，作为原始类型java.util.Set的成员


我也尝试过不使用泛型，因此我可以只具有抽象的TreeStructure，然后从其他类进行扩展，但是由于无法从OneToMany / ManyToOne引用中引用MappedSuperclass，所以对父/子有问题。

所以，最后要指出的是：是否有以更好/更清洁的方式实施此操作？这是有意义的警告，还是Intellij不够聪明？

最佳答案

问题不在于JPA，而在于泛型的使用。

首先，更改您的抽象类签名，使其具有递归类型：

public abstract class TreeStructure<T extends TreeStructure<T>>

接下来，您无法引用“ this”，因为您不知道“ this”的实现，因此您可以将其强制转换为“ T”或添加具有如下签名的抽象方法：

public abstract T getImpl();

在实现中只需返回“ this”。

public T getImpl() {
  return this;
}

在侧节点上，访问类中的父类实例变量可能不是一个好主意。向TreeStructure类添加addChild和removeChild方法可能是一个更好的主意。