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

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

[nospor]

ale trzeba by tam dac ostatni id z bazy

Ale po co? Po co wogole warunek na te id, skoro mają byc losowane wszystkie rekordy? wystarczy warunek na wyswietlenia.

[dawhol]

no spoko ale jak nie bylo tego to mi losowal liczbe której niema w bazie a moze mialem inny blad (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
musze to sprawdzic (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) poziej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) bo teraz lece do pracy (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[nospor]

no spoko ale jak nie bylo tego to mi losowal liczbe której niema w bazie

jak moze losowac ci z bazy cos czego nie ma? Losuje po rekordach, ktore są

[Astarot]

W swoim rotatorze nie uwzględniłeś danych osadzonych w skryptach JS, a także banerków flashowych. Pozatym dodaj alt="coś" bo walidator będzie krzyczał.

<img src="'.$dane['obrazek'].'" alt="cos" />

pozdr.

Ten post edytował Astarot 25.07.2006, 09:27:42

[dawhol]

~nospor moze i masz racje widocznie mialem inny blad (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) i jak go poprawilem to zapomnialem tego zmienic (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) no ale w koncu ucze sie od paru dni wiec odrazy nie napisze skryptół bezbłędnyc po to je tu daje zeby je ocenic i ew. podac jakies uwagi (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

~astarot fatk zapomniałem o tym (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) zdarza sie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) trzn w pierwszej wersji to było ale w pierwszej wersji nie mialem za to ilosci wyswietlen a teraz dodalem i zapomnialem o tym (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Nightwalker]

w Twoim kodzie nie ma jakże ważnego

mysql_free_result" title="Zobacz w manualu php" target="_manual

[dawhol]

~Nightwalker zaraz to dodam u siebie (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) dzieki za poinformowanie mnie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) nawet nie znalem takiej funkcji (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale teraz juz bede o niej pamietał (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) w koncu czlowiek uczy sie na blędach (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

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

[Nightwalker]

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

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

Takie coś rodzi złe nawyki, dlatego to napisałem (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Edit do posta powyżej: źle się wyraziłem, ale lepiej opróżniać ZAWSZE i później nie zapominać, niż na odwrót (przynajmniej takie jest moje zdanie w tej sprawie) (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował Nightwalker 25.07.2006, 17:33:59

[Turgon]

Według mnie przydałoby się mysql_num_rows, gdyż wykonujemy selecta id z bazy bez warunków, czyli mamy wszystkie rekordy kolumny id. Następnie ta prosta komenda i skrypt wie spośród jakich liczb losowac.

[dawhol]

Dobra to powiem co poprawilem i moderator moze zamknac temat (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) jak chce ofc (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

1) usubołem warunek id BETWEEN 1 AND ... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) i działą moj bład był jakis (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
2) dodałem pole opis w bazie oraz alt w obrazku i title w linku (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
3) zrobiłem echo mysql_error w funkcji warunkowej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) przy mysql_query (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) tak jak radzil ~nospor
4) dodałem w tabeli pole max wyswietlen i zamiast 1000 w pierwsyzsm zapytaniu sql dalem 'maxwyswietlen' tzn:

[SQL] pobierz, plaintext 
SELECT * FROM cms_bannery WHERE wyswietlen<maxwyswietlen ORDER BY rand() LIMIT 1
[SQL] pobierz, plaintext 

działa pięknie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) jeszcze pozniej sie pobawei aby wyszczególnic banery gif od flasha i innych (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) ale to pozniej (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował dawhol 25.07.2006, 17:19:51

[Nightwalker]

I jeszcze jedno, dlaczego wyciągasz wszystko z bazy (SELECT *) zamiast wyciągnąć tylko potrzebne pola, przykładowo

[SQL] pobierz, plaintext 
SELECT `pole1`, `pole2`, `pole3` FROM `tabela`
[SQL] pobierz, plaintext 

?

Komentarz do wpisu poniżej: nie znam struktury bazy, a przeważnie potrzebne jest tylko kilka pól...

Ten post edytował Nightwalker 25.07.2006, 18:52:49

[dawhol]

no bo wszystko jest mi potrzbne (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) to chyba logiczne (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)