Question

更新

根据下面接受的答案的建议，我测试了一个负数：

$negativeInt = -1;
Cache::write('my-test-count', $negativeInt, 'short');
$readVal = Cache::read('my-test-count', 'short');
debug($readVal);
exit;

尝试读取任何负数时，unseralize错误会一致地重现。它现在是一个被接受的错误，我认为它将由2.8.1

解决

原始问题：

我一直收到此错误，但无法弄清楚为什么甚至如何进一步排查问题。

如果Cache::read()返回false，则只会触发抛出错误的行。但是，如果我没有在它之前放置@，那么该行本身会抛出一个反序列化的错误。

问题： 如何在不定期发布反序列化通知的情况下可靠地使用Redis进行计数？如果密钥中的数据是＆＃34;坏＆＃34;，我怎么知道没有收到通知只是做::read。我已经尝试确保我的数据是（int）（见下文），但这似乎没什么帮助。

注意（8）：unserialize（）：偏移量为0的2字节错误 [APP /供应商/ pear-pear.cakephp.org / CakePHP的/蛋糕/缓存/引擎/ RedisEngine.php，第136行

检查错误后：

> unserialize - [internal], line ??
> RedisEngine::read() - APP/Vendor/pear-pear.cakephp.org/CakePHP/Cake/Cache/Engine/RedisEngine.php, line 136
> Cache::read() - APP/Vendor/pear-pear.cakephp.org/CakePHP/Cake/Cache/Cache.php, line 358
> Cache::remember() - APP/Vendor/pear-pear.cakephp.org/CakePHP/Cake/Cache/Cache.php, line 567
> Item::getCount() - APP/Model/Item.php, line 812

它似乎来自这个功能：

public function getCount($id) {
    $model = $this;

    // here is where the Cache::read() and debug are in the update below

    return Cache::remember('item' . $id. '_count', function() use ($model, $id) {
        $count = $model->find('count', array(
            'conditions' => array(
                $model->alias . '.status' => 1,
                $model->alias . '.id' => $id
            )
        ));
        return ($count === false) ? 0 : (int)$count;
    }, 'my_counts'); // THIS IS LINE 812
}

public function decrementCount($id, $offset = 1) {
    if(empty($id)) return false;
    $count = @Cache::read('item' . $id . '_count', 'my_counts');
    if($count === false) {
        $this->getCount($id);
    } else {
        Cache::decrement('item' . $id . '_count', $offset, 'my_counts');
    }
}

public function incrementCount($id, $offset = 1) {
    if(empty($id)) return false;
    $count = @Cache::read('item' . $id. '_count', 'my_counts');
    if($count === false) {
        $this->getCount($id);
    } else {
        Cache::increment('item' . $id. '_count', $offset, 'my_counts');
    }
}

更新

此函数在循环中运行（通过1-20项）。当我在Cache::remember(...：

之前添加以下内容时

$toReturn = Cache::read('item' . $id. '_count', 'my_counts');
debug($toReturn);

它给出了这个：

debug：（int）0
debug：（int）0
注意（8）：unserialize（）：错误在...（来自Cache :: read）
debug：false
注意（8）：unserialize（）：错误在...（来自Cache :: remember
debug：（int）0
注意（8）：unserialize（）：错误在...（来自Cache :: read）
debug：false
注意（8）：unserialize（）：错误在...（来自Cache :: remember
0
3
1
1 ...

Answer 1

这似乎是一个核心错误。我或许可以回答为什么会出现这个问题。

我猜这个问题是由int(-1)引起的。在写入数据时，如果数据是整数，则RedisEngine不会对数据进行序列化。因此，int(-1)将在不调用serialize()的情况下保存。

public function write($key, $value, $duration) {
    if (!is_int($value)) {
        $value = serialize($value);
    }
    ...
}

但在阅读数据时，实施似乎是不对称的。我不太了解Redis，但我猜Redis将int(-1)存储为string(-1)。当ctype_digit("-1")返回false时，string(-1)会被错误地反序列化。

public function read($key) {
    $value = $this->_Redis->get($key);
    if (ctype_digit($value)) { // ctype_digit("-1") === false
        $value = (int)$value;
    }
    if ($value !== false && is_string($value)) { // is_string("-1") === true
        $value = unserialize($value);
    }
    ...
}

因此，您将看到＆＃34;注意（8）：unserialize（）：偏移0处的错误为2个字节＆＃34;。 string(-1)的大小为2个字节。

如果你可以复制写作int(-1)并阅读它会引发通知，那么在github上打开一个问题呢。

除此之外，解决方法将停止使用Cache::decrement()。在竞争条件下，该方法将存储int(-1)。如果它不是重要数据，您可以使用Cache::write()。例如：

$count = Cache::read('item' . $id. '_count', 'my_counts');
if ($count === false || 0 > $count - $offset) {
    $this->getCount($id);
} else {
    Cache::write('item' . $id. '_count', $count - $offset, 'my_counts');
}

但如果它是重要数据，那么您可能需要实现独占锁定/解锁。

感谢您阅读此内容，如果我错了，请抱歉。

Answer 2

解决方案：

return ($count === false) ? 0 : (int)$count;

长版：

您不应该递增/递减序列化值。

Cache :: remember会做这样的事情：

$key = 'key';
$value = 10;
$serialized = serialize($value); // $serialized = 'i:10;'
$redis->set($key, $serialized);

Cache :: read做的事情如下：

$key = 'key';
$serialized = $redis->get($key); // $serialized = 'i:10;'
$value = unserialize($serialized); // $value = (int) 10
return $value;

但是Cache :: decrement做了这样的事情：

$key = 'key';
$count = 1;
$redis->decrby($key, $count); // Error, decrby expects plain value, "10", not "i:10;" ( http://redis.io/commands/decrby )

此外，如果您成功减少，那么您将在redis中具有普通值，因此结果将为：

$key = 'key';
$serialized = $redis->get($key); // $serialized = '10'
$value = unserialize($serialized); // error, cannot unserialize '10'

修改检查来源后： https://github.com/cakephp/cakephp/blob/master/src/Cache/Engine/RedisEngine.php

我看到Cake会检查：

if (!is_int($value)) { $value = serialize($value); }

因此，为避免序列化，您需要做的就是确保存储（int）。

Answer 3

只需删除第三个参数，它应该是正常工作

return Cache::remember('item' . $id. '_count', function() use ($model, $id) {
    $count = $model->find('count', array(
        'conditions' => array(
            $model->alias . '.status' => 1,
            $model->alias . '.id' => $id
        )
    ));
    return ($count === false) ? 0 : (int)$count;
});

注意通过Redis

3 个答案: