[Résolu] Héritage

Et l'eau,

Suivant ce que tu dis, c'est louche puisqu'une class fille récupère les attributs parents ($this->client_name). Ensuite, dans ton code, on ne voit pas le tenant et l'aboutissant. Donc, je peux pas vraiment t'aider. Pour finir, l'héritage, c'est quelque chose de bien mais ça apporte son lot de problèmes. Donc, tes classes qui héritent de plusieurs parents, ça me parait suspect comme architecture....

Merci niuxe je t'avoue que c'est pas très clair lol

Mes classes n'héritent pas de plusieurs parents, il s'agit en fait d'une cascade de bas en haut

Dans mon programme en fonction du lien on détermine un controleur et une fonction

if(isset($_GET['p']))
	{
    $page = $_GET['p'];
	}
else
	{
    $page = 'sundries.index';
	}

$page = explode('.', $page);

//ON PREND COMME EXEMPLE account.details.index&id=4


if($page[0] == 'admin')
	{
    $controller = '\App\Controller\Admin\\' . ucfirst($page[1]) . 'Controller';
    $ref_page=$page[0].$page[1].$page[2];
    //$controller = new $controller();
    $action = $page[2];
	}
elseif($page[0] == 'account')
 	{
    // ucfirst($page[1]) CORRESPOND A details ET devient Details   
    $controller = '\App\Controller\Account\\' . ucfirst($page[1]) . 'Controller';
    $ref_page=$page[0].$page[1].$page[2];
    //$controller = new $controller($role=null,$name=null);
    $action = $page[2]; 
    }	
 else
 	{
    $controller = '\App\Controller\\' . ucfirst($page[0]) . 'Controller';
    //$ref_page=$page[1].$page[2];
    //$controller = new $controller();
    $action = $page[1]; /*ex Index*/
	}

$controller = new $controller();

//DEVIENT : $controller = new \App\Controller\Account\DetailsController

$controller->$action(); /*TRANSFORME ACTION PRECEDENT EN FONCTION EX FUNCTION INDEX*/

En étant sur la page

http://www.hervedunoyer.com/index.php?p=account.details.index&id=4

On a un controleur :

DetailsController.php

et une fonction index

public function index(){

        $pageId=29;
        $page_required = $this->Detail->findPage2($pageId)  ;

        $level=$this->Detail->level($pageId);
        $level_id=$level->level;

        $name= $this->getName();
        
        $role= $this->getRole();
                                
        if ($role > $level_id)
            {
            $this->habilitation();
            }
                 
        $account = $this->Detail->allById($_GET['id']);

        $this->render('account.details.index', compact('account','page_required','role','name'));
    }

J'ai modifié par rapport à ce qui est écrit précédemment au niveau de la classe mère

<?php

namespace App\Controller\Account;

use \App;

class AppController extends \App\Controller\AppController{

    protected $client_id;
    protected $client_role;
    protected $client_name;

    public function __construct(){

        parent::__construct();
            
        if(!$_SESSION['auth'])
            {
            $this->forbidden(); //Core/Controller
            }  
        else
            {
            $client_id = $_SESSION['auth'] ;
            $client_role = $_SESSION['role'];
            $client_name= $_SESSION['name'];

            $this->client_role = $client_role;
            $this->client_name = $client_name;
            } 


?>
        <!--<script type="text/javascript">
            setTimeout("window.location='index.php?p=clients.logout'","600000");//En MilliSecondes
        </script>-->

<?php  

    }

    public function getName(){
            return $this->client_name;
            }

    public function getRole(){
            return $this->client_role;
            }        
   
//FIN CLASSE
}

que je récupère dans ma fonction index

Cela fonctionne mais je ne sais pas si c'est correct.

Mon problème est que pour toutes les fonctions de ma classe je répète à chaque fois

$name= $this->getName();
        
        $role= $this->getRole();
                                
        if ($role > $level_id)
            {
            $this->habilitation();
            }

J'aimerai que cela soit fait une fois pour toute dans la classe mère par exemple ou au niveau du constructeur et ne pas à avoir à le répéter dans chaque fonction.

En plus comme tu peux le voir je répète aussi dans chaque fonction en fonction d'un $pageId qui me permet à partir d'une base de données de retrouver les informations de chaque page et de trouver le degré d'habilitation

J'aimerai donc factoriser cela mais je n'y arrive pas

J'espère que c'est un peu plus clair

Merci encore pour ton aide

Hervé

Bonjour Jencal et merci pour ton intervention

J'ai modifié ma classe mère ainsi :

class AppController extends \App\Controller\AppController{

    protected $name;
    protected $role;

     public function __construct(){
 
        parent::__construct();
            
        if(!$_SESSION['auth'])
            {
            $this->forbidden(); //Core/Controller
            }  
        else
            {
            $this->role = $_SESSION['role'];
            $this->name = $_SESSION['name'];
          
            }
 //FIN CONSTRUCT           
    }
                
         
//FIN CLASSE
}

Ainsi que ma classe fille :

class DetailsController extends AppController{

    public function __construct(){
        parent::__construct();
        $this->loadModel('Detail');
    }
       
    public function index(){

        $pageId=29;

        $role= $this->role;
        $name= $this->name;
  
        $page_detail = $this->Detail->findPage2($pageId)  ; 

        $title=$page_detail->title;

        $level_id=$page_detail->level;
                 
       if ($role > $level_id)
            {
            $this->habilitation();
            }
                 
        $account = $this->Detail->allById($_GET['id']);


        $this->render('account.details.index', compact('account','title','role','name'));
    }

Je ne sais pas si cela est correct mais cela marche

Par contre ce que je souhaiterai faire c'est ne pas avoir à répéter pour chaque fonction la même chose :

       $pageId=29;
   
        $role= $this->role;
        $name= $this->name;
  
        $page_detail = $this->Detail->findPage2($pageId)  ; 

        $title=$page_detail->title;

        $level_id=$page_detail->level;
                 
       if ($role > $level_id)
            {
            $this->habilitation();
            }

Seul le numéro de page change

J'avais pensé à créer une classe pour cela ou une fonction (mais où la placer ?)

J'espère que c'est plus clair

Merci d'avance pour votre aide
Modifié par dudu22 (07 Sep 2020 - 20:03)

Oui je suis d'accord avec toi index.php sert de pseudo routeur et c'est pas top

Pour les Controleur effectivement le même nom c'est source d'erreur

Pour créer un vrai router aurais tu un tuto à me conseiller.

Merci encore pour ton aide

OK je vais essayer avec quelques tutos trouvés

Merci pour ton aide

Je marque comme résolu