collectors, new router, errors handlers, refactoring

mirzaev/minimal/system/controller.php

@@ -4,47 +4,50 @@ declare(strict_types=1);
 
 namespace mirzaev\minimal;
 
-// Файлы проекта
+// Files of the project
-use mirzaev\minimal\model;
+use mirzaev\minimal\model,
+	mirzaev\minimal\traits\magic;
 
 // Встроенные библиотеки
 use exception;
 
 /**
- * Контроллер
+ * Controller (base)
  *
  * @package mirzaev\minimal
+ *
+ * @license http://www.wtfpl.net/ Do What The Fuck You Want To Public License
  * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
  */
 class controller
 {
-  /**
+	use magic;
-   * Постфикс
-   */
-  public const POSTFIX = '_controller';
 
 	/**
-   * Инстанция модели
+	 * Postfix of file names
+	 */
+	public const string POSTFIX = '_controller';
+
+	/**
+	 * Instance of the model connected in the router
 	 */
 	protected model $model;
 
 	/**
-   * Инстанция шаблонизатора представления
+	 * View template engine instance (twig)
 	 */
 	protected object $view;
 
 	/**
-   * Конструктор
+	 * Constructor
 	 */
-  public function __construct()
+	public function __construct() {}
-  {
-  }
 
 	/**
-   * Записать свойство
+	 * Write property
 	 *
-   * @param string $name Название
+	 * @param string $name Name of the property
-   * @param mixed $value Значение
+	 * @param mixed $value Value of the property
 	 *
 	 * @return void
 	 */
@@ -52,71 +55,40 @@ class controller
 	{
 		match ($name) {
 			'model' => (function () use ($value) {
-        if ($this->__isset('model')) throw new exception('Запрещено реинициализировать свойство с инстанцией модели ($this->model)', 500);
+				if ($this->__isset('model')) throw new exception('Can not reinitialize property: ' . static::class . '::$model', 500);
 				else {
-          // Свойство не инициализировано
+					// Property not initialized 
 
 					if (is_object($value)) $this->model = $value;
-          else throw new exception('Свойство $this->model должно хранить инстанцию модели (объект)', 500);
+					else throw new exception('Property "' . static::class . '::view" should store an instance of a model', 500);
 				}
 			})(),
 			'view' => (function () use ($value) {
-        if ($this->__isset('view')) throw new exception('Запрещено реинициализировать свойство с инстанцией шаблонизатора представления ($this->view)', 500);
+				if ($this->__isset('view')) throw new exception('Can not reinitialize property: ' . static::class . '::$view', 500);
 				else {
-          // Свойство не инициализировано
+					// Property not initialized 
 
 					if (is_object($value)) $this->view = $value;
-          else throw new exception('Свойство $this->view должно хранить инстанцию шаблонизатора представления (объект)', 500);
+					else throw new exception('Property "' . static::class . '::view" should store an instance of a view template engine', 500);
 				}
 			})(),
-      default => throw new exception("Свойство \"\$$name\" не найдено", 404)
+			default => throw new exception('Property "' . static::class . "::\$$name\" not found", 404)
 		};
 	}
 
 	/**
-   * Прочитать свойство
+	 * Read property
 	 *
-   * @param string $name Название
+	 * @param string $name Name of the property
 	 *
-   * @return mixed Содержимое
+	 * @return mixed Value of the property
 	 */
 	public function __get(string $name): mixed
 	{
 		return match ($name) {
-      'model' => $this->model ?? throw new exception("Свойство \"\$model\" не инициализировано", 500),
+			'model' => $this->model ?? throw new exception('Property "' . static::class . '::$model" is not initialized', 500),
-      'view' => $this->view ?? throw new exception("Свойство \"\$view\" не инициализировано", 500),
+			'view' => $this->view ?? throw new exception('Property "' . static::class . '::$view" is not initialized', 500),
-      default => throw new exception("Свойство \"\$$name\" не обнаружено", 404)
+			default => throw new exception('Property "' . static::class . "::\$$name\" not found", 404)
-    };
-  }
-
-  /**
-   * Проверить свойство на инициализированность
-   *
-   * @param string $name Название
-   * 
-   * @return bool Инициализировано свойство?
-   */
-  public function __isset(string $name): bool
-  {
-    return match ($name) {
-      default => isset($this->{$name})
-    };
-  }
-
-  /**
-   * Удалить свойство
-   *
-   * @param string $name Название
-   *
-   * @return void
-   */
-  public function __unset(string $name): void
-  {
-    match ($name) {
-      default => (function () use ($name) {
-        // Удаление
-        unset($this->{$name});
-      })()
 		};
 	}
 }

