PHP中If/While循环内变量$i出现Undefined Index错误求助

嘿Sean，作为PHP新手碰到这种变量错误确实头疼，别慌，咱们一步步拆解问题，先搞定错误，再解答你提到的那些疑问。

为啥会出现$i相关的未定义错误？ #

首先得澄清下：你说的"Undefined Index"一般是指访问数组不存在的键，但你的代码里$i是做计数器用的，所以更可能是**Undefined Variable（变量未定义）**的错误。大概率是这两个原因：

• 语法错误导致脚本提前中断：你代码里$title的赋值写法完全错了，PHP解析到这儿直接报错，后面的$i++根本没机会执行，甚至可能让$i的定义逻辑被跳过；
• 变量类型问题：你把$i定义成了字符串"0"，后面做数值计算时可能出现隐性类型转换，导致判断逻辑出错，间接引发错误。

修复后的完整代码 #

我把你的代码改好了，标注了关键修改点，你直接用就行：

<?php 
$servername = "server"; 
$dbusername = "username"; 
$dbpassword = "password"; 
$dbname = "database"; 

// 初始化数据库连接
$con = mysqli_init(); 
if (!$con) { 
    die("mysqli_init failed"); 
} 

if (!mysqli_real_connect($con, $servername, $dbusername, $dbpassword, $dbname)) { 
    die("连接失败: " . mysqli_connect_error()); 
} 

// 查询商品数据
$query = "SELECT year, brand, model, class, stocknr, price, status, pic FROM stock WHERE class = 'car' ORDER BY brand ASC"; 
$result = mysqli_query($con, $query); 

// 提前定义计数器为整数（别用字符串，避免类型坑）
$i = 0; 

// 检查查询是否成功（这里改成查询错误提示，更精准）
if (!$result) { 
    die("查询错误: " . mysqli_error($con)); 
} 

// 先输出表格标签（你之前漏了，HTML结构会炸）
echo '<table>';

while ($row = mysqli_fetch_array($result)) { 
    // 修正$title的拼接语法，之前的括号引号全错了
    $title = $row['brand'] . " " . $row['model']; 
    $uppic = $row['pic']; 
    $price = $row['price']; 
    $stock = $row['stocknr']; 

    // 每4个商品开新行（$i从0开始，所以判断$i%4==0）
    if ($i % 4 == 0) { 
        echo '<tr>' . "\n"; 
    } 

    echo "<td>"; 
    // 修复HTML结构：<td>里不能嵌套<tr>！改用div排版
    echo '<div style="width:250px; text-align:center; padding:10px;">';
    echo '<a href="details.php?stocknr=' . $stock . '"><span class="fs13">' . $title . '</span></a><br>';
    echo '<a href="details.php?stocknr=' . $stock . '"><img src="upload/imagesize.php?w=220&h=250&img=' . $uppic . '.jpg" border="0" alt="' . $title . '"></a><br>';
    echo '<a href="details.php?stocknr=' . $stock . '"><span class="sapri">R' . $price . '</span></a>';
    echo '</div>';
    echo "</td>"; 

    // 每4个商品闭合当前行（$i+1是第N个商品，所以判断($i+1)%4==0）
    if (($i + 1) % 4 == 0) { 
        echo '</tr>' . "\n"; 
    } 

    $i++; 
} 

// 填充剩余空单元格，保证每行4个
while (($i) % 4 != 0) { 
    echo '<td><div style="width:250px; padding:10px;"></div></td>'; 
    if (($i + 1) % 4 == 0) { 
        echo '</tr>' . "\n"; 
    } 
    $i++; 
} 

echo '</table>'; 

// 记得关闭数据库连接，养成好习惯
mysqli_close($con);
?>

关键修改说明： #

1. 修复$title的语法错误：之前的括号+引号混用是致命错误，现在改成标准的字符串拼接；
2. 调整计数器逻辑：把$i设为整数0，修正换行判断条件，避免类型转换坑；
3. 修复HTML结构：<td>里嵌套<tr>是违反HTML规范的，改用<div>控制每个商品的布局；
4. 精准错误提示：查询失败时显示查询错误，而不是连接错误，方便排查问题；
5. 补充数据库连接关闭：避免资源泄漏。

解答你的其他疑问 #

1. 怎么设置全局变量？ #

你的代码里$i已经是全局变量了，因为它定义在函数外面。如果以后把代码封装到函数里，要在函数内部用全局$i的话，可以这么写：

function renderProducts($con) {
    global $i; // 声明使用全局的$i
    $i = 0;
    // 后续逻辑...
}

不过更推荐用函数参数或返回值传递变量，尽量少用全局变量，避免代码耦合。

2. isset()或empty()有用吗？ #

对于计数器$i来说，isset($i)可以用来检查变量是否定义，比如在使用前加个判断：

if (!isset($i)) {
    $i = 0;
}

能避免未定义错误，但最好的方式还是提前定义变量，别依赖isset()兜底。empty()的话，$i=0时它会返回true，不太适合判断计数器是否存在。

3. 用函数有帮助吗？ #

当然有！把商品渲染逻辑封装成函数，不仅代码更清晰，还能避免作用域问题。比如写个renderProductItem()渲染单个商品，再写个renderProductTable()处理整个表格，可读性和可维护性都会提升很多。

4. 可以用$i = count($row);吗？ #

count($row)返回的是当前行的字段数量（你的查询有8个字段，所以会返回8或16，因为mysqli_fetch_array()默认返回关联+索引数组），这对你的计数器逻辑没用。如果想获取总商品数，可以用mysqli_num_rows($result)：

$totalProducts = mysqli_num_rows($result);

但计数器$i还是得从0开始递增，用来跟踪当前渲染的是第几个商品。

应该能帮你搞定问题了，要是还有疑问随时问！

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