我怎么能用大量的isset检查和重复的生成参数来减少PHP代码？

Question

问候。 我正在努力减少看起来相当冗长的代码段，让我不相信它的必要性。 它是一个生成用于填充表单的多个会话数组的函数，它让我验证参数数组中是否存在某些值，每个单独的值都可以生成所请求的数组。 开始：

function prepOptional($formData) {

    $baseInfo=getBaseInfo();

    $_SESSION['fooData'] = (isset($formData['cbFoo']) ? prepBaseForm($baseInfo, 'foo',
            'Option foo') : '');

    $_SESSION['opt1Data'] = (isset($formData['cbOpt1']) ? prepBaseForm($baseInfo, 'opt1',
            'Option 1') : '');

    $_SESSION['barData'] = (isset($formData['cbBar']) ? prepBaseForm(prepOtherArray($formData), 'bar',
            'Option bar', 'Optional argument') : '');

}

该函数接受formdata数组作为参数，并根据包含的值在会话中生成相应的数组; 只有在检查匹配标志的存在之后才会发生这种情况，因此是issets和三元运算符。

被调用函数prepBaseForm将第二个和第三个参数（对应于文件名和友好名称）的附加分隔到第一个参数，即数组; 它还接受一个可选的第四个参数，指示要连接到填充表单的文件。

如果数组中不存在该值，则代码会故意将相应的会话变量清除为''以保持生成顺序不变（为此：有更好的方法可以做到这一点，考虑到有些情况，其中给定的生成顺序会被打破吗？）。

实际的代码有大约二十个具有这种格式的生成块，并且它变得非常冗长; 我想到的一个优化是循环遍历给定的字段并概括生成过程，但正如您所看到的，很多时候参数不同，或者必须以另一种方式生成prepBaseForm函数的输入数组。 关于如何解决这种重构的任何想法？ 在此先感谢您的回复。

Answer 1

一种选择是提供默认值，如果使用正常条件更改值，则运行它。 然后你可以通过将所有内容放入一个简单的循环来进一步清理：

function prepOptional($formData) {
    $baseInfo=getBaseInfo();

    $options = array(
        'foo' => 'Option foo', 
        'opt1' => 'Option Opt1',
        'bar' => 'Option Bar',
    );
    foreach ($options as $name => $text) {
        //Add defaults for formData and Session arrays)
        $formData += array('cb' . ucfirst($name) => '');
        $_SESSION += array($name . 'Data' => '');

        if ($formData['cb' . ucfirst($name)]) {
            $_SESSION[$name . 'Data'] = prepBaseForm(
                $baseInfo, 
                $name, 
                $text
            );
        }
    }
}