2
if(!isset($_SESSION["TIP"])){
    header("location:../index.php?msg=1");
}else if(isset($_SESSION["TIP"])){
    if($_SESSION["TIP"]!="Normal"){
        header("location:../index.php?msg=3");  
    }else if ($_SESSION["TIP"]!="Gold") {
        header("location:../index.php?msg=3");                  
    }
}

Tengo este código para validar si el usuario que entra al programa tiene los permisos correspondientes, el problema es que mi variable $_SESSION["TIP"] si es igual a "Normal" pero me la toma como si no lo fuera.

Esto solo pasa cuando pongo el else if o unos || en la condición, alguien sabe donde esta mi error?

2
  • 1
    Trata de mejorar la forma en la que implementas las condiciones ya que se puede hacer mucho más simple. Primero haces un if para negar y luego un else para afirmar. Otra cosa también es el hecho de que cuando veríficas si tu variable de sesión es distinto a "normal" se puede dar el caso de que normal != gold y ni siquiera tomaría el siguiente else if. Commented el 13 dic. 2018 a las 6:16
  • Hola, podrías mejorar la edición para aclarar mejor la duda, cuando dices que la condición es si es igual a "Normal" pero veo que estás haciendo lo contrario; es decir si no es igual a "Normal".
    – Edgard
    Commented el 13 dic. 2018 a las 15:09

4 respuestas 4

1

bueno al punto, dices que $_SESSION["TIP"] si es igual a "Normal". Pero nunca le dices que haría si así fuera. Tu código solo tiene comparaciones en caso de que sea distinto a "normal" y a "gold".

0

En ningún momento defines la acción que tomaría en caso de que $_SESSION["TIP"] == "Normal".

Una solución sería definirlo de la siguiente forma:

if(!isset($_SESSION["TIP"])){
    // Variable de sesión no establecida

} else {
    if( $_SESSION["TIP"] == "Normal" ){
        // Establecida, es Normal

    } else if ( $_SESSION["TIP"] == "Gold") {
        // Establecida, es Gold 

    } else {
        // Establecida, pero no es ninguna de las anteriores

    }
}

Espero que esto sea lo que buscas!!

3
  • 1
    Gracias por responder, ese el es punto, si $_SESSION["TIP"]=="Normal" no quiero que haga nada, que se quede así como está. Yo solo quiero que se desencadene cuando no sea igual a "Normal" Commented el 13 dic. 2018 a las 6:26
  • Entonces creo que tu error puede estar en que usas en el segundo condicional else, else if en vez de elseif: php.net/manual/es/control-structures.elseif.php
    – BladeMaker
    Commented el 13 dic. 2018 a las 6:37
  • 1
    La solución era más fácil, la condición del if que está dentro del if(isset($_SESSION["TIP"])) tenía que ser: $_SESSION["TIP"]!="Normal" && $_SESSION["TIP"]!="Gold" Tenía que ser con && no con || Commented el 13 dic. 2018 a las 13:45
0

La solución era más fácil de lo que parecía:

if(!isset($_SESSION["TIP"])){
    header("location:../index.php?msg=1");
}else if(isset($_SESSION["TIP"])){
    if($_SESSION["TIP"]!="Normal" && $_SESSION["TIP"]!="Gold"){
        header("location:../index.php?msg=3");
    }               
}

Solo tuve que cambiar los || por &&

2
  • y por que cambiarlos, antes era alguna de las dos opciones y ahora pides que ambas condiciones se cumplan?
    – user75901
    Commented el 13 dic. 2018 a las 13:51
  • había puesto al inicio que pasaba con los if o si ponía un || en la condición Commented el 13 dic. 2018 a las 14:00
0

es ta simple como:

$session = $_SESSION["TIP"] ?? false;
if(!$session){
   header("location:../index.php?msg=3"); 
}

if ($session != "Gold" || $session != "Normal" ) {
   header("location:../index.php?msg=3");                  
}



// continuamos ejecucion normal  porque es gold o normal
2
  • Por favor agrega una descripción a tu respuesta. Commented el 13 dic. 2018 a las 15:38
  • ¿Y eso que hace? Commented el 13 dic. 2018 a las 18:36

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.