[youheardit]

Magnus_A: Tack så mycket, detta var klockrent!

Obe: Som bjorne säger så är ju det problemet att alla artiklar (43955st) som inte har kommentar skrivs också ut. vilket jag inte vill, jag var nog lite otydlig i min första tråd, ber om ursäkt isåfall. men tack ändå

[obe]

Ursprungligen postat av Bjorne

Ditt sätt är dock felaktigt. Tänk på vad som händer ifall en artikel har 0 kommentarer.

Ok, men en WHERE-sats är ju inte så svårt att lägga till. Generellt sett är joins bättre än subquerys.

[Bjorne]

Det hjälper inte med en WHERE sats, trust me. Med LEFT JOIN:en kommer 1 rad för varje artikel att väljas för alla artiklar som har 0 eller 1 kommentar. Alltså kommer COUNT(*) ge 1 de fallen. Man kan lösa det genom att använda en INNER JOIN istället som gör att bara artiklar som har 1 eller flera kommetarer väljs. Men då listas ju inga artiklar med 0 kommentarer.

[FredrikMH]

Bjorne, jo men det går att lösa. Fast det kanske är långsamt?

SELECT a.*, c.ncomments FROM artiklar a
LEFT JOIN (SELECT pid, count(*) AS ncomments FROM comments GROUP BY pid) AS c
ON c.pid=a.pid

Hur som helst så innebär en kombination av GROUP BY och ORDER BY att en temporär tabell skapas. Det bästa hade varit att spara antalet kommentarer i en egen kolumn i tabellen för artiklarna.

[obe]

Ursprungligen postat av Bjorne

Det hjälper inte med en WHERE sats, trust me. Med LEFT JOIN:en kommer 1 rad för varje artikel att väljas för alla artiklar som har 0 eller 1 kommentar. Alltså kommer COUNT(*) ge 1 de fallen. Man kan lösa det genom att använda en INNER JOIN istället som gör att bara artiklar som har 1 eller flera kommetarer väljs. Men då listas ju inga artiklar med 0 kommentarer.

Ok, jag var tvungen att prova den och den fungerar. Du missade nog "GROUP BY".

[Magnus_A]

select count(kommentartabellen.postid) as antal_kommentarer,
kommentartabellen.postid,
en_annan_tabell.annat_fält
from kommentartabellen, en_annan_tabell
where kommentartabellen.postid=en_annan_tabell.postid
group by kommentartabellen.postid
order by antal_kommentarer desc limit [antal i topplistan]

Bättre så?

[FredrikMH]

Den bästa lösningen är ändå att köra en seperat kolumn i artikel-tabellen och räkna antalet kommentarer där. Blir ju en extra SQL-fråga vid in/borttagning av kommentar men slipper köra en GROUP BY/ORDER BY.

[Bjorne]

Ursprungligen postat av FredrikMH

Bjorne, jo men det går att lösa. Fast det kanske är långsamt?

SELECT a.*, c.ncomments FROM artiklar a
LEFT JOIN (SELECT pid, count(*) AS ncomments FROM comments GROUP BY pid) AS c
ON c.pid=a.pid

Hur som helst så innebär en kombination av GROUP BY och ORDER BY att en temporär tabell skapas. Det bästa hade varit att spara antalet kommentarer i en egen kolumn i tabellen för artiklarna.

Jo fast om man ska vara petnoga (vilket jag tycker man ska vara när det gäller sql) ska det vara:

SELECT a.*, COALESCE(c.ncomments, 0) FROM ...

Annars får du NULL istället för 0 ifall en artikel inte har några kommentarer. Men sen specificierade ju youheardit att topplistan aldrig ska innehålla artiklar med 0 kommentarer och då är ju obes ursprungliga version effektivast igen.