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

bez przesady... jak juz podajecie linki to czytajcie chociaz:

mysql_free_result() używa się tylko w wypadkach obawy zajęcia zbyt dużej ilości pamięci przez zapytania zwracające duże ilości danych. Cała pamięć przydzielona wynikowi skojarzonemu z podanym identyfikatorem wyniku będzie automatycznie zwolniona.

Skrypt tutaj zwraca jeden rekord, ktory ma sie nijak do " duże ilości danych."

edit (po poscie ponizej): z tresci posta wnioskowalem jednak ze to jest wymog dla tego skryptu a nie dobry nawyk (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)