代码优化,重构

今天把09年写的一个类拿来使用,浏览了一下,看着之前写的烂代码挺难读的,随手就优化了一下,直接看代码。

老代码:

$getStr = '';
if($_GET) {
    $getStr = '?';
    foreach ($_GET as $k => $v) {
        if($k != 'p' && $k != 'ps') {
            if(!in_array($k,$this->mIgnoreGet)) {
                $getStr .= $k . '=' . $v . '&';
            }
        }
    }
    if($getStr == '?') {
        $getStr = '';
    }
}

优化之后的代码:

parse_str($_SERVER['QUERY_STRING'], $query_array);
if(!empty($query_array)) {
    unset($query_array['p']);
    foreach (array_keys($this->mIgnoreGet) as $v) {
        unset[$v];
    }
}
$getStr = http_build_query($query_array);
$getStr = empty($getStr) ? '' : '?' . $getStr;

对于结构的优化就不写了。

这就是一次很小的重构。真的很小,用时很短。但是真实的改善了代码结构, 减少代码行数,提升代码的可读性。