Image.Save(..) lève une exception GDI+ car le flux de mémoire est fermé.

Question

J'ai des données binaires que je veux sauvegarder comme une image. Lorsque j'essaie d'enregistrer l'image, une exception est levée si le flux de mémoire utilisé pour créer l'image a été fermé avant l'enregistrement. La raison pour laquelle je fais cela est que je crée des images de façon dynamique et qu'en tant que tel, je dois utiliser un flux de mémoire.

voici le code :

[TestMethod]
public void TestMethod1()
{
    // Grab the binary data.
    byte[] data = File.ReadAllBytes("Chick.jpg");

    // Read in the data but do not close, before using the stream.
    Stream originalBinaryDataStream = new MemoryStream(data);
    Bitmap image = new Bitmap(originalBinaryDataStream);
    image.Save(@"c:\test.jpg");
    originalBinaryDataStream.Dispose();

    // Now lets use a nice dispose, etc...
    Bitmap2 image2;
    using (Stream originalBinaryDataStream2 = new MemoryStream(data))
    {
        image2 = new Bitmap(originalBinaryDataStream2);
    }

    image2.Save(@"C:\temp\pewpew.jpg"); // This throws the GDI+ exception.
}

Quelqu'un a-t-il une idée de la façon dont je pourrais sauvegarder une image avec le flux fermé ? Je ne peux pas compter sur les développeurs pour se souvenir de fermer le flux après l'enregistrement de l'image. En fait, le développeur n'aurait AUCUNE idée que l'image a été générée en utilisant un flux de mémoire (parce que cela se produit dans un autre code, ailleurs).

Je suis vraiment confus :(

Answer 1

Comme il s'agit d'un MemoryStream, vous n'avez pas vraiment besoin de pour fermer le flux - rien de mal ne se passera si vous ne le faites pas, même si c'est une bonne pratique de se débarrasser de tout ce qui est jetable de toute façon. (Voir cette question pour en savoir plus à ce sujet).

Cependant, vous devrait de disposer du Bitmap - et cela fermera le flux pour vous. En fait, dès que vous donnez un flux au constructeur du Bitmap, il "possède" le flux et vous ne devez pas le fermer. Comme les docs pour ce constructeur dites :

Vous devez garder le flux ouvert pendant la vie du Bitmap.

Je n'ai pas trouvé de documentation promettant de fermer le flux lorsque vous disposez du bitmap, mais vous devriez être en mesure de le vérifier assez facilement.

Answer 2

Une erreur générique s'est produite dans GDI+. Peut également être causé par la fourniture chemin de sauvegarde incorrect ! Il m'a fallu une demi-journée pour le remarquer. Donc, assurez-vous que vous avez vérifié le chemin de sauvegarde aussi..,

Answer 3

Il est peut-être utile de mentionner que si le C:\Temp n'existe pas, il lancera également cette exception même si votre flux est toujours existant.

Answer 4

Copiez le Bitmap. Vous devez garder le flux ouvert pendant toute la durée de vie du bitmap.

Lors du dessin d'une image : System.Runtime.InteropServices.ExternalException : Une erreur générique s'est produite dans GDI

    public static Image ToImage(this byte[] bytes)
    {
        using (var stream = new MemoryStream(bytes))
        using (var image = Image.FromStream(stream, false, true))
        {
            return new Bitmap(image);
        }
    }

    [Test]
    public void ShouldCreateImageThatCanBeSavedWithoutOpenStream()
    {
        var imageBytes = File.ReadAllBytes("bitmap.bmp");

        var image = imageBytes.ToImage();

        image.Save("output.bmp");
    }

Answer 5

Ce que Jon veut dire, c'est qu'après :

using(Stream.....){
}// here the stream is closed so when you try to save - you get an exception

Il serait également utile que vous ne mettiez pas le code dans une image et que vous le copiez-colliez ici.

EDIT : Je suppose que je n'ai pas vraiment comprendre ce que Jon disait après tout :). - C'était assez proche, je pense.