C# fuite de Mémoire dans le Bitmap

J'ai eu une fuite de mémoire dans mon application dans ces lignes.. Si je prends un coup d'oeil dans le gestionnaire de tâche, à chaque fois que ce processus est déclenché, la mémoire RAM est en hausse de +- 300 MO..

Bitmap bmp1 = new Bitmap(2480, 3508);
panel1.DrawToBitmap(bmp1, new Rectangle(0, 0, 2480, 3508));
pictureBox2.Image = bmp1;

Quelqu'un peut m'aider avec sa fuite? Si j'utilise:

bmp1.Dispose();

J'obtiens une exception dans le "Programme.cs" à cette ligne: Application.Run(new Form1());
Et après cela, l'application est arrêtée en cours d'exécution...

Écran de l'application:
C# fuite de Mémoire dans le Bitmap

Et l'exception est? ObjectDisposedExeption?
Offtopic Mais éviter de regarder dans le gestionnaire des tâches et découvrez l'analyseur de Performances à la place
Vous devez disposer de l'image une fois que vous avez fini avec elle. Est-il un point dans votre application lorsque vous n'avez plus l'affichage de l'image? Si oui, c'est là que vous devez vous en débarrasser.
Je pense qu'il est de répéter le processus et la vieille image n'est pas nettoyé (au moins pas tout de suite).
L'image est toujours dans l'écran dans la picturebox1
Aucune exception sont les suivants: paramètre non Valide.

InformationsquelleAutor Ferry Jongmans | 2013-02-19

bitmap c#image

9

Mise à jour: Vous n'avez pas une fuite de mémoire soi, vous avez juste à attendre pour le Garbage Collector pour libérer les ressources.

Si vous ne voulez faire le garbage collector collect bien, vous pouvez faire ceci:
```
System.GC.Collect();
System.GC.WaitForPendingFinalizers();
```
Pourquoi avez-vous besoin de disposer de l'image bitmap? Si votre PictureBox est de l'utiliser, alors vous avez besoin de l'image bitmap. Si vous êtes à changer beaucoup de choses, peut-être que vous devez remplacer l'ancienne image bitmap pour un nouveau et de disposer de l'ancien:
```
Bitmap bmp1 = new Bitmap(2480, 3508);
panel1.DrawToBitmap(bmp1, new Rectangle(0, 0, 2480, 3508));
Image img = pictureBox1.Image;
pictureBox1.Image = bmp1;
if (img != null) img.Dispose(); //the first time it'll be null
```
- C'est la même chose que l'appel à la Dispose qui est apparemment provoquer une exception.
- J'obtiens toujours le même résultat que: bmp1.Dispose(); et l'application s'est arrêté.
- n'est-t-il le même que l'invocation bmp1.Dispose(); directement?
- Il est, mais il vaut mieux utiliser le formulaire de using.
- Mmm lorsque j'utilise la fonction clone, j'obtiens une erreur encore une fois.. "Erreur 1 Impossible de convertir implicitement le type de l '"objet" à " Système.De dessin.Image'. Une conversion explicite existe (vous manque un plâtre?) C:\Users\Ferry\Documents\Visual Studio 2012\Projects\Dirk Beprijzing\Dirk Beprijzing\Form1.cs 73 37 Dirk Beprijzing"
- Désolé, mon mal, devrait être pictureBox2.Image = (Bitmap)bmp1.Clone();. Cela étant dit, la deuxième solution est probablement mieux que je ne pense pas que le premier en fait des réponses à votre question. J'ai édité la réponse pour faire plus de sens.
- Je suis encore à la découverte de l'augmentation de mon mémoire de travail de l'application, à chaque fois quand je déclencheur de ce morceau de code. Maintenant, j'ai: aide (var bmp1 = new Bitmap(2480, 3508)) { panel1.DrawToBitmap(bmp1, nouveau Rectangle(0, 0, 2480, 3508)); pictureBox2.Image = (Bitmap)bmp1.Clone(); bmp1.Dispose(); }
- Oui, le Clone() méthode ne va pas se débarrasser de votre vieille image, voir ma nouvelle mise à jour du code et essayer, laissez-moi savoir comment cela fonctionne. Remarque, vous n'utilisez pas un using bloc maintenant.
- Celui-ci: Bitmap bmp1 = new Bitmap(2480, 3508); panel1.DrawToBitmap(bmp1, nouveau Rectangle(0, 0, 2480, 3508)); Image img = pictureBox1.Image; pictureBox1.Image = bmp1; if (img != null) img.Dispose(); // la première fois, il va être nul, Est l'augmentation de la mémoire RAM ainsi
- Apparemment Bitmap and Image never close the underlying stream. You'll have an eternal lock on that file, until the GC cares about collecting the floating reference. Si vous êtes en s'appuyant sur la GC encore.
- Voir ma mise à jour sur le lancement le garbage collector en colère.
- C'est la solution, je vous remercie!
- Vous êtes les bienvenus. 🙂
- Je pense que u peut directement appeler juste: si(pictureBox1.Image !=null){ pictureBox1.Image.Dispose() ; } :au lieu de (les 3 dernières lignes) affectation de l'image dans une autre Image img référence et le disposer. @AdamKDean ce do u think? J'ai été réussi par l'intermédiaire de ce fait.
InformationsquelleAutor Adam K Dean
3

Je suppose que vous jetez seulement l'image que vous n'avez pas besoin de plus. Vous avez encore besoin de la bmp1 créé, il suffit de fixer le contenu de la pictureBox2.Image champ . Essayez quelque chose le long de ces lignes:
```
Bitmap bmp1 = new Bitmap(2480, 3508);
panel1.DrawToBitmap(bmp1, new Rectangle(0, 0, 2480, 3508));
Bitmap bmp2 = (Bitmap)pictureBox2.Image;
pictureBox2.Image = bmp1;
bmp2.Dispose();
```
_{Disclaimer: je ne suis pas expérimenté avec C#, donc j'ai peut-être mal...}
- Mmm, à cette ligne: Bitmap bmp2 = pictureBox2.De l'Image; il dit: "Erreur 1 Impossible de convertir implicitement le type de Système.De dessin.Image' de 'Système.De dessin.Bitmap'. Une conversion explicite existe (vous manque un plâtre?)"
- Ne fait pas de sens, en fait, vous avez besoin de l'image Bitmap en vie aussi longtemps que la picturebox est de l'afficher
- regardez à nouveau le code: c'est ce que je voulais faire. Il ne dispose que de l'image qui ne s'affiche pas plus... je n'ai pas le C#, mais comme ma réponse est fondamentalement la même que Adam K du Doyen accepté de répondre, je pense qu'il a du sens.
- J'ai mis à jour le code avec le cast explicite, mais un peu googler aurait résolu le problème pour vous - que le message d'erreur n'est pas une chose rare...
InformationsquelleAutor ppeterka

Vous devez utiliser externes gdi32.dll pour éviter bitmap fuites de mémoire

[System.Runtime.InteropServices.DllImport("gdi32.dll")] 
public static extern bool DeleteObject(IntPtr hObject);
...
//delete bitmap handle
DeleteObject((IntPtr)hBitmap);

InformationsquelleAutor user1722669

-1

    Bitmap copy_Screen()
    {
        try
        {
            var bmpScreenshot = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height, PixelFormat.Format32bppArgb);
            var gfxScreenshot = Graphics.FromImage(bmpScreenshot);
            try
            {
                gfxScreenshot.CopyFromScreen(Screen.PrimaryScreen.Bounds.X, Screen.PrimaryScreen.Bounds.Y, 0, 0, Screen.PrimaryScreen.Bounds.Size, CopyPixelOperation.SourceCopy);
            }
            catch (Exception) { }
            return bmpScreenshot;
        }
        catch (Exception) { }
        return new Bitmap(10, 10, PixelFormat.Format32bppArgb);
    }

    private void button5_Click(object sender, EventArgs e)
    {
        //Start Stop timer
        if (timer1.Enabled == false) { timer1.Enabled = true; } else { timer1.Enabled = false; }
    }

    private void timer1_Tick(object sender, EventArgs e)
    {
        //memory leak solve
        System.GC.Collect();
        System.GC.WaitForPendingFinalizers();

        Bitmap BM = copy_Screen();
        if (pictureBox1.Image != null)
        {
            pictureBox1.Image.Dispose();
        }
        pictureBox1.Image = BM;

    }

Pourquoi avez-vous post ce code? Ce problème permet-il de résoudre?

InformationsquelleAutor Ercan ILIK

Vous devez vous connecter pour publier un commentaire.