Como validar funciones a los usuarios

[anacona16]

Hola foreros de ForosdelWeb mi pregunta es la siguiente:

La cuestion es que al validar los usuarios que ingresan a mi sitema utilizando variables de session y datos de una tabla en MySQL en la que tengo un campo que es rollusua que es el Roll del usuario que me devuelve Administrador o Usuario segun el caso.

Lo que hago es

Código PHP:

  if ($_SESSION['rollusua'] == "Administrador"){
$link = "<a href='administrarusuarios.php'">Administrar usuarios</a> | <a href='logout.php'">cerrar sesion</a>";
}else{
$link = "<a href='logout.php'">cerrar sesion</a>";
}
 
echo $link; 
   

Bueno y de esta manera es que valido las funciones o acciones que pueden realizar los usuarios que inicien session en mi sistema, ¿Esta bien esto? ¿Se puede hacer mejor, como?

En mi opinion es algo muy mal hecho pero es lo que se me ocurre.

[millan2525]

Código PHP:

if ($_SESSION['rollusua'] == "Administrador"){ 
$link = "<a href='administrarusuarios.php'>Administrar usuarios</a> | <a href='logout.php'>cerrar sesion</a>"; 
}else{ 
$link = "<a href='logout.php'>cerrar sesion</a>"; 
} 
 
echo $link;
 

No sé para que le pusiste comillas dobles por en medio, pues puedes no ponerselas, y de la otra forma creo que podría darte algún error(al no escaparlas).

No obstante también te digo que en la página "administrarusuarios.php" tendrás que poner para que si no eres "Administrador" no te ponga nada. Así:

Código PHP:

if ($_SESSION['rollusua'] == "Administrador"){ 
echo 'lo que quieres que ponga';
} else {
echo 'Esta página no es visible para ti, en 3 segundos serás redireccionado a la página principal';
}
 

[anacona16]

Ok, gracias por tu respuesta, lo de las comillas no hay problema, el error lo hice cuando postee el mensaje.

Ahora, ¿es seguro como estoy validando estas sessiones?, o ¿Hago un script (administrarusuarios.php) para el administrador y otro para usuarios (usuario.php)?