Ребята привет! У меня есть некий action класс, который имеет

Question

Ребята привет! У меня есть некий action класс, который имеет

метод run. Сначала этот метод имел несколко аргументов, но потом их стало много и я заменил их на dto.

private SomeDto $dto;
public run(SomeDto $dto) {
$this->dto = $dto;
... work
}

и дальше все приватные методы обращаются к $this->dto...

С одной стороны так не надо дублировать поля SomeDto, а с другой создаю сильную связанность.
Как быть в данном случае?

#laravel #programming #russian

0

07.11.2023

16 ответов

29 просмотров

Dmitriy Автор вопроса

Николай Яргин
Беглым взглядом могу сказать, что тут пригодится о...

отлично, а можно больше деталей?)

0

07.11.2023

Dmitriy Автор вопроса

Николай Яргин
Беглым взглядом могу сказать, что тут пригодится о...

какой именно?

0

07.11.2023

Николай Яргин

Dmitriy
отлично, а можно больше деталей?)

Момент

0

07.11.2023

Николай Яргин

Dmitriy
какой именно?

На позицию эксперта не претендую, однако на данный момент у тебя уже используется один из методов SOLID — инверсия зависимостей, что уже хорошо. Единственное ты можешь "прокачать" его использование. Сделать это можно, создав интерфейс с методом run(), а затем класс Action расширить от этого интерфейса: class Action implements [имя_интерфейса].

0

07.11.2023

Dmitriy Автор вопроса

Николай Яргин
На позицию эксперта не претендую, однако на данный...

ясно, спасибо. В данном случае это излишне. Меня смутило, что идет много обращений к $this->dto, но по сути этот dto написан специально для этого класса и больше нигде использоваться не будет

0

07.11.2023

Egor Gruzdev

зачем здесь: $this->dto =

0

07.11.2023

Dmitriy Автор вопроса

Egor Gruzdev
зачем здесь: $this->dto =

другие приватные методы его юзают

0

07.11.2023

Dmytro 🇺🇦

а кто мешает разделить дто в конструкторе? public run(SomeDto $dto) { $this->value1 = $dto->value1; ... work }

0

07.11.2023

Egor Gruzdev

другие приватные методы должны его получить как параметр функции

0

07.11.2023

Dmitriy Автор вопроса

Dmytro 🇺🇦
а кто мешает разделить дто в конструкторе? public ...

да, сначала тоже об этом подумал. Но вопрос зачем дублировать все что уже написано в dto

0

07.11.2023

Dmitriy Автор вопроса

Egor Gruzdev
другие приватные методы должны его получить как па...

почему?

0

07.11.2023

Dmitriy Автор вопроса

Egor Gruzdev
другие приватные методы должны его получить как па...

чем плоха такая реализация?

0

07.11.2023

Dmitriy Автор вопроса

Egor Gruzdev
другие приватные методы должны его получить как па...

по сути это так же как если бы приватный метод обращался к какому-то свойству класса через $this

0

07.11.2023

Egor Gruzdev

Dmitriy
почему?

как минимум когда у тебя метод run будет выглядить: public function run(Dto $dto): mixed { $this->dto = $dto; $this->method1(); $this->method2(); $this->method3(); } трудно читать код и не понятно от чего зависит метод класса, для меня следующий код более понятен для восприятия public function run(Dto $dto): mixed { $this->method1($dto); $this->method2($dto); $this->method3($dto); } также при таком способе может и методы не придется делать приватными

0

07.11.2023

Dmitriy Автор вопроса

Egor Gruzdev
как минимум когда у тебя метод run будет выглядить...

согласен полностью, спасибо!

0

07.11.2023