Problema al validar form

[ferwall]

Hola, tengo un problema con validar el formulario, agregue un if para que me de un error en otra pagina "completetodo.html" el problema es que al agregar el if donde sea que lo ponga, cada vez que completo el form aunque lo complete bien me lleva a la pagina completetodo.html, me anula el envio del formulario. Alguien sabe donde debería colocar dicho if? o de que otra forma debería colocarlo? esta marcado con un comentario en el código. Gracias

Código PHP:

  <?php 
$nombre=$_POST['nombre']; 
$nombreempresa=$_POST['nombreempresa']; 
$correo=$_POST['correo']; 
$direccion=$_POST['direccion']; 
$telefono=$_POST['telefono']; 
$celular=$_POST['celular']; 
$comentarios=$_POST['comentarios']; 
$opcion="";  
foreach ($_POST['opcion'] as $casilla){  
$opcion.=$casilla.", ";  
} 
$proveedisenio="";  
foreach ($_POST['radio0'] as $radio0){  
$proveedisenio.=$radio0.", ";  
} 
$comentarios1=$_POST['comentarios1']; 
$comentarios2=$_POST['comentarios2']; 
$comentarios3=$_POST['comentarios3']; 
$publicacupon="";  
foreach ($_POST['radio1'] as $radio1){  
$publicacupon.=$radio1.", ";  
} 
//este es el if para validar el form 
if ($nombre == "" or $nombreempresa == "" or $telefono == "" or $direcion == "" or $correo == "" or $celular== "")  
 
header("Location: completetodo.html");  
 
else { 
function form_mail($sPara, $sAsunto, $sTexto, $sDe) 
{ 
$bHayFicheros = 0;  
$sCabeceraTexto = "";  
$sAdjuntos = ""; 
 
if ($sDe)$sCabeceras = "From:".$sDe."\n";  
else $sCabeceras = "";  
$sCabeceras .= "MIME-version: 1.0\n";  
foreach ($_POST as $sNombre => $sValor)  
$sTexto = $sTexto."\n".$sNombre." = ".$sValor; 
 
foreach ($_FILES as $vAdjunto) 
{  
if ($bHayFicheros == 0) 
{  
$bHayFicheros = 1;  
$sCabeceras .= "Content-type: multipart/mixed;";  
$sCabeceras .= "boundary=\"--_Separador-de-mensajes_--\"\n"; 
 
$sCabeceraTexto = "----_Separador-de-mensajes_--\n";  
$sCabeceraTexto .= "Content-type: text/plain;charset=iso-8859-1\n";  
$sCabeceraTexto .= "Content-transfer-encoding: 7BIT\n"; 
 
$sTexto = $sCabeceraTexto.$sTexto;  
}  
if ($vAdjunto["size"] > 0) 
{  
$sAdjuntos .= "\n\n----_Separador-de-mensajes_--\n";  
$sAdjuntos .= "Content-type: ".$vAdjunto["type"].";name=\"".$vAdjunto["name"]."\"\n";;  
$sAdjuntos .= "Content-Transfer-Encoding: BASE64\n";  
$sAdjuntos .= "Content-disposition: attachment;filename=\"".$vAdjunto["name"]."\"\n\n"; 
 
$oFichero = fopen($vAdjunto["tmp_name"], 'r');  
$sContenido = fread($oFichero, filesize($vAdjunto["tmp_name"]));  
$sAdjuntos .= chunk_split(base64_encode($sContenido));  
fclose($oFichero);  
}  
} 
if ($bHayFicheros)  
$sTexto .= $sAdjuntos."\n\n----_Separador-de-mensajes_----\n";  
return(mail($sPara, $sAsunto, $sTexto, $sCabeceras));  
} 
 
if (form_mail("[email protected]", $_POST[asunto], 
"Los datos introducidos en el formulario son:\n\n", $_POST[email]))  
 
if (form_mail("[email protected]", $_POST[asunto], 
"Los datos introducidos en el formulario son:\n\n", $_POST[email])) 
header ("Location: gracias.html"); 
}//aca cierro el if para validar 
?>   

[MaNuX0218]

En el if indicas si algún campo se encuentra vacio o sin rellenar te envia a completetodo.html, si no, no.

Igualmente te falta parentesis en

if ($nombre == "" or $nombreempresa == "" or $telefono == "" or $direcion == "" or $correo == "" or $celular== "") {

Y

} else {

Saludos.

[ferwall]

Antes que nada gracias por responder, agregue las llaves, pero sigue sin funcionar, es como que anule el envío, me lleva siempre a la pagina de error, aunque el formulario este bien completado. Si borro el if completo se envia sin ningun problema.

Ya no se que hacer trate de validarlo con un sript pero tampoco pude, haga lo que haga, se me anula el envio y me muestra siempre el error, o lo envia directamente aunque haya campos incompletos

[herzbazi]

yo en lo personal lo haria con isset

para validar desde un principio y lo que haria es tirar despues una alerta con js

[Drowna]

La mejor forma es como te dice herzbazi, pon un if que encapsule todo y ponle como condición un isset que compruebe que todas las variables tienen contenido.

Código PHP:

if(isset($variable1) && isset($variable2) && resto..){
// Script de manipulación del formulario.
 
} else {
 
header('Donde quieras');
}
 

Lo que hace el isset es enviar true si la variable contiene algo. En la condicional con el && le indicamos que para que se ejecute el script de control del script todas las variables tienen que tener contenido. No sé si te funcione, espero que si.

[Middrel]

Corríjanme si me equivoco...

En ocasiones a mi no me funciona usar "or" o "and", me funciona mejor si uso "&&" o "||".

Y podrías usar también, como dice herzbazi,

Código PHP:

  if (!isset($nombre) || !isset($nombreempresa) || !isset($telefono) || !isset($direcion) || !isset($correo) || !isset($celular)) { 
 
Y 
 
} else {  
   

o en lugar de isset, empty

Código PHP:

  if (empty($nombre) || empty($nombreempresa) || empty($telefono) || empty($direcion) || empty($correo) || empty($celular)) { 
 
Y 
 
} else {  
   

[ferwall]

Ahi lo solucione!! muchas gracias por sus aportes!!! mas tarde subo la solucion como me quedo. muchas gracias a todos