Правильная обработка ajax методов?

Question

[typeof]

У меня есть приложение в котором много ajax обработчиков, и все эти обработчики посылают запросы на один файл - ajax.php.
Там запрос обрабатывается и стоит большой свитч по типу -

$_m = $_POST['method'];
switch ($_m){
  case "save": 
    // cose
     break;
  case "delete":
    // code
    break;
   // и так далее где-то ~40 кейсов  
}

Я хотел бы узнать, есть какие либо приемы или паттерны которые помогут избавится от такой конструкции, или сделать ее гибче и проще, можно конечно передавать метод специальному классу, и в классе создавать инстанс класса methodClass, но такой подход слишком громоздкий.

Answer 1

[Илья Плотников]

class AjaxController 
{
    publc function actionSave()
    {
        return .....;
    }
}

$controller = new AjaxController();
if (method_exists($controller, "action".ucfirst($_POST['method'])))
{
    return $controller->"action".ucfirst($_POST['method'])();
}

Если по душе подобные конструкции, открывайте yiiframework.ru и начинайте изучать. Там все так-же удобно сделано.

Comments

[Илья Плотников]

Только не советую делать функции без классов и префиксов. Не редки случаи, когда через url можно вызвать exec с любым содержимым, например rm...

[rie]

То есть по сути, ничего не изменится, и теперь вместо громадного свитча будет громадный класс на все руки?

[Илья Плотников]

@rie Изменится. С функциями класс а удобнее работать, чем с кейсами, классов может быть несколько (или у вас только ajax контроллер?). В базовом классе можно написать систему контроля прав доступа и потом в контроллерах просто ее наследовать. Код будет выглядеть опрятно. Если что-то не нравится, критикуйте, предлагайте свои методы. Я люблю критику.

Answer 2

[Сергей Протько]

Да, это называется маршрутизацией запросов. Если не хотите писать велосипеды, рекомендую взять какой микро фреймворк, например Silex (хотя бы посмотрите примеры) и не париться.