Maldito if

[SirDuque]

Hola gente, tengo un if bastante denso.

Código PHP:

if ( ($socio!=NULL and $socio==$row['CODIGO'])and ($apyno!=NULL and $apyno!=$row['APYNO']) and ($direc!=NULL and $direc==$row['DIREC']) and ($postal!=NULL and $postal==$row['CODPOS']) and ($local!=NULL and $local==$row['LOCAL']) and ($telef!=NULL and $telef==$row['TELEF']) and ($estado!=NULL or $estado==$row['ESTADO']) or ($fenac!=NULL and $fenac==$row['FENAC']) and ($fein!=NULL and $fein==$row['FEING']) or ($feegr!=NULL and $feegr==$row['FEEGR']) or ($catego!=NULL and $catego==$row['CATEGO']) and ($nrdoc!=NULL and $nrdoc==$row['NRDOC']) and ($legajo!=NULL and $legajo==$row['LEGAJO']) or ($email!=NULL and $email==$row['EMIAL']) or ($inter!=NULL and $inter==$row['TELELAB']) and ($cremax!=NULL and $cremax==$row['CREMAX']) and ($id!=NULL and $id==$row['ID_SOPRO']))
    {
        echo "se actualizo el Apellido y Nombre<br>";
    }
 

y este

Código PHP:

if ( ($socio!=NULL and $socio==$row['CODIGO'])and ($apyno!=NULL and $apyno==$row['APYNO']) and ($direc!=NULL and $direc!=$row['DIREC']) and ($postal!=NULL and $postal==$row['CODPOS']) and ($local!=NULL and $local==$row['LOCAL']) and ($telef!=NULL and $telef==$row['TELEF']) and ($estado!=NULL or $estado==$row['ESTADO']) or ($fenac!=NULL and $fenac==$row['FENAC']) and ($fein!=NULL and $fein==$row['FEING']) or ($feegr!=NULL and $feegr==$row['FEEGR']) or ($catego!=NULL and $catego==$row['CATEGO']) and ($nrdoc!=NULL and $nrdoc==$row['NRDOC']) and ($legajo!=NULL and $legajo==$row['LEGAJO']) or ($email!=NULL and $email==$row['EMIAL']) or ($inter!=NULL and $inter==$row['TELELAB']) and ($cremax!=NULL and $cremax==$row['CREMAX']) and ($id!=NULL and $id==$row['ID_SOPRO']))
    {
        echo "Cambia direccion<br>";
    }
 

Si le prestan atencion las diferencias son:


($apyno!=NULL and $apyno!=$row['APYNO']) and ($direc!=NULL and $direc==$row['DIREC'])

($apyno!=NULL and $apyno==$row['APYNO']) and ($direc!=NULL and $direc!=$row['DIREC'])

Para las dos imprimen.
resultado:

echo "se actualizo el Apellido y Nombre<br>";
echo "Cambia direccion<br>";


Alguna idea?

[Sourcegeek]

Y...
Que es lo que quieres que hagamos?

[SirDuque]

Iniciado por Sourcegeek

Y...
Que es lo que quieres que hagamos?

Que si $apyno!=$row['APYNO'] se da

imprima:
echo "se actualizo el Apellido y Nombre<br>";


Y como $direc!=$row['DIREC'] no se da.

NO imprima:
echo "Cambia direccion<br>";

y en cambio estan imprimiendo las dos cosas. tambien invirtiendo los caso.

[Triby]

Algo tienes mal en el planteamiento de tu script, lo mas facil seria evitar todos esos $variable == NULL inicializando cada una, suponiendo que vienen desde POST:

Código PHP:

// Con IF convencional:
if(isset($_POST['variable']))
    $variable = $_POST['variable'];
else
    $variable = 0; // Puede ser false, cadena vacia o lo que corresponda, segun el tipo
 
// Lo mismo, pero con asignacion ternaria:
$variable = (isset($_POST['variable'])) ? $_POST['variable'] : 0;
 
// Despues solamente deberias comparar:
if($variable == $row['campo'] && bla bla bla) {
}
 

Por cierto, tambien deberias considerar usar && en vez de AND, ya que este ultimo, segun el manual de PHP tiene "menor peso" incluso que || (OR).

No, los IF no estan malditos, simplemente evaluan las comparaciones que escribimos dentro de los parentesis, si no lo hacemos bien... seguro que no funciona correctamente!

[Sourcegeek]

Como punto a marcar... No necesitas checar si una variable es diferente a NULL y checar si esa variable tiene X valor..
Es obvio que si la variable tiene X valor, no es nula...
Me explico;

Código PHP:

  //Erroneo checar dos veces
if ($socio!=NULL and $socio==$row['CODIGO']) {}

//Correcto
if ($socio==$row['CODIGO']) {}  
   

[SirDuque]

Iniciado por Sourcegeek

Como punto a marcar... No necesitas checar si una variable es diferente a NULL y checar si esa variable tiene X valor..
Es obvio que si la variable tiene X valor, no es nula...
Me explico;

Código PHP:

  //Erroneo checar dos veces 
if ($socio!=NULL and $socio==$row['CODIGO']) {} 
 
//Correcto 
if ($socio==$row['CODIGO']) {}  
   

Gracias Capote! lo edite como dijiste y lo soluciono


SOLUCIONADO

[Sourcegeek]

De nada :D

//Cof cof.. Un poquitin de karma... Cof.. Cof..... Jaja :P