klurig mySQL fråga (php)
 

Hejsan!
Har nu stött på ett problem... jag driver sidan nyhetsarkiv.se och i systemet finns det tillgänglighet att kommentera en artikel.
Artiklarna ligger i en tabell och kommentarerna i en.
Men nu vill jag göra en lista över mest kommenterade artiklarna i form av "Mest diskuterade"... jag har googlat i säkert någon timme utan resultat.
kommentarerna har en kolumn "postid" som är id't på artikeln som kommentaren tillhör.

Någon som har en lösning? jag har försökt med array_sort() mm...
men får inte riktigt till det så den räknar av antal kommentarer per artikel...

Tacksam för svar!
Med vänliga hälsningar!
//Philip Andersson!

Något sånt här borde fungera:
select count(postid) as antal_kommentarer,
postid
from kommentartabellen
group by postid

$sql = "select count(pid) as antalet,
pid
from comments
group by pid";
while($line = mysql_fetch_assoc(mysql_query($sql))){
echo $line['pid'];
echo "<br>";
}
?>[/CODE]
pid = postid som i förra posten...

Resultatet blir en oändligt lång lista med samma pid som i första kommentaren...
har jag gjort fel ? eller är mysql-frågan fel ställd?

tack! :)

jag skulle kunna tänka mig att det borde funka med att först använda någon form av array sort by count eller något och sedan köra array_unique på dem så får man ju fram listan...

Du borde kunna köra något liknande:

select
pid,
(select count(*) from comments where pid=c.pid) as antalet
from comments c
order by antalet desc

Det kan vara snabbare än group by förutsatt att du har bra indexering..

Får samma resultat:
Känns som jag gör fel?
Är allting rätt när jag skriver ut resultatet?

select a.*, count(*) as ncomments from artiklar a left join comments c on c.pid=a.pid group by a.pid order by a.pid desc

??

har jag gjort fel? you bet.
Gör du på detta viset så ställer du frågan till databasen om och om igen och visar resultatet för första raden ett oändligt antal gånger.
Den sql-fråga du fått ger det riktiga svaret, OM du ställer frågan rätt till databasen och sen tar reda på svaret och skriver ut det ordentligt.
Pröva med att ställa frågan En(1) gång till databasen, ta hand om resultatet och lista det på ett riktigt sätt:
Ser du skillnaden?

Dock är jag ganska övertygad om att mitt sätt är snabbare, dessutom får du även ut artikeldatan i samma fråga :)

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

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å :)

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

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.

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.

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

Bättre så?

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.

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.