PHP OO $_session

[Feligo]

Hola, empezé a ocupar las sesiones en PHP, y he seguido ejemplos, pero no me funciona.

Resulta que en una página creo todo un IF recibiendo información de un formulario y comparando con la BD para ver si coinciden, de ser así se almacenan los datos en la $_SESSION, así:

Código PHP:

<?php
session_start();
if($_POST['Entrar']){
    $ok = "todo ok";
    $usu = $_POST["txt_usuario"];
    $pass = $_POST["txt_contrasena"];
    $consultar = pg_query($con,"select * from usuario where nombre = '$usu' and clave = '$pass'");
    $rs = pg_fetch_array($consultar);
    if($usu == $rs["nombre"] && $pass == $rs["clave"]){
        //todo el proceso de guardado
    }else{
        $ok = "no todo ok";
    }
}
if($ok == "todo ok"){
    $HT = $_SESSION['HT'];
    $_SESSION['USUARIO'] = $usu;
    sleep(3);
    header('Location:home.php');
}
?>
 

En la otra página (home.php) quiero mostrar el nombre del usuario que se conectó que estaría guardado en la $_SESSION así:

Código PHP:

<?php
session_start;
if (isset ($_SESSION['USUARIO']) ){
    echo "bien";
    echo $_SESSION['USUARIO'];
}else{
    echo "error";
}
?>
 

Y resulta que no entra a este último IF y me tira error... ¿qué me falta?

[vgonga1986]

Concretamente te faltan dos paréntesis, en:
session_start();

Aprovecho para comentarte un par de cosas
1) Esto es un poco loco, no?:

Código PHP:

$ok = "todo ok";
 

No conoces las variables boolean? true y false

2) Esta línea te va a dar error si el usuario no está en la base de datos:

Código PHP:

if($usu == $rs["nombre"] && $pass == $rs["clave"]){
 

Ya que el fetch no devolverá ningún valor y al intentar hacer $rs['nombre'] te dirá que no existe el índice.

3) Esta línea no sé para qué está, se supone que estás creando ahora la sesión:

Código PHP:

$HT = $_SESSION['HT'];
 

Yo cambiaría algunas cosas:

Código PHP:

<?php
session_start();
if ($_POST['Entrar']) {
    $ok = true;
    $usu = $_POST["txt_usuario"];
    $pass = $_POST["txt_contrasena"];
    $consultar = pg_query($con, "select * from usuario where nombre = '$usu' and clave = '$pass'");
    if (pg_fetch_array($consultar)) { // Esta función devuelve false si no hay ningún registro en la consulta
        //todo el proceso de guardado
    }else{
        $ok = false;
    }
    if ($ok) { // $ok == true
        $HT = $_SESSION['HT']; // La dejo, pero no sé para qué sirve.
        $_SESSION['USUARIO'] = $usu;
        sleep(3);
        header('Location:home.php');
    }
}
?>
 

Un saludo.

[Feligo]

Gracias, eso me faltaba, los ().

Con relación al "switch" creo que es cosa de gustos, el mío cumple la función y el boolean también. Igual decidí un string porque después mando esa variable a otra página donde mostraría "todo ok"...

Y lo que tú dices que no va es porque no mostré todo el código completo :)

Gracias igual por tu ayuda.