mirzaev/minimal/system/core.php

@@ -14,13 +14,12 @@ use exception,
 	ReflectionClass as reflection;
 
 /**
- * Ядро
+ * Core
  *
  * @package mirzaev\minimal
- * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
  *
- * @todo
+ * @license http://www.wtfpl.net/ Do What The Fuck You Want To Public License
- * 1. Добавить __isset() и __unset()
+ * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
  */
 final class core
 {
@@ -81,9 +80,7 @@ final class core
 	 * Деструктор
 	 *
 	 */
-  public function __destruct()
+	public function __destruct() {}
-  {
-  }
 
 	/**
 	 * Запуск

mirzaev/minimal/system/model.php

@@ -4,86 +4,31 @@ declare(strict_types=1);
 
 namespace mirzaev\minimal;
 
-// Встроенные библиотеки
+// Files of the project
+use	mirzaev\minimal\traits\magic;
+
+// Built-in libraries
 use exception;
 
 /**
- * Модель
+ * Model (base)
  *
  * @package mirzaev\minimal
+ *
+ * @license http://www.wtfpl.net/ Do What The Fuck You Want To Public License
  * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
  */
 class model
 {
-  /**
+	use magic;
-   * Постфикс
-   */
-  public const POSTFIX = '_model';
 
 	/**
-   * Конструктор
+	 * Postfix of file names
 	 */
-  public function __construct()
+	public const string POSTFIX = '_model';
-  {
-  }
 
 	/**
-   * Записать свойство
+	 * Constructor
-   *
-   * @param string $name Название
-   * @param mixed $value Содержимое
-   *
-   * @return void
 	 */
-  public function __set(string $name, mixed $value = null): void
+	public function __construct() {}
-  {
-    match ($name) {
-      default => throw new exception("Свойство \"\$$name\" не найдено", 404)
-    };
-  }
-
-  /**
-   * Прочитать свойство
-   *
-   * @param string $name Название
-   *
-   * @return mixed Содержимое
-   */
-  public function __get(string $name): mixed
-  {
-    return match ($name) {
-      default => throw new exception("Свойство \"\$$name\" не обнаружено", 404)
-    };
-  }
-
-  /**
-   * Проверить свойство на инициализированность
-   *
-   * @param string $name Название
-   *
-   * @return bool Инициализировано свойство?
-   */
-  public function __isset(string $name): bool
-  {
-    return match ($name) {
-      default => isset($this->{$name})
-    };
-  }
-
-  /**
-   * Удалить свойство
-   *
-   * @param string $name Название
-   *
-   * @return void
-   */
-  public function __unset(string $name): void
-  {
-    match ($name) {
-      default => (function () use ($name) {
-        // Удаление
-        unset($this->{$name});
-      })()
-    };
-  }
 }

mirzaev/minimal/system/router.php

@@ -11,6 +11,8 @@ use mirzaev\minimal\core;
  * Маршрутизатор
  *
  * @package mirzaev\minimal
+ *
+ * @license http://www.wtfpl.net/ Do What The Fuck You Want To Public License
  * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
  */
 final class router
@@ -18,7 +20,8 @@ final class router
 	/**
 	 * @var array $router Реестр маршрутов
 	 */
-  protected array $routes = [];
+	/* protected array $routes = []; */
+	public array $routes = [];
 
 	/**
 	 * Записать маршрут
@@ -28,114 +31,190 @@ final class router
 	 * @param ?string $method Вызываемый метод в инстанции контроллера обработчика
 	 * @param ?string $request HTTP-метод запроса (GET, POST, PUT...)
 	 * @param ?string $model Инстанция модели (переопределение инстанции модели в $target)
+	 * @param array $variables 
 	 *
-   * @return void
+	 * @return static The instance from which the method was called (fluent interface)
 	 */
 	public function write(
 		string $route,
 		string $handler,
 		?string $method = 'index',
 		?string $request = 'GET',
-    ?string $model = null
+		?string $model = null,
-  ): void {
+		array $variables = []
+	): static {
 		// Запись в реестр
 		$this->routes[$route][$request] = [
 			'controller' => $handler,
 			'model' => $model ?? $handler,
-      'method' => $method
+			'method' => $method,
+			'variables' => $variables
 		];
+
+		// Exit (success) (fluent interface)
+		return $this;
 	}
 
 	/**
-   * Обработать маршрут
+	 * Handle a request
 	 *
-   * @param ?string $uri URI запроса (https://domain.com/foo/bar)
+	 * @param string|null $uri URI (protocol://domain/foo/bar)
-   * @param ?string $method Метод запроса (GET, POST, PUT...)
+	 * @param string|null $method Method (GET, POST, PUT...)
-	 * @param ?core $core Инстанция системного ядра
+	 * @param core|null $core Instence of the system core
 	 *
-	 * @return string|int|null Ответ
+	 * @return string|int|null Response
 	 */
-  public function handle(?string $uri = null, ?string $method = null, ?core $core = new core): string|int|null
+	public function handle(
-  {
+		?string $uri = null,
-    // Инициализация значений по умолчанию
+		?string $method = null,
-    $uri ??= $_SERVER['REQUEST_URI'] ?? '/';
+		?core $core = new core
-    $method ??= $_SERVER["REQUEST_METHOD"] ?? 'GET';
+	): string|int|null {
-
+		// Declaration of the registry of routes directoies 
-    // Инициализация URL запроса (/foo/bar)
+		$routes = [];
-    $url = parse_url($uri, PHP_URL_PATH);
-
-    // Универсализация маршрута
-    $url = self::universalize($url);
-
-    // Сортировка реестра маршрутов от большего ключа к меньшему (кешируется)
-    krsort($this->routes);
-
-    // Поиск директорий в ссылке
-    preg_match_all('/[^\/]+/', $url, $directories);
-
-    // Инициализация директорий
-    $directories = $directories[0];
 
 		foreach ($this->routes as $route => $data) {
-      // Перебор маршрутов
+			// Iteration over routes
 
-      // Универсализация маршрута
+			// Search directories of route (explode() creates empty value in array)
+			preg_match_all('/(^\/$|[^\/]+)/', $route, $data['directories']);
+			$routes[$route] = $data['directories'][0];
+		}
+
+		if (count($routes) === count($this->routes)) {
+			// Initialized the registry of routes directoies 
+
+			// Initializing default values		
+			$uri ??= $_SERVER['REQUEST_URI'] ?? '/';
+			$method ??= $_SERVER["REQUEST_METHOD"];
+
+			// Initializing of URN (/foo/bar)
+			$urn = parse_url(urldecode($uri), PHP_URL_PATH);
+
+			// Universalization of URN
+			$urn = self::universalize($urn);
+
+			// Search directories of URN (explode() creates empty value in array)
+			preg_match_all('/(^\/$|[^\/]+)/', $urn, $directories);
+			$directories = $directories[0];
+
+			/**
+			 * Initialization of the route
+			 */
+
+			// Initializing the buffer of matches of route directories with URN directories
+			$matches = [];
+
+			foreach ($directories as $i => $urn_directory) {
+				// Iteration over directories of URN 
+
+				foreach ($this->routes as $route => $data) {
+					// Iteration over routes
+
+					if (isset($data[$method])) {
+						// The request method matches the route method
+
+						// Universalization of route
 						$route = self::universalize($route);
 
-      // Поиск директорий
+						// Skipping unmatched routes based on results of previous iterations
-      preg_match_all('/[^\/]+/', $route, $data['directories']);
+						if (isset($matches[$route]) && $matches[$route] === false) continue;
 
-      // Инициализация директорий
+						// Initializing of route directory
-      $data['directories'] = $data['directories'][0];
+						$route_directory = $routes[$route][$i] ?? null;
 
-      if (count($directories) === count($data['directories'])) {
+						if (isset($route_directory)) {
-        // Входит в диапазон маршрут (совпадает количество директорий у ссылки и маршрута)
+							// Initialized of route directory
 
-        // Инициализация реестра переменных
+							if ($urn_directory === $route_directory) {
-        $data['vars'] = [];
+								// The directory of URN is identical to the directory of route
 
-        foreach ($data['directories'] as $index => &$directory) {
+								// Writing: end of URN directories XNOR end of route directories
-          // Перебор директорий
+								$matches[$route] = !(isset($directories[$_i = $i + 1]) xor isset($routes[$route][$_i]));
+							} else if (preg_match('/^\$([a-zA-Z_\x80-\xff]+)$/', $route_directory) === 1) {
+								// The directory of route is a variable ($variable)
 
-          if (preg_match('/\$([a-zA-Z_\x80-\xff][a-zA-Z0-9_\x80-\xff]+)/', $directory) === 1) {
+								// Writing: end of URN directories XNOR end of route directories
-            // Директория является переменной (.../$variable/...)
+								$matches[$route] = !(isset($directories[$_i = $i + 1]) xor isset($routes[$route][$_i]));
+							} else if (
+								!isset($routes[$route][$i + 1])
+								&& preg_match('/^\$([a-zA-Z_\x80-\xff]+\.\.\.)$/', $route_directory) === 1
+							) {
+								// The directory of route is a collector ($variable...)
+								// AND this is the end of route directories
+
+								// Writing
+								$matches[$route] = 'collector';
+							} else $matches[$route] = false;
+						} else if ($matches[$route] === 'collector') {
+						} else $matches[$route] = false;
+					}
+				}
+			}
+
+			// Finding a priority route from match results
+			foreach ($matches as $route => $match) if ($match !== false) break;
+
+			if ($route && !empty($data = $this->routes[$route])) {
+				// Route found
+
+				// Universalization of route
+				$route = self::universalize($route);
+
+				/**
+				 * Initialization of route variables
+				 */
+
+				foreach ($routes[$route] as $i => $route_directory) {
+					// Iteration over directories of route
+
+					if (preg_match('/^\$([a-zA-Z_\x80-\xff]+)$/', $route_directory) === 1) {
+						// The directory is a variable ($variable)
+
+						// Запись в реестр переменных и перещапись директории в маршруте
+						$data[$method]['variables'][trim($route_directory, '$')] = $directories[$i];
+					} else if (preg_match('/^\$([a-zA-Z_\x80-\xff]+\.\.\.)$/', $route_directory) === 1) {
+						// The directory of route is a collector ($variable...)
+
+						// Инициализаия ссылки на массив сборщика
+						$collector = &$data[$method]['variables'][trim($route_directory, '$.')];
+
+						// Инициализаия массива сборщика
+						$collector ??= [];
 
 						// Запись в реестр переменных
-            $directory = $data['vars'][trim($directory, '$')] = $directories[$index];
+						$collector[] = $directories[$i];
+
+						foreach (array_slice($directories, ++$i, preserve_keys: true) as &$urn_directory) {
+							// Перебор директорий URN
+
+							// Запись в реестр переменных
+							$collector[] = $urn_directory;
+						}
+
+						break;
 					}
 				}
 
-        // Реиницилазция маршрута
+				/**
-        $route = self::universalize(implode('/', $data['directories']));
+				 * Initialization of route handlers
-
+				 */
-        // Проверка на пустой маршрут
-        if (empty($route)) $route = '/';
-
-        if (mb_stripos($route, $url, 0, "UTF-8") === 0 && mb_strlen($route, 'UTF-8') <= mb_strlen($url, 'UTF-8')) {
-          // Идентифицирован маршрут (длина не меньше длины запрошенного URL)
 
 				if (array_key_exists($method, $data)) {
 					// Идентифицирован метод маршрута (GET, POST, PUT...)
 
-						$route = $data[$method];
+					// Инициализация обработчиков (controller, model, method)
+					$handlers = $data[$method];
 
-            if (class_exists($controller = $core->namespace . '\\controllers\\' . $route['controller'] . $core->controller::POSTFIX)) {
+					if (class_exists($controller = $core->namespace . '\\controllers\\' . $handlers['controller'] . $core->controller::POSTFIX)) {
 						// Найден контроллер
 
 						// Инициализация инстанции ядра контроллера
 						$controller = new $controller;
 
-              // Инициализация инстанции ядра модели
+						// Вызов связанного с маршрутом метода и возврат (успех)
-              if (class_exists($model = $core->namespace . '\\models\\' . $route['model'] . $core->model::POSTFIX));
+						return $controller->{$handlers['method']}($handlers['variables'] + $_REQUEST, $_FILES, file_get_contents('php://input'));
-
-              // Вызов связанного с маршрутом методв и возврат (успех)
-              return $controller->{$route['method']}($data['vars'] + $_REQUEST, $_FILES);
 					}
 				}
-
-          // Выход из цикла (провал)
-          break;
-        }
 			}
 		}
 
@@ -143,6 +222,68 @@ final class router
 		return $this->error($core);
 	}
 
+	/**
+	 * Sorting routes
+	 *
+	 * 1. Short routes
+	 * 2. Long routes
+	 * 3. Short routes with variables (position of variables from "right" to "left")
+	 * 4. Long routes with variables (position of variables from "right" to "left")
+	 * 5. Short routes with collector
+	 * 6. Long routes with collector
+	 * 7. Short routes with variables and collector (position of variables from "right" to "left" then by amount)
+	 * 8. Long routes with variables and collector (position of variables from "right" to "left")
+	 *
+	 * Добавить чтобы сначала текст потом переменная затем после переменной первыми тексты и в конце перменные опять и так рекурсивно
+	 *
+	 * @return static The instance from which the method was called (fluent interface)
+   *
+	 * @todo ПЕРЕДЕЛАТЬ ПОЛНОСТЬЮ
+	 */
+	public function sort(): static
+	{
+		uksort($this->routes, function (string $a, string $b) {
+			// Sorting routes
+
+			// Initialization of string lengths (multibyte-safe)
+			$length = [
+				$a => mb_strlen($a),
+				$b => mb_strlen($b)
+			];
+
+			// Initialization of the presence of variables
+			$variables = [
+				$a => preg_match('/\$([a-zA-Z_\x80-\xff]+)(\/|$)/', $a) === 1,
+				$b => preg_match('/\$([a-zA-Z_\x80-\xff]+)(\/|$)/', $b) === 1
+			];
+
+			// Initialization of the presence of collectors
+			$collectors = [
+				$a => preg_match('/\$([a-zA-Z_\x80-\xff]+)\.\.\.$/', $a) === 1,
+				$b => preg_match('/\$([a-zA-Z_\x80-\xff]+)\.\.\.$/', $b) === 1
+			];
+
+
+			if ($variables[$a] && !$variables[$b]) return 1;
+			else if (!$variables[$a] && $variables[$b]) return -1;
+			else if ($variables[$a] && $variables[$b]) {
+			} else if ($collectors[$a] && !$collectors[$b]) return 1;
+			else if (!$collectors[$a] && $collectors[$b]) return -1;
+			else {
+				// NOR variables and XAND collectors (both are not found or both are found)
+
+				// The routes are the same length (no changes)
+				if ($length[$a] === $length[$b]) return 0;
+
+				// The longer route moves to the end
+				return $length[$a] > $length[$b] ? 1 : -1;
+			}
+		});
+
+		// Exit (success) (fluent interface)
+		return $this;
+	}
+
 	/**
 	 * Сгенерировать ответ с ошибкой
 	 *
@@ -161,15 +302,17 @@ final class router
 	}
 
 	/**
-   * Универсализировать маршрут 
+	 * Universalize URN 
 	 *
-   * @param string $route Маршрут
+	 * Always "/" at the beginning and never "/" at the end
 	 *
-   * @return string Универсализированный маршрут
+	 * @param string &$urn URN (/foo/bar)
+	 *
+	 * @return string Universalized URN 
 	 */
-  private function universalize(string $route): string
+	private function universalize(string $urn): string
 	{
-    // Если не записан "/" в начале, то записать, затем, если записан "/" в конце, то удалить
+		// Universalization of URN and exit (success)
-    return preg_replace('/(.+)\/$/', '$1', preg_replace('/^([^\/])/', '/$1', $route));
+		return (string) preg_replace('/(.+)\/$/', '$1', preg_replace('/^([^\/])/', '/$1', $urn));
 	}
 }

mirzaev/minimal/system/traits/magic.php

@@ -0,0 +1,81 @@
+<?php
+
+declare(strict_types=1);
+
+namespace mirzaev\minimal\traits;
+
+// Built-in libraries
+use exception;
+
+/**
+ * Trait of magical methods
+ *
+ * @method void __set(string $name, mixed $value = null) Write property
+ * @method mixed __get(string $name) Read property
+ * @method void __unset(string $name) Delete property
+ * @method bool __isset(string $name) Check property for initialization
+ *
+ * @package mirzaev\minimal\traits
+ * @author Arsen Mirzaev Tatyano-Muradovich <arsen@mirzaev.sexy>
+ */
+trait magic
+{
+	/**
+	 * Write property
+	 *
+	 * @param string $name Name of the property
+	 * @param mixed $value Value of the property
+	 *
+	 * @return void
+	 */
+	public function __set(string $name, mixed $value = null): void
+	{
+		match ($name) {
+			default => throw new exception('Property "' . static::class . "::\$$name\" not found", 404)
+		};
+	}
+
+	/**
+	 * Read property
+	 *
+	 * @param string $name Name of the property
+	 *
+	 * @return mixed Value of the property
+	 */
+	public function __get(string $name): mixed
+	{
+		return match ($name) {
+			default => throw new exception('Property "' . static::class . "::\$$name\" not found", 404)
+		};
+	}
+
+	/**
+	 * Delete property
+	 *
+	 * @param string $name Name of the property
+	 *
+	 * @return void
+	 */
+	public function __unset(string $name): void
+	{
+		match ($name) {
+			default => (function () use ($name) {
+				unset($this->{$name});
+			})()
+		};
+	}
+
+	/**
+	 * Check property for initialization
+	 *
+	 * @param string $name Name of the property
+	 *
+	 * @return bool Is the property initialized?
+	 */
+	public function __isset(string $name): bool
+	{
+		return match ($name) {
+			default => isset($this->{$name})
+		};
+	}
+}