0

以下のコードを単純化して、switch ステートメントを 1 つだけ使用することはできますか? 3つ目のスイッチが必要なようですので、できれば1つだけにしたいです。

    $input_separator= $_REQUEST['input_separator'];

    switch ($input_separator) {  
        case "new_line":
            $input_separator="\n";
            break;
        case "comma":
            $input_separator=",";
            break;
        case "none":
            $input_separator="";
            break;
    }



    $output_separator= $_REQUEST['output_separator'];

    switch ($output_separator) {
        case "new_line":
            $output_separator="\n";
            break;
        case "comma":
            $output_separator=",";
            break;
        case "none":
            $output_separator="";
            break;
    }
4

2 に答える 2

2

ステートメントは必要ない ようです:switch

$input_separator = $_REQUEST['input_separator'] == "new_line" ? "\n" : "";
$output_separator = $_REQUEST['output_separator'] == "new_line" ? "\n" : "";

編集:これを試してください:

$separators = Array(
    "new_line"=>"\n",
    "comma"=>",",
    "none"=>""
);
$input_separator = $separators[$_REQUEST['input_separator']];
$output_separator = $separators[$_REQUEST['output_separator']];
于 2013-04-28T01:24:03.987 に答える
1

シンプルな機能を使ってみませんか？

function convert_seperator($seperator){
    $ret = '';
    switch ($seperator) {  
        case "new_line":
            $ret = "\n"; // or $ret = PHP_EOL;
            break;
        case "comma":
            $ret = ",";
            break;
        case "none":
            $ret = "";
            break;
        default:
            exit('Invalid seperator');
    }
    return $ret;
}
$input_separator = convert_seperator($_REQUEST['input_separator']);
$output_separator = convert_seperator($_REQUEST['output_separator']);
于 2013-04-28T01:37:26.147 に答える