我来帮你把现有的图片上传+数据库记录逻辑封装到服务层，这样控制器就能更轻量化，逻辑也更易复用和测试。下面是具体的实现步骤：

1. 定义服务接口（面向接口编程） #

首先创建一个服务接口，明确我们需要的功能方法，这样后续可以轻松替换实现或者做单元测试：

public interface IImageService
{
    // 传入视图模型，返回新增记录的主键
    Task<int> UploadImageAndCreateRecordAsync(ImageUploadViewModel model);
}

这里的ImageUploadViewModel就是你已经在用的那个，包含图片文件和名称的模型，比如：

public class ImageUploadViewModel
{
    [Required]
    public string ImageName { get; set; }
    
    [Required]
    [DataType(DataType.Upload)]
    [AllowedExtensions(new string[] { ".jpg", ".png", ".jpeg" })]
    public IFormFile ImageFile { get; set; }
}

2. 实现服务类 #

接下来创建服务的具体实现类，注入你的DbContext和IWebHostEnvironment（用来获取wwwroot的路径）：

public class ImageService : IImageService
{
    private readonly AppDbContext _dbContext;
    private readonly IWebHostEnvironment _webHostEnvironment;

    // 通过构造函数注入依赖
    public ImageService(AppDbContext dbContext, IWebHostEnvironment webHostEnvironment)
    {
        _dbContext = dbContext;
        _webHostEnvironment = webHostEnvironment;
    }

    public async Task<int> UploadImageAndCreateRecordAsync(ImageUploadViewModel model)
    {
        // 1. 先向数据库添加记录（此时还没保存，主键未生成）
        var imageRecord = new Image
        {
            Name = model.ImageName,
            // 这里先留空，等保存后再设置文件名
            FileName = string.Empty
        };

        _dbContext.Images.Add(imageRecord);
        await _dbContext.SaveChangesAsync(); // 保存后，imageRecord.Id会被自动赋值

        // 2. 处理文件存储
        var uploadsFolderPath = Path.Combine(_webHostEnvironment.WebRootPath, "images");
        // 确保目录存在，避免报错
        if (!Directory.Exists(uploadsFolderPath))
        {
            Directory.CreateDirectory(uploadsFolderPath);
        }

        // 获取文件扩展名，保持原格式
        var fileExtension = Path.GetExtension(model.ImageFile.FileName).ToLower();
        // 用主键作为文件名
        var fileName = $"{imageRecord.Id}{fileExtension}";
        var filePath = Path.Combine(uploadsFolderPath, fileName);

        // 保存文件到wwwroot/images
        using (var stream = new FileStream(filePath, FileMode.Create))
        {
            await model.ImageFile.CopyToAsync(stream);
        }

        // 3. 更新数据库记录的文件名
        imageRecord.FileName = fileName;
        await _dbContext.SaveChangesAsync();

        return imageRecord.Id;
    }
}

注意：这里的Image是你的实体类，对应数据库的图片表，包含Id、Name、FileName这些核心字段。

3. 注册服务到依赖注入容器 #

在Startup.cs的ConfigureServices方法里，把我们的服务注册进去：

public void ConfigureServices(IServiceCollection services)
{
    // 注册DbContext（你应该已经有这行代码了）
    services.AddDbContext<AppDbContext>(options =>
        options.UseSqlServer(Configuration.GetConnectionString("DefaultConnection")));

    // 注册图片服务，用Scoped生命周期（和DbContext一致）
    services.AddScoped<IImageService, ImageService>();

    // 其他MVC相关注册...
    services.AddMvc();
}

4. 修改控制器，调用服务 #

现在你的控制器就可以简化了，只需要注入服务，处理模型验证，然后调用服务方法：

public class ImagesController : Controller
{
    private readonly IImageService _imageService;

    public ImagesController(IImageService imageService)
    {
        _imageService = imageService;
    }

    [HttpGet]
    public IActionResult Upload()
    {
        return View();
    }

    [HttpPost]
    public async Task<IActionResult> Upload(ImageUploadViewModel model)
    {
        if (!ModelState.IsValid)
        {
            return View(model);
        }

        try
        {
            var imageId = await _imageService.UploadImageAndCreateRecordAsync(model);
            return RedirectToAction("Details", new { id = imageId });
        }
        catch (Exception ex)
        {
            ModelState.AddModelError(string.Empty, $"上传失败：{ex.Message}");
            return View(model);
        }
    }
}

额外提示 #

• 可以在服务里添加更多的验证逻辑，比如检查文件大小（比如限制在5MB以内），避免过大的文件上传。
• 如果需要支持更多图片格式，修改ImageUploadViewModel里的AllowedExtensions属性即可。
• 后续如果要更换存储方式（比如改成云存储），只需要重新实现IImageService接口，不需要修改控制器代码，这就是面向接口编程的优势。

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