Pregunta de if else

[miros84]

Código PHP:

  if($ime!="" and $mail!="" and $mensaje!="") 
{  
mysql_query("INSERT INTO formulario (ime, mail, mensaje) 
VALUES ('$ime', '$mail', '$mensaje')");  
}  
   

Tengo un formulario en donde los usuarios pueden dejar opinion, pero al dar F5, si has dejado antes alguna opinion, se vuelve a insertar la misma opninion. Por eso he hecho un if-else para que no me mete datos en la tabla si no se rellenan los campos, pero es que no me funcciona. Tengo algun error?

[pateketrueke]

en ves de eso, evitalo desde otro punto...

cuando insertes correctamente, redirige a la misma página para borrar el post...

y siempre, comprueba que exista el POST antes de insertar/actualizar ... y sobre todo, valida todo!

Código PHP:

  if ( ! empty($_POST))
{
  $id = ! empty($_POST['id'])? $_POST['id']: '';
  $msg = ! empty($_POST['msg'])? $_POST['msg']: '';

  if ( ! empty($id) && ! empty($msg))
  {
    // ...

    $sql = 'INSERT ...';

    header("Location: $_SERVER[PHP_SELF]");
  }
}  
   

espero comprendas, suerte!

[miros84]

Código PHP:

  <?php 
ini_set('error_reporting', E_ALL);
$self=$_SERVER['PHP_SELF'];
$con = mysql_connect("456","web","dfgdfgdfg");
if (!$con)
  {
  die('Could not connect: ' . mysql_error());
  }

mysql_select_db("mitabla", $con);


if(isset($_POST["ime"])) {$ime=addslashes(trim($_POST["ime"]));}
if(isset($_POST["mail"])) {$mail=addslashes(trim($_POST["mail"]));}
if(isset($_POST["mensaje"])) {$mensaje=addslashes(trim($_POST["mensaje"]));}
if(isset($_POST["spam"])) {$spam=addslashes(trim($_POST["spam"]));}


if($ime!="" and $mail!="" and $mensaje!="")
{ 
if($spam==78){ 
mysql_query("INSERT INTO formulario (ime, mail, mensaje)
VALUES ('$ime', '$mail', '$mensaje')"); 
} 
} 

$result = mysql_query("SELECT * FROM formulario");
while($row = mysql_fetch_array($result))
{
aqui va el texto de la tabla
}
?>   

Echa un vistazo si tengo la suficiente seguridad.
La linea if($spam==78){} es porque tengo in input con antispam pregunta, donde si respondes con 78 se inserta, si no, no se inserta nada. Donde meteria el codigo que tu me proporcionaste?

[pateketrueke]

no lo tienes que meter en ningún lado !!!

es solo un ejemplo, analizalo... y aplicalo a tu propio script.....


suerte!

[miros84]

Es que se me hace mas complejo de lo que yo llego. Algo mas sencillo es imposible? La manera que yo utilizé deberia funcionar, no? No veo el error...

[Ronruby]

El ejemplo que te proporciono pateketrueke solo revisa si las variables están vacías etc ...

Pero, básicamente... para redireccionar lo que tienes que hacer es:
header("Location: $_SERVER[PHP_SELF]");

Nota, no puedes imprimir nada ante de enviar cabeceras al navegador

[acoevil]

Basicamente lo que te estan diciendo es que revices los valores que te llegan para insertarlos en tu base de datos

por ejemplo

$resultado=mysql_query("INSERT INTO formulario (ime, mail, mensaje)
VALUES ('$ime', '$mail', '$mensaje')");

Entonces colocas asi

if($resultado) // si los datos se insertan correctamente
{
header("Location: $_SERVER[PHP_SELF]");

}
else
{
...
}

Salu2

[miros84]

pero no se trata de insertar los datos corectamente. No me entendeis? Os dejo el enlace. http://www.ley-dominical.websphp.com/
Alguien deja una opinion, luego refresca con F5 y esa misma opninion se inserta otra vez. He dejado el codigo de como lo hago mas arriba. Abajo de todo he hecho un echo($ime); echo($mail); echo($mensaje);
y he visto que se me mantienen los valores de las varibales.

[GatorV]

Es por eso que como te decia pateketrueke, tienes que usar un header() para redireccionar al final para limpiar los datos que vienen por $_POST y así evitar la doble inserción.

Saludos.

[miros84]

Vale. Esta claro. Lo unico que no entiendo es como poner un header()
Se que tiene que ir al principio de todo para que no me de error, pero yo lo nececito poner despues del "Mysql query insert into"
Como lo hago?

[Ronruby]

Se que tiene que ir al principio de todo para que no me de error, pero yo lo nececito poner despues del "Mysql query insert into"

No al principio de TODO, sino antes de cualquier salida.

Código PHP:

  <?php
echo "Hola mundo";
header("Location: hola.html"); //da error
?>   

Código PHP:

  <?php
mysql_query(/* ... */);
header("Location: hola.html"); //no da error, porque no hemos impreso nada aun
?>   

[GatorV]

No, no tiene que ir al principio de todo, tiene que ir antes de cualquier salida de texto, por lo que perfectamente lo puedes poner después de un mysql_query, solo verifica que no haya ninguna salida de texto antes para que no te salga error.

Saludos.

[miros84]

Lo he puesto asi:

Código PHP:

  mysql_query("INSERT INTO form(ime, mail, mensaje)
VALUES ('$ime', '$mail', '$mensaje')"); 
}
$result = mysql_query("SELECT * FROM form");
if($result)
{header("Location:http://ley-dominical.websphp.com/");}
while($row = mysql_fetch_array($result))  
   

Y me da este error:
Cannot modify header information - headers already sent by (output started at /../../../ley-dominical/index.php:6) in /../../../ley-dominical/index.php on line 129

las primeras 10 lineas son:

Código PHP:

  <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>LEY DOMINICAL</title>
<style type="text/css">
<!--
.tr54 {
    text-align: center;
    color: #006;  
   

Es decir en linea 6 tengo <style type="text/css">
No entiendo este problema.

[GatorV]

No puedes enviar nada, nada de contenido de texto antes de poder enviar un header() en este caso todo tu HTML es texto, por eso tienes que mover el proceso a antes:

Código PHP:

  <?php
if( hay_datos_post() ) {
      // compruebas todos los datos
      insertar_bdd();
      if( insertar_exito() ) {
              redirigir();
      } 
}
// a partir de aqui todo tu HTML
?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">  

Saludos.

[miros84]

No consigo que me vaya