使用Image.FromFile加载图片触发OutOfMemoryException问题求助

首先，你碰到的System.OutOfMemoryException和图片无法打开的核心问题其实是异步下载还没完成，你就急着去读文件了——这是个很容易踩的异步坑！

你用了web.DownloadFileAsync()，这个方法是后台异步执行的，代码会直接跳到下一行继续跑。当你调用Image.FromFile()的时候，文件可能才下载了一半，内容根本不完整。GDI+（.NET处理图片的底层组件）遇到这种残缺的JPG文件，不会直接告诉你“文件坏了”，反而会抛出OutOfMemoryException——这是个历史遗留的错误提示问题，确实容易误导人。而你在资源管理器里打不开图片，也是因为文件本身没下载完，是个残缺的文件。

接下来给你两种解决思路，还有关于Stream的疑问解答：

一、先解决异步下载的时机问题（两种方案） #

方案1：改用同步下载（简单直接，适合小图片） #

如果你的图片不大，直接用同步的DownloadFile()就好，它会等文件完全下载完再继续执行后续代码，不会出现文件不完整的情况：

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    Uri imageUrl = new Uri("https://edge.bonnieplants.com/www/img/products/artichokes-400px-30.jpg");
    string fileName = Path.GetFileName(imageUrl.LocalPath);
    fileName = fileName.Replace("-400px-30", "");
    string filePath = Path.Combine(Application.StartupPath, fileName); // 用Path.Combine避免路径拼接出错

    // 同步下载，确保文件100%下载完成
    web.DownloadFile(imageUrl, filePath);

    // 现在安全加载图片，记得释放旧资源
    if (pictureBox1.Image != null)
    {
        pictureBox1.Image.Dispose();
    }
    pictureBox1.Image = Image.FromFile(filePath);
}

方案2：继续用异步，但监听下载完成事件（不卡UI，适合大图片） #

如果不想让UI因为下载卡住，那就得给WebClient加个下载完成的事件监听，等下载真的结束了再加载图片：

public Form1()
{
    InitializeComponent();
    // ... 你的其他初始化代码 ...
    // 提前订阅下载完成事件
    web.DownloadFileCompleted += Web_DownloadFileCompleted;
}

private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    Uri imageUrl = new Uri("https://edge.bonnieplants.com/www/img/products/artichokes-400px-30.jpg");
    string fileName = Path.GetFileName(imageUrl.LocalPath);
    fileName = fileName.Replace("-400px-30", "");
    string filePath = Path.Combine(Application.StartupPath, fileName);

    // 异步下载，把文件路径作为参数传递给事件处理方法
    web.DownloadFileAsync(imageUrl, filePath, filePath);
}

private void Web_DownloadFileCompleted(object sender, AsyncCompletedEventArgs e)
{
    // 先检查有没有下载错误
    if (e.Error != null)
    {
        MessageBox.Show($"下载失败：{e.Error.Message}");
        return;
    }

    // 获取之前传递的文件路径
    string filePath = (string)e.UserState;
    // WinForms控件必须在UI线程更新，所以用Invoke切换线程
    pictureBox1.Invoke(new Action(() =>
    {
        if (pictureBox1.Image != null)
        {
            pictureBox1.Image.Dispose();
        }
        pictureBox1.Image = Image.FromFile(filePath);
    }));
}

二、关于是否需要用Stream的问题 #

你现在的问题根源不是Stream，但Stream确实是个更好的实践——Image.FromFile()会一直锁定文件，直到你Dispose掉这个Image，如果你后续想删除或者修改这个文件会报错。用Stream的话就不会有这个问题，还能更灵活处理图片内容：

比如直接从网络流加载（不用存本地文件）：

// 同步从网络流加载
using (var stream = web.OpenRead(imageUrl))
{
    if (pictureBox1.Image != null)
    {
        pictureBox1.Image.Dispose();
    }
    pictureBox1.Image = Image.FromStream(stream);
}

或者下载完文件后用Stream读取：

using (var stream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
{
    if (pictureBox1.Image != null)
    {
        pictureBox1.Image.Dispose();
    }
    pictureBox1.Image = Image.FromStream(stream);
}

这种方式不仅避免了文件锁定，还能减少磁盘IO（如果不需要保存图片到本地的话）。

额外提醒 #

• 永远用Path.Combine()拼接路径，手动加@"\"很容易因为路径末尾有没有斜杠、空格等问题出错。
• 每次切换图片时记得Dispose旧的Image，不然会积累内存泄漏。
• 记得处理网络异常，比如断网、404等情况，避免程序崩溃。

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