[SOLUCIONADO] Problema con operadores lógicos

[MeDxEc]

Buenas tardes, tengo el siguiente problema:

Tengo una variable de sesión llamada nivel que contiene dos posibles valores traídos desde mysql, "admin" o "editor". Admin tiene permisos para editar o borrar un articulo mientras que "editor" solo puede editar el articulo. Tengo el siguiente código que a mi parecer debería funcionar:

Código PHP:

<?php
 
                    $nivel = $_SESSION['nivel'];
 
                    if (isset($nivel) === 'editor' or 'admin') {
                        echo '<small><a href="editar.php?id='.$id.'">Editar</a></small>';
                    };
 
                    if (isset($nivel) === 'admin') {
                        echo ' | <small><a href="borrar.php?id='.$id.'">Borrar</a></small>';
                    };
                    
                ?>
 

He intentado cambiar el "or" por "||", "and", "&&", "xOr" y nada, no funciona. ¿Alguna solución?

[stramin]

Debes inicializar las sesiones:

Código PHP:

session_start()
 

Para consultar por la igualdad de 2 variables debes poner la pregunta completa

Código PHP:

if (isset($nivel) and $nivel === 'editor' or $nivel === 'admin') {
 

Lo mismo para la segunda condición:

Código PHP:

if (isset($nivel) and $nivel === 'admin')
 

[Erick_MD9]

Hola

La funcion "isset()" te regresa un valor booleano true/false dependiendo si la variable esta definida y no es null.

Lo correcto deberia ser.

if ($nivel === 'editor' or $nivel==='admin') {
//codigo
}

Saludos

[pateketrueke]

Ya lo has dicho tú "a tu parecer", pero tú no eres la ley, la ley es lógica, no tú.

Para empezar isset() devuelve un booleano, por lo tanto tu comparación está mal

Ahora, observa esto:

Código PHP:

// MAL
if (isset($nivel) === 'editor' or 'admin')
 
// BIEN
if ($nivel === 'editor' || $nivel === 'admin')
 

¿Notas las diferencias?

He intentado cambiar el "or" por "||", "and", "&&", "xOr" y nada, no funciona. ¿Alguna solución?

Te sugiero leer un manual de programación general pues la precedencia de operadores es importante, además un manual de PHP básico para que tengas opiniones fundamentadas, no solo a tu parecer.

[stramin]

Tiene toda la razón Erick_MD9, corrijo la primera condicion tal como la planteó:

Código PHP:

if ($nivel === 'editor' or $nivel === 'admin') {
 

y aprovecho de cambiar la segunda:

Código PHP:

if ($nivel === 'admin')
 

[Alexis88]

También puede ser así:

Código PHP:

if (isset($nivel)){
    echo '<a href="editar.php?id='.$id.'"><small>Editar</small></a>';
 
    if ($nivel === 'admin')       
        echo '<a href="borrar.php?id='.$id.'"><small>Borrar</small></a>';
}
else
    echo "La variable no está definida";
 

Como solamente hay dos valores posibles, si fuera 'admin', también se mostraría el botón "borrar", sino, solamente el de "editar".

Saludos

[MeDxEc]

Iniciado por pateketrueke

Ya lo has dicho tú "a tu parecer", pero tú no eres la ley, la ley es lógica, no tú.

Para empezar isset() devuelve un booleano, por lo tanto tu comparación está mal

Ahora, observa esto:

Código PHP:

// MAL
if (isset($nivel) === 'editor' or 'admin')
 
// BIEN
if ($nivel === 'editor' || $nivel === 'admin')
 

¿Notas las diferencias?

Sí, la ley es lógica y si lees con lógica lo que estaba haciendo le verás el sentido a lo que estoy preguntando ;)

Te sugiero leer un manual de programación general pues la precedencia de operadores es importante, además un manual de PHP básico para que tengas opiniones fundamentadas, no solo a tu parecer.

Consulté el manual de PHP, por eso dije que había intentado con los operadores. Mira un ejemplo de lo que te planteo:

$variable = $_POST['variable'];

Esta puede contener dos valores, o "editor" o "admin"

LEELO CON LOGICA, (un poco de pseudocódigo)

If $variable = 'admin' then -> mostrar('usted tiene permisos de editor más los permisos especiales').

If $variable = 'editor' then -> mostrar('usted solo puede editar')

Ahora la comparación:

If $variable es = a 'admin' o 'editor' -> acción

¿Lo leíste con lógica? Bueno, entonces verás que no está mal. Mi error está en la mal utilización del isset, no en la lógica. Coloqué los demás operadores simplemente porque también los estaba usando en el momento, error de despiste.

----

Gracias a los demás que respondieron. El uso que le dí a isset() era para verificar que el campo tuviese contenido y en caso de que no pues mostrara error. Lo pensé pero no era la forma correcta de ejecutarlo así que pido disculpas por mi error de novato.

[pateketrueke]

Así es, tu explicación es lógica y se entendió desde el inicio, pero no había lógica en tu forma de hacerlo, pues no respeta la sintaxis misma de los operadores.

Es decir, para que lo siguiente sea posible:

If $variable es = a 'admin' o 'editor' -> acción

Debería ser así:

Código PHP:

if (in_array($variable, array('admin', 'editor')))
 

Eso es bastante claro a comparación de lo que hacías inicialmente, que sea lógico no significa que debas hacerlo así.

Es decir, tu "seudo lógica" está bien para explicar tu problema, pero no está bien para expresar con programación el mismo.

No es igual pensarlo que plasmarlo.

[razierplus2]

Ya lo has dicho tú "a tu parecer", pero tú no eres la ley, la ley es lógica, no tú.

Pateketrueke, eso creo que sobraba, recuerda, tu tuviste un inicio como todo el mundo.

El hombre ha cometido un error y por eso recurre al foro, es de "novato", pues si, pero recuerdo aquellos tiempos donde los arrays me daban dolor de cabeza y los foreach no los entendia bien.

No te lo tomes a mal, pero a la gente, ese tipo de respuestas puede sentarles mal, sobre todo si el no ha faltado a nadie.

No es igual pensarlo que plasmarlo.

Ahi tienes toda la razón

Con respecto al problema, los compañeros te han respondido, y ahi tienes los errores. Saludos.

[MeDxEc]

@pateketrueke Por eso mismo recurrí al foro, mi lógica estaba bien pero necesitaba ayuda en la forma de llevarla acabo :v

Gracias a todos los que respondieron, ya está funcionando perfectamente :)