Принципы SOLID: извлечение кода в суперклассе

Ваши изменения в коде не нарушают принципы SOLID. Извлечение общего кода в суперкласс и вызов его внутри дочерних классов соответствует принципам.

Вот почему:

Принцип единой ответственности (SRP): Каждый класс имеет одну ответственность. Сериализатор отвечает за форматирование, а JSONSerializer и XMLSerializer отвечают за создание своих определенные типы сериализации.

Принцип открытости-закрытости (OCP). Класс сериализатора открыт для расширение, но закрыто для модификации. Вы можете добавлять новые классы, которые наследуется от сериализатора, но вы не изменяете класс сериализатора каждый раз, когда вам нужен новый тип сериализатора.

Принцип замены Лискова (LSP): дочерние классы могут использоваться в место родительских классов, не вызывая проблем. Вы не нарушаете это потому, что функция сериализации в дочерних классах не выполняет все, что могло бы нарушить функцию, как она определена в Класс сериализатора.

Принцип разделения интерфейсов (ISP): клиенты не должны быть принуждены зависят от интерфейсов, которые они не используют. Мы не видим никакого интерфейса зависимости здесь, поэтому этот принцип не нарушается.

Принцип инверсии зависимостей (DIP). Модули высокого уровня не должны зависят от низкоуровневых модулей. Оба должны зависеть от абстракций. метод сериализации высокого уровня в сериализаторе не зависит от Подробности метода низкоуровневого форматирования.

Кроме того, хорошо применяется принцип «сухого» («не повторяйся»), перемещая общий формат кода в сериализатор родительского класса, тем самым повышая удобство сопровождения кода и уменьшая потенциальные ошибки.

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

Интерфейс против наследования реализации

Наследование интерфейса (также известное как подтипирование) — это когда класс предоставляет реализацию методов интерфейса. Этот тип наследования представляет собой просто «контракт», который гарантирует, что класс придерживается определенной структуры. В этом случае класс соглашается реализовать ряд методов с определенными типами ввода и вывода.

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

Ваш пример больше касается наследования реализации, поскольку вы создаете подклассы JSONSerializer и XMLSerializer, которые переносят метод формата из родительского Serializer класса.

Теперь, с точки зрения принципов SOLID, обычно рекомендуется отдавать предпочтение композиции наследованию и полагаться на интерфейсы (наследование интерфейсов). Этот принцип известен как принцип инверсии зависимостей, буква «D» в SOLID. Этот принцип говорит нам, что модули высокого уровня не должны напрямую зависеть от модулей низкого уровня, а должны зависеть от абстракций (обычно интерфейсов). Но каждый случай имеет свой собственный контекст и потребности, поэтому могут быть ситуации, когда наследование реализации более применимо.

Чтобы применить composition в вашем случае, вы можете:

1. Извлечение общего метода format() в отдельный класс Formatter:

class Formatter {
    format = () => console.info("Formatting...")
}

2. Вместо расширения Serializer наши классы JSONSerializer и XMLSerializer будут включать экземпляр Formatter для обработки форматирования:

class JSONSerializer {
    constructor(){
       this.formatter = new Formatter(); 
    }

    serialize = () => {
        this.formatter.format();
        // ... JSON specific serialization
    }
}

class XMLSerializer {
    constructor(){
       this.formatter = new Formatter(); 
    }

    serialize = () => {
        this.formatter.format();
        // ... XML specific serialization
    }
}

Или, если вы предпочитаете внедрение зависимостей:

class Formatter {
    format = () => console.info("Formatting...")
}

class JSONSerializer {
    constructor(formatter){
       this.formatter = formatter;
    }

    serialize = () => {
        this.formatter.format();
        // ... JSON specific serialization
    }
}

class XMLSerializer {
    constructor(formatter){
       this.formatter = formatter; 
    }

    serialize = () => {
        this.formatter.format();
        // ... XML specific serialization
    }
}

const formatter = new Formatter();
const jsonSerializer = new JSONSerializer(formatter);
const xmlSerializer = new XMLSerializer(formatter);