PHP|Débutant :: Forums

Pas vraiment le temps pour le moment de regarder tout le code mais effectivement l'objet n'était pas très utile dans ce cas là.
Une simple fonction aurait fait l'affaire.

Ok !

Maintenant, sur le contenu :

Aurait-il pu être "mieux" de creer une class PagesAutorisees, PageDemande, Inclusion (la première version que j'ai fait avait cette forme)...enfin de séparer en plusieurs classes courtes ?
Les fonctions sont-elles logiques ?

Plus particulièrement : toute critique est attendue ^^

Ok merci !

Je vais revenir sur quelques points :

2. Quelle est cette notation à dos de chameaux ?

3. De façon générale, quel "forme" de noms devrais-je donner à mes variables ? ($mavariable, $MaVariable, $Ma_Variable, $ma_variable, ($my_variable), ...). De meme, que faire pour les noms de fonctions ?

Par ailleurs, j'avais essayé le plus possible de faire des fonctions sans argument : cela a-t-il un interet ?

Et si je sépare ma class en deux, à savoir une class PageDemandee et une class PagesAutorisees, où devrait avoir lieu le test "savoir si la page est autorisé ? dans la class PageDemandee où je construit une fonction avec argument PagesAutorisees ? Dans la class PagesAutorisees ? Dans une autre class ? Ou alors dans PageDemandee crée une nouvelle variable $autorisee ?

Merci !

PS : je vais modifier mon code est proposé autre chose d'équivalent

Dernière modification par moijhd (04-11-2009 19:37:51)

Saluton,
Si je souscris bien volontiers à la plupart des recommandations de Maskime, j'émets une grosse réserve :
- sur la recommandation n°1

en tant que militant espérantiste contre l'hégémonie de l'anglais, je ne pouvais laisser passer cela.

Pour le reste, c'est affaire, comme souvent, de bon sens. La POO, c'est super sympa, mais ce n'est pas la panacée à mettre à toutes les sauces, pour tout, et n'importe quoi.
Je vais me citer, je dis souvent dans mes interventions de formateur la chose suivante :

Nouvelle version :

[code php]

<?php
class Page{

    // VARS
   
    public $Name = NULL;
   
    // CONST
   
    const PAGE_URL_PARAM = 'page';
    const DEFAULT_NAME = 'accueil';
   
    // FUNC
   
    // Construct
    public function __construct($Name = NULL){
        $this->NewName($Name);
    }

    // Ajoute le nom
    public function NewName($Name){
        // on regarde si le nom est vide
        if(empty($Name)){
            // Le nom est vide
            // On regarde si un nom est passé en argument
            $this->NewNameFromUrl();
        }
        else{
            // On donne un nom à la page
            $this->Name = $Name;
        }
    }
   
    // Ajoute le nom à la page par récupération de l'url
    public function NewNameFromUrl(){
        // On regarde si un nom de page est passé dans l'url
        if(isset($_GET[self::PAGE_URL_PARAM]) and !empty($_GET[self::PAGE_URL_PARAM])){
            // On donne le nom dans l'url à cette page
            $this->NewName($_GET[self::PAGE_URL_PARAM]);
        }
        else{
            // On ne fait rien
            $this->NewDefaultName();
        }
    }
   
    // Donne le nom par defaut à la page
    public function NewDefaultName(){
        $this->Name = self::DEFAULT_NAME;
    }
}
?>

<?php
class PageControler{

    // VARS
   
    public $AuthorisedPageNames = array();
   
    // CONST
       
    // FUNC
   
    // Constructeur
    public function __construct($PagesToBeAuthorised = NULL, $PageToBeIncluded = NULL){
        // On récupère le tableau de pages ou la page à autoriser
        $this->NewAuthorisedPages($PagesToBeAuthorised);
        // On regarde si la page à inclure est effectivement une page
        if($this->IsPage($PageToBeIncluded)){
            // C'est une page
            // On tente de l'inclure
            $this->IncludePage($PageToBeIncluded);
        }
        else{
            // Ce n'est pas une page
            // On tente d'inclure une page vide URL
            $this->IncludePage(new Page());
        }
    }
   
    // Ajoute un nouvelle page à autoriser
    public function NewAuthorisedPage($PageToBeAuthorised){
        // On regarde si la nouvelle page à autoriser est effectivement une page
        if($this->IsPage($PageToBeAuthorised)){
            // C'est une page
            // On l'ajoute
            $this->NewAuthorisedPageName($PageToBeAuthorised->Name);
        }       
    }
   
    // Ajoute un tableau de nouvelles pages à autoriser
    public function NewAuthorisedPages($PagesToBeAuthorised){
        // on regarde si une page ou si un tableau de pages est passé en argument
        if(!empty($PagesToBeAuthorised)){
            // On regarde si c'est un tableau de page
            if(is_array($PagesToBeAuthorised)){
                // Pour chaque page du tableau
                foreach($PagesToBeAuthorised as $PageToBeAuthorised){
                    // On tente d'autoriser la page
                    $this->NewAuthorisedPage($PageToBeAuthorised);
                }       
            }
            else{
                // Pour l'unique page
                // On tente de l'autoriser
                $this->NewAuthorisedPage($PagesToBeAuthorised);
            }   
        }   
    }
   
    // Ajoute un nouveau nom de page à autoriser
    public function NewAuthorisedPageName($PageNameToBeAuthorised){
        // Si le nom n'est pas vide
        if(!empty($PageNameToBeAuthorised)){
            // On ajoute le nom à la liste
            array_push($this->AuthorisedPageNames, $PageNameToBeAuthorised);
        }       
    }

    // Vérifie qu'un nom est autorisé
    public function IsPageNameAuthorised($PageName){
        return in_array($PageName, $this->AuthorisedPageNames);
    }

    // Vérifie qu'un nom est autorisé
    public function IsPageAuthorised($Page){
        return $this->IsPageNameAuthorised($Page->Name);
    }

    // Inclure une page
    public function IncludePage($PageToBeIncluded){
        // On vérifie que $PageToInclude est bien un objet de type page
        if($this->IsPage($PageToBeIncluded)){
            // On tente d'inclure la page par son nom
            $this->IncludePageFromPageName($PageToBeIncluded->Name);
        }
        else{
            // On crée une nouvelle page à inclure
            $this->IncludePage(new Page);
        }
    }

    // Inclure une page à partir de son nom
    public function IncludePageFromPageName($PageToBeIncludedName){
        // On vérifie que le nom de la page est autorisé
        if($this->IsPageNameAuthorised($PageToBeIncludedName)){
            // On ne fait rien
            // Le nom du fichier
            $FileName = $PageToBeIncludedName.'.php';           
        }
        else{
            // On appelle la page par defaut
            // Le nom du fichier
            $FileName = Page::DEFAULT_NAME.'.php';           
        }   
        // On inclut
        include($FileName);   
    }
   
    // Verifie que des noms sont autorisés
    public function IsEmptyAuthorisedPageNames(){
        return empty($this->AuthorisedPageNames);
    }
   
    // Vérifie qu'une variable est un objet page
    public function IsPage($Object){
        return is_a($Object, 'Page');
    }
}
?>

<?php
class PageToBeIncluded extends Page{
   
    // VARS
   
    public $Title = NULL;
   
    // CONST
   
    const DEFAULT_TITLE = 'Titre par défaut';
   
    // FUNC
   
    // Constructeur
    public function __construct($Name = NULL, $Title = NULL){
        $this->NewName($Name);
        $this->NewTitle($Title);
    }
   
    // Donne un titre à la page à afficher
    public function NewTitle($Title){
        // On regarde si le titre est vide
        if(empty($Title)){
            // Le titre est vide
            // On donne le titre par defaut
            $this->NewDefaultTitle();
        }
        else{
            // On donne le titre
            $this->Title = $Title;
        }
    }
   
    // Donne le titre par défaut à la page
    public function NewDefaultTitle(){
        $this->NewTitle(self::DEFAULT_TITLE);
    }
   
}
?>

[/code]

On critiquera le tout

Dernière modification par moijhd (07-11-2009 01:12:36)

Ce n'est pas très constructif !

Considérons que le but est de "faire de la poo', que ce que l'on fait n'est qu'un prétexte et que l'on peut essayer de le faire bien quand même.

Donc il n'y a rien à redire sur tout ce qu'il y a écrit ? Même pas un petit détail ? Même pas une amélioration à apporter (et pas à ajouter) ?

Saluton,
A ta décharge, Pierrot, POO ou pas, mais encore plus en POO, les commentaires explicitant les finalités d'une classe constituent un élément déterminant de son utilisabilité.
Une classe est censée être, a minima, une boîte noire avec laquelle on peut dialoguer via ses méthodes et attributs publics.
Cette "API" doit faire l'objet d'un commentaire exhaustif en début de déclaration de classe.
Et je ne parle même pas des possibilités d'extension de la classe.

2. PageToBeIncluded sert juste à dire que j'ai d'une part les pages autorisées et d'autre part, la page que je veux afficher, et donc, pour la page à afficher, je vais pouvoir donner plus d'informations à la page à savoir, pour l'exemple, le titre.

3. J'ai bien evidement fait des tests, je publierais donc un "usage type" ou du moins, la façon dont je désire l'utiliser !

Enfin, je me pose le problème suivant :
Il ne me semble pas très logique de vouloir pour chaque nouveau changement de page charger, c'est-à-dire ré-éxécuter, la partie où "j'ajouterais les pages autorisées". Je pense donc, à la "première connexion", chargée les pages, puis mette mon objet dans une session pour le réutiliser dans les pages suivants ?

Je développerais cela dans mon prochain message