Où devrait fonctionner une certaine fonction go-model ou controller? CakePHP

Question

Trips hasMany Legs hasMany Segments

Dans mon application de recherche de vols, j'ai une fonction qui retourne des Leg.destination (s) uniques. Cette fonction est pour une méthode de contrôleur de déclenchement. Est-ce que je mets la fonction dans le modèle de voyage ou de jambe? Si dans le modèle de jambe je l'appelle avec $ this-> Trip-> Leg-> findUniqueDests .....? Je demande parce que je veux coller à la convention de CakePHP. Merci!

//code that finds ALL destinations for each Leg 
$destinations=$this->Trip->Leg->find('all', array('limit'=>100,'fields'=>'Leg.destination')); 

//code that finds the unique destinations (to be used to search all flights for a particular city 
function findUniqueDests($destinations){ 
    $unique_destinations = array(); 
    foreach ($destinations as $dest) 
    { 
     if(!in_array($dest, $unique_destinations)) 
     { 
      $unique_destinations[] = $dest; 
      sort($unique_destinations); 
     } 
    } 
    return $unique_destinations; 

}

Answer 1

Oui, vous mettre dans le modèle Leg. Cela vous permettra d'appeler la méthode de tout autre modèle connexe:

// Trip Controller 
$this->Trip->Leg->findUniqueDests($destinations); 

// Leg Controller 
$this->Leg->findUniqueDests($destinations); 

// Segment Controller 
$this->Segment->Leg->findUniqueDests($destinations); 

Kudos à vous pour savoir qu'il devrait être dans un modèle. Beaucoup de gens commençant par CakePHP entassent toutes leurs méthodes dans les contrôleurs. Le fait de le faire dans le Modèle de cette façon vous permet de réutiliser le code partout dans l'application. En réalité, ce type de fonction d'utilité pourrait être placé dans n'importe quel modèle. Mais puisqu'il s'agit de Legs, la maison la plus logique serait le modèle Leg. Question: Pourquoi trier chaque fois qu'une destination est ajoutée à la baie? Ce serait plus optimisé:

function findUniqueDests($destinations) { 
    $unique_destinations = array(); 
    foreach ($destinations as $dest) { 
     if(!in_array($dest, $unique_destinations)) { 
      $unique_destinations[] = $dest; 
     } 
    } 
    return sort($unique_destinations); 
}