Lista tematów na forum Opcje

[snapshot]

Mam tabele topics i posts. Topics zawiera klucz obcy idPost z posts (są tam dane z "postu zakładającego temat"), a posts zawiera idTopics łączący z Topics. Wyciągam listę tematów z tytułem, datą założenia tematu, liczbą postów i ostatnią odpowiedzią:

[SQL] pobierz, plaintext 
SELECT `t`.*, `p`.`dateCreate`, `p`.`idPost`, `p`.`idUser`, `u`.`login`, `u`.`idUser`, `u`.`isAdmin`, COUNT(p2.idPost) AS `posts`, MAX(p2.dateCreate) AS `dateReply`, `c`.`code` AS `countryCode`, `c`.`name` AS `countryName`
FROM `topics` AS `t`
INNER JOIN `posts` AS `p` ON t.idPost = p.idPost
INNER JOIN `users` AS `u` ON p.idUser = u.idUser
LEFT JOIN `posts` AS `p2` ON t.idTopic = p2.idTopic
LEFT JOIN `countries` AS `c` ON u.userCountry = c.idCountry
WHERE (t.idForum = 2) AND (p.idPost != p2.idPost)
GROUP BY `t`.`idTopic`
ORDER BY `dateReply` DESC
[SQL] pobierz, plaintext 

Nie wyświetlają mi się natomiast tematy nie mające odpowiedzi. Próbowałem kombinować z instrukcją warunkową IF(), ale do niczego nie doszedłem.

[nieraczek]

A po co w topics idPost ? Dane z postu zakładającego temat mogłeś dać do topics. I wtedy:

[SQL] pobierz, plaintext 
LEFT JOIN `posts` AS `p` ON t.idTopic = p.idTopic
INNER JOIN `users` AS `u` ON p.idUser = u.idUser
LEFT JOIN `countries` AS `c` ON u.userCountry = c.idCountry
[SQL] pobierz, plaintext 

Ci pewnie nie chciało się dwa razy wstawiać identycznych kolumn w dwóch tabelach i zapewne dlatego tak zrobiłeś, ale już przy zwykłym wyszukaniu tematów, w których coś tam, coś tam - będziesz musiał łączyć dwie tabele i w ogóle zapytania się skomplikują.

Z drugiej strony nie trzeba dwa razy dawać takich samych kolumn dla dwóch tabel. Możesz spróbować tak:

[SQL] pobierz, plaintext 
LEFT JOIN `posts` AS `p` ON (t.idTopic = p.idTopic AND t.idPost = p.idPost)
INNER JOIN `users` AS `u` ON p.idUser = u.idUser
LEFT JOIN `countries` AS `c` ON u.userCountry = c.idCountry
[SQL] pobierz, plaintext 

Ten post edytował nieraczek 22.05.2009, 08:18:27

[snapshot]

Działa:

[SQL] pobierz, plaintext 
SELECT
  `t`.*, `p`.`dateCreate`, `p`.`idPost`, `p`.`idUser`, `u`.`login`, `u`.`idUser`, `u`.`isAdmin`,
  COUNT(p2.idPost)-1 AS `posts`,
  IF(COUNT(p2.idPost)-1>0,MAX(p2.dateCreate),NULL) AS `dateReply`,
  `c`.`code` AS `countryCode`, `c`.`name` AS `countryName`
FROM `topics` AS `t`
INNER JOIN `posts` AS `p` ON t.idPost = p.idPost
INNER JOIN `users` AS `u` ON p.idUser = u.idUser
INNER JOIN `posts` AS `p2` ON t.idTopic = p2.idTopic
LEFT JOIN `countries` AS `c` ON u.userCountry = c.idCountry
WHERE (t.idForum = 2)
GROUP BY `t`.`idTopic`
ORDER BY MAX(p2.dateCreate) DESC
[SQL] pobierz, plaintext 

Mam tylko pytanie, czy wywoływanie po 2 razy MAX i COUNT jest dobre, czy jednak da się to zrobić optymalniej?

Ten post edytował snapshot 22.05.2009, 13:37:09

[erix]

Nie jestem pewien, ale czy by nie wystarczyło użycie samego aliasu zamiast drugiego count, ew. zmiennej?