PHP中Undefined variable: current错误修复方案正确性咨询

问题描述 #

原始PHP代码运行时在LineA处报错：Undefined variable: current abc.php at LineA，代码如下：

if (isset($helloWorld[$this->index])) {
    $current = $helloWorld[$this->index];
}

if (isset($current)) {
    if (!is_array($current)) {
        return $list;
    }
}
foreach ($current as $key => $values) {                     // LineA
    $attName = $helloChamp[$key];

    foreach ($values as $value) {
        $valName = $helloHow[$key][$value];
        $list[$attName][] = $valName;
    }
}

尝试将foreach循环放入if(isset($current))代码块中（LineZ处），修改后的代码如下，询问该方案是否能正确解决问题：

if (isset($helloWorld[$this->index])) {
    $current = $helloWorld[$this->index];
}

if (isset($current)) {
    if (!is_array($current)) {
        return $list;
    }
}
if (isset($current)) {                                           // Line Z
    foreach ($current as $key => $values) {                     // LineA
        $attName = $helloChamp[$key];

        foreach ($values as $value) {
            $valName = $helloHow[$key][$value];
            $list[$attName][] = $valName;
        }
    }
}

回答 #

这个方案可以正确解决LineA处的未定义变量问题。

原因说明 #

原始代码中，$current仅在$helloWorld[$this->index]存在时才会被定义。如果这个条件不满足，$current就未初始化，直接进入foreach循环就会触发未定义变量的错误。

修改后，foreach循环被包裹在if(isset($current))代码块内，只有当$current确实被定义时，才会执行循环逻辑，从根源上避免了未定义变量的情况。

代码优化建议 #

可以合并重复的isset($current)判断，让代码更简洁：

if (isset($helloWorld[$this->index])) {
    $current = $helloWorld[$this->index];
}

if (isset($current)) {
    if (!is_array($current)) {
        return $list;
    }
    // 直接将foreach放在这里，避免重复判断
    foreach ($current as $key => $values) {
        $attName = $helloChamp[$key];

        foreach ($values as $value) {
            $valName = $helloHow[$key][$value];
            $list[$attName][] = $valName;
        }
    }
}

这样既保证了逻辑正确性，又减少了冗余的判断步骤。

内容的提问来源于stack exchange，提问作者flash