Simplifier cette méthode avec récursion

Question

je dois simplifier cette méthode avec une récursion pour se débarrasser de la logique métier en double mais je ne peux pas comprendre comment faire:

public function compute() 
{ 
    $ret = array(); 
    foreach ($this->_items as $item) { 
     $ret[] = array($item); 
    } 
    foreach ($this->_items as $item) { 
     foreach ($this->_items as $item2) { 
      $tmp = array($item, $item2); 
      if (count($tmp) === count(array_unique($tmp))) { 
       $ret[] = $tmp; 
      } 
     } 
    } 
    foreach ($this->_items as $item) { 
     foreach ($this->_items as $item2) { 
      foreach ($this->_items as $item3) { 
       $tmp = array($item, $item2, $item3); 
       if (count($tmp) === count(array_unique($tmp))) { 
        $ret[] = $tmp; 
       } 
      } 
     } 
    } 
    return $ret; 
} 

EDIT:

Cette méthode est censé revenir toutes les combinaisons d'éléments du tableau, donc si vous avez tableau comme:

[a, b, c] 

Il retournera:

[ 
    [a], 
    [b], 
    [c], 
    [a, b], 
    [a, c], 
    [b, a], 
    [b, c], 
    [a, b, c], 
    [a, c, b], 
    [b, a, c], 
    [b, c, a], 
    [c, a, b], 
    [c, b, a] 
] 

Answer 1

Pour votre calcul, il n'y a pas besoin de récursion pour rationaliser ce que vous appelez la logique métier ici. Au moins pas au début. C'est déjà suffisant pour déplacer le code en double dans une fonction de son propre et ensuite faire le traitement.

Je suggère également que cette première étape en raison de l'ordre d'exécution que vous avez ici:

public function compute() 
{ 

    $ret = array(); 

    foreach ($this->_items as $item) { 
     $ret[] = array($item); 
    } 

    $each = function(array $tmp) use (&$ret) { 
     if (count($tmp) === count(array_unique($tmp))) { 
      $ret[] = $tmp; 
     } 
    } 

    foreach ($this->_items as $item) { 
     foreach ($this->_items as $item2) { 
      $each(array($item, $item2)); 
     } 
    } 

    foreach ($this->_items as $item) { 
     foreach ($this->_items as $item2) { 
      foreach ($this->_items as $item3) { 
       $each(array($item, $item2, $item3)); 
      } 
     } 
    } 

    return $ret; 
}