Есть ли более чистый способ построения класса MappedSuperclass Tree в Spring JPA?

Question

В настоящее время у меня есть несколько сущностей, которые ведут себя как дерево и должны сохранить их в БД.

Итак, чтобы не иметь дублирующегося кода, я построил этот класс:

@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 высвечивает некоторые ошибки:

mappedBy = "parent" -> Невозможно разрешить атрибут parent.

children.iterator (). Next (). SetParent (parent) -> непроверенный вызов setParent (T) как члена необработанного типа TreeStructure

pParent.children.add (this) -> непроверенный вызов add (E) в качестве члена типа raw java.util.Set

Я также пытался не использовать обобщения, поэтому я мог просто создать абстрактную TreeStructure, а затем расширить ее из других классов, но у меня возникли проблемы с parent / children, поскольку вы не можете ссылаться на MappedSuperclass из ссылок OneToMany / ManyToOne.

Итак, наконец, мы подошли к делу: есть ли способ реализовать это лучше / чище? Являются ли эти предупреждения значимыми или просто Интеллидж недостаточно умён?

Answer 1

Проблема не в JPA, а в использовании обобщений.

Сначала измените сигнатуру абстрактного класса, чтобы она имела рекурсивный тип:

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

Далее,вы не можете ссылаться на 'this', так как вы не знаете реализацию 'this', поэтому вы можете либо просто привести его к 'T', либо добавить абстрактный метод с такой подписью:

public abstract T getImpl();

А в реализации просто вернуть 'this'.

public T getImpl() {
  return this;
}

На боковом узле, вероятно, не очень хорошая идея получить доступ к переменным экземпляра родительских классов в вашем классе.Возможно, было бы лучше добавить методы addChild и removeChild в ваш класс TreeStructure.

Answer 2

У меня был очень похожий сценарий, и я не использовал T. Вместо этого у меня был только абстрактный класс, потому что мне не требовалась гибкость типизированных детей, и у меня не было приведений. насколько я понимаю, это может вас заземлить (общий код), но я не знаю, есть ли у вас другие требования.

Еще одним отличием в моем случае было то, что абстрактный класс - это не отображенный суперкласс, а @Inheritance(strategy = InheritanceType.SINGLE_TABLE).

Если это может помочь, вы можете найти полный рабочий пример в этом хранилище

@Inheritance(strategy = InheritanceType.SINGLE_TABLE)
public abstract class TreeStructure {

    ...

    @ManyToOne(cascade = CascadeType.PERSIST)
    private TreeStructure  parent;

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