fixed some error

README.md

@@ -5,9 +5,7 @@
 
 [![packagist-badge](https://img.shields.io/packagist/v/swooletw/laravel-swoole.svg)](https://packagist.org/packages/swooletw/laravel-swoole)
 [![Total Downloads](https://poser.pugx.org/swooletw/laravel-swoole/downloads)](https://packagist.org/packages/swooletw/laravel-swoole)
-[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/swooletw/laravel-swoole/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/swooletw/laravel-swoole/?branch=master)
-[![travis-badge](https://api.travis-ci.org/swooletw/laravel-swoole.svg?branch=master)](https://travis-ci.org/swooletw/laravel-swoole)
-
+[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/learning-student/owuan-swoole/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/learning-student/owuan-swoole/?branch=master)[![Build Status](https://travis-ci.com/learning-student/owuan-swoole.svg?branch=master)](https://travis-ci.com/learning-student/owuan-swoole)
 This package provides a high performance HTTP server to speed up your Laravel/Lumen application based on [Swoole](http://www.swoole.com/).
 
 ## Version Compatibility

routes/websocket.php

@@ -17,7 +17,7 @@
     // called while socket on connect
 });
 
-Websocket::on('disconnect', function ($websocket) {
+Websocket::on('disconnect', function () {
     // called while socket on disconnect
 });
 

src/Cache/Caching.php

@@ -4,11 +4,7 @@
 namespace SwooleTW\Http\Cache;
 
 use Anonym\Components\Cache\Cache;
-use Anonym\Components\Cache\CacheInterface;
-use Anonym\Components\Cache\DriverAdapterInterface;
 use Anonym\Components\Cache\DriverNotInstalledException;
-use function foo\func;
-use Illuminate\Http\Request;
 use Illuminate\Http\Response;
 
 /**
@@ -121,27 +117,41 @@ private function checkCacheExists($request): bool
     /**
      * @param $request
      * @return Response
+     * @throws \RuntimeException
      */
     private function getCachedResponse($request): Response
     {
         $uri = $request->server['request_uri'];
 
-        return $this->caching->get($uri);
+        $response = $this->caching->get($uri);
 
+
+        if (!$response instanceof Response) {
+            throw new \RuntimeException('Cache removed');
+        }
+
+        return $response;
     }
 
+
     /**
      * @return \Closure
      */
     public function cachePreEvent(): callable
     {
         return function ($request) {
 
+
             if (!$this->checkCacheExists($request)) {
                 return false;
             }
 
-            return $this->getCachedResponse($request);
+            try {
+                return $this->getCachedResponse($request);
+
+            } catch (\RuntimeException $exception) {
+                return false;
+            }
 
         };
     }

src/Cache/SwooleLumenRouter.php

@@ -23,9 +23,7 @@ public function addRoute($method, $uri, $action)
 
         // save only GET routes
         if ($method === 'GET' && is_array($action) && isset($action['cache'])) {
-
-            $caching = $this->app->make(Caching::class)->cacheRoute($uri);
-
+            $this->app->make(Caching::class)->cacheRoute($uri);
         }
     }
 }

src/Commands/HttpServerCommand.php

@@ -65,7 +65,11 @@ class HttpServerCommand extends Command
      */
     public function handle()
     {
-        $this->checkEnvironment();
+        if (!$this->checkEnvironment()) {
+            return;
+        }
+
+
         $this->loadConfigs();
         $this->initAction();
         $this->runAction();
@@ -135,7 +139,7 @@ protected function stop()
     {
         $pid = $this->getCurrentPid();
 
-        if (! $this->isRunning($pid)) {
+        if (!$this->isRunning($pid)) {
             $this->error("Failed! There is no swoole_http_server process running.");
 
             return;
@@ -179,7 +183,7 @@ protected function reload()
     {
         $pid = $this->getCurrentPid();
 
-        if (! $this->isRunning($pid)) {
+        if (!$this->isRunning($pid)) {
             $this->error("Failed! There is no swoole_http_server process running.");
 
             return;
@@ -189,7 +193,7 @@ protected function reload()
 
         $isRunning = $this->killProcess($pid, SIGUSR1);
 
-        if (! $isRunning) {
+        if (!$isRunning) {
             $this->error('> failure');
 
             return;
@@ -246,7 +250,7 @@ protected function initAction()
     {
         $this->action = $this->argument('action');
 
-        if (! in_array($this->action, ['start', 'stop', 'restart', 'reload', 'infos'], true)) {
+        if (!in_array($this->action, ['start', 'stop', 'restart', 'reload', 'infos'], true)) {
             $this->error(
                 "Invalid argument '{$this->action}'. Expected 'start', 'stop', 'restart', 'reload' or 'infos'."
             );
@@ -256,12 +260,13 @@ protected function initAction()
     }
 
     /**
-     * @param \SwooleTW\Http\Server\Facades\Server $server
+     * @param Server $server
      *
      * @return \Swoole\Process
      */
     protected function getHotReloadProcess($server)
     {
+
         $recursively = Arr::get($this->config, 'hot_reload.recursively');
         $directory = Arr::get($this->config, 'hot_reload.directory');
         $filter = Arr::get($this->config, 'hot_reload.filter');
@@ -284,7 +289,7 @@ protected function getHotReloadProcess($server)
      */
     protected function isRunning($pid)
     {
-        if (! $pid) {
+        if (!$pid) {
             return false;
         }
 
@@ -312,7 +317,7 @@ protected function killProcess($pid, $sig, $wait = 0)
             $start = time();
 
             do {
-                if (! $this->isRunning($pid)) {
+                if (!$this->isRunning($pid)) {
                     break;
                 }
 
@@ -337,7 +342,7 @@ protected function getCurrentPid()
         $path = $this->getPidPath();
 
         return $this->currentPid = file_exists($path)
-            ? (int) file_get_contents($path) ?? $this->removePidFile()
+            ? (int)file_get_contents($path) ?? $this->removePidFile()
             : null;
     }
 
@@ -372,25 +377,28 @@ protected function isDaemon(): bool
     /**
      * Check running enironment.
      */
-    protected function checkEnvironment()
+    protected function checkEnvironment(): bool
     {
         if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') {
             $this->error("Swoole extension doesn't support Windows OS yet.");
 
-            exit(1);
+            return false;
         }
 
-        if (! extension_loaded('swoole')) {
+        if (!extension_loaded('swoole')) {
             $this->error("Can't detect Swoole extension installed.");
 
-            exit(1);
+            return false;
         }
 
-        if (! version_compare(swoole_version(), '4.0.0', 'ge')) {
+        if (!version_compare(swoole_version(), '4.0.0', 'ge')) {
             $this->error("Your Swoole version must be higher than 4.0 to use coroutine.");
 
-            exit(1);
+            return false;
         }
+
+        return true;
+
     }
 
     /**

src/Server/Facades/Server.php

@@ -8,6 +8,7 @@
  * Class Server
  *
  * @mixin \Swoole\Http\Server
+ * @method void reload()
  */
 class Server extends Facade
 {

src/Server/helpers.php

@@ -3,6 +3,15 @@
 use SwooleTW\Http\Event\EventBase;
 use App\Events\Event;
 
+/**
+ *  if swoole_version not provided,
+ */
+if (!function_exists('swoole_version')) {
+    function swoole_version(): string {
+        return '0';
+    }
+}
+
 /**
  * This is only for `function not exists` in config/swoole_http.php.
  */