Приведите пример реализации MVC + Service Layer + Repository?

Question

[Ruslan Ruslanov]

Может кто-то привести толковый пример реализации MVC + Service Layer + Repository?
А то я не совсем понимаю, что тогда должна делать модель в таком случае, и с чем работает контроллер.

Comments

[Дмитрий]

Контроллер работает с request и response, модель работает с данными

[Ruslan Ruslanov]

Дмитрий, гениально.
но я просил пример реализации.
что с чем работает я понимаю.

[Дмитрий]

Ruslan Ruslanov,
https://docs.laminas.dev/laminas-mvc/
https://docs.laminas.dev/laminas-db/

Answer 1

[dmitriy]

Модель раскладывается на три составляющие: класс сущности (Entity), класс работы с бд(Repository) и класс для бизнес логики (Service)

а пример простой

<?php
class Controller
{

	private $userService;

	function __construct(UserService $userService)
	{
	
		$this->userService = $userService;
	}
	
	public function show(int $id){
	
		try{
			$user = $this->userService->getById($id);
		}
		catch(UserNotFound $ex){
			return new Response(null,404);
		}
	
		return new JsonResponse($user->toArray(),200);
	}
	
}

class UserService
{


	private $userRepository;

	function __construct(UserRepository $userRepository)
	{
	
		$this->userRepository = $userRepository;
	}

	public function getById(int $id): User
	{
	
		if(!$user = $this->userRepository->findById($id))
		{
			throw new UserNotFound($id);
		}
		
		return $user;
	}
}

Comments

[Ruslan Ruslanov]

Приведите пример реализации

[dmitriy]

привел

[Ruslan Ruslanov]

dmitriy, а классы Responce, Service я сам додумать должен?
ну сори, если бы я мог, то не написал бы такой вопрос.

[dmitriy]

Service дополнил

[dmitriy]

Response думаю вы должны знать что такое

[FanatPHP]

ой.
Ruslan Ruslanov, ты имел в виду тестовое задание целиком за тебя сделать?
при чем здесь response?

[Ruslan Ruslanov]

dmitriy, а модели в такой схеме где находятся? и где используются? какая вообще роль у модели тогда?

[dmitriy]

https://www.php-fig.org/psr/psr-7/

[dmitriy]

Ruslan Ruslanov, ну если из модели вытащили бизнес логику и работу с хранилищем, что осталось?

[Ruslan Ruslanov]

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

[Ruslan Ruslanov]

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

[dmitriy]

Ruslan Ruslanov,
там кнопочка,нажмите, отметить решением, спасибо)

[Евгений Ромашкан]

Ruslan Ruslanov, У модели нет обязанностей кроме бизнес-логики.
Размазанная по сервисам логика меняющая общее состояние - очень плохой способ делать дела с позиции инкапсуляции/связности, хотя и все с этого начинают.

[Ruslan Ruslanov]

Евгений Ромашкан, а если необходимо изменить состояние у множества сущностей, то как тогда быть?

[Евгений Ромашкан]

Ruslan Ruslanov, Если их много - что-то у вас не так спроектировано.
А так - из контроллера или сервиса вызовите две подряд и ок. Рано вам усложнять.
Важно правда понимать, что своё состояние сущность контроллирует сама(никаких сеттеров)

[Ruslan Ruslanov]

Евгений Ромашкан, не соглашусь.
1. Если сущностей много, то это не обязательно косяк. Точнее, если много экземпляров однотипных сущностей. Как же тот же findAll? Как это все обрабатывать? Как массив? Ну, если только так.
2. Вызов нескольких моделей подряд работает, только если их до 5 штук. Далее нужно как-то выкручиваться, иначе контроллер/сервис распухнет (с чем я и столкнулся).
3. А когда будет пора усложнять? Может мне стоит по совету анона с тостера перестать разрабатывать проект, потому что рано? Может мне стоит оставить разработку вообще и пойти в другую проф. область? Может посоветуете еще, что мне съесть на ужин? Или во сколько спать лечь?
4. Почему никаких сеттеров? Да, это выглядит как костыль, но фактически публичный сеттер не дает доступ к свойству, и принцип закрытости/инкапсуляция не нарушаются.

[Евгений Ромашкан]

Ruslan Ruslanov, Тот факт что за вопросами вы на Тостер пришли за примерами, а не изучать соотв-ю теорию говорит от том что усложнять вам что-либо рано. Ну и культ карго в придачу.

Почему никаких сеттеров?

Потому что размызываются бизнес-процессы(нет единого источника правды), потому что нет контроля за состоянием(инвариантами), потому что выше связность, потому что инкапсуляция таки нарушается если логика вытекает из объекта.
Я б ещё накинул что это(геттеры/сеттеры) процедурное программирование, но на самом деле полезнее будет найти что-то более объективно определяющее хорошее и плохое, чем баззворды(это к слову о культе карго).

Да, это выглядит как костыль, но фактически публичный сеттер не дает доступ к свойству, и принцип закрытости/инкапсуляция не нарушаются.

Наоборот, выглядит так, будто не нарушается, а по сути - костыль.

[batyrmastyr]

Евгений Ромашкан, «фиканье» в отношении процедурного программирования - тоже карго-культ. В php геттеры и сеттеры банально медлительны и, в нынешних средах разработки, неудобны, но инкапсуляцию не нарушают.