Joindre la dernière occurrence d'une table

**Romalafrite** · 09/08/2012, 11h41

Bonjour à tous,

j'ai une table ps_address contenant différentes adresses et la table ps_customer contenant différents clients. J'aimerais joindre au customer la dernière adresse qu'il a créée.

j'essaie de faire l'équivalent de ceci :

Code :

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

1
2
3
LEFT JOIN 
        (SELECT * FROM ps_address) as A 
		ON A.id_customer=C.id_customer AND A.id_address IN (SELECT MAX(id_address) FROM ps_address A GROUP BY id_customer)

mais vu le nombre d'enregistrements, c'est extrêmement lent, forcément.

Avez-vous une idée de comment faire ?

Merci d'avance

**lola06** · 09/08/2012, 11h54

Il est possible de faire une jointure, à vérifier si c'est vraiment mieux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
LEFT JOIN (
    SELECT A1.id_customer, A1.id_adress
    FROM ps_address A1    
    JOIN (
       SELECT id_customer, MAX(id_address) id_address
       FROM ps_address
       GROUP BY id_customer
    ) A2
    ON A1.id_customer = A2.id_customer
    AND A1.id_address= A2.id_address
) as A 
ON A.id_customer=C.id_customer

**Romalafrite** · 09/08/2012, 12h02

ça ne marche pas mieux malheureusement, ça mouline..

**lola06** · 09/08/2012, 12h04

Il serait peut-être intéressant de voir toute la requête à optimiser. Et les index.

De plus au bout d'un moment si vraiment il y a beaucoup d'enregistrement il faut bien que ça réfléchisse.

**Romalafrite** · 09/08/2012, 12h06

j'ai essayé ça sans que ça marche mieux aussi :

Code :

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

INNER JOIN ps_address A ON A.id_customer=C.id_customer AND A.id_address IN (SELECT MAX(id_address) FROM ps_address GROUP BY id_customer)

