1. PHP / Говнокод #18204

    +142

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    $act = isset($_REQUEST['act']) ? $_REQUEST['act'] : '';
	
	if ($act == 'details') {
		orders_details();
	} elseif ($act == 'details_xls') {
		orders_details_xls();
	} elseif ($act == 'set_status') {
		orders_set_status();
	// ... ещё строчек 30
	} else {
		orders_list();
	}

    Так мы реализуем роутер

    Запостил: Tairesh, 20 Мая 2015

    Комментарии (5) RSS

    • ava Fike 20.05.2015 12:58 # 0

      ето REST, вы не понимаети
      Ответить

    • ava Vindicar 20.05.2015 13:20 # 0

      Хмммм...
      handlers = array('details'=>'order_details',);
$handlers[( (isset($_REQUEST['act']) and array_key_exists($_REQUEST['act'], $handlers) ? $_REQUEST['act'] : 'list' )]();

      ...блин, так еще говнокоднее
      Ответить

    • ava Vasiliy 20.05.2015 13:28 # +1

      Если кода не много рефакторится в 15 минут.
      Делается класс в него все что в ифах в методы.
      В точке входа по параметрам определяем какой экземпляр класса нужен. Делаем экземпляр рефликсией получаем список аргументов метода все, что по именам совпадает с параметрами запроса передаем в метод.
      ???
      Профит.

      Если кода много тоже самое только ебатни много лень.
      Ответить

    • ava bormand 20.05.2015 13:31 # 0

      Вот как надо роутеры писать: http://govnokod.ru/16668
      Ответить

    Добавить комментарий