Телеграмм чат группы yii3ru страница 2304

Извини, не очень понял

14:49пожаловаться #1

DS

PHP Sandbox, test PHP online, PHP tester

Eugene She

Ребята вопрос по SOLID имменно LSP
Есть код

http://sandbox.onlinephpfunctions.com/code/8ca46c480a49134d5edb4fa408835c55d6bf2869

Нарушает ли он принцип? Поидее да, но есть мнение что на конструкторы не распространяется.

Подскажите пожалуйста

Onlinephpfunctions

PHP Sandbox. Run, execute and test PHP code from your browser. Versions: 8.0.0, 7.4.13, 7.4.7, 7.4.0, 7.3.25, 7.3.19, 7.3.12, 7.3.5, 7.2.31, 7.2.25, 7.2.18, 7.2.4, 7.1.33, 7.1.29, 7.1.0, 7.0.14, 7.0.5, 7.0.4, 7.0.3, 7.0.2, 7.0.1, 5.6.29, 5.6.20, 5.6.19, 5.6.18, 5.6.17, 5.6.2, 5.5.34, 5.5.33, 5.5.32, 5.5.31, 5.5.18, 5.5.5, 5.5.0.a6, 5.5.0.a.5, 5.5.0.a.2, 5.4.34, 5.4.21, 5.4.13, 5.4.12, 5.4.11, 5.4.10, 5.4.9, 5.4.8, 5.4.7, 5.4.6, 5.4.5, 5.4.4, 5.4.3, 5.4.2, 5.4.1, 5.4.0, 5.3.29, 5.3.27, 5.3.23, 5.3.22, 5.3.21, 5.3.20, 5.3.19, 5.3.18, 5.3.17, 5.3.16, 5.3.15, 5.3.14, 5.3.13, 5.3.12, 5.3.11, 5.3.10, 5.3.2, 5.3.1, 5.3.0, 5.2.17, 5.2.16, 5.1.6, 5.1.5, 5.0.5, 5.0.4, 4.4.9

Так там методов нет, что там смотреть?

14:49пожаловаться #2

ES

Dmitriy S

Так там методов нет, что там смотреть?

Так суть вопроса в этом и есть - на конструкторы распространяется LSP или нет

14:50пожаловаться #3

DS

Вообще не нарушает, конструктор там ни при чем. LSP - это возможность заменить один класс другим без потери функционала.

14:50пожаловаться #4

DS

Типа у тебя стоит MessageNotifier, все работает, потом ты меняешь его на DealIndexationResult и все должно продолжать работать

14:52пожаловаться #5

ES

Dmitriy S

Вообще не нарушает, конструктор там ни при чем. LSP - это возможность заменить один класс другим без потери функционала.

То есть получается создание объекта может быть любым каким угодно, главное чтобы интерфейс был соблюден

14:53пожаловаться #6

DS

Eugene She

То есть получается создание объекта может быть любым каким угодно, главное чтобы интерфейс был соблюден

Ну да

14:53пожаловаться #7

ES

@yiiliveext Спасибо

14:53пожаловаться #8

PC

Pavel Chaplygin in Yii Framework 3

Имхо для таких случаев лучше использовать фабричный метод

class DealIndexationResult extends MessageNotifier
{
    private Deal $deal;
    
    public static function fromDeal(Deal $deal): static
    {
        $instance = new static($this->getMessage($deal));
        $instance->deal = $deal;
        return $instance;
    }
}

Хотя да, на конструкторы можно не распространять LSP, тут уже зависит от контекста

14:54пожаловаться #9

DS

Dmitriy S

Ну да

Собственно, это наследование, так что там публичный интерфейс в любом случае будет соблюден

14:56пожаловаться #10

ES

Pavel Chaplygin

Имхо для таких случаев лучше использовать фабричный метод

class DealIndexationResult extends MessageNotifier
{
    private Deal $deal;
    
    public static function fromDeal(Deal $deal): static
    {
        $instance = new static($this->getMessage($deal));
        $instance->deal = $deal;
        return $instance;
    }
}

Хотя да, на конструкторы можно не распространять LSP, тут уже зависит от контекста

Спасибо за ответ

14:56пожаловаться #11

Д

Pavel Chaplygin

Имхо для таких случаев лучше использовать фабричный метод

class DealIndexationResult extends MessageNotifier
{
    private Deal $deal;
    
    public static function fromDeal(Deal $deal): static
    {
        $instance = new static($this->getMessage($deal));
        $instance->deal = $deal;
        return $instance;
    }
}

Хотя да, на конструкторы можно не распространять LSP, тут уже зависит от контекста

Всё равно лучше определить свои параметры конструктора. Из-за того, что родительский интерфейс конструктора может сломаться, дочерний класс становится зависимым и хрупким.

14:57пожаловаться #12

ES

Дмитрий

Всё равно лучше определить свои параметры конструктора. Из-за того, что родительский интерфейс конструктора может сломаться, дочерний класс становится зависимым и хрупким.

Имеется ввиду копипастить родительский конструктор внутрь дочернего класса?

15:01пожаловаться #13

Д

Скорее переопределять, если есть различия.
Если всё может создаваться только по Deal, то и фабричный метод не нужен будет. В конструктор передаем объект Deal, а он уже распаковывает себя.
В том примере можно сделать new DealIndexationResult($msg) и класс будет невалидным, т.к. свойство private Deal $deal будет отсутствовать.

15:05пожаловаться #14

DS

Дмитрий

Скорее переопределять, если есть различия.
Если всё может создаваться только по Deal, то и фабричный метод не нужен будет. В конструктор передаем объект Deal, а он уже распаковывает себя.
В том примере можно сделать new DealIndexationResult($msg) и класс будет невалидным, т.к. свойство private Deal $deal будет отсутствовать.

Там Deal в конструкторе, как ты туда $msg передашь?

15:07пожаловаться #15

Д

Pavel Chaplygin

Имхо для таких случаев лучше использовать фабричный метод

class DealIndexationResult extends MessageNotifier
{
    private Deal $deal;
    
    public static function fromDeal(Deal $deal): static
    {
        $instance = new static($this->getMessage($deal));
        $instance->deal = $deal;
        return $instance;
    }
}

Хотя да, на конструкторы можно не распространять LSP, тут уже зависит от контекста

Здесь конструктор от родителя, а там было message

15:07пожаловаться #16

DS

А, ты о коде с фабричным методом. Там приватный конструктор надо

15:09пожаловаться #17

Д