嘿，我来帮你搞定这个多维数组的未定义索引问题！你的代码里主要有两个核心问题：一是直接访问嵌套数组的title等索引时没做存在性检查，二是三元表达式的赋值写法冗余且容易出错。咱们一步步来优化：

核心问题分析 #

1. 未检查title索引：直接用$iva_all_dpt_info[$weekdays[$j]][$k]['title']当数组键，如果这个title不存在，就会抛出未定义索引错误。
2. 冗余的三元赋值：你的三元表达式里重复写了$iva_departments[...][] = ...，这完全没必要——我们可以先安全获取值，再添加到数组里。
3. 重复嵌套访问：多次写超长的数组路径不仅难看，还容易写错，也不利于维护。

优化后的完整代码 #

for ($k = 0; $k < count($iva_all_dpt_info[$weekdays[$j]]); $k++) {
    // 用null合并运算符获取当前项，避免嵌套访问报错
    $currentItem = $iva_all_dpt_info[$weekdays[$j]][$k] ?? [];
    
    // 先检查title是否存在且有效，无效直接跳过当前循环
    $title = $currentItem['title'] ?? null;
    if (!$title) {
        continue;
    }
    
    // 确保目标数组结构已初始化，避免后续赋值时出现未定义索引
    if (!isset($iva_departments[$title])) {
        $iva_departments[$title] = [
            'doctors_id' => [],
            'doctors' => [],
            'department' => [],
            'specialty' => [],
            'hours' => []
        ];
    }
    
    // 安全获取各字段值，空值用''替代
    $iva_departments[$title]['doctors_id'][] = $currentItem['doctor_id'] ?? '';
    $iva_departments[$title]['doctors'][] = $currentItem['doctors'] ?? '';
    $iva_departments[$title]['department'][] = $currentItem['department'] ?? '';
    $iva_departments[$title]['specialty'][] = $currentItem['specialty'] ?? '';
    
    // 处理时间格式，同时检查starttime和endtime是否存在
    $startTime = $currentItem['starttime'] ?? '';
    $endTime = $currentItem['endtime'] ?? '';
    if (in_array($startTime, $hours)) { // 把in_array检查移到这里更合理
        if ($iva_timeformat == 12 && $startTime && $endTime) {
            $iva_departments[$title]['hours'][] = date('h.i a', strtotime($startTime)) . ' - ' . date('h.i a', strtotime($endTime));
        } else {
            $iva_departments[$title]['hours'][] = $startTime . ' - ' . $endTime;
        }
    }
}

关键优化点说明 #

• 提取当前元素：把$iva_all_dpt_info[$weekdays[$j]][$k]赋值给$currentItem，并用?? []确保即使原索引不存在也不会报错，后续访问字段更简洁。
• 检查title有效性：先确认title存在且不为空，否则直接跳过当前循环——没有有效的title，后续的数组分组也没有意义。
• 初始化目标数组结构：提前创建$iva_departments[$title]的数组结构，避免往未定义的数组键里添加元素时抛出错误。
• 用null合并运算符简化空值处理：$currentItem['doctor_id'] ?? ''等价于isset($currentItem['doctor_id']) ? $currentItem['doctor_id'] : ''，写法更简洁，是PHP7+推荐的语法。
• 调整in_array位置：把in_array检查移到获取startTime之后，逻辑更清晰，也避免了重复访问嵌套索引。

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