Récupération informations sur plusieurs tables BDD

**Demirdiouff** · 22/11/2019, 11h38

Bonjour

Je me permets de vous demander de l'aide par rapport a l'utilisation de l'inner join que je dois pas bien réussir à faire...

J'ai une page, qui affiche un tableau avec plusieurs lignes concernant des "clients".
Chaque ligne récupère des informations via une entrée en BDD, jusqu'ici tout va bien;

Sauf que dans ce tableau, j'ai 4 colonnes, dont 1 qui détient des informations sur une table différente que les autres. J'ai cru comprendre qu'avec un inner join je pouvais effectuer une requête SQL pour récupérer des informations dans les deux donc voici ma requête :

Code sql :

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

SELECT customersiso.customeriso_id, vendorsiso.vendoriso_lastname, vendorsiso.vendoriso_firstname FROM customersiso INNER JOIN vendorsiso ON customersiso.vendoriso_id = vendorsiso.vendoriso_id

Cette requête (après test sur phpmyadmin) m'affiche bien les informations sur chaque ligne du tableau, pour avoir le vendeur associé a chaque customeriso_id.

Mon problème est le suivant :
je n'arrive pas a utiliser cette requête correctement pour récupérer le nom et prénom du vendeur associé a la ligne du tableau en question.

Code :

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

1
2
3
4
5
6
7
8
9
10
					<tbody>
						<?php foreach ($customersIso as $customer): ?>
						<tr>
							<td><a href="index.php?action=showCustomerIsoPage&customerIsoId=<?= $customer->customerIsoId()?>&lastname=<?= $customer->lastname()?>"><?= $customer->lastname()?> <?= $customer->firstname() ?></a></td>
							<td><?= $customer->city()?></td>
							<td><!-- ici devrait apparaître le nom du vendeur --></td>
							<td><?= $customer->date()?></td>
						</tr>
						<?php endforeach; ?>
					</tbody>

Est-il possible d'utiliser deux foreach en un par exemple ?

Merci beaucoup pour votre aide

**mathieu** · 22/11/2019, 12h34

éditer votre message pour nous montrer le code PHP que vous utilisez pour la requête SQL.

**darkstar123456** · 22/11/2019, 12h34

Salut,

Difficile de répondre précisément à cette question vu que tu ne passes pas par les méthodes "classiques" pour tes requêtes mais bien par un objet.
Vu qu'on ne sait pas comment est construit l'objet, on ne peut pas te dire comment l'afficher ^^

La première chose à faire est d'afficher les valeurs pour savoir comment elles sont enregistrées (et donc voir comment les afficher)

Donc 2 solutions : afficher tout l'objet, ou -à chaque ligne- afficher l'objet de la ligne

Tu peux mettre ça dans ta boucle foreach()

Code php :

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

echo PHP_EOL . '<pre>$customer: ' . print_r($customer, true) . '</pre>' . PHP_EOL;

Pour répondre à ta seconde question, tu peux imbriquer autant de boucles que nécessaires.
Attention cependant que ça complexifie pas mal la lecture mais aussi l'interprétation du code (cette dernière est appelée "Grand O") !

**Demirdiouff** · 22/11/2019, 13h00

Je savais pas si je devais éditer ou poster un nouveau message, mais voici le cheminement de mon code :

Model :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
    public function getVendorIsoByIdCustomer()
    {
        $vendorsIso = [];
        $db = $this->getDb();
        $getVendorsIso = $db->query('SELECT customersiso.customeriso_id, vendorsiso.vendoriso_lastname, vendorsiso.vendoriso_firstname FROM customersiso INNER JOIN vendorsiso ON customersiso.vendoriso_id = vendorsiso.vendoriso_id');
        while ($vendorIso = $getVendorsIso->fetch(PDO::FETCH_ASSOC))
        {
            $vendorsIso[] = new VendorIso($vendorIso);
        }
        return $vendorsIso;
    }

Controlleur :

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
    public function administrationIsoPage()
    {
        if (!isset($_SESSION['username']))
        {
            header("Location: index.php?");
            exit();
        }
        $customersIso = $this->customerIsoManager->getCustomersIso();
        $vendorsIso = $this->customerIsoManager->getVendorIsoByIdCustomer();
        $view = new view("AdministrationIsoPage", "Page d'administration Isolation");
        $view->generateAdministrationPage(array(
            'customersIso' => $customersIso,
            'vendorsIso' => $vendorsIso
        ));
    }

mes entités sont faites comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class VendorIso extends AbstractEntity
{
    private $vendoriso_id;
    private $vendoriso_lastname;
    private $vendoriso_firstname;
    private $vendoriso_address;
    private $vendoriso_postal_code;
    private $vendoriso_city;
    private $vendoriso_date_of_birth;
    private $vendoriso_place_of_birth;
    private $vendoriso_nationnality;
    private $vendoriso_social_insurance_number;
    private $vendoriso_code;

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
34
35
36
37
class CustomerIso extends AbstractEntity
{
    private $customeriso_id;
    private $vendoriso_id;
    private $lastname;
    private $firstname;
    private $address;
    private $locality;
    private $city;
    private $postal_code;
    private $phone_number_p;
    private $phone_number_f;
    private $email;
    private $situation;
    private $area_attic;
    private $area_wall;
    private $area_flooring;
    private $iso_type;
    private $iso_type1;
    private $house_age;
    private $attic_status;
    private $attic_status1;
    private $attic_status2;
    private $window;
    private $attic_access;
    private $attic_access1;
    private $attic_access2;
    private $heat_type;
    private $heat_type1;
    private $heat_type2;
    private $heat_type3;
    private $financing;
    private $population;
    private $documents;
    private $documents1;
    private $documents2;
    private $date;

et la vue vous l'avez plus haut...
Lorsque j'ai réussi a faire une tentative qui fonctionnait, malheureusement, je n'ai pas réussi a avoir un seul "vendeur" pour une ligne, mais j'avais à chaque fois toute la liste disponible, donc ça m'effectuait pas le tri...

Invité · 22/11/2019, 13h22

Bonjour,

... (erreur)...

**Demirdiouff** · 22/11/2019, 13h38

Bonjour jreaux62,

Je n'ai pas bien saisi le sens de votre message ? Vous parlez de messages d'erreurs que j'ai en exécutant le code ?
Si c'est le cas, alors j'en ai pas...

Invité · 22/11/2019, 14h06

Non, en fait, c'était une erreur de ma part...
(Je ne maitrise ni la POO, ni le MVC.)

Par contre, j'ai l'impression que tu t'y prends mal :

Code :

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

            $vendorsIso[] = new VendorIso($vendorIso);

Quel rapport avec la requête ?
Qu'espères-tu récupérer ainsi ?

Tu devrais au moins vérifier ce que tu récupères...

**Demirdiouff** · 22/11/2019, 14h20

Effectivement, ça n'a pas trop de rapport... Mais si je conservais l'entité "CustomerIso" je peux pas accéder aux objets à l'intérieur de VendorIso (ce qui me paraît normal) donc la est mon soucis également...
J’espérais pouvoir instancier l'entité pour pouvoir ensuite accéder aux objets à l'intérieur avec le return... mais je crois que je me suis perdu

A l'heure actuelle, lorsque je passe par "VendorIso" pour ta citation, j'arrive tout de même à avoir les valeurs Lastname() et Firstname() de l'entité VendorIso. Donc semblerait être sur la bonne voie... par contre, j'ai dans chaque ligne du tableau, TOUTES les informations qui s'affichent et non celui correspondant à l'ID "customeriso_id".

**rawsrc** · 22/11/2019, 15h05

Salut,

j'essaie de comprendre où tu veux en venir mais j'ai du mal.
Plusieurs points :
- pourquoi utiliser des entités alors qu'un simple tableau suffirait
- tu peux tout récupérer en une seule fois (je pars du principe qu'il est possible d'avoir des clients sans vendeur LEFT JOIN):

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT
    c.customeriso_id, 
    c.customeriso_lastname,
    c.customeriso_firstname,
    c.customeriso_city,
    c.cutomeriso_date,
    v.vendoriso_id,
    v.vendoriso_lastname,
    v.vendoriso_firstname
FROM
    customeriso AS c LEFT JOIN vendoriso AS v ON c.vendoriso_id = v.customeriso_id
ORDER BY
    c.customeriso ASC

Après tes traitements sont d'une extrême lourdeur et puis à l'affichage, rien est sécurisé !

J'ai une question : tu suis un framework MVC existant ou tout ça c'est de ton cru ?

**Demirdiouff** · 22/11/2019, 15h30

Salut rawsrc,

Je te remercie pour ta réponse,

Tout mon traitement se faisait jusqu'à maintenant via des entités donc j'ai continué sur cette voie la... et en ce qui concerne les clients, ils seront obligatoirement associés à un vendeur, mais pour ça je devrais m'en sortir.

Que veux-tu dire par extrême lourdeur ?
Et concernant la sécurité c'est "normal" on va dire, car je m'en suis pas du tout préoccupé pour le moment j'avais décidé de laisser ça pour la fin afin de pouvoir me concentrer sur une seule chose à la fois, n'étant déjà pas à l'aise du tout avec chacun des différents aspects.

Je ne suis pas de MVC existant, disons que c'est un entraînement sur long terme et que j'essaye de bidouiller ce que je peux en apprenant de nouvelles choses tous les jours.

(PS: je suis tout ouïe niveau conseils/remarques pour m'améliorer.)

(Concernant ta réponse, j'essaye de l'appliquer dès que je peux et te tiens informé.)

**rawsrc** · 22/11/2019, 15h38

Envoyé par Demirdiouff

Et concernant la sécurité c'est "normal" on va dire, car je m'en suis pas du tout préoccupé pour le moment j'avais décidé de laisser ça pour la fin

euh, ça va être chaud ! T'imagine la reprise de code que cela va te faire !!! Sans compter les oublis (forcément, y en aura).

Par extrême lourdeur, quand je vois ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
class VendorIso extends AbstractEntity
{
    private $vendoriso_id;
    private $vendoriso_lastname;
    private $vendoriso_firstname;
    private $vendoriso_address;
    private $vendoriso_postal_code;
    private $vendoriso_city;
    private $vendoriso_date_of_birth;
    private $vendoriso_place_of_birth;
    private $vendoriso_nationnality;
    private $vendoriso_social_insurance_number;
    private $vendoriso_code;

j'ai mal à l'aorte, vois-tu

Tu es allé te promener sur mon blog DVP, parce que question MVC y a de quoi se repaître

**Demirdiouff** · 25/11/2019, 10h49

Effectivement, je vais peut être faire une pause dès que j'aurais résolu mon problème en reprenant la partie sécurité sérieusement...

Je me suis penché durant ce week-end sur ton blog, ce qui m'a permis d'apprendre pas mal d'éléments supplémentaires, je te remercie pour l'info.

Cependant, quand je reprend la requête SQL (après quelques corrections):

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
SELECT 
c.customeriso_id, 
c.lastname, 
c.firstname, 
c.city, 
c.date, 
v.vendoriso_id,
v.vendoriso_lastname, 
v.vendoriso_firstname 
FROM 
customersiso AS c LEFT JOIN vendorsiso AS v ON c.vendoriso_id = v.customeriso_id 
ORDER BY 
c.customersiso ASC