• Je me demandais si je devais faire une méthode qui vérifiait si la Seelie en question n'était-elle pas déjà dans un Vol par mesure de sécurité ?
  
  Je m'explique : la classe Orpheline n'est chargée que quand le constructeur de la classe mère Seelie aura récolté les informations sur la Seelies et aura indiqué si la Seelies est Orpheline, Membre d'un Vol ou Fondatrice d'un Vol (pour instancier l'extension de classe adéquate, j'aimerai au passage avoir votre avis sur cette façon de faire). Mais la page d'action du formulaire de création pourrait-être détourné !
  
  
• Dois-je vérifier les données en appelant des méthodes dédiées pour ensuite, si chaque donnée est correcte, appeler la méthode qui va les utiliser ?
  
  Exemple : J'ai une méthode createFlight qui permet d'initier la création d'un Vol. Dois-je faire dans la page quelque chose comme :
  
  Code PHP :

  <?php 
  
  $orpheline = new Orpheline($_SESSION['seelie']); // On instancie uniquement si la Seelies est réellement orpheline.
  
  if($orpheline->isValidName($_POST['name']) && $orpheline->isValidDescription($_POST['description'])){
  
      $orpheline->createFlight($_POST['name'], $_POST['description']);
  
  } 
  
  

  
  
• Ou bien ces vérifications doivent-elle être faîtes dans la méthode createFlight ?
  
  
• Ensuite, je me demandais dans quelle classe mettre la méthode createFlight. En effet, c'est une action qui n'est accessible qu'à une Seelie Orpheline, mais c'est à la fois une action qui concerne les Vols. ?
  

Citation :

• Seelies Orphelines
  • Fonder un Vol ;
    
  • Accepter ou décliner l'invitation d'un Vol ;
    
• Seelies membres
  • Inviter une Seelie orpheline dans le Vol ;
    
  • Quitter le Vol ;
    
  • Poster sur les canaux de discussion du Vol ;
    
  • Émettre un avis favorable ou défavorable à l'intégration d'une Seelie ;
    
  • Demander l'exclusion ou la démission d'une Seelie membre ou Fondratrice ;
    
• Seelies Fondatrices
  • Hérite des possibilités des Seelies membres
    
  • Émettre un avis sur l'exclusion ou la démission d'une Seelie ;
    
  • Demander un remplacement à la place de Fondatrice (nécessaire pour quitter un Vol) ;
    

<?php 

//code original

$orpheline = new Orpheline($_SESSION['seelie']);

if($orpheline->isValidName($_POST['name']) && $orpheline->isValidDescription($_POST['description'])){

    $orpheline->createFlight($_POST['name'], $_POST['description']);

} 

<?php 

//code plus évolutif

$orpheline = new Orpheline($_SESSION['seelie']);

if( $orpheline->isValid($_POST) ){

    $orpheline->createFlight($_POST);

} 

<?php 

function createFlight( array $arr )

{

  // test qui veut dire : si on a utilisé isValid et que les données sont valides

  if( $this->createFlightFlag == TRUE )

  {

  // code pour creer la vol

  }

} 

oxman a écrit :Je n'y connais rien en POO, mais ça me semble une absurdité de devoir appeler une méthode de vérification des données avant.

A mes yeux c'est createFlight qui doit s'en occuper.

oxman a écrit :Je ne sais pas, mais il doit y avoir plus propre je pense