PHP数组未定义偏移量错误修复：如何正确显示USSD菜单？

错误根源分析 #

报错Undefined offset: 4的核心原因：

• $items数组共4个元素，索引范围是0-3，但toString中$bufferLimit被设置为getBufferLimit()+1，导致循环条件$i <= $bufferLimit允许$i取值到4，此时访问$items[4]触发数组越界。
• $lastMenu的判断逻辑错误：用$bufferLimit == count($items)（即4==4）判定为最后一页，但数组最大索引是3，导致循环会尝试访问不存在的元素。

代码修复步骤 #

1. 修正recurseMenu方法的循环与分页逻辑 #

修改循环边界、越界判断及分页序号逻辑，避免报错同时优化用户体验：

function recurseMenu($items,$bufferLimit) {
    $objectString="<strong>". $this->getTitle() . "</strong>" . PHP_EOL;
    $lastMenu=false;
    $totalItems = count($items); // 缓存总元素数
    var_dump($items);           //DEBUG "items"
    
    if($totalItems>0) {
        // 修正循环条件：确保i不超过数组最大索引，同时不超过bufferLimit
        for ($i = $this->index; $i <= $bufferLimit && $i < $totalItems; $i++) {
            // 用isset判断元素是否存在，避免越界报错
            if (isset($items[$i])) {
                $item = $items[$i];
                // 修正序号：显示当前页的本地序号（1、2...）而非全局序号
                $num = ($i - $this->index) + 1;
                // 获取节点数据
                $userSessions = $_SESSION['userSessions'];
                $currUserSession = $userSessions[$this->address];
                $node = $currUserSession->getNode($item);
                $title = $node->getTitle();
                $objectString .= PHP_EOL . $num . '. ' . $title;
            }
        }
    } else {
        $objectString=$objectString.PHP_EOL . 'NO DATA AVAILABLE, TRY AGAIN LATER';
    }
    
    // 修正最后一页判断：当前页已显示完所有剩余元素即为最后一页
    $lastMenu = ($this->index + ($bufferLimit - $this->index + 1) >= $totalItems);
    
    $objectString=$objectString . PHP_EOL . PHP_EOL . "<strong>0. Exit</strong>";
    if($this->getParent() != '0'){
        $objectString=$objectString . PHP_EOL . "<strong>#. Back</strong>";
    }
    
    if($lastMenu===false){
        // 修正剩余元素数计算：显示未显示的剩余元素数量
        $rem = $totalItems - ($bufferLimit + 1);
        $objectString=$objectString . PHP_EOL . "<strong>99. Next (".$rem.")</strong>";
    }
    return $objectString;
}

2. 修正toString中的bufferLimit边界处理 #

保留+1逻辑的同时，确保bufferLimit不超过数组最大索引，从源头避免越界：

// 原toString中的bufferLimit计算
// $bufferLimit = $this->getBufferLimit() + 1;
// 修改为：
$totalItems = count($items);
$bufferLimit = min($this->getBufferLimit() + 1, $totalItems - 1);

这样既保留原有分页逻辑，又避免了数组越界，同时解决按下99键的延迟问题（不再循环访问不存在的元素，减少无效执行步骤）。

3. 可选：优化getNode方法容错性 #

为避免节点名称错误导致崩溃，在UssdTree.php中增加存在性判断：

function getNode($name){
    // 检查节点是否存在，避免Undefined index报错
    if(isset($this->treeMenu[$name])){
        return $this->treeMenu[$name];
    }
    // 节点不存在时返回默认值或抛出异常，根据业务需求调整
    return null;
}

关键修改点说明 #

• 循环条件增加$i < $totalItems，彻底避免数组越界访问
• 用isset($items[$i])替代直接判断元素值，消除空元素或越界引发的警告
• 修正菜单序号为当前页本地序号，提升分页后的用户体验
• 调整lastMenu判断逻辑，确保分页状态判断准确
• 修正剩余元素数计算，显示真实未加载的元素数量
• 对bufferLimit做边界处理，从调用层规避越界风险

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