PHP OOP文件上传__construct构造函数失效问题求助

让我们一步步拆解你遇到的__construct失效问题：

核心错误点 #

1. 构造函数参数不匹配 #

你在实例化Imageupload类时传入的是一个数组$args：

$image = new Imageupload($args);

但你的构造函数定义的是接受单个字符串参数$string：

public function __construct($string){ ... }

这种参数类型不匹配会导致构造函数无法正确获取到你要处理的文件字段名，直接引发逻辑错误。

2. 单引号内的变量无法被解析 #

构造函数里的$_FILES['$string']存在语法问题：PHP中单引号包裹的字符串不会解析变量，这里的$string不会被替换成传入的参数值，而是被当作字面量'$string'去访问$_FILES，显然这个键不存在，导致所有属性赋值失败。

3. 不必要的$args['file_upload']错误赋值 #

你在fileupload.php里写了：

$args['file_upload'] = $_FILES['file_upload'][''] ?? NULL;

这里$_FILES['file_upload']['']是无效的写法，$_FILES的结构是$_FILES['字段名']['name/tmp_name/type/size']，空字符串的键不存在，这行代码完全没必要保留。

修正后的完整代码 #

fileupload.class.php #

调整构造函数，使其接受数组参数（和你传入的$args匹配），同时修正变量解析的问题：

class Imageupload {
    public $filename;
    public $temp_path;
    public $type;
    public $size;
    public $id;
    public $prod_name;

    public function __construct($args = []){
        // 先处理产品名称
        $this->prod_name = $args['prod_name'] ?? NULL;
        // 处理文件上传字段，这里固定字段名为'file_upload'，或者也可以从$args传入
        $file_field = 'file_upload';
        // 修正变量解析，直接使用变量而非单引号包裹
        $this->filename = $_FILES[$file_field]['name'][0]; 
        $this->temp_path = $_FILES[$file_field]['tmp_name'][0]; 
        $this->type = $_FILES[$file_field]['type'][0]; 
        $this->size = $_FILES[$file_field]['size'][0]; 
    } 

    public function create() { 
        // 完善文件移动逻辑
        if(move_uploaded_file($this->temp_path, '你的目标存储路径/' . $this->filename)){
            // 假设此处完成数据库插入并获取ID
            $this->id = 1; // 示例值，实际根据数据库操作调整
            return true;
        } else {
            return false;
        }
    }
}

fileupload.php #

简化$args的构建，去掉无效的file_upload赋值：

session_start(); // 记得开启Session，否则$_SESSION无法使用

function is_post_request(){
    return $_SERVER['REQUEST_METHOD'] === 'POST';
}

if(is_post_request()) { 
    $args = []; 
    $args['prod_name'] = $_POST['prod_name'] ?? NULL; 
    // 直接传入$args数组到构造函数
    $image = new Imageupload($args); 
    $result = $image->create(); 
    if($result === true) { 
        $new_id = $image->id; 
        $_SESSION['message'] = 'The image was uploaded.'; 
        // 可在此添加跳转逻辑，比如：header("Location: success.php"); exit;
    } else { 
        // 显示错误信息，比如：$_SESSION['error'] = 'Upload failed.';
    } 
} else { 
    $image = []; 
}

HTML表单 #

保持原表单结构，但必须添加enctype="multipart/form-data"属性（文件上传必填）：

<form method="post" enctype="multipart/form-data">
    <p><input name="file_upload[]" type="file" id="file_upload" value=""></p> 
    <p>Product name: <input type="text" name="prod_name" value="" /></p>
    <button type="submit">Upload</button>
</form>

额外优化建议 #

如果你希望构造函数更灵活，支持动态传入文件字段名，可以修改构造函数为：

public function __construct($args = []){
    $this->prod_name = $args['prod_name'] ?? NULL;
    // 从$args获取文件字段名，默认值为'file_upload'
    $file_field = $args['file_field'] ?? 'file_upload';
    $this->filename = $_FILES[$file_field]['name'][0]; 
    // ... 其他属性赋值
}

实例化时传入字段名即可：

$args['file_field'] = 'file_upload';
$image = new Imageupload($args);

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