Programm zeigt "OutOfMemory"

  • C#
  • .NET (FX) 1.0–2.0

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

    Programm zeigt "OutOfMemory"

    Hallo,
    ich habe eine einfache Slide Show geschrieben. In dieser wird in einer Picturebox alle paar Sekundne ein neues Bild angezeigt.

    Leider kommt nach einiger Zeit eine OutOfMemory Fehlermeldung. Ich finde leider den Fehler nicht. Also wodurch genau dieser Fehler ausgelöst wird. Laut TaskManager nutzt das Programm max. 50MB des RAMs.

    Hier mal der wichtigste Teil aus dem Programm:


    C#-Quellcode

    1. ​
    2. //Wähle Verzeichniss aus für Bilder
    3. private void verzeichnissAuswählenToolStripMenuItem_Click(object sender, EventArgs e)
    4. {
    5. DialogResult result = folderBrowserDialog.ShowDialog();
    6. if (result == DialogResult.OK)
    7. {
    8. Dirpath = folderBrowserDialog.SelectedPath;
    9. Properties.Settings.Default.Pfad = Dirpath.ToString();
    11. Properties.Settings.Default.Save();
    13. }
    14. }
    16. //Erstelle Timer
    17. private void slideshowStartenToolStripMenuItem_Click(object sender, EventArgs e)
    18. {
    19. if(Properties.Settings.Default.Pfad == "")
    20. {
    21. MessageBox.Show("Bitte wählen Sie zuerst ein Verzeichnis aus.");
    22. DialogResult result = folderBrowserDialog.ShowDialog();
    23. if (result == DialogResult.OK)
    24. {
    25. Dirpath = folderBrowserDialog.SelectedPath;
    26. Properties.Settings.Default.Pfad = Dirpath.ToString();
    28. Properties.Settings.Default.Save();
    30. }
    31. }
    33. base.SetVisibleCore(true);
    34. string pfad = Properties.Settings.Default.Pfad;
    36. System.Windows.Forms.Timer t = new System.Windows.Forms.Timer();
    37. t.Interval = 1000 * Convert.ToInt32(Properties.Settings.Default.Sekunden); // Intervall
    38. t.Tick += new EventHandler(timer_Tick);
    39. t.Start();
    41. images = Directory.GetFiles(pfad, "*.jpg", SearchOption.AllDirectories);
    43. }



    C#-Quellcode

    1. //Ändere Bild alle X Sekunden
    3. Random random = new Random();
    4. void timer_Tick(object sender, EventArgs e)
    5. {
    6. //Zufallwiedergabe an
    7. if (Properties.Settings.Default.Zufall)
    8. {
    9. int interval = random.Next(0, images.Length);
    10. if (pictureBox1.Image != null)
    11. {
    12. pictureBox1.Image.Dispose();
    13. }
    14. pictureBox1.Image = Image.FromFile(images[interval]);
    15. }
    16. //Zufallwiedergabe aus
    17. else
    18. {
    19. if (pictureBox1.Image != null)
    20. {
    21. pictureBox1.Image.Dispose();
    22. }
    23. pictureBox1.Image = Image.FromFile(images[counter++]);
    24. }
    26. }



    Wäre nett wenn sich jemand mal den Code ansehn könnte und mir einen Tipp gibt. Habe das Projekt im Anhang, benutzt wird Visual Studio 2015.
    Dateien

    • Slide Show.rar

      (2,09 MB, 79 mal heruntergeladen, zuletzt: )

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

    @King2k7 Memory nicht, wohl aber eine IndexOutOfRangeException, weil Du den counter gnadenlos erhöhst, ohne ihn auf die Anzahl der vorhandenen Bilder zu beschränken.

    Dies hilft:

    C#-Quellcode

    1. counter += 1;
    2. if (counter >= images.Length)
    3. {
    4. counter = 0;
    5. }
    6. pictureBox1.Image = Image.FromFile(images[counter]);
    OutOfMemory kam bis jetzt bei mir noch nicht.
    Beim wievielten Bild kam diese Exception bei Dir?
    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!

    @King2k7 Du musst zunächst den Absturz sicher reproduzieren, und dass nach einer minimalen Laufzeit.
    Dann setz mal die Zeit zwischen zwei Bildern drastisch runter, dann sollte es schneller gehen. Vielleicht suchst Du noch Bilder raus, die sehr groß sind, die kannst Du ja in ein separates Verzeichnis packen.
    Dein Code an sich sieht eigentlich gut aus.

    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!

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

Facebook
Telefonspende