Question

我正在创建这个简单的PHP，尝试使用Predis库在Redis上缓存请求，我在类外部实例化了$ redis对象，并且需要从类函数内部调用它。但是我不断收到Undefined变量：redis消息。

我已经尝试过从类范围的内部和外部调用缓存验证功能。

代码如下：

<?php
require "predis/src/Autoloader.php";
Predis\Autoloader::register();

 global $redis;

 $redis = new Predis\Client();

class SimpleJsonRequest
{
    private $redisLocal;

    public function __construct() {
        global $redis;
        $this->redisLocal =& $redis;
    }

    private static function makeRequest(string $method, string $url, array $parameters = null, array $data = null)
    {
        $opts = [
            'http' => [
                'method'  => $method,
                'header'  => 'Content-type: application/json',
                'content' => $data ? json_encode($data) : null
            ]
        ];
        $url .= ($parameters ? '?' . http_build_query($parameters) : '');
        return file_get_contents($url, false, stream_context_create($opts));
    }
    public static function get(string $url, array $parameters = null)
    {
        if(validateCache('GET', $url, $parameters))
            return json_decode(self::makeRequest('GET', $url, $parameters));
    }
    public static function post(string $url, array $parameters = null, array $data)
    {
        if(validateCache('POST', $url, $parameters, $data))
            return json_decode(self::makeRequest('POST', $url, $parameters, $data));
    }
    public static function put(string $url, array $parameters = null, array $data)
    {
        if(validateCache('PUT', $url, $parameters, $data))
            return json_decode(self::makeRequest('PUT', $url, $parameters, $data));
    }   
    public static function patch(string $url, array $parameters = null, array $data)
    {
        if(validateCache('PATCH', $url, $parameters, $data))
            return json_decode(self::makeRequest('PATCH', $url, $parameters, $data));
    }
    public static function delete(string $url, array $parameters = null, array $data = null)
    {
        if(validateCache('DELETE', $url, $parameters, $data))
            return json_decode(self::makeRequest('DELETE', $url, $parameters, $data));
    }
}

function validateCache(string $method, string $url, array $parameters = null, array $data = null)
{
    if($redis->exists($method)
        && $redis->get($method, 'url') == $url 
        && json_decode($redis->get($method, 'parameters')) == $parameters
        && json_decode($redis->get($method, 'data')) == $data)
    {
        echo'request cached';
        return false;
    }
    else
    {
        $redis->hMset($method, [
                        'url' => $url,
                        'parameters' => $parameters = null ? null : json_encode($parameters),
                        'data' => $data = null ? null : json_encode($data)
                    ]);

        $redis->expire($method, 10);

        echo 'not cached';
        return true;
    }
}

$test = new SimpleJsonRequest();
print_r($redis);
echo $test->get('1');
echo $test->get('1');

我希望它先打印“未缓存”，然后再打印“请求缓存”，但我一直收到未定义变量错误。

Answer 1

您的validateCache函数在引用$redis时没有像在SimpleJsonRequest构造函数中那样先声明为全局变量。

Answer 2

停止使用global。这是普遍的坏习惯，而且在课堂上太可怕了。只需传递您需要的内容即可：

class SimpleJsonRequest
{
    public function __construct($redis) {
        $this->redis = $redis;
    }
}

$redis = new Predis\Client();
$test = new SimpleJsonRequest($redis);

然后在需要的地方使用$this->redis，例如validateCache：

if($this->redis->exists($method)
    && $this->redis->get($method, 'url') == $url 
    && json_decode($this->redis->get($method, 'parameters')) == $parameters
    && json_decode($this->redis->get($method, 'data')) == $data)
{
    // rest of code
}

如何在类内部调用全局对象函数，得到“未定义变量”错误

2 个答案: