PHP类构造方法强制校验文件扩展名为docx的实现问题求助

原有代码问题修正 #

你写的逻辑存在几个核心问题需要调整：

• 条件运算符错误：当前使用的&&（逻辑与）需要同时满足「扩展名非docx」和「文件不存在」才会触发拦截，不符合你「任意一个条件满足就报错」的需求，需要替换为||（逻辑或）
• 校验顺序错误：原有代码先给属性赋值再做校验，会导致无效路径被存入属性，建议先校验通过再赋值
• pathinfo调用优化：直接传入PATHINFO_EXTENSION参数可直接获取扩展名字符串，避免数组键不存在的报错，同时增加小写转换兼容大写后缀的场景（比如.DOCX）

修正后完整代码 #

class Abc {
    public string $path;

    public function __construct(string $path) {
        // 提取扩展名并转小写，兼容大小写后缀
        $extension = strtolower(pathinfo($path, PATHINFO_EXTENSION));
        // 只要扩展名不匹配或者文件不存在，直接抛出异常
        if ($extension !== 'docx' || !file_exists($path)) {
            throw new \InvalidArgumentException('参数错误：请传入存在的docx格式文件路径');
        }
        // 校验通过后再给属性赋值
        $this->path = $path;
    }
}

调用示例 #

构造方法抛出异常后不会生成实例对象，自然无法调用后续对象方法，调用时建议捕获异常处理错误：

try {
    $docxObj = new Abc('./test/file.docx');
    // 校验通过，后续可正常调用对象方法
} catch (\InvalidArgumentException $e) {
    // 处理错误逻辑，比如输出提示
    echo '文件校验失败：' . $e->getMessage();
}

可选优化 #

如果要避免把名为xxx.docx的文件夹误识别为有效文件，可以把判断条件里的!file_exists($path)替换为!is_file($path)，确保路径指向的是真实文件。

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