Remplacer un switch php

[sff]

Bonjour,

Je me demande si il y a pas possibilité de remplacer un switch de ce genre :

Code: Tout sélectionner

if(isset($_GET['action'])){
   switch ($_GET['action']) {
   case 'add_sale': add_sale(); break;
   case 'add_sale_crystals': add_sale_crystals(); break;
   case 'buy': buy(); break;
   case 'edit': edit(); break;
   case 'delete': delete(); break;
   case 'finalize': finalize(); break;
      default : lists();
   }
}else{
   lists();
}

Par une ligne de code plus simple. La "case" est toujours la même que la fonction à insérer. Pouvez vous m'aider ?

Merci

[UsagiYojimbo]

Ca devrait être possible en effet : http://www.manuelphp.com/php/functions.variable-functions.php

Par contre je te conseille tout de même de garder ton switch, qui permet de vérifier que tu ne récupères pas n'importe quoi du $_GET. Sinon bonjour les injections...

[rudddy]

$action = $_get['action'];

$$action();



un truc dans ce genre devrait faire ton bonheur

[tabouet]

Bonsoir,

A ta place je garderai le switch, il a le mérite d'être "clair".

A bientôt,
Alexis

[Sullyvan]

$action = $_get['action'];

$$action();



un truc dans ce genre devrait faire ton bonheur

Code: Tout sélectionner

call_user_func($_GET['action'])

me semble un peu plus approprié.

Il reste avant tout à vérifier que $_get['action'] n'est pas n'importe quoi avec un in_array par exemple.

[DadouDuck]

Si c'est des fonctions "normales" :

Code: Tout sélectionner

if (function_exists($_GET['action'])) {
    call_user_func_array($_GET['action']);
}


Si c'est une méthode dans une classe

Code: Tout sélectionner

if (method_exists($_GET['action'])) {
    call_user_func_array($_GET['action']);
}