Какие пробелы в моем классе для сравнения значений?

Question

[Антон Шаманов]

Запилил класс для гибкого сравнения значений. Постарался учесть все случаи для которых не подходит нативное сравнение. Посмотрите пожалуйста - может что-то не учел.
Код класса

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

Comments

[Дмитрий]

а зачем сравнивать ресурсы? и только стримы?

[Антон Шаманов]

Дмитрий, это нужно для сравнения массивов/объектов. Стримы сравниваются по мета данным, для других ресурсов подобных мета данных нет.
Что-то вроде

class File {
    /**
     * @var resource
     */
    public $handler;
    public function __construct(string $path) { 
           $this->handler = fopen($path, 'r');
    }
}

$comparator = new Comparator(Comparator::EQUAL_OBJECT | Comparator::EQUAL_STREAM);
$comparator->compare(new File('foo/bar.php'), new File('foo/bar.php'));

Answer 1

[smple]

compare должна не просто говорить равно или не равно, а должна еще сообщать какое число больше или меньше, поэтому возвращать она должна не bool, а больше или меньше, обычно делают возвращаемый тип int (0 если равны и -1 и 1 в случае если одно из чисел больше другова)

как сделать такое сравнение для произвольных массивов ? это спорный вопрос и универсального ответа нет.

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

другое мелко улучшение, я бы убрал все публичные методы по изменению класса после создания, оставил бы только один публичный метод это compare, чтобы было видно цель этого класса, а какие то методы бы просто удалил (getFlags например)

ну и в целом такое не особо нужно в php, непонятно зачем это применять, код нормальный, понятный, даже есть тесты.