BFW\Traits\Memcache

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	ifExists()	0	13	3
A	majExpire()	0	21	2

<?php

namespace BFW\Traits;

trait Memcache
{
    /**
     * Permet de savoir si la clé existe
     * 
     * @param string $key la clé disignant les infos concernées
     * 
     * @throws \Exception Erreur dsans les paramètres donnée à la méthode
     * 
     * @return bool
     */
    public function ifExists($key)
    {
        $verifParams = verifTypeData([['type' => 'string', 'data' => $key]]);
        if (!$verifParams) {
            throw new \Exception('The $key parameters must be a string');
        }

        if ($this->get($key) === false) {

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return false;
        }

        return true;
    }

    /**
     * On modifie le temps avant expiration des infos sur
     * le serveur memcached pour une clé choisie.
     * 
     * @param string $key    la clé disignant les infos concerné
     * @param int    $expire le nouveau temps avant expiration
     *                          (0: pas d'expiration, max 30jours)
     * 
     * @throws \Exception Erreur dsans les paramètres donnée à la méthode
     * 
     * @return boolean|null
     */
    public function majExpire($key, $expire)
    {
        $verifParams = verifTypeData([
            ['type' => 'string', 'data' => $key],
            ['type' => 'int', 'data' => $expire]
        ]);

        if (!$verifParams) {
            throw new \Exception(
                'Once of parameters $key or $expire not have a correct type.'
            );
        }

        $value = $this->get($key); //Récupère la valeur

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        
        //On la "modifie" en remettant la même valeur mais en changeant
        //le temps avant expiration
        $this->replace($key, $value, 0, $expire);

It seems like replace() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        
        return true;
    }
}