Рефакторинг

Author: ProklUng

psalm.xml

@@ -6,7 +6,6 @@
     xmlns="https://getpsalm.org/schema/config"
     xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
     allowStringToStandInForClass="true"
-    allowCoercionFromStringToClassConst="true"
     findUnusedPsalmSuppress="true"
     skipChecksOnUnresolvableIncludes="true"
 >
@@ -53,6 +52,11 @@
                 <directory name="/"/>
             </errorLevel>
         </InvalidThrow>
+        <UnresolvableInclude>
+            <errorLevel type="suppress">
+                <directory name="/"/>
+            </errorLevel>
+        </UnresolvableInclude>
         <UndefinedMagicMethod>
             <errorLevel type="suppress">
                 <directory name="/"/>

src/CompilerContainer.php

@@ -185,14 +185,14 @@ public function cacheContainer(
         require_once $compiledContainerFile;
 
         $classCompiledContainerName = '\\'.$classCompiledContainerName;
-
+        /** @psalm-suppress LessSpecificReturnStatement */
         return new $classCompiledContainerName();
     }
 
     /**
      * Задать ID модуля.
      *
-     * @param string $moduleId
+     * @param string $moduleId ID модуля.
      *
      * @return CompilerContainer
      */
@@ -218,7 +218,7 @@ private function createCacheDirectory(string $dir) : void
     }
 
     /**
-     * Gets the container class.
+     * Класс контейнера.
      *
      * @param string  $env   Окружение.
      * @param boolean $debug Режим отладки.
@@ -234,7 +234,10 @@ private function getContainerClass(string $env, bool $debug) : string
 
         if (!preg_match('/^[a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*$/', $class)) {
             throw new InvalidArgumentException(
-                sprintf('The environment "%s" contains invalid characters, it can only contain characters allowed in PHP class names.', $this->environment)
+                sprintf(
+                    'The environment "%s" contains invalid characters, it can only contain characters allowed in PHP class names.',
+                    $env
+                )
             );
         }
 
@@ -260,7 +263,9 @@ private function dumpContainer(ConfigCache $cache, ContainerBuilder $container,
         }
 
         $dumper = new PhpDumper($container);
+        /** @psalm-suppress UndefinedClass */
         if (class_exists(\ProxyManager\Configuration::class) && class_exists(ProxyDumper::class)) {
+            /** @psalm-suppress InvalidArgument */
             $dumper->setProxyDumper(new ProxyDumper());
         }
 

src/DI/AbstractServiceContainer.php

@@ -4,6 +4,7 @@
 
 use Closure;
 use Exception;
+use LogicException;
 use ProklUng\ContainerBoilerplate\CompilerContainer;
 use ProklUng\ContainerBoilerplate\Utils\BitrixSettingsDiAdapter;
 use Symfony\Component\DependencyInjection\Container;
@@ -57,7 +58,7 @@ abstract class AbstractServiceContainer
      */
     public function __construct()
     {
-        $this->debug = (bool)$_ENV['DEBUG'] ?? true;
+        $this->debug = !array_key_exists('DEBUG', $_ENV) ? true : (bool)$_ENV['DEBUG'];
         $this->environment = $this->debug ? 'dev' : 'prod';
     }
 
@@ -73,7 +74,7 @@ abstract  public function initContainer() : void;
      * Загрузка всего хозяйства.
      *
      * @return void
-     * @throws Exception
+     * @throws Exception | LogicException
      */
     public function load() : void
     {
@@ -83,6 +84,11 @@ public function load() : void
 
         $this->createContainer();
         $compilerContainer = new CompilerContainer($_SERVER['DOCUMENT_ROOT']);
+
+        if (!$this->moduleId) {
+            throw new LogicException('Children of AbstractServiceContainer must define moduleId property.');
+        }
+
         $compilerContainer->setModuleId($this->moduleId);
 
         // Кэшировать контейнер?
@@ -130,9 +136,10 @@ public static function getInstance() : Container
     /**
      * Экземпляр контейнера.
      *
-     * @return Container
+     * @return Container|null
+     * @throws Exception
      */
-    public function getContainer(): Container
+    public function getContainer(): ?Container
     {
         return static::$container;
     }

src/Resource/FileBitrixSettingsResource.php

@@ -40,22 +40,22 @@ public function __construct(string $resource)
      */
     public function __toString(): string
     {
-        return $this->resource;
+        return (string)$this->resource;
     }
 
     /**
      * @return string The canonicalized, absolute path to the resource.
      */
     public function getResource(): string
     {
-        return $this->resource;
+        return (string)$this->resource;
     }
 
     /**
      * {@inheritdoc}
      */
     public function isFresh(int $timestamp): bool
     {
-        return false !== @filemtime($this->resource) && $this->timestamp >= $timestamp;
+        return false !== @filemtime((string)$this->resource) && $this->timestamp >= $timestamp;
     }
 }