Что это значит и как исправить проблему сонара Java? «Циклы между пакетами должны быть удалены» (squid: CycleBetweenPackages)

Question

Циклы существуют между пакетами, когда существуют зависимости использования или импорта вида между классами в этих пакетах.

Рассмотрим следующий пример. Пусть есть 4 класса: Truck и интерфейс Car в org.example.car пакет и Navigation и CPU в пакете org.example.part. В пакетах org.example.car и org.example.part мы используем отношения между классами Truck --> Car и классами Navigation --> CPU. Предположим, что класс Truck использует класс Navigation, поэтому у нас есть связь между пакетами org.example.car --> org.example.part. Тогда предположим, что класс CPU также использует интерфейс Car, который дает нам соотношение между пакетами org.example.part --> org.example.car. Хотя класс Navigation не использует класс пакетов org.example.car, у нас есть цикл соседних классов в пакетах (CPU класс).

Этот пример представлен на рисунке ниже:

Когда несколько пакетов участвуют в цикле, это означает, что эти пакеты высоко в сочетании, и не существует никакого способа повторного использования/извлечения одного из тех, пакетов без импорта всех других пакетов. Такой цикл может быстро увеличить усилия, необходимые для поддержания приложения и принятия изменений в бизнесе.

Answer 1

Чтобы исправить цикл зависимостей в ваших пакетах, вы должны ввести дополнительный пакет и переместить пару зависимых классов/интерфейсов в этот вновь созданный пакет.

Это представлено на рисунке ниже:

В нашем примере, давайте решили переместить интерфейс Car вновь созданного пакета org.example.api. Это нарушает цикл зависимостей и снижает технический уровень в нашем приложении. Теперь все пакеты имеют зависимости только одним способом, и при необходимости их можно разбить на разные артефакты.