[MySQL][php] Rotator Banerów =) Oceńcie i sprawdzcie :) Opcje

[dawhol]

W tabeli mamy trzy pola:
"adres" - adres do strony
"obrazek" - adres do obrazka
"wyswietlen" - ilos wyswietlen (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Pliku z formularzem nie daje bo chyba kazdy wie jak wygląda (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
Pliku gdzie jest wykonywana funkcja która dodaje do bazy MySQL tez nie daje poniewaz tego nie chce abyscie oceniali bo niema tam co zwykle zapytani (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) zalezy mi na ocenie skryptu który wyświetla banery (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) rotacyjnie co odswierzenie strony (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . Oto jego kod:

[PHP] pobierz, plaintext 
<?php
$zapytanie = "SELECT * FROM cms_bannery WHERE id BETWEEN 1 AND 3 AND wyswietlen<1000 ORDER BY rand() LIMIT 1";
$wykonaj = mysql_query($zapytanie);
 
if($dane=mysql_fetch_array($wykonaj)) 
{
	echo ('<a href="'.$dane['adres'].'"><img src="'.$dane['obrazek'].'"></a>');
} 
else 
	 echo mysql_error();
 
$wysw = $dane['wyswietlen'];
$adres = $dane['adres'];
$up = 1;
 
$ilwysw = $wysw+$up;
 
$zapytanie2 = "UPDATE cms_bannery SET wyswietlen='$ilwysw' WHERE adres='$adres'";
$wykonaj2 = mysql_query($zapytanie2);
 
if($wykonaj2) 
{
	echo ('');
} 
else 
	echo mysql_error();
?>
[PHP] pobierz, plaintext 

Oczywiscie skrypt po kazdym wyswietleniu banera dodaje 1 do jego liczby wyswietlen (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Prosze o oceny i ew. rady co i jak poprawic (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) moj drugi skrypcik po 4-5 dniach nauki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował dawhol 25.07.2006, 08:14:14

[nospor]

[PHP] pobierz, plaintext 
<?php
if($dane=mysql_fetch_array($wykonaj)) 
{
	echo ('<a href="'.$dane['adres'].'"><img src="'.$dane['obrazek'].'"></a>');
} 
else 
	 echo mysql_error();
?>
[PHP] pobierz, plaintext 

Te mysql_error() powinno byc po mysql_query a nie po mysql_fetch_array(). W przypadku bledu zapytania dostaniesz jeszcze niepotrzebne bledy przy mysql_fetch_array() gdyz wykonywac sie bedzie na blednych danych.
Gdy zapytanie nic ci nie zwroci, to wygenereuje sie mysql_error(), czyli nic sie nie wygeneruje, gdy errora mysql nie bedzie.
Krotko mowiac: w zlym miejscu dalej mysql_error() (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[SQL] pobierz, plaintext 
...id BETWEEN 1 AND 3...
[SQL] pobierz, plaintext 

czemu tak? mozna tylko 3 banery losowac i to tylko z zadanym id?