feat: ✨ Add exception and change start session

Author: syfxlin

.env.example

@@ -1,4 +1,6 @@
-APP_KEY=
+APP_ENV=production
+APP_NAME=XK-PHP
+APP_URL=http://localhost
 
 DB_CONNECTION=mysql
 DB_HOST=127.0.0.1

app/Application.php

@@ -3,6 +3,7 @@
 namespace App;
 
 use App\Bootstrap\BootProviders;
+use App\Bootstrap\HandleExceptions;
 use App\Bootstrap\LoadConfiguration;
 use App\Bootstrap\LoadEnvironmentVariables;
 use App\Bootstrap\RegisterFacades;
@@ -11,13 +12,11 @@
 use App\Kernel\ProviderManager;
 use App\Kernel\Container;
 use App\Kernel\RouteManager;
-use Laminas\HttpHandlerRunner\Emitter\SapiEmitter;
-use Psr\Http\Message\ResponseInterface;
 use function app_path;
 use function array_walk;
 use function base_path;
-use function config;
 use function config_path;
+use function env;
 use function public_path;
 use function realpath;
 use function storage_path;
@@ -43,6 +42,8 @@ class Application extends Container
         LoadEnvironmentVariables::class,
         // 加载配置
         LoadConfiguration::class,
+        // 注册异常处理
+        HandleExceptions::class,
         // 注册门面
         RegisterFacades::class,
         // 注册服务提供者管理器
@@ -104,7 +105,7 @@ protected function dispatchToEmit(): void
         $response = $this->make(RouteManager::class)->dispatch($request);
 
         // 发送响应
-        (new SapiEmitter())->emit($response);
+        $response->send();
     }
 
     /**
@@ -141,4 +142,13 @@ public static function setInstance(Application $application): void
     {
         self::$app = $application;
     }
+
+    public function environment(string $env = null)
+    {
+        $app_env = env('APP_ENV', 'production');
+        if ($env === null) {
+            return $app_env;
+        }
+        return $app_env === $env;
+    }
 }

app/Bootstrap/HandleExceptions.php

@@ -0,0 +1,95 @@
+<?php
+
+namespace App\Bootstrap;
+
+use App\Exceptions\Exception;
+use App\Http\Request;
+use App\Http\Response;
+use ErrorException;
+use Throwable;
+use function in_array;
+use function response;
+use function view;
+
+class HandleExceptions extends Bootstrap
+{
+    public function boot(): void
+    {
+        if (!$this->app->environment('testing')) {
+            error_reporting(-1);
+            set_error_handler([$this, 'handleError']);
+            set_exception_handler([$this, 'handleException']);
+            register_shutdown_function([$this, 'handleDown']);
+            ini_set('display_errors', 'Off');
+        }
+    }
+
+    public function handleError(
+        $level,
+        $message,
+        $file = '',
+        $line = 0,
+        $context = []
+    ): void {
+        if (error_reporting() & $level) {
+            throw new ErrorException($message, 0, $level, $file, $line);
+        }
+    }
+
+    public function handleException(Throwable $e): void
+    {
+        $this->renderResponse($e)->send();
+    }
+
+    protected function renderResponse(Throwable $e): Response
+    {
+        $request = $this->app->make(Request::class);
+        if ($e instanceof Exception) {
+            return $e->render();
+        }
+        return $this->renderExceptionResponse($request, $e);
+    }
+
+    protected function renderExceptionResponse(
+        Request $request,
+        Throwable $e
+    ): Response {
+        $code = $e->getCode() === 0 ? 500 : $e->getCode();
+        $message =
+            $e->getMessage() === ''
+                ? Response::$phrases[$code]
+                : $e->getMessage();
+        $content = [
+            'status' => $code,
+            'message' => $message,
+            'errors' => $message
+        ];
+        if (!$request->ajax()) {
+            $content = view('errors/errors', $content);
+        }
+        return response($content, $code);
+    }
+
+    public function handleDown(): void
+    {
+        $error = error_get_last();
+        if (
+            $error !== null &&
+            in_array(
+                $error['type'],
+                [E_COMPILE_ERROR, E_CORE_ERROR, E_ERROR, E_PARSE],
+                true
+            )
+        ) {
+            $this->handleException(
+                new ErrorException(
+                    $error['message'],
+                    $error['type'],
+                    0,
+                    $error['file'],
+                    $error['line']
+                )
+            );
+        }
+    }
+}

app/Controllers/HomeController.php

@@ -11,6 +11,7 @@
 use App\Annotations\Route;
 use App\Annotations\Autowired\Autowired;
 use ReflectionClass;
+use RuntimeException;
 
 class HomeController
 {
@@ -66,4 +67,15 @@ public function get(Request $request): bool
     {
         return true;
     }
+
+    /**
+     * @param Request $request
+     * @return bool
+     *
+     * @Route\Get("/exce")
+     */
+    public function exception(Request $request): bool
+    {
+        throw new RuntimeException('Error');
+    }
 }

app/Exceptions/Exception.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace App\Exceptions;
+
+use App\Http\Response;
+use RuntimeException;
+
+abstract class Exception extends RuntimeException
+{
+    abstract public function render(): Response;
+}

app/Exceptions/HttpException.php

@@ -0,0 +1,34 @@
+<?php
+
+namespace App\Exceptions;
+
+use App\Http\Response;
+use Throwable;
+
+class HttpException extends Exception
+{
+    /**
+     * @var Response
+     */
+    protected $response;
+
+    public function __construct(
+        Response $response,
+        $message = '',
+        $code = 0,
+        Throwable $previous = null
+    ) {
+        $this->response = $response;
+        parent::__construct($message, $code, $previous);
+    }
+
+    public function getResponse(): Response
+    {
+        return $this->response;
+    }
+
+    public function render(): Response
+    {
+        return $this->getResponse();
+    }
+}

app/Http/CookieManager.php

@@ -14,14 +14,6 @@ class CookieManager
      */
     private $response_cookies = [];
 
-    /**
-     * @return CookieManager
-     */
-    public static function make(): CookieManager
-    {
-        return new static();
-    }
-
     /**
      * 判断 Cookie 是否存在，并是否为空
      *

app/Http/Response.php

@@ -4,6 +4,7 @@
 
 use App\Facades\App;
 use App\Kernel\View;
+use Laminas\HttpHandlerRunner\Emitter\SapiEmitter;
 use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\StreamInterface;
 use function get_class;
@@ -109,11 +110,6 @@ class Response implements ResponseInterface
      */
     private $cookies;
 
-    /**
-     * @var int[]
-     */
-    private $accept_code = [];
-
     /**
      * Response constructor.
      * @param string|StreamInterface $content
@@ -141,12 +137,6 @@ public function __construct(
             $this->stream = $content;
         }
         $this->cookies = Cookie::makeFromArray($this->getHeader('Set-Cookie'));
-        // Has session
-        $session_cookie = SessionManager::makeCookie();
-        if ($session_cookie !== null) {
-            $this->cookies[$session_cookie->getName()] = $session_cookie;
-            $this->updateCookieHeader();
-        }
     }
 
     /**
@@ -234,6 +224,11 @@ public static function make(
         return new static($content, $status, $headers);
     }
 
+    public function send(): void
+    {
+        (new SapiEmitter())->emit($this);
+    }
+
     /**
      * @inheritDoc
      */
@@ -357,10 +352,10 @@ public function hasCookie(string $name): bool
      * @param Cookie $cookie
      * @return Response
      */
-    public function withCookie(string $name, Cookie $cookie): Response
+    public function withCookie(Cookie $cookie): Response
     {
         $new = clone $this;
-        $new->cookies[$name] = $cookie;
+        $new->cookies[$cookie->getName()] = $cookie;
         $new->updateCookieHeader();
         return $new;
     }
@@ -398,7 +393,6 @@ public function cookie(
         bool $http_only = false
     ): Response {
         return $this->withCookie(
-            $name,
             Cookie::make($name, $value)
                 ->withMaxAge($expire)
                 ->withPath($path)
@@ -416,7 +410,7 @@ public function withCookies(array $cookies): Response
     {
         $result = clone $this;
         foreach ($cookies as $cookie) {
-            $result = $result->withCookie($cookie->getName(), $cookie);
+            $result = $result->withCookie($cookie);
         }
         return $result;
     }
@@ -430,13 +424,6 @@ public function withCookies(array $cookies): Response
      */
     private function convert($content): string
     {
-        if (
-            !in_array($this->status, $this->accept_code) &&
-            $this->status >= 400 &&
-            !App::make(Request::class)->ajax()
-        ) {
-            return view('errors/errors', $content)->render();
-        }
         if ($content === null) {
             return '';
         }