problemas con los encabezados

dieguicho · #1 (**permalink**) 02/11/2005, 12:11

hola estimados... tengo esta funcion en un archivo que se llama global.php

Código PHP:

  function cargarVisita($IDforoActualizar){

global $connection;

$cookie_foro = $_COOKIE["foro_".$IDforoActualizar]; 

    if(!isset($cookie_foro)){

        $visitas = mysql_query("UPDATE foros set visitas = visitas+1 WHERE IDforo =$IDforoActualizar", $connection);

        setcookie("foro_$IDforoActualizar", time(), time()+86400);

    }

}

el cual incluyo desde la pagina respuestas de esta manera
require("lib/global.php");

y al final de respuestas.php hago esto
cargarVisita($IDforo);

$IDforo es la variable del foro en el que estoy
no me deja cargar la cookie!! me da el error de que los encabezados ya estan seteados.
alguien podria enseñarme como manejar esto? siempre me pasa cuando quiero hacer redirects o setear cookies
gracias!

jam1138 · #2 (**permalink**) 02/11/2005, 14:19

Cita:

no me deja cargar la cookie!! me da el error de que los encabezados ya estan seteados.

Pués eso... no puedes setear una cookie (ni iniciar sesión, ni utilizar header()) si antes tienes una salida al navegador.... estos 3 procesos tienes que hacerlo antes de eso. Simplemente organizate.

www.php.net/setcookie

dieguicho · #3 (**permalink**) 02/11/2005, 18:01

pero la salida la tengo dentro de un if!
por ej
if(!is_numeric($IDforo)){
header("Location: /error");
} else {
setcookie("IDusuario", $usuarioActual['IDusuario'], time()+86400);
}

ahi ya me da error, como deberia de organizarme para eso?

jam1138 · #4 (**permalink**) 03/11/2005, 00:34

Cita:

pero la salida la tengo dentro de un if!

Supongo entendiste por "salida" a lo "devuelto" por la función... no es así. Cuando dije salida al navegdor me refería a la entrega de código... algún echo(), espacio en blanco antes de iniciar PHP.. o que este emebebido dentro de HTML. Eso es lo que debes evitar.

Esto esta mal:

Código PHP:

   echo "<html><head></head><body>"; // entrega de código

header("Location: blablabla.php"); // setcookie() ó session_start()

echo "</body></html>";

Simplemente debes organizarte... Esto es correcto:

Código PHP:

  header("Location: blablabla.php"); // setcookie() ó session_start()
// antes de cualquier salida... si husas header("Locaton:..") es bueno poner un exit() inmediatamente después.
echo "<html><head></head><body>"; // entrega de código
echo "</body></html>";

Espero quede claro...

Cluster · #5 (**permalink**) 03/11/2005, 10:30

Cita:

Iniciado por jam1138

Supongo entendiste por "salida" a lo "devuelto" por la función... no es así. Cuando dije salida al navegdor me refería a la entrega de código... algún echo(), espacio en blanco antes de iniciar PHP.. o que este emebebido dentro de HTML. Eso es lo que debes evitar.

Esto esta mal:

Código PHP:

   echo "<html><head></head><body>"; // entrega de código

header("Location: blablabla.php"); // setcookie() ó session_start()

echo "</body></html>";

Simplemente debes organizarte... Esto es correcto:

Código PHP:

  header("Location: blablabla.php"); // setcookie() ó session_start()
// antes de cualquier salida... si husas header("Locaton:..") es bueno poner un exit() inmediatamente después.
echo "<html><head></head><body>"; // entrega de código
echo "</body></html>";

Espero quede claro...

Te faltó un

exit;

despues del header() ..

No es que necesariamente cada vez que se usa "header()" se tenga que terminar la ejecución del script en ese momento (lo que hace exit;) .. pero, en este caso estamos "redireccionado" ("Location") hacia otro script/página .. por ende, la ejecución del resto de -ese- escript en concreto ya no nos hace falta, además que nos puede interferir con el resto de código sub-siguiente si es que no empezamos a tomar en cuenta condicionales (if .. else ...) para que el flujo del código nos lleve al final sin pasar por nada que no nos interese antes.

Un saludo,

jam1138 · #6 (**permalink**) 03/11/2005, 19:30

Cita:

Iniciado por Cluster

Te faltó un

exit;

despues del header() ..

No lo puse en el código para ser "ilustrativo" (solo mostrar el ordenamiento del código) pero lo menciono entre comentarios

Cita:

Iniciado por jam1138

(...) si husas header("Locaton:..") es bueno poner un exit() inmediatamente después.

No hay aclaraciones demás.