[Magnus_A]

Ursprungligen postat av youheardit

$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...

40
40
40
40
40
osv....

har jag gjort fel ? eller är mysql-frågan fel ställd?

tack!

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:

(fråga som ovan)
$result=mysql_query($sql);
while($line = mysql_fetch_assoc($result)){
print_r($line);
echo "<br>";
}

Ser du skillnaden?

[obe]

Ursprungligen postat av Magnus_A

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

[Bjorne]

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

[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.

[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".