given solution #1

KVytyagov · 2018-10-10T18:54:56Z

No description provided.

KVytyagov · 2018-10-10T18:58:22Z

src/Decorator/DecoratorManager.php

@@ -0,0 +1,65 @@
+<?php
+


не хватает объявления declare(strict_types=1); для строгой проверки входящих параметров функций

KVytyagov · 2018-10-10T19:03:26Z

src/Decorator/DecoratorManager.php

+
+class DecoratorManager extends DataProvider
+{
+    public $cache;


нет phpDoc для свойств

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

аналогично для свойства $logger - у него уже есть сеттер

KVytyagov · 2018-10-10T19:09:52Z

src/Decorator/DecoratorManager.php

+use Psr\Log\LoggerInterface;
+use src\Integration\DataProvider;
+
+class DecoratorManager extends DataProvider


Плохое название класса. По сути, это DataProvider с возможностью использования кэша. Скорее всего это какой-нибудь CachedDataProvider

KVytyagov · 2018-10-10T19:11:51Z

src/Decorator/DecoratorManager.php

+use Psr\Log\LoggerInterface;
+use src\Integration\DataProvider;
+
+class DecoratorManager extends DataProvider


Лучше использовать не наследование, а композицию. Увеличивает переиспользуемость кода

KVytyagov · 2018-10-10T19:16:16Z

src/Decorator/DecoratorManager.php

+     * @param string $password
+     * @param CacheItemPoolInterface $cache
+     */
+    public function __construct($host, $user, $password, CacheItemPoolInterface $cache)


нет типизации на аргументах $host, $user, $password

в логике реализации заложено обязательное наличие свойства $logger в объекте, поэтому его стоит вынести в конструктор

В случае с использованием композиции. конструктор будет выглядеть так:
public function __construct(DataProviderInterface $dataProvider,CacheItemPoolInterface $cache, LoggerInterface $logger)

KVytyagov · 2018-10-10T19:49:02Z