Arbeitsspeicher läuft voll durch Sceenshots erstellen

  • C#
  • .NET (FX) 4.5–4.8

Es gibt 7 Antworten in diesem Thema. Der letzte Beitrag () ist von King2k7.

    Arbeitsspeicher läuft voll durch Sceenshots erstellen

    Hallo,
    ich schreibe gerade ein Programm das regelmäßig ein Screenshot vom Desktop macht. Der Code läuft in einem Timer und wiederholt sich alle paar Minuten.
    Jetzt habe ich das Problem das mir der Arbeitsspeicher vollläuft, durch den Code mit dem die Screenshots gemacht werden.

    Weiß jemand wie ich dem Problem entgegen wirke bzw. Arbeitsspeicher wieder freigeben wenn der Code im Timer einmal durchgelaufen ist.

    C#-Quellcode

    1. ​ void t1_Tick(object sender, EventArgs e)
    2. {
    4. // Capture the screen
    5. Bitmap screenshot = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height);
    6. Graphics graphics = Graphics.FromImage(screenshot);
    7. graphics.CopyFromScreen(0, 0, 0, 0, screenshot.Size);
    9. //Farbcode auslesen
    10. Bitmap b = new Bitmap(screenshot);
    11. Color c = b.GetPixel(1333, 972);
    15. if (c.R.ToString() + c.G.ToString() + c.B.ToString() == "25079")
    16. {
    18. t1.Stop();
    20. Return();
    21. }
    25. }

    Edit: Habe jetzt etwas mit using herumprobiert. Damit ist die Arbeitsspeicherauslastung um einiges besser geworden. Aber nach wie vor steigt der verbrauch. Ich denke ich habe noch einen Fehler in der Verwendung.
    Gibt es verbesserungsvorschläge bezüglich des Codes um die Arbeitsspeicherauslastung zu minimieren?

    C-Quellcode

    1. ​ void t1_Tick(object sender, EventArgs e)
    2. {
    3. System.Threading.Thread.Sleep(1500);
    6. // Capture the screen
    7. Bitmap screenshot = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height);
    9. using (screenshot)
    10. {
    11. Graphics graphics = Graphics.FromImage(screenshot);
    13. using (graphics)
    14. {
    15. graphics.CopyFromScreen(0, 0, 0, 0, screenshot.Size);
    17. //Farbcode auslesen
    18. Bitmap b = new Bitmap(screenshot);
    19. Color c = b.GetPixel(1333, 972);
    21. if (c.R.ToString() + c.G.ToString() + c.B.ToString() == "25079")
    22. {
    24. t1.Stop();
    26. ReturnToMenu();
    27. }
    28. }
    31. }

    Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von „King2k7“ ()

    Unter der Annahme, dass sich dein PrimaryScreen nicht ändert, probier mal:

    VB.NET-Quellcode

    1. Private screenshot As Bitmap = New Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height)
    2. Private graphics As Graphics = Graphics.FromImage(screenshot)
    4. Private Sub t1_Tick(sender As Object, e As EventArgs) Handles t1.Tick
    5. graphics.CopyFromScreen(0, 0, 0, 0, screenshot.Size)
    6. Dim c As Color = screenshot.GetPixel(1333, 972)
    7. If $"{c.R}{c.G}{c.B}" = "25079" Then
    8. t1.Stop()
    9. ReturnToMenu()
    10. EndIf
    11. End Sub

    c# Telerik-konvertiert

    C#-Quellcode

    1. private Bitmap screenshot = new Bitmap(Screen.PrimaryScreen.Bounds.Width, Screen.PrimaryScreen.Bounds.Height);
    2. private Graphics graphics = Graphics.FromImage(screenshot);
    4. private void t1_Tick(object sender, EventArgs e)
    5. {
    6. graphics.CopyFromScreen(0, 0, 0, 0, screenshot.Size);
    7. Color c = screenshot.GetPixel(1333, 972);
    8. if ($"{c.R}{c.G}{c.B}" == "25079")
    9. {
    10. t1.Stop();
    11. ReturnToMenu();
    12. }
    13. }

    Du machst immer wieder zwei neue BitMaps, die auch noch jeweils immer die gleichen sind am Ende.
    Stattdessen mach eine BitMap und mal immer wieder in diese eine rein.
    Using ist somit hier nicht notwendig, da die Disposables sowieso zur gesamten Laufzeit bereit stehen sollen.

    Da du ja scheinbar nur einen bestimmten Pixel prüfst, kannst du außerdem den Screenshot, den machst noch deutlich verkleinern. Aber gut das ist weniger relevant für den Arbeitsspeicher.

    Dieser Beitrag wurde bereits 6 mal editiert, zuletzt von „Haudruferzappeltnoch“ ()

    was machst du denn mit den Bitmaps? Wenn du die irgendwie in eine list speicherst etc. ist klar das irgendwann der Arbeitsspeicher voll ist.
    Meine Projekte Genesis Game Engine | GFX | smartli.me - Der smarte URL shortener

    King2k7 schrieb:

    C#-Quellcode

    1. Bitmap b = new Bitmap(screenshot);
    Diese Instanz musst Du noch zerstören.
    Und

    C#-Quellcode

    1. if (c.R.ToString() + c.G.ToString() + c.B.ToString() == "25079")
    da machst Du Color.ToArgb() draus.
    Falls Du diesen Code kopierst, achte auf die C&P-Bremse.
    Jede einzelne Zeile Deines Programms, die Du nicht explizit getestet hast, ist falsch :!:
    Ein guter .NET-Snippetkonverter (der ist verfügbar).
    Programmierfragen über PN / Konversation werden ignoriert!