Algo mal en este codigo???

[Ne0Xer0]

Estoy haciendo un form, con algunos campos como nombre, telefono y algo asi, y que eso me lo envie con la funcion "mail()" a una direccion especificada con un listbox. hize este codigo y tb no he recivido el mail, ¿que esta mal con este codigo?.

if ($quien = 1)
{$email = "[e-mail]";}
elseif ($quien = 2)
{$email = "cablecolor@xxxcl";}
elseif ($quien = 3)
{$email = " [e-mail]";}
elseif ($quien = 4)
{$email = "[e-mail],[e-mail]";}
$mandarmail = mail($email,"Consulta web",$nombre.", ".$mail.", ".$rut.", ".$dir." le ha enviado esto:\n".$comentario);
if ($mandarmail = true)
{echo "\nSu comentario ha sido enviado, en poco tiempo le tendremos su respuesta";}
else
{echo "\nSu comentario no ha podido ser enviado, pruebe mas tarde";}

[Cluster]

if ($mandarmail = true)

eso sería:
if ($mandarmail)

(si quieres usar el true deberias usar el == de comparación no el = de asignación)

if ($mandarmail == true)

Luego .. el monton de if elseif que usas quedaria mas legible el código si usases:

Código PHP:

  switch ($quien) {
   case 1:
       $email="[e-mail]";
   case 2:
       $email="[e-mail]";
   case 3:
       $email="[e-mail]";
   default:
       $email="[e-mail]";
} 
   

Y sobre el uso de mail() .. mejor revisa la sintaxis correcta de mail():
http://us.php.net/manual/en/function.mail.php


Tambien como observación:

if ($mandarmail = true)
{echo "\nSu comentario ha sido enviado, en poco tiempo le tendremos su respuesta";}
else
{echo "\nSu comentario no ha podido ser enviado, pruebe mas tarde";}

eso despista bastante a la hora de leer codigo usa mejor:

if ($mandarmail = true){
echo "Su comentario ha sido enviado, en poco tiempo le tendremos su respuesta<br>";
} else {
echo "Su comentario no ha podido ser enviado, pruebe mas tarde<br>";
}

Se ve mas facil así donde termina y empieza un bloque {} .. y mas claro se ve si tuvieras mas lineas de código por bloque.

Y .. los saltos de linea \n q haces al prinpico e cada frase "echo" .. deberian ser al final .. En este caso .. ese salto de linea \n no es válido para HTML .. en HTML el salto de linea es \n (suponemos q estas dando salida a un navegador no ? ..)

Un saludo,

[Ne0Xer0]

vaya, muchas gracias, necesitaba que me corrigieran todo eso.

[josemi]

Hola,

Otra opcion para la seleccion del email es usar arrays y olvidarse de ifs y switchs. Pe:

Código PHP:

  $emails=(1=>"[e-mail]",2=>"[e-mail]",3=>"[e-mail]");
$email=$emails[$quien]; 
   

Es otra opcion. Habria que añadirle codigo para asegurarse que $quien tiene un valor valido (con array_key_exists()) y darle un valor por defecto.

Espero haber sido de ayuda.

PD: Respuesta patrocinada por AAA (Asociacion de Amigos del Array)