Discussion :

[dancom5]

Code :

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

1
2
3
4
5
SELECT ALL * , COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY idimg
ORDER BY i.id ASC

résultat :
aaa_0000.jpg 8
qqq_5555.jpg 2
sdf_3522.jpg 0
hau_2233.jpg 5

[Séb.]

GROUP BY idimg <= Ceci est faux, il faut grouper par image, donc sur i.id

[dancom5]

Ça sort tous les résultats :

Code :

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

1
2
3
4
5
SELECT ALL * , COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC

Vraiment MERCI Séb! Je suis content. Vous avez eu une patience énorme pour moi!

ça sort tout:

Code :

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

1
2
3
4
5
SELECT ALL * , COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC

Merci Séb, vous avez été très généreux avec moi. MERCI!

Je vais intégrer ça dans ma page.. je vais espérer que ça va
bien rouler étant donné que j ai aussi le page par page dans
la requête.
ça devrait aller ca fonctionnait avant.

La requête qui était ça

Code :

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

SELECT * FROM ori_images ORDER BY id ASC LIMIT $ledebut,$nb_affichage_par_page

est devenu ça :

Code :

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

1
2
3
4
5
SELECT ALL * , COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC $ledebut,$nb_affichage_par_page

ça changé de quoi dans mon affichage.

Le ID, le TITRE et la DESC des images ont changés pour ceux
des commentaires. Je tente les ajustement. Si c'est concluant, je mets en résolu après.

Si je pourrais mettre comme ceci :

Code :

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

1
2
 
$ligne['ori_image.titre']

?

[Séb.]

Code :

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

ORDER BY i.id ASC $ledebut,$nb_affichage_par_page

Il manque la clause LIMIT.

Le ID, le TITRE et la DESC des images ont changés pour ceux
des commentaires. Je tente les ajustement.

Parce que tes tables ont des champs de mêmes noms, ce qui n'est pas une bonne pratique.
Regarde dans l'exemple donné dans mon 1er message, chaque champ a un nom unique

Si je pourrais mettre comme ceci :

Code :

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

1
2
 
$ligne['ori_image.titre']

?

Plutôt $ligne['i.titre'] à cause des AS de la jointure.

[dancom5]

La solution des jointure fonctionnerait si mes champs des tables aurait été tous différents. J'ai en commun (id, titre et desc) dans mes deux tables. Il faudrait que je change à pas mal de place le code.

Alors, je tests la première solution:

Code :

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

1
2
3
4
5
SELECT COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC

Affiche bien tous les nombre de commentaires.

Mais, si j'ajoute :

Code :

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

UPDATE ori_images SET img_id =

qui devient :

Code :

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

1
2
3
4
5
6
7
UPDATE ori_image
SET idimg =
(SELECT COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC)

J'obtiens le message suivant dans phpmyadmin:

Code :

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

#1093 - You can't specify target table 'ori_images' for update in FROM clause

[Séb.]

La solution des jointure fonctionnerait si mes champs des tables aurait été tous différents. J'ai en commun (id, titre et desc) dans mes deux tables. Il faudrait que je change à pas mal de place le code.

Tu ne devrais pas avoir tant de scripts que ça à corriger.

Alors, je tests la première solution:

Mauvaise idée.

Code :

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

UPDATE ori_images SET img_id =

qui devient :

Code :

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

1
2
3
4
5
6
7
UPDATE ori_image
SET idimg =
(SELECT COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC)

Cela ne respecte pas ce que je t'avais donné dans mon 1er message.
Il faut que tu reprennes ma requête et que tu l'adaptes à ta BdD, il n'y a rien (et surtout pas de jointure) à ajouter.

[dancom5]

J'ai beaucoup de fichier qui utilise les mêmes tables.

J'essais ceci :

Code :

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

1
2
3
4
5
UPDATE ori_images
SET img_id =
(SELECT COUNT(*)
FROM ori_comen
WHERE idimg = id)

mais je suis un peu perdu avec la clause where

Le seul moyen d'afficher le nombre de commentaires avec les 0 :

Code :

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

1
2
3
4
5
SELECT COUNT( c.id ) AS n
FROM ori_images AS i
LEFT OUTER JOIN ori_comen AS c ON i.id = c.idimg
GROUP BY i.id
ORDER BY i.id ASC

[Séb.]

Code :

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

1
2
3
4
5
UPDATE ori_images
SET img_id = # Pourquoi changer l'identifiant de l'image ?
(SELECT COUNT(*)
FROM ori_comen
WHERE idimg = id) # id est ambigu

Essaie ceci :

Code :

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

1
2
3
4
5
6
7
UPDATE ori_images
SET nb_commentaires = # Champ nb_commentaires à créer
(
    SELEC COUNT(*)
    FROM ori_commen
    WHERE ori_commen.idimg = ori_images.id
)

[dancom5]

Dans phpmyadmin, ça fonctionne:

Code :

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

1
2
3
4
5
6
7
UPDATE ori_images
SET img_id =
(
    SELECT COUNT(*)
    FROM ori_comen
    WHERE ori_comen.idimg = ori_images.id
)

J'ai laissé img_id étant donné que ce champ est justement pour stocker les nombres.

Je vais faire la version en PHP.. Merci Séb!

[Séb.]

J'ai laissé img_id étant donné que ce champ est justement pour stocker les nombres.

Ah, pas sûr que le nom du champ soit bien choisi

[dancom5]

F Séb!

Fantastique!

En plus, cette solution est simple. Rien à changer.

J'ai appris quelque chose et ça va me servir dans le future.