Nieskonczona Funkcja, Co to ma byc Opcje

[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

[PHP] pobierz, plaintext 
<?php
  if ($baner['baner_max_show']==-1 OR $baner['baner_show']<$baner['baner_max_show']){
	  if ($baner['baner_typ']=='FLASH'){
		  echo $baner['baner_flash'];
		  $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
		  $db->sql_query($sql);
		  $isset_ip[]=$losowy;
		  } else{
		  echo '<a href=\"'.$baner['baner_url'].'\" target=\"_top\"><img src=\"'.$baner['baner_img'].'\" border=\"0\" alt=\"'.$baner['baner_alt'].'\"></a>';
		  $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
		  $db->sql_query($sql);
		  $isset_ip[]=$losowy;
}
}else{
 baner($typ);
}
?>
[PHP] pobierz, plaintext 

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

W tym ifie mam jakiąś błąd

[PHP] pobierz, plaintext 
<?php
if ($baner['baner_max_show']==-1 OR $baner['baner_show']<$baner['baner_max_show']){...}else{
 baner($typ);
}
?>
[PHP] pobierz, plaintext 

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 (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[kszychu]

OR jest operatorem bitowym, użyj operatora logicznego ||

[sajborg]

Niestety nic sie nie zmieniło (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[Luke]

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 ...

[PHP] pobierz, plaintext 
<?php
  if ($baner['baner_max_show']== '-1' || $baner['baner_show']<$baner['baner_max_show']){
	  if ($baner['baner_typ']=='FLASH'){
		  echo $baner['baner_flash'];
		  $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
		  $db->sql_query($sql);
		  $isset_ip[]=$losowy;
		  } else{
		  echo '<a href=\"'.$baner['baner_url'].'\" target=\"_top\"><img src=\"'.$baner['baner_img'].'\" border=\"0\" alt=\"'.$baner['baner_alt'].'\"></a>';
		  $sql=&#092;"UPDATE homp_baner SET baner_show=baner_show+1 WHERE baner_id='\".$losowy.\"'\";
		  $db->sql_query($sql);
		  $isset_ip[]=$losowy;
}
}else{
baner($typ);
}
?>
[PHP] pobierz, plaintext 

[kszychu]

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

[PHP] pobierz, plaintext 
<?php
if ($baner['baner_max_show']== '0' || $baner['baner_show']<$baner['baner_max_show']){
?>
[PHP] pobierz, plaintext 

[tsufu]

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

[PHP] pobierz, plaintext 
<?php
if ($baner['baner_max_show']== '0' || $baner['baner_show']<$baner['baner_max_show']){
?>
[PHP] pobierz, plaintext 

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 (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) a najlepiej to chyba jesli bylo mozliwe zbudowac if w ten sposob if ($baner['baner_max_show']<0 ) imho (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)