Stig Sørensen wrote:
> Ja naturligvis, tusinde tak igen Johan.
Det var så lidt... iøvrigt kom jeg til at kigge lidt på forspørgslen med
friske øjne - og underede mig lidt...
tjek lige om du ikke kan skifte
SELECT SUM(php_garage_rating.rating)/COUNT(php_garage_rating.rating) as
rt FROM phpbb_garage, php_garage_rating WHERE phpbb_garage.id = $id AND
phpbb_garage_rating.garage_id = $id
ud med
SELECT SUM(rating)/COUNT(rating) as rt FROM php_garage_rating WHERE
phpbb_garage_rating.garage_id = $id
Så vidt jeg kan se bruger du slet ikke data fra phpbb_garage
> Lidt OT spørgsmål, hvis det er i orden.
>
> Ved mange af mine SQL spørgelser har jeg følgende efter:
>
> while($row = mysql_fetch_assoc($query_vis_nyhed))
>
> feks:
>
> $query_vis_nyhed = mysql_query("SELECT *, DATE_FORMAT(tid, '%d.%m.%Y -
> kl: %H:%i') AS tid from nyheder WHERE id = '".$_GET['id']."'");
> while($row = mysql_fetch_assoc($query_vis_nyhed))
> {
>
> Grunden til at der i min rating-script, længere oppe, ikke er while
> indbundet, er det fordi der alligevel kun er én række vi skal have
> hævet ud?
> Bare så jeg har forstået det korrekt ;)
Ja - det er helt korrekt - while kører indtil condition returnerer
false... Derfor vil while løkken stoppe når der ikke er flere rows
tilbage at hælde ud af requested (da mysql_fetch_* returner false når
der ikke er flere rows tilbage).
Reelt set kan du dog sagnes bruge (når der blot er en enkelt felt eller
2 der skal hives ud) mysql_result - det er dog en dårlig vane (imho) med
mindre man har rimelig styr på det :)
Så generelt - brug mysql_fetch_assoc - alternativt mysql_fetch_row (som
dog gør koden mindre læselig).
--
Johan Holst Nielsen
Freelance PHP Developer -
http://phpgeek.dk