Ayuda codigo

[JCGX]

Bien, partiendo de que la variable upload="ok" el script funciona a la perfeccion y ejecuta todo este codigo. El problema es que si no es ="ok" sigue ejecutando la parte de html del codigo y se salta la parte de php. ¿Donde esta el problema?


Codigo php:

<?
if ($upload="ok")
{
//Ahora comprobamos si es necesario redimensionar la imagen

//Esto solo lo ejecuta si se cumple la condicion

if ($ancho>200 or $alto>200)
{
if ($ancho>$alto)
{
$compresion=200/$ancho;
$neoancho=200;
$neoalto=$alto*$compresion;

}
elseif ($ancho<$alto)
{
$compresion=200/$ancho;
$neoalto=200;
$neoancho=$alto*$compresion;
}
else
{
$neoalto=200;
$neoancho=200;
}

$neoalto=round($neoalto,0);
$neoancho=round($neoancho,0);
}
else
{
$neoalto=$alto;
$neoancho=$ancho;
}
?>

//esta parte la ejecuta tanto si se cumple la condicion como si no

<p><img src=<? echo "$path/imagen.$var[1]"?> width="<? echo $neoancho?>" height="<? echo $neoalto?>"></img></p>
<table width="400" border="0" cellspacing="0" cellpadding="0">
<tr>
<td>Informacion de:<? echo " $nombre" ?><br></td>
</tr>
<tr>
<td></td>
</tr>
<tr>
<td>Tama&ntilde;o:<? echo " $tamanio" ?></td>
</tr>
<tr>
<td>Tipo:<? echo " $tipo" ?></td>
</tr>
<tr>
<td>Ancho:<? echo " $ancho" ?></td>
</tr>
<tr>
<td>Alto:<? echo " $alto" ?></td>
</tr>
</table>
<p>&iquest;Esta seguro de que desea enviar esta imagen?</p>
<p>SI ----------------------- NO</p>
</div>

<?
}
?>

[Jordi1]

creo que deberia ir asi:

tu as puesto
if ($upload="ok")eso es igualdad

deberia ser
if ($upload=="ok") condicion

[josemi]

Hola,

Es lo que comenta Jordi1, pero creo que es mas claro decir que = es asignacion y == comparacion.

Yo para evitar estos despistes procuro poner en los casos de comparar con una constante la sentencia asi:

Código PHP:

  if ("ok"==$upload) 
   

porque si se me olvida un = PHP me da un mensaje de error, porque es ilegal asignar valroes a las constantes.

Saludos.