WN

WN (https://www.wn.se/forum/index.php)
-   Serversidans teknologier (https://www.wn.se/forum/forumdisplay.php?f=4)
-   -   klurig mySQL fråga (php) (https://www.wn.se/forum/showthread.php?t=1039423)

youheardit 2009-12-04 08:29

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 2009-12-04 10:45

Citat:

Ursprungligen postat av Bjorne (Inlägg 20333985)
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 2009-12-04 11:34

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 2009-12-04 11:53

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 2009-12-04 12:06

Citat:

Ursprungligen postat av Bjorne (Inlägg 20334038)
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 2009-12-04 21:57

Citat:

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 2009-12-05 05:54

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 2009-12-05 15:59

Citat:

Ursprungligen postat av FredrikMH (Inlägg 20334044)
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.


Alla tider är GMT +2. Klockan är nu 19:49.

Programvara från: vBulletin® Version 3.8.2
Copyright ©2000 - 2025, Jelsoft Enterprises Ltd.
Svensk översättning av: Anders Pettersson