Ревью 🔍

Несколько ключевых моментов, которые помогут в процессе проверки:

⏱️ Уровень сложности ревью: 4 🔵🔵🔵🔵⚪

🧪 Запрос содержит тесты

🔒 Проблем с безопасностью не выявлено

⚡ Рекомендовано к рассмотрению

Возможная проблема

Интерфейс Figure разрешает реализации Circle и Square, но при этом класс Rectangle наследуется от Square, что может привести к нарушению принципа подстановки Лисков, так как Square не должен иметь подтипов, изменяющих его поведение (например, по соотношению сторон).

public sealed interface Figure permits Circle, Square {

Возможная проблема

В тесте создаётся объект Circle с радиусом 1, ожидается площадь 3, но расчёт через Math.round(3.14 * 1) даёт 3 — это верно, однако при использовании Math.PI результат может отличаться. Также путаница в типах: square = new Circle(1), что вводит в заблуждение.

assertEquals("Circle: 3", figureService.getArea(square));

Design Issue

Интерфейс BusinessOperation разрешает только один класс AbstractBusinessOperation, что делает sealed-ограничение избыточным. Лучше использовать абстрактный класс напрямую или пересмотреть иерархию разрешённых типов.

public sealed interface BusinessOperation permits AbstractBusinessOperation {

java-praсtice-ru

Популярные репозитории