Es seguro este metodo de control de acceso?

[jrBarcelo]

Buenas a todos! Estoy haciendo un proyecto, y tengo una duda...
Necesito que alguna paginas sean privadas, eso lo consigo creando una sesion "nivel" que la creo cuando inician sesion los usuarios. Ahora lo que necesito es que si no estas registrado no veas las paginas privadas.

Y lo estoy haciendo de la siguiente manera, a ver que les parece:

Código PHP:

  function is_User() {
    if ($_SESSION['nivel'] <> '') {
        if ($_SESSION['nivel'] <> '1') {
            header("Location: http://xxxxx.es");
        }
    } else if ($_SESSION['nivel'] == '') {
        header("Location: http://xxxxx.es");
    }
}  
   

Despues esta funcion la llamo en cada archivo que necesito, de esta manera:

Código PHP:

  <?php
require 'include/config.php';//Conexion a la base de datos y SESSION START
require 'include/func.php';//FUNCIONES DE LOGIN,DE USER ETC


is_User();
require 'header.php';
?>   

Que les parece?Algun consejo?Muchas Gracias

[h2swider]

Esta bien lo que haces, solo le cambiaria algunos detalles al codigo.
Primero que nada, luego de los header, deberias poner un exit, ya que php continua la ejecución de la pagina y te puede llegar a generar algun problema. Luego validaria solo por si acaso, que las variables de session siempre existan. Con respecto al else if, yo no lo veo necesario, directamente tendria que entrar si la condicion inicial no se cumple

Código PHP:

function is_User() {
    if (isset($_SESSION['nivel']) && $_SESSION['nivel'] <> '') {
        if ($_SESSION['nivel'] <> '1') {
            header("Location: http://xxxxx.es");
            exit;
        }
    } else {
        header("Location: http://xxxxx.es");
        exit;
    }
}
 

[eits]

te iba a sugerir lo mismo que h2swider pero ya no tiene caso, ahora, otro punto no se que tan avanzado estés en php y si conoces o puedes trabajar con poo y que tanto de privacidad necesitas, si la privacidad no es compleja, con lo que te recomiendan bastará, pero si necesitas algo mas avanzado te recomiendo leer sobre las listas de control de acceso