Merge pull request #15 from byjg/5.0

Add GarbageCollectorInterface.php

.github/workflows/phpunit.yml

@@ -24,8 +24,12 @@ jobs:
     services:
       memcached:
         image: memcached
+        ports:
+          - "11211:11211"
       redis:
         image: redis
+        ports:
+          - "6379:6379"
         options: >-
           --health-cmd "redis-cli ping"
           --health-interval 10s

README.md

@@ -37,7 +37,7 @@ PSR-6 Getting Started: [here](docs/basic-usage-psr6-cachepool.md)
 | [\ByJG\Cache\Psr16\TmpfsCacheEngine](docs/class-tmpfs-cache-engine.md)                           | Uses the Tmpfs as the cache engine                                     |
 | [\ByJG\Cache\Psr16\RedisCachedEngine](docs/class-redis-cache-engine.md)                          | uses the Redis as cache                                                |
 | [\ByJG\Cache\Psr16\SessionCachedEngine](docs/class-session-cache-engine.md)                      | uses the PHP session as cache                                          |
-| [\ByJG\Cache\Psr16\ShmopCachedEngine](docs/class-shmop-cache-engine.md)                          | uses the shared memory area for cache                                  |
+| [\ByJG\Cache\Psr16\ShmopCacheEngine](docs/class-shmop-cache-engine.md) (deprecated)              | uses the shared memory area for cache. Use TmpfsCacheEngine.           |
 
 
 ## Logging cache commands
@@ -52,6 +52,17 @@ You can use a PSR-11 compatible to retrieve the cache keys.
 
 See more [here](docs/psr11-usage.md)
 
+## Beyond the PSR protocol
+
+The PSR protocol is a good way to standardize the cache access, 
+but sometimes you need to go beyond the protocol.
+
+Some cache engines have additional features that are not covered by the PSR protocol.
+
+Some examples are:
+- [Atomic Operations](docs/atomic-operations.md)
+- [Garbage Collection](docs/garbage-collection.md)
+
 ## Install
 
 Just type: 

docker-compose.yml

@@ -0,0 +1,17 @@
+services:
+  memcached:
+    image: memcached
+    container_name: memcached
+    ports:
+      - "11211:11211"
+
+  redis:
+    image: redis
+    container_name: redis
+    ports:
+      - "6379:6379"
+    healthcheck:
+      test: ["CMD", "redis-cli", "ping"]
+      interval: 10s
+      timeout: 5s
+      retries: 5

docs/atomic-operations.md

@@ -0,0 +1,56 @@
+# Atomic Operations
+
+Some cache engines allow you to do atomic operations such as incrementing or decrementing a value.
+
+Besides this is not cache operation, it is a common operation in cache engines.
+
+The advantage of using atomic operations is that you can avoid race conditions when multiple processes 
+are trying to update the same value.
+
+The atomic operations are:
+- Increment: Increment a value by a given number
+- Decrement: Decrement a value by a given number
+- Add: Add a value to a list in the cache
+
+The engines that support atomic operations have to implement the `AtomicOperationInterface`.
+
+Some engines that support atomic operations are:
+- RedisCachedEngine
+- MemcachedEngine
+- TmpfsCacheEngine
+- FileSystemCacheEngine
+
+## Increment
+
+The increment operation is used to increment a value by a given number.
+
+```php
+<?php
+/** @var \ByJG\Cache\AtomicOperationInterface $cache */
+$cache->increment('my-key', 1);
+```
+
+## Decrement
+
+The decrement operation is used to decrement a value by a given number.
+
+```php
+<?php
+/** @var \ByJG\Cache\AtomicOperationInterface $cache */
+$cache->decrement('my-key', 1);
+```
+
+## Add
+
+The add operation is used to add a value to a list in the cache.
+
+```php
+<?php
+/** @var \ByJG\Cache\AtomicOperationInterface $cache */
+$cache->add('my-key', 'value1');
+$cache->add('my-key', 'value2');
+$cache->add('my-key', 'value3');
+
+print_r($cache->get('my-key')); // ['value1', 'value2', 'value3']
+```
+

docs/garbage-collection.md

@@ -0,0 +1,30 @@
+# Garbage Collection
+
+Some cache engines need to have a garbage collection process to remove the expired keys.
+
+In some engines like `Memcached` and `Redis` the garbage collection is done automatically by the engine itself.
+
+In other engines like `FileSystem` and `Array` there is no such process. The current implementation
+is based on the Best Effort. It means an expired key is removed only when you try to access it.
+
+If the cache engine has a low hit rate, it is recommended to run a garbage collection process
+to avoid the cache to grow indefinitely.
+
+The classes that implement the `GarbageCollectionInterface` have the method `collectGarbage()`.
+
+Some engines that support garbage collection are:
+- FileSystemCacheEngine
+- ArrayCacheEngine
+- TmpfsCacheEngine
+
+## Example
+
+```php
+<?php
+/** @var \ByJG\Cache\GarbageCollectionInterface $cache */
+$cache->collectGarbage();
+```
+
+Note: The garbage collection process is blocking. 
+It means the process will be slow if you have a lot of keys to remove.
+

src/AtomicOperationInterface.php

@@ -0,0 +1,14 @@
+<?php
+
+namespace ByJG\Cache;
+
+use DateInterval;
+
+interface AtomicOperationInterface
+{
+    public function increment(string $key, int $value = 1, DateInterval|int|null $ttl = null): int;
+
+    public function decrement(string $key, int $value = 1, DateInterval|int|null $ttl = null): int;
+
+    public function add(string $key, $value, DateInterval|int|null $ttl = null): array;
+}

src/Factory.php

@@ -70,10 +70,10 @@ public static function createRedisCacheEngine(?string $servers = null, ?string $
         );
     }
 
-    public static function createTmpfsCachePool(?LoggerInterface $logger = null): CachePool
+    public static function createTmpfsCachePool(string $prefix = 'cache', ?LoggerInterface $logger = null): CachePool
     {
         return new CachePool(
-            new TmpfsCacheEngine($logger)
+            new TmpfsCacheEngine($prefix, $logger)
         );
     }
 

src/GarbageCollectorInterface.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace ByJG\Cache;
+
+interface GarbageCollectorInterface
+{
+    public function collectGarbage();
+
+    public function getTtl(string $key): ?int;
+}

src/Psr16/ArrayCacheEngine.php

@@ -3,16 +3,19 @@
 namespace ByJG\Cache\Psr16;
 
 use ByJG\Cache\Exception\InvalidArgumentException;
+use ByJG\Cache\GarbageCollectorInterface;
 use DateInterval;
 use Psr\Container\ContainerExceptionInterface;
 use Psr\Container\NotFoundExceptionInterface;
 use Psr\Log\LoggerInterface;
 use Psr\Log\NullLogger;
 
-class ArrayCacheEngine extends BaseCacheEngine
+class ArrayCacheEngine extends BaseCacheEngine implements GarbageCollectorInterface
 {
 
-    protected array $cache = [];
+    protected array $cache = [
+        "ttl" => []
+    ];
 
     protected LoggerInterface|null $logger = null;
 
@@ -41,7 +44,7 @@ public function has(string $key): bool
     {
         $key = $this->getKeyFromContainer($key);
         if (isset($this->cache[$key])) {
-            if (isset($this->cache["$key.ttl"]) && time() >= $this->cache["$key.ttl"]) {
+            if (isset($this->cache['ttl']["$key"]) && time() >= $this->cache["ttl"]["$key"]) {
                 $this->delete($key);
                 return false;
             }
@@ -93,7 +96,7 @@ public function set(string $key, mixed $value, DateInterval|int|null $ttl = null
 
         $this->cache[$key] = serialize($value);
         if (!empty($ttl)) {
-            $this->cache["$key.ttl"] = $this->addToNow($ttl);
+            $this->cache["ttl"]["$key"] = $this->addToNow($ttl);
         }
 
         return true;
@@ -116,12 +119,32 @@ public function delete(string $key): bool
         $key = $this->getKeyFromContainer($key);
 
         unset($this->cache[$key]);
-        unset($this->cache["$key.ttl"]);
+        unset($this->cache["ttl"]["$key"]);
         return true;
     }
 
     public function isAvailable(): bool
     {
         return true;
     }
+
+    public function collectGarbage()
+    {
+        foreach ($this->cache["ttl"] as $key => $ttl) {
+            if (time() >= $ttl) {
+                unset($this->cache[$key]);
+                unset($this->cache["ttl"]["$key"]);
+            }
+        }
+    }
+
+    public function getTtl(string $key): ?int
+    {
+        $key = $this->getKeyFromContainer($key);
+        if (isset($this->cache["ttl"]["$key"])) {
+            return $this->cache["ttl"]["$key"];
+        }
+
+        return null;
+    }
 }