PHP语法错误求助：Parse error: unexpected end of file

一、报错原因：未闭合的foreach循环 #

你遇到的Parse error: Syntax error, unexpected end of file是因为PHP的替代语法foreach未闭合。

代码中使用了foreach ($productCodes as $productCode):这种带冒号的替代语法，但没有添加对应的endforeach;来结束循环。PHP解析器会一直寻找闭合标记，直到文件末尾都没找到，因此抛出"意外的文件结束"错误。

修复方法：在</option>之后、</select>之前添加<?php endforeach; ?>，完整的下拉框代码如下：

<select name="productCode">
    <?php foreach ($productCodes as $productCode): ?>
    <option value="<?php echo htmlspecialchars($productCode['productCode']); ?>">
        <?php echo htmlspecialchars($productCode['name']); ?>
    </option>
    <?php endforeach; ?>
</select>

二、通用代码优化建议（针对PHP入门8周的学习阶段） #

1. 完善数据验证逻辑 #

• 邮箱验证：目前只判断了非空，建议用filter_input(INPUT_POST, 'email', FILTER_VALIDATE_EMAIL)验证邮箱格式，避免无效格式的邮箱提交。
• 查询结果判断：$customer = $statement->fetch()如果没有匹配到数据，会返回false，此时直接访问$customer['email']会触发报错。应该先判断查询结果是否有效：

  $customer = $statement->fetch();
  $statement->closeCursor();
  
  if (!$customer) {
      $error = 'No matching email. Please enter a valid email.';
      require_once('index.php');
      exit();
  }
  

2. 增强数据库操作的健壮性 #

• 开启PDO错误模式：在数据库连接后添加$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);，这样数据库操作出错时会抛出异常，方便你调试问题。
• 复用数据库操作逻辑：可以把prepare、execute、closeCursor这些重复步骤封装成函数，减少代码冗余。

3. 代码可读性与规范 #

• 统一缩进：保持代码缩进一致（比如用4个空格），让循环、条件语句的层级更清晰。
• 添加注释：对关键逻辑（比如查询用户、获取产品列表）添加简短注释，方便后续维护。
• 避免冗余代码：比如<p></p>是空标签，可以直接删除。

4. 安全细节强化 #

• 你已经用htmlspecialchars()转义输出内容，这点做得很好，继续保持——这能有效防止XSS攻击。
• 隐藏字段的安全：customerID隐藏字段可以用htmlspecialchars()转义，虽然是内部值，但养成转义所有输出的习惯更安全：

  <input type="hidden" name="customerID" value="<?php echo htmlspecialchars($customer['customerID']); ?>">
  

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