Amélioration d'un script pour les sessions

Bonjour,

Tout d'abord, tu dis que les membres ont le champs rang a 1 et pourtant tu met une variable admin a 1, c'est ambigue.
Tu devrais plutot mettre la variable admin a 0 et faire le if avec un == plutot qu'un !=
Avec un != le jour ou tu rajoute un autre rang, (modérateur, VIP, etc...) et qu'ils auront un rang superieur a 1, il auront les droits admin puisque tu test la différence a 1 et donc 0 est vrai tels que 2, 3, etc sont vrai aussi.

Dans le code suivant, je test $_SESSION, si c'est vide alors je redirige vers index. En faisant sa, je n'ai plus besoins de else.
Si c'est faux et que $_SESSION n'est pas vide, alors le script continue et test l'existence de rang dans la variable. Si c'est vrai alors je test si dans la variable, c'est le rang admin (0) sinon je redirige.

<?php 
    $admin = 0; 
    if (empty($_SESSION)) {
        header('Location:../index.php');
        die();
    }
    if (array_key_exists('rang', $_SESSION)) {
        if ($_SESSION'rang'] == $admin) {
            echo 'Welcome';
        }else{
            header('Location:../index.php');
            die();
        }
    }
?>

Désolé pour les fautes mais le français et moi sa fait 4.

Voilà, j'ai fait une petite fonction (je suis fier de moi !)

function autorisation($id){
    $id = $_SESSION'rang'];
    $admin = 0;
    if(isset($admin) && $admin == $id){
        return true;
    }
    header('Location:../index.php');
    die();
}

Donc les pages ou je souhaite autoriser l'admin je mets
autorisation($_SESSION'rang']);

Si tu veux en version petite :

function autorisation(){
    if(array_key_exists('rang', $_SESSION))
        return $_SESSION'rang'] == 0 ? true : header('Location:../index.php');
    header('Location:../index.php');
    die();
}

Ici je test l'existance de rang dans $_SESSION, si c'est vrai alors je test si oui ou non $_SESSION'rang'] = 0, si c'est vrai, cela retourn true et sinon sa redirige
Si l'existance de rang dans $_SESSION est faux, il redirige avec le deuxieme header juste au dessus du die.
Pour information, a coté du return c'est du ternaire

$_SESSION'rang'] == 0 ? true : header('Location:../index.php');
//est exactement la même chose que
if($_SESSION'rang'] == 0)
    return true;
else
    return header('Location:../index.php');

Imaginons que tu veuille avertir que quelqu'un n'a pas les droits d'acces a ta page car il n'est pas admin, tu vas devoir modifier ton code pour mettre ce qui faut a l'interieur de la fonction.
Avec la fonction de Pewel
tu n'aurais qu'a mettre le code dans le else qui test l'autorisation. ( if(autorisation()) )

Gyuki a tout dit :D

Quelque chose qu'il faut savoir aussi pour essayer de faire le code le plus propre possible :
-une fonction/méthode/classe doit avoir une et une seule fonctionnalité.

Je vais commencer par la classe.
Imaginons que tu ait une classe "Voiture". Elle ne va pas gérer des "Scooter" ou encore moins des "Camions".
C'est pourquoi un nom comme "Véhicule" peut être ambigüe parce qu'on ne sait pas trop son comportement.

Idem pour les fonctions/méthodes. Il faut essayer de créer des fonctions courtes, qui ont une fonctionnalité.
Si tu dois avoir une fonction qui test une valeur et après faire un traitement suivant cette valeur, il vaut mieux passer par plusieurs méthodes.

Exemple :

$voitures = array("BMW", "Mercedes", "Peugeot", "Fiat");
function getProprietaire($voitures, $key) {
    if (array_key_exists($key , $voitures)) {
        //requête sql pour aller chercher autre chose liée à la voiture en base (le propriétaire par exemple)
        $result = mysql_query("SELECT .....");
        $row = mysql_fetch_row($result);

        if($row[0] == "Durand") {
           ...
        }
    }
    else {
        //résultat non trouvé, autre traitement...
    }
}

Tu vois, cette fonction elle :
-vérifie l'existance de la clé dans la tableau
-va taper dans la bdd pour aller prendre un résultat
-fait autre chose si le résultat n'est pas trouvé

Ca fait beaucoup trop de traitement pour une seule et même fonction.
C'est pourquoi c'est plus lisible et plus facilement maintenable de découper tout ça en plusieurs fonctions :
-une fonction getProprietaire qui va retourner le proprietaire
-une fonction searchKeyInArray qui va faire une recherche dans le tableau
-une fonction selectInDataBase qui va interroger la Bdd

Exemple :

$voitures = array("BMW", "Mercedes", "Peugeot", "Fiat");
function getProprietaire($voitures, $key) {
    if (searchKeyInArray($key , $voitures)) {
        $results = selectInDataBase($params);
    }
    else {
        //résultat non trouvé, autre traitement...
    }
}
function searchKeyInArray($key, $tab) {
    //on peut immaginer faire un traitement sur un tableau beaucoup plus complexe
    return array_key_exists($key , $voitures);
}
function selectInDataBase($params) {
    //requête sql pour aller chercher autre chose liée à la voiture en base (le propriétaire par exemple)
    $result = mysql_query("SELECT .....");
    $row = mysql_fetch_row($result);

    checkProprietaire($row[0]);
}
function checkProprietaire($name) {
    if($name == "Durand") {
        //do something
    }
    else {
        //do another thing
    }
}

Voilà, j'ai pris des exemples simples mais plus ton code est fourni, plus il sera maintenable avec ces petites astuces.