problema con codigo ayuda porfavor!

[la.terrible]

Hola,

bueno, aqui va el codigo que me esta dando problemas :S

Código PHP:

  if(strlen($tour && $dia)>0) 
{ 
    if(!$sendmail) 
        echo $mailbad; 
    else 
    { 
        echo $mailok; 
        $sendmail; 
    } 
     
} 
else 
    echo $formbad;  
   

el problema esta en que si las variables $tour y $dia no son mas grandes que 0, igual me envía el mail, siendo que no debería porque si no se cumple el primer IF debería ejecutarse el segundo ELSE (el ultimo, el echo $formbad;)

lo raro es que si no se cumple el primer IF, se ejecuta el ultimo ELSE, pero el mail se envía de todas formas :S

quien será el bueno que le hechará una mano a este newbie =) ?


chao!

[santuguerra]

Código PHP:

  if(strlen($tour)>0 && strlen($dia)>0)
{
    if(!$sendmail)
        echo $mailbad;
    else
    {
        echo $mailok;
        $sendmail;
    }
    
}
else
    echo $formbad;  
   

Con eso te deberia andar

Suerte!

[la.terrible]

no me anda =(
me tira el mismo problema que antes =(

[santuguerra]

De donde viene el $tour y $dia?
pregunto porque quizas deberias usar el $_POST['tour'] y $_POST['dia']...

[la.terrible]

asi lo tengo $tour = $_POST['tour'] y $dia = $_POST['dia']

:S

[Bellenger]

Hola, deberias completar tu codigo cuando usas las llevas, es una buena practica de programacion...

Código PHP:

  if(strlen($tour)>0 && strlen($dia)>0){ 
    if(!$sendmail){ 
        echo $mailbad; 
    }else{ 
        echo $mailok; 
        $sendmail; 
    } 
     
}else{ 
    echo $formbad;  
}  
   

ahora si no debe fallar...

[la.terrible]

lo solucioné ya =O!

aqui esta el codigo terminado, aunque tengo una duda,

Código PHP:

  // RESERVA 
if(strlen ($tour and $dia and $pasajeros and $hotel and $nombre and $direccion and $ciudad and $pais and $email)>0 && $datoradiobutton == reserva ){ 
    if(@mail($para,$asunto,$mensaje,$headers)){ 
        echo $mailok; 
        } 
    else { 
        echo $mailbad; 
        } 
} 
 
//INFO     
elseif (strlen ($iemail and $inombre and $icoment)>0 && $datoradiobutton == info) { 
    if (@mail ($para,$iasunto,$imensaje,$headers)){ 
        echo $mailok; 
    } 
    else { 
        echo $mailbad; 
    } 
} 
else { 
    echo $formbad; 
    exit; 
}  
   

como hago para agrupar esas variables en vez de colocarlas todas y ponerle and $variable and $variable etc, no se pueden agrupar todas esas?

gracias chao!

[GatorV]

Creo no estas usando la mejor forma para strlen, si te fijas strlen espera un string, y al tu hacer $iemail && $inombre, estas convirtiendo a un booleano y luego nuevamente a strlen.

Si quieres comparar que ciertas variables no esten vacias puedes hacer esto:

Código PHP:

  $required = array( "iemail", "inombre", "icoment" );
$empty = false;
foreach( $required as $var ) {
      if( empty( $$var ) ) {
            $empty = true;
      }
}
if( !$empty ) {
     mail... 
} else {
     echo $formbad;
     exit;
}  
   

Saludos.