Корпуса выключателей общего назначения

Question

У меня есть переключатель со многими корпусами. Все они работают по одной схеме: case -> с тем же именем и поместите результат в результат var.

switch ($_POST["field"]) { 
    case "keyword1": $result = keyword1(); break; 
    case "keyword2": $result = keyword2(); break; 
    case "keyword3": $result = keyword3(); break; 
} 

Можно ли это сделать с помощью общего кода или я должен сделать это - как показано выше - для каждого случая вручную?

Answer 1

Если имена так же, как это, а также имена функций вы могли бы сделать что-то вроде этого:

$result = $_POST["field"](); 

И проверить, если функция существует или отозваны вы можете использовать: is_callable() и function_exists().

Кроме того, чтобы пользователь не мог вызывать каждую функцию, вы могли бы просто создать белый список, в который будут включены имена функций, которые пользователь может вызвать.

Answer 2

Вы можете использовать имя переменной, но убедитесь, что вы проверяете формат значения $ _POST перед его использованием, иначе вы создаете утечку безопасности.

$functionName = $_POST['field']; 

if(!preg_match("/^keyword\d+$/", $functionName) { 
    die('Invalid value for POSTDATA entry "field"'); 
} 

$functionName(); 

Answer 3

call_user_func() было бы намного приятнее. Проверьте это в руководстве: http://php.net/manual/en/function.call-user-func.php

Но все-таки вы должны фильтровать входящие значения или Ваш код будет очень небезопасно ... Например, Вы можете предоставить множество допустимых функций и проверить, если данный $_POST['field'] значение в этом массиве.

Answer 4

Вы можете использовать call_user_func с белым списком имен функций:

$funcs = ['func1','funct2'...]; 
if(in_array($_POST['key'], $funcs) 
    $result = call_user_func($_POST['key']); 

Хотя я склонен думать, дело переключателя более читаемый