EDIT : (je précise que je n'ai pas d'autre jointure ou quoique ce soit d'autre qui puisse faire ramer ; avant ça, ya juste un SELECT * FROM ps_customer C)

**lola06** · 09/08/2012, 12h10

Pourriez-vous mettre l'explain plan de la requête ?

**Romalafrite** · 09/08/2012, 12h13

Envoyé par lola06

Il serait peut-être intéressant de voir toute la requête à optimiser. Et les index.

De plus au bout d'un moment si vraiment il y a beaucoup d'enregistrement il faut bien que ça réfléchisse.

Pas de souci d'index, voici ce que dit le EXPLAIN :

Code :

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

1
2
3
4
5
6
7
8
 
+----+--------------------+------------+-------+----------------------------+------------------+---------+-----------------------+-------+-------------+
| id | select_type        | table      | type  | possible_keys              | key              | key_len | ref                   | rows  | Extra       |
+----+--------------------+------------+-------+----------------------------+------------------+---------+-----------------------+-------+-------------+
|  1 | PRIMARY            | C          | ALL   | PRIMARY,id_customer_passwd | NULL             | NULL    | NULL                  | {DESDIZAINESDEMILLIERS} |             |
|  1 | PRIMARY            | A          | ref   | address_customer           | address_customer | 4       | xx.C.id_customer      |     1 | Using where |
|  2 | DEPENDENT SUBQUERY | ps_address | index | NULL                       | address_customer | 4       | NULL                  |     1 | Using index |
+----+--------------------+------------+-------+----------------------------+------------------+---------+-----------------------+-------+-------------+

Merci pour ton aide,

**SQLpro** · 09/08/2012, 12h27

De toutes façon se baser sur un ID pour trouver le dernier est parfaitement stupide. Rien ne garantie que ce soit bien la dernière adresse !
En effet, on peut toujours forcer un auto incrément ou encore décrémenter....

La seule solution viable consiste à rajouter une colonne DATE dans votre table, avec la date de validité de l'adresse.

A +

**Romalafrite** · 09/08/2012, 12h39

Envoyé par SQLpro

De toutes façon se baser sur un ID pour trouver le dernier est parfaitement stupide. Rien ne garantie que ce soit bien la dernière adresse !
En effet, on peut toujours forcer un auto incrément ou encore décrémenter....

La seule solution viable consiste à rajouter une colonne DATE dans votre table, avec la date de validité de l'adresse.

A +

Tout d'abord, merci pour la réponse, quoiqu'un brin agressive pour un modérateur.

D'un point de vue purement SGBD (quand on ne maîtrise pas ce qui peut arriver en face) c'est peut-être une hérésie, je ne sais pas .

Dans notre système, le dernier ID est forcément la dernière insertion. Et on ne s'amuse pas à forcer un auto-incrément ou à décrémenter donc l'ID est le plus efficace à mon sens.

Toutefois, si vous avez une solution, même avec la date, je suis preneur.

**Romalafrite** · 09/08/2012, 12h43

je précise que j'ai essayé ceci également et que ça ne marche pas mieux (même en laissant tourner 5mn)

Code :

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

1
2
3
4
 
 
INNER JOIN ps_address A ON A.id_customer=C.id_customer 
WHERE A.id_address IN (SELECT MAX(id_address) FROM ps_address GROUP BY id_customer)

**Romalafrite** · 09/08/2012, 12h53

j'ai l'impression que ceci marche beaucoup mieux !

Code :

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

1
2
INNER JOIN (SELECT * FROM ps_address ORDER BY id_address DESC) A ON A.id_customer=C.id_customer 
GROUP BY C.id_customer

ça me gêne de faire un GROUP BY à la fin mais c'est une solution extrêmement rapide ! (par contre : Using temporary; Using filesort)

je précise que c'est pour une utilisation "deux/trois fois par jour" donc ce n'est pas grave mais si quelqu'un a besoin de solution pour une requête à utiliser au moins une fois par chargement de page, ça n'ira pas du coup

je vais passer en "résolu" mais attention aux copieurs, la solution qui marche est à analyse de votre côté avec une option SQL_NO_CACHE et un EXPLAIN pour bien se rendre compte de la consommation de votre côté

Merci à tous pour votre aide

**lola06** · 09/08/2012, 13h10

Heu, je ne comprend pas trop to GROUP BY ici.

Pourrais-tu mettre la requête entière ?

**Fred_34** · 09/08/2012, 14h21

Je suis un peu de l'avis de lola06, j'ai de gros doutes sur ta requête.

Que donne ceci ? :

Code :

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

1
2
3
4
INNER JOIN (SELECT id_customer,MAX(id_address) AS id_address FROM ps_address GROUP BY id_customer) T
ON T.id_customer=C.id_customer 
INNER JOIN ps_address 
ON T.id_address=ps_address.id_address

**Romalafrite** · 09/08/2012, 14h56

la requête s'exécute assez rapidement mais ceci ne marche pas car j'ai besoin de tous les champs de l'adresse, pas seulement le MAX(id), ce qui complique les choses.

Le Group By peut avoir une logique :

je joins toutes les adresses au client (classées par ID DESC)
je groupe par client pour n'avoir que la première

La requête complète (modèle prestashop) qui fonctionne :

Code :

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

1
2
3
4
5
6
SELECT 
*
 FROM ps_customer C
 
INNER JOIN (SELECT * FROM ps_address ORDER BY id_address DESC) A ON A.id_customer=C.id_customer 
GROUP BY C.id_customer

**lola06** · 09/08/2012, 15h04

Envoyé par Romalafrite

la requête s'exécute assez rapidement mais ceci ne marche pas car j'ai besoin de tous les champs de l'adresse, pas seulement le MAX(id), ce qui complique les choses.

Sa requête est correcte, car si tu regardes attentivement il fait une autre jointure avec la table ps_address pour récupérer toutes les infos.

Envoyé par Romalafrite

Le Group By peut avoir une logique :

je joins toutes les adresses au client (classées par ID DESC)
je groupe par client pour n'avoir que la première

Le group by doit être utilisé avec une fonction d'agrégation.
Qu'est ce qui te garanti ici que c'est bien la première qui est prise ? Il n'y a aucun contrôle.

Je viens de réagir un peu tard, mais plus haut tu nous as mis l'EXPLAIN, or ici c'est l'EXPLAIN PLAN de la requête qui est intéressant.

**tatayo** · 09/08/2012, 15h28

Bonjour,
J'ajoute mon grain de sel, avec une petite réflexion: la dernière adresse, c'est celle pour laquelle il n'en existe pas d'autre "après".
Donc on peut aussi le faire avec une jointure:

Code SQL :

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

1
2
3
4
5
6
 
select customer.*
from customer
inner join adresse as a1 on a1.id_cust = customer.id
left outer join adresse as a2 on a2.id_cust = customer.id and a2.id > a1.id
where a2.id is null

Bon, j'ai un peu appliqué l'hypothèse du monde clos, car je ne renvoie pas de client sans adresse, mais c'est pour l'idée.

Tatayo.

**Romalafrite** · 09/08/2012, 15h45

Envoyé par lola06

Sa requête est correcte, car si tu regardes attentivement il fait une autre jointure avec la table ps_address pour récupérer toutes les infos.

Le group by doit être utilisé avec une fonction d'agrégation.
Qu'est ce qui te garanti ici que c'est bien la première qui est prise ? Il n'y a aucun contrôle.

Je viens de réagir un peu tard, mais plus haut tu nous as mis l'EXPLAIN, or ici c'est l'EXPLAIN PLAN de la requête qui est intéressant.

Le fonctionnement du group by est bien par défaut de prendre la première occurence ? ou je me trompe ? je n'ai mis aucune fonction d'agrégation et la requête fonctionne quand même en prenant la valeur que j'attendais.

**lola06** · 09/08/2012, 16h01

Pour rappel un tutoriel sur l'utilisation du GROUP BY.

Ce que tu décris est peut-être un effet de bord du GROUP BY, mais comme ça n'est pas le fonctionnement normalisé tu ne peux pas garantir tes résultats en l'utilisant de la sorte.

**Romalafrite** · 09/08/2012, 16h14

Effectivement, même la doc MySQL ne précise rien là-dessus..

Merci pour votre aide en tout cas, très intéressante cette discussion !

**StringBuilder** · 09/08/2012, 17h09

Tatayo > Si je ne m'abuse, si tu remplaces le premier inner join par un left outer join, tu ramènes exactement ce que romalafrite cherche à faire.

Et c'est à mon avis la requête la plus simple et la plus propre.

Joindre la dernière occurrence d'une table

Langage SQL

Discussions similaires

Partager

Partager