4

我很好奇执行下面的代码的更好方法，我发现它重复并且想减少它，有什么建议吗？

我试图用可变变量做一些事情，但我没有让它发挥作用。

所以基本上我有一堆颜色名称$_GET[color-name-here']

目标是将颜色代码设置为新颜色代码。因此，使用 URL，我可以将颜色代码设置为so的red颜色代码greenred's value00FF00

// Get color and color replacement values from URL
// get_value_or is ran through this code...
// isset($_GET[$key]) && !empty($_GET[$key]) ? $_GET[$key] : $default;
$red = get_value_or('red', null);
$orange = get_value_or('orange', null);
$yellow = get_value_or('yellow', null);
$green = get_value_or('green', null);
$turquoise = get_value_or('turquise', null);
$blue = get_value_or('blue', null);
$purple = get_value_or('purple', null);
$pink = get_value_or('pink', null);
$white = get_value_or('white', null);


// Define Default Color Name and Hexcode values
$colorsArray = array(
    'red' => 'FF0000',
    'orange' => 'FF5000',
    'yellow' => 'FFF200',
    'green' => '00FF00',
    'turquoise' => '00F0C8',
    'blue' => '0064FF',
    'purple' => '9F00FF',
    'pink' => 'FF0082',
    'white' => 'FFFFFF'
);



// Iterate Color Array and Set New Color Values if they exist
foreach($colorsArray as $colorName => $colorCode){

    // Do something to set each color Name with a New color code, if that color name has a value set

}


// Right now I am doing it manually for each color name, all 9+ like this...

//Set Reds NEW color value
if(isset($red)){
    $colorsArray['red'] = $colorsArray[$red];
}

//Set oranges NEW color value
if(isset($orange)){
    $colorsArray['orange'] = $colorsArray[$orange];
}

//Set yellows NEW color value
if(isset($yellow)){
    $colorsArray['yellow'] = $colorsArray[$yellow];
}

那么任何想法如何用更少的代码设置所有颜色？

仅当该颜色在 URL 中使用 $_GET 变量设置了新值时，才应更新该颜色的代码

PS）我不确定这个问题的好标题，如果你有更好的标题，请随时更改，谢谢

4

2 回答 2

5

如果我是你，我会把作业放在循环中：

$colorsArray = array(
    'red' => 'FF0000',
    'orange' => 'FF5000',
    'yellow' => 'FFF200',
    'green' => '00FF00',
    'turquoise' => '00F0C8',
    'blue' => '0064FF',
    'purple' => '9F00FF',
    'pink' => 'FF0082',
    'white' => 'FFFFFF'
);
foreach ($colorsArray as $colorName => $colorCode) {
    $colorsArray[$colorName] = get_value_or($colorName, $colorCode);
}

它非常整洁，但我不确定它是否适用于您的真实代码。

编辑 我更新了代码，因为我意识到数组$color_names是不必要的，您将它们放在$colorsArray.

编辑 再次更新了代码，因为if循环中的 in 也是不必要的。

于 2013-08-02T01:31:24.320 回答
3

您可以使用 super global 访问所有全局变量$GLOBALS。所以你可以这样做：

foreach ($colorsArray as $colorName => $colorCode) {
    if (isset($GLOBALS[$colorName]) {
        $colorsArray[$colorName] = $GLOBALS[$colorName];
    }
}

有关 $GLOBALS 的更多信息。

于 2013-08-02T01:06:09.787 回答