[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

[dawhol]

stwierdzilem terasz ze trzeba by poprawic to tak aby wszystko wzgledem ID bylo albo obrazka a nie adresu bo czasem mozna dac dwa rozne banery pod jeden adres =)

w sumie trzeba by to popraiwc tam jes BETWEN 1 and 3 poniewaz chce aby losowal mi liczbe pomiedzy tymi trzema bo obecnie mam 3 banery w bazie ale trzeba by tam dac ostatni id z bazy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) zamiast tej 3 zapomniałem to zrobic (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)