Discussion :

[robin52]

Bonjour,

mon problème est assez spécifique donc je vais essayer de détailler au max


j'ai deux classes distinctes:
la 1er me servant a afficher une liste de personne selon deux paramètres ($mois/$annee);
la seconde me servant à afficher des détails sur une personne avec 3 parametres ($id/$mois/$annee)
la sortie pour les deux est un pdf fait avec la classe FPDF
les deux marches sans soucis !

La construction des classes est assez similaire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
 
require_once '/../classePourMonPdf.php';
require_once '/../4autresclasses.php';
require_once '/../fpdf.php';
require_once '/classe2.php';
 
class classe1 extends classePourAfficherPdf{
 
public function NomDeMaClasse($parametres){
$annee = $parametre['annee'];
$mois = $parametre['mois'];
$detail = $parametre['detail']; // true ou false pour savoir si je dois ajouter la seconde classe.
 
// ici je met mes paramètres pour la construction du pdf
 
$pdf=new FPDF();
$pdf->AddPage();
$this->function2($annee, $mois);
}
 
private function function2($annee, $mois){
 
// ici l'appel à ma BDD avec les paramètres $annee et $mois
// ici la construction de mon pdf
//dans mon switch je récupère un tableau avec le $id de toute la liste
 
if($detail = true){
   foreach($tableau as $id){
      //ici ça coince :aie: je souhaiterais appeler ma 2sd classe pour chaque $id
   }
}
}

voili voila pour la 1er classe, la 2eme est sur le même principe:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
require_once '/../classePourMonPdf.php';
require_once '/../4autresclasses.php';
require_once '/../fpdf.php';
 
class classe2 extends classePourAfficherPdf{
 
public function NomDeMaClasse($parametres){
$id = $parametre['id'];
$annee = $parametre['annee'];
$mois = $parametre['mois'];
if(array_key_exists('detail', $Parametres)){ // si j'appel directement la classe pour juste avoir le pdf d'une personne
   $detail = $objParametres['detail']; 
}else{
   $detail = false;
}
// ici je met mes paramètres pour la construction du pdf
if($detail != true){
   $pdf=new FPDF();
   $pdf->AddPage();
}else{
   $pdf->AddPage();
}
$this->function2($id, $annee, $mois);
 
private function function2($id, $annee, $mois){
// ici l'appel à ma BDD avec les paramètres $id, $annee et $mois
// ici la construction de mon pdf
}

une solution ? une idée ? une insulte ?

[robin52]

Bon j'ai créé une solution que je trouve un peu placebo,

j'ai réuni dans une nouvelle classe les 4 fonctions et en supprimant le paramètre 'detail',

ça marche parfaitement mais une classe Ctrl+C Ctrl+V de deux autres je trouve ça moche

Si quelqu'un trouve mieux je suis preneur ...

[Celira]

Donc la question c'était "comment réunir le code des deux classes en une seule ?" ?

Au passage if($detail = true){ est toujours vrai, car tu utilises = (affectation) à la place de == (comparaison d'égalité) ou === (comparaison d'égalité stricte).

[robin52]

bien vu pour le '=', dans mon code je l'avais passé en '!= false' donc je n'avais pas vraiment fait gaffe ^^'

et pour réunir les deux codes c'est globalement l'idée oui , sauf qu'il faut aussi que je puisse appeler les codes séparément car
si j'ai besoin de la liste des personnes, je n'ai pas forcément envie d'avoir le listing détaillé de toutes les personnes et
si je veux avoir le détail d'une personne je n'ai pas besoin d'avoir le listing détaillé de tout le service ça fait des économies d'encre.

[Celira]

Ah, j'avais pas vu le commentaire "ça coince ici". A priori, tu as juste à appeler ta 2e classe là où tu en as besoin. Tu as essayé ? tu as un message d'erreur ?
Sinon, la solution "une classe, plusieurs fonctions" est parfaitement viable.

[robin52]

si j'envoie dans mon foreach

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$instance->__constructorClasse2($annee, $mois, $id, $detail);

ma fonction attend juste '$paramètre'

et donc je pars en erreur

[robin52]

...et après une lute acharnée, le hareng n'a pas réu... non rien à avoir

j'ai réussi à me dépatouiller!

classe1:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
//dans mon switch:
$tableau[]=array( $data['id'], $annee, $mois, $detail, $this->pdf);
 
//après
if($detail != 'false'){           
            foreach($tableau as $utilisateur){
                $instance = new classe2($utilisateur, $annee, $mois, $detail, $this->pdf);
            }        
        }

classe2:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
public function function1Classe2($parametres){
        if(array_key_exists('id', $parametres)){            
            $id = $parametres['id'];
            $annee = $parametres['annee'];
            $mois = $parametres['mois'];
            if(array_key_exists('detail', $parametres)){
               $detail = $parametres['detail']; 
            }else{
               $detail = 'false';
            }
        }else{
 
            $id = $parametres[0];
            $annee = $parametres[1];
            $mois = $parametres[2];
            $detail = $parametres[3];
            $this->pdf = $parametres[4];
        }

Voili Voila pu de soushi (vous connaissez désormais la fin du hareng )