Algo mal en este codigo???

[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="[email protected]"; 
   case 2: 
       $email="[email protected]"; 
   case 3: 
       $email="[email protected]"; 
   default: 
       $email="[email protected]"; 
}  
   

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,