ASP.Net Core 5 MVC表单未选图片时加载wwwroot默认图片方案

• 逻辑触发条件缺失：如果表单完全没选上传文件，Pet1ChipImage会是null或者空集合，根本进不了foreach循环，写的else分支完全没有执行机会。
• 分支内逻辑错误：即使进入长度为0的else分支，加载了默认图片对象后，并没有将其转为字节数组赋值给petReg.Pet1ChipImage，反而还在对长度为0的上传对象执行流读取，最终拿到的是空字节。
• 用法不符合ASP.NET Core规范：MyServer.MapPath是传统.NET Framework的用法，Core下应该通过注入IWebHostEnvironment获取wwwroot路径；System.Drawing.Image.FromFile依赖Windows专属的GDI+，跨平台部署会直接报错，处理简单读文件场景完全没必要引入这个依赖。
• 多余参数：Action里的byte[] bytes参数不会被模型绑定自动赋值，属于无效参数。

第一步：注入路径服务 #

在控制器构造函数注入IWebHostEnvironment，用于获取wwwroot文件夹的物理路径：

private readonly IWebHostEnvironment _webHostEnv;

// 控制器构造函数
public PetController(IWebHostEnvironment webHostEnv)
{
    _webHostEnv = webHostEnv;
}

第二步：修正上传判断逻辑 #

不要在遍历上传文件的循环里做空判断，优先判断是否存在有效上传文件，不存在直接读取默认图赋值：

public async Task<IActionResult> Create(PetReg petReg, List<IFormFile> Pet1ChipImage)
{
    // 校验是否存在有效上传图片
    bool hasUploadImg = Pet1ChipImage != null 
        && Pet1ChipImage.Any(file => file != null && file.Length > 0);

    if (hasUploadImg)
    {
        // 取第一个有效上传文件转字节数组，多图场景可自行调整遍历逻辑
        var uploadFile = Pet1ChipImage.First(file => file.Length > 0);
        using var ms = new MemoryStream();
        await uploadFile.CopyToAsync(ms);
        petReg.Pet1ChipImage = ms.ToArray();
    }
    else
    {
        // 拼接默认图片的物理路径，和你wwwroot下的实际存放路径对应即可
        string defaultImgPath = Path.Combine(_webHostEnv.WebRootPath, "Images", "Default.png");
        // 直接读取默认图的字节数组赋值，无第三方依赖、跨平台兼容
        petReg.Pet1ChipImage = await File.ReadAllBytesAsync(defaultImgPath);
    }

    // 此处保留你原有的数据库保存、页面跳转等业务逻辑即可
    // 例如：_context.PetRegs.Add(petReg); await _context.SaveChangesAsync();
    return RedirectToAction(nameof(Index));
}

• 提前确认wwwroot/Images路径下存在Default.png文件，发布项目时要将该文件设置为"如果较新则复制"，避免部署后找不到文件抛出IO异常。
• 单图存储场景不需要遍历整个上传文件集合，直接取第一个有效文件即可，减少不必要的性能开销。
• 不要用System.Drawing.Common类库处理这类简单的文件读取场景，该类库非Windows环境需要额外安装系统依赖，且存在内存泄漏风险，直接读文件字节是最轻量可靠的方案。

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