Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Nieskonczona Funkcja
Forum PHP.pl > Forum > PHP
sajborg
Mam sobie funkcje wyswietlającą banery Mam w bazie maxymalną liczbe wyswietleń i ilość jaką juz zostały wyswietlone oto fragmęt w którym mam błąd

  1. <?php
  2. if ($baner['baner_max_show']==-1 OR $baner['baner_show']<$baner['baner_max_show']){
  3. if ($baner['baner_typ']=='FLASH'){
  4. echo $baner['baner_flash'];
  5. $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
  6. $db->sql_query($sql);
  7. $isset_ip[]=$losowy;
  8. } else{
  9. echo '<a href=\"'.$baner['baner_url'].'\" target=\"_top\"><img src=\"'.$baner['baner_img'].'\" border=\"0\" alt=\"'.$baner['baner_alt'].'\"></a>';
  10. $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
  11. $db->sql_query($sql);
  12. $isset_ip[]=$losowy;
  13. }
  14. }else{
  15.  baner($typ);
  16. }
  17. ?>


Banery które mają max wiswietleń na -1 znaczy to ze mają nieograniczoną liczbe .

W tym ifie mam jakiąś błąd

  1. <?php
  2. if ($baner['baner_max_show']==-1 OR $baner['baner_show']<$baner['baner_max_show']){...}else{
  3.  baner($typ);
  4. }
  5. ?>


Nie działa to tylko cały czas funkcja baner sie powtarza i www sie zawiesza.

Mam w bazie powiedzmy 3 wpisy

1 Wyswietlen 5120 Max wyswietlen 1000
2 Wyswietlen 2342 Max wyswietlen -1
3 Wyswietlen 23 Max wyswietlen 1000

Więc powinno wyswietlić któryś z 2 banerów albo 2 albo 3 a tu nic sad.gif
kszychu
OR jest operatorem bitowym, użyj operatora logicznego ||
sajborg
Niestety nic sie nie zmieniło sad.gif
Luke
Cytat
OR jest operatorem bitowym, użyj operatora logicznego ||


bzdura ... oba operatory sa logiczne, roznia sie priorytetem wykonania ...

http://www.php.net/manual/pl/language.operators.logical.php

może tak ...

  1. <?php
  2. if ($baner['baner_max_show']== '-1' || $baner['baner_show']<$baner['baner_max_show']){
  3. if ($baner['baner_typ']=='FLASH'){
  4. echo $baner['baner_flash'];
  5. $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
  6. $db->sql_query($sql);
  7. $isset_ip[]=$losowy;
  8. } else{
  9. echo '<a href=\"'.$baner['baner_url'].'\" target=\"_top\"><img src=\"'.$baner['baner_img'].'\" border=\"0\" alt=\"'.$baner['baner_alt'].'\"></a>';
  10. $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
  11. $db->sql_query($sql);
  12. $isset_ip[]=$losowy;
  13. }
  14. }else{
  15. baner($typ);
  16. }
  17. ?>
kszychu
Cytat(Luca @ 2004-08-25 11:25:21)
Cytat
OR jest operatorem bitowym, użyj operatora logicznego ||


bzdura ... oba operatory sa logiczne, roznia sie priorytetem wykonania ...

Mea culpa
sajborg
Nie kumam tego ale działa zamieniłem -1 na 0 i smiga kompletnie tego nie rozumiem

  1. <?php
  2. if ($baner['baner_max_show']== '0' || $baner['baner_show']<$baner['baner_max_show']){
  3. ?>
tsufu
Cytat(paweb @ 2004-08-25 14:04:34)
Nie kumam tego ale działa zamieniłem -1 na 0 i smiga kompletnie tego nie rozumiem

  1. <?php
  2. if ($baner['baner_max_show']== '0' || $baner['baner_show']<$baner['baner_max_show']){
  3. ?>

mozliwe iz:
$baner['baner_max_show'] jest traktowany jako string a nie jako powiedzmy liczba czy cos takiego
po zapisie w ciapkach ' ' wlasnie te wartosci sa porownywane jako stringi wiec bedzie dzialac winksmiley.jpg a najlepiej to chyba jesli bylo mozliwe zbudowac if w ten sposob if ($baner['baner_max_show']<0 ) imho winksmiley.jpg
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.