CRUD系统编辑记录时文件上传功能报错问题求助

问题概述 #

开发简单CRUD系统时，新增记录的文件上传功能正常，但复用代码到编辑模块后，出现以下警告：

Warning: Undefined array key "file" in C:\xampp\htdocs\crud\edit.php on line 12
Warning: Trying to access array offset on value of type null in C:\xampp\htdocs\crud\edit.php on line 12

核心问题分析 #

1. 页面加载时提前读取$_FILES：未提交表单时，$_FILES数组中不存在"file"键，直接读取会触发未定义警告
2. 提交判断不匹配：编辑表单提交按钮name="update"，但代码中判断的是isset($_POST["submit"])，条件永远不成立
3. 逻辑结构错误：文件上传代码未包裹在正确的条件判断内，导致页面加载时就执行
4. 未处理无文件更新场景：用户不上传新文件时，未保留原文件记录
5. SQL注入风险：直接拼接用户输入到SQL语句中

修复后的完整edit.php代码 #

<?php
// 引入数据库连接文件
include_once("config.php");
include 'db.php';
$statusMsg = '';

// 仅在表单提交时处理更新逻辑
if(isset($_POST['update']))
{
    $id = $_POST['id'];
    $toolnr = $_POST['toolnr'];
    $status = $_POST['status'];
    $toolname = $_POST['toolname'];
    $serial = $_POST['serial'];
    $usedat = $_POST['usedat'];
    $owner = $_POST['owner'];
    $calibrated = $_POST['calibrated'];
    $nextcalibration = $_POST['nextcalibration'];
    $vendors = $_POST['vendors'];

    // 默认保留原文件名，用户上传新文件时才更新
    $fileName = $_POST['old_file'];
    $targetDir = "uploads/";

    // 检查是否有新文件上传
    if(isset($_FILES["file"]) && !empty($_FILES["file"]["name"])){
        $fileName = basename($_FILES["file"]["name"]);
        $targetFilePath = $targetDir . $fileName;
        $fileType = pathinfo($targetFilePath,PATHINFO_EXTENSION);
        
        // 允许的文件格式列表
        $allowTypes = array('jpg','png','jpeg','gif','pdf','doc','xlsx');
        if(in_array($fileType, $allowTypes)){
            // 上传文件到服务器
            if(move_uploaded_file($_FILES["file"]["tmp_name"], $targetFilePath)){
                $statusMsg = "文件上传成功";
            } else {
                $statusMsg = "文件上传失败，请重试";
            }
        } else {
            $statusMsg = "仅允许上传jpg、png、jpeg、gif、pdf、doc、xlsx格式的文件";
        }
    }

    // 使用预处理语句防止SQL注入
    $stmt = $mysqli->prepare("UPDATE tools SET toolnr=?, status=?, toolname=?, serial=?, usedat=?, owner=?, calibrated=?, nextcalibration=?, vendors=?, file_name=? WHERE id=?");
    $stmt->bind_param("ssssssssssi", $toolnr, $status, $toolname, $serial, $usedat, $owner, $calibrated, $nextcalibration, $vendors, $fileName, $id);
    
    if($stmt->execute()){
        header("Location: index.php");
        exit;
    } else {
        $statusMsg = "更新失败: " . $stmt->error;
    }
    $stmt->close();
}

// 获取要编辑的记录数据
$id = $_GET['id'];
$result = mysqli_query($mysqli, "SELECT * FROM tools WHERE id=$id");
$user_data = mysqli_fetch_array($result);

$toolnr = $user_data['toolnr'];
$status = $user_data['status'];
$toolname = $user_data['toolname'];
$serial = $user_data['serial'];
$usedat = $user_data['usedat'];
$owner = $user_data['owner'];
$calibrated = $user_data['calibrated'];
$nextcalibration = $user_data['nextcalibration'];
$vendors = $user_data['vendors'];
$old_file = $user_data['file_name'];
?>

<!-- 表单必须添加enctype属性才能支持文件上传 -->
<form method="post" action="" enctype="multipart/form-data">
    <table border="0">
        <tr>
            <td>工具编号</td>
            <td><input type="text" name="toolnr" value="<?php echo $toolnr; ?>"></td>
        </tr>
        <tr>
            <td>状态</td>
            <td><input type="text" name="status" value="<?php echo $status; ?>"></td>
        </tr>
        <tr>
            <td>工具名称</td>
            <td><input type="text" name="toolname" value="<?php echo $toolname; ?>"></td>
        </tr>
        <tr>
            <td>序列号</td>
            <td><input type="text" name="serial" value="<?php echo $serial; ?>"></td>
        </tr>
        <tr>
            <td>使用地点</td>
            <td><input type="text" name="usedat" value="<?php echo $usedat; ?>"></td>
        </tr>
        <tr>
            <td>负责人</td>
            <td><input type="text" name="owner" value="<?php echo $owner; ?>"></td>
        </tr>
        <tr>
            <td>校准日期</td>
            <td><input type="text" name="calibrated" value="<?php echo $calibrated; ?>"></td>
        </tr>
        <tr>
            <td>下次校准日期</td>
            <td><input type="text" name="nextcalibration" value="<?php echo $nextcalibration; ?>"></td>
        </tr>
        <tr>
            <td>供应商</td>
            <td><input type="text" name="vendors" value="<?php echo $vendors; ?>"></td>
        </tr>
        <tr>
            <td>当前文件</td>
            <td>
                <?php if($old_file): ?>
                    <a href="uploads/<?php echo $old_file; ?>"><?php echo $old_file; ?></a>
                <?php else: ?>
                    无关联文件
                <?php endif; ?>
                <input type="hidden" name="old_file" value="<?php echo $old_file; ?>">
            </td>
        </tr>
        <tr>
            <td>上传新文件</td>
            <td><input type="file" name="file"></td>
        </tr>
        <tr>
            <td><input type="hidden" name="id" value="<?php echo $_GET['id']; ?>"></td>
            <td><input type="submit" name="update" value="更新记录"></td>
        </tr>
    </table>
</form>

<?php if($statusMsg): ?>
    <p style="color: red;"><?php echo $statusMsg; ?></p>
<?php endif; ?>

关键修复说明 #

• 延迟文件处理逻辑：将文件上传代码移到update提交判断内，避免页面加载时读取不存在的$_FILES键
• 匹配提交按钮标识：将判断条件改为isset($_POST['update'])，与表单提交按钮的name属性一致
• 保留原文件记录：通过隐藏字段存储原文件名，用户不上传新文件时自动保留原有文件
• 添加表单必要属性：给表单添加enctype="multipart/form-data"，确保文件能正常上传
• 修复SQL注入风险：使用预处理语句替代直接拼接SQL，提升系统安全性
• 增加状态提示：添加文件上传和记录更新的状态反馈，便于调试和用户操作感知

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