c# - 覆盖现有的 Jpeg 文件/用编辑后的 ​​JPEG 文件替换现有的 JPEG 文件

Question

我已经构建了一个程序，允许我通过 System.Image.Drawing 插入图像的评论和标题，所以现在，我无法尝试用一个覆盖现有的 Jpeg 文件添加了评论和标题，但我在删除文件时遇到错误，所以我不确定该怎么做，因为我已经尝试处理该文件，但在那种情况下我无法保存它，因为我处理得太早了，但我无法保存它，因为现有的文件名没有被删除，所以我现在有点卡在中间。

这是我的代码:

public string EditComment(string OriginalFilepath, string newFilename)
{
       
    image = System.Drawing.Image.FromFile(OriginalFilepath);
    PropertyItem propItem = image.PropertyItems[0];
    using (var file = System.Drawing.Image.FromFile(OriginalFilepath))
    {
        propItem.Id = 0x9286;  // this is the id for 'UserComment'
        propItem.Type = 2;
        propItem.Value = System.Text.Encoding.UTF8.GetBytes("HelloWorld\0");
        propItem.Len = propItem.Value.Length;
        file.SetPropertyItem(propItem);
        PropertyItem propItem1 = file.PropertyItems[file.PropertyItems.Count() - 1];
        file.Dispose();
        image.Dispose();
        string filepath = Filepath;
        if (File.Exists(@"C:\Desktop\Metadata"))
        {
            System.IO.File.Delete(@"C:\Desktop\Metadata");
        }
        string newFilepath = filepath + newFilename;
        file.Save(newFilepath, ImageFormat.Jpeg);//error appears here
        return filepath;      
     }
}

显示的错误是:

An exception of type 'System.ArgumentException' occurred in System.Drawing.dll but was not handled in user code

Additional information: Parameter is not valid.

Answer 1

问题是从文件中打开图像会锁定文件。您可以通过将文件读入字节数组，从中创建内存流，然后从该流中打开图像来解决这个问题:

public string EditComment(string originalFilepath, string newFilename)
{
    Byte[] bytes = File.ReadAllBytes(originalFilepath);
    using (MemoryStream stream = new MemoryStream(bytes))
    using (Bitmap image = new Bitmap(stream))
    {
        PropertyItem propItem = image.PropertyItems[0];
        // Processing code
        propItem.Id = 0x9286;  // this is the id for 'UserComment'
        propItem.Type = 2;
        propItem.Value = System.Text.Encoding.UTF8.GetBytes("HelloWorld\0");
        propItem.Len = propItem.Value.Length;
        image.SetPropertyItem(propItem);
        // Not sure where your FilePath comes from but I'm just
        // putting it in the same folder with the new name.
        String newFilepath;
        if (newFilename == null)
            newFilepath = originalFilePath;
        else
            newFilepath = Path.Combine(Path.GetDirectory(originalFilepath), newFilename);
        image.Save(newFilepath, ImageFormat.Jpeg);
        return newFilepath;
    }
}

确保您不像在测试代码中那样将图像对象放置在using block 中。不仅 using block 完全存在，因此您不必手动处理，而且将不再存在于内存中的图像保存到磁盘也相当困难。同样，您似乎从文件中打开图像两次。我只是假设所有这些都是试图解决问题的实验，但一定要确保清理掉它们。

打开图像时要记住的基本规则是:

• 从文件创建的 Image 对象 will lock the file during the life cycle of the image object ，防止文件被覆盖或删除，直到图像被处理。
• 从流创建的 Image 对象 will need the stream to remain open for the entire life cycle of the image object .与文件不同，没有任何主动强制这样做，但在流关闭后，图像在保存、克隆或以其他方式操作时会出错。

与某些人的看法相反，对图像对象的基本 .Clone() 调用将不会改变此行为。克隆的对象仍将保留对原始源的引用。

---

请注意，如果您确实想要一个未包含在 using block 中的可用图像对象，you can use LockBits and Marshal.Copy to copy the byte data of the image object into a new image with the same dimensions and the same PixelFormat ，有效地制作了原始图像的完整数据克隆。 (注意:我认为这不适用于动画 GIF 文件)完成之后，您可以安全地处理原始文件并只使用新的干净克隆版本。

还有一些其他解决方法可以实际输出图像，但我见过的大多数都不是最佳方法。以下是锁定问题的两个最常见的其他有效解决方法:

• 使用 Bitmap(Image image) 构造函数从文件加载的图像创建一个新的 Bitmap。这个新对象将不具有指向该文件的链接，让您可以自由处置锁定文件的对象。这非常有效，但它会将图像的颜色深度更改为 32 位 ARGB，这可能不是您想要的。不过，如果您只需要在 UI 上显示图像，这是一个很好的解决方案。
• 如我的代码所示创建一个 MemoryStream，但不是在 using block 中，根据需要保持流打开。让流打开对我来说并不是一个好主意。尽管有些人说过，由于 MemoryStream 仅由一个简单的数组支持，而不是某些外部资源，垃圾收集器显然可以很好地处理这种情况...

我也看到有人使用 System.Drawing.ImageConverter 从字节转换，但是 I looked into the internals of that process , 它所做的实际上与这里的最后一个方法相同，它使内存流保持打开状态。