Image.FromFile / Image.FromStream giver OutOfMemoryException

1. Jeg ville bruge 'using' alle steder hvor det er muligt. Så er du sikker på at du får lukket instanserne når de ikke længere skal bruges.

2. Du får ikke lukket dit image objekt i dit loop.
Det er derfor du får out of memory exceptions.


Din kode skal se sådan ud:

if (folderBrowserDialog1.ShowDialog() == DialogResult.OK)
            {
                string foldername = folderBrowserDialog1.SelectedPath;
                string[] folderFiles = Directory.GetFiles(foldername, "*.jpg");

                listboxBilder.Items.Clear();
                galleryList = new List<GalleryImage>();
                Image img = null;

                foreach (string f in folderFiles)
                {
                    using (FileStream fs = new FileStream(f, FileMode.Open, FileAccess.Read))
                    {
                        img = Image.FromStream(fs);
                    }
                    GalleryImage newObj = new GalleryImage(f, img, DateTime.Now);
                    galleryList.Add(newObj);
                    listboxBilder.Items.Add(newObj);
                    img.Dispose();
                }
            }



Mvh.
Lasse

Smider også lige et svar med det samme.

Jeg har forsøgt at bruge img.dispose(), men så kan den ikke længere læse fra det Image objekt jeg har tilføjet galleryList når jeg senere forsøger at tilgå denne.

Jeg tror ikke det er en god ide at loade så mange billeder ind i memory på en gang. Du skal huske på, at de fleste billeder fylder meget mere i memory, end de gør på harddisken, da de er komprimeret på harddisken.

I memory fylder alle billeder ca det samme som et bitmap billede på disken.

Havde det ikke været en ide, bare at gemme på referencen til billedet i memory, og loade det ind i memory bare når det skal bruges?

Det kan selvfølgelig godt være... Jeg bruger listbox'en til at fremvise det enkelte billede i en picturebox, men jeg kan selvfølgelig bare loade billedet på nyt hver gang det skal vises i stedet for at gemme det i memory.

...men så får jeg jo bare samme problem hvis man viser mange billeder efter hinanden. For jeg kan jo ikke dispose billedet jeg lige har assignet til picturebox'en.

Det er selvfølgelig rigtigt at du ikke kan bruge objektet længere når du har disposet det.

Men så kunne du gøre noget helt andet.

Du kunne erstatte dit Image i galleryimage med en stream. Det fylder ikke så meget, og så kunne du oprette dit billede ud fra stream objektet.


F.eks. sådan her:
public partial class Form1 : Form
    {
        List<GalleryImage> galleryList = new List<GalleryImage>();
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            if (folderBrowserDialog1.ShowDialog() == DialogResult.OK)
            {
                string foldername = folderBrowserDialog1.SelectedPath;
                string[] folderFiles = Directory.GetFiles(foldername, "*.jpg");

                listboxBilder.Items.Clear();
                galleryList = new List<GalleryImage>();

                foreach (string f in folderFiles)
                {
                    FileStream fs = new FileStream(f, FileMode.Open, FileAccess.Read);
                    GalleryImage newObj = new GalleryImage(f, fs, DateTime.Now);
                    galleryList.Add(newObj);
                    listboxBilder.Items.Add(newObj);
                }
            }
        }

        private void button2_Click(object sender, EventArgs e)
        {
            GalleryImage gm = (GalleryImage)listboxBilder.SelectedItem;
            Image img = Image.FromStream(gm.stream);
        }

    }

    public class GalleryImage
    {
        public string file { get; set; }
        public FileStream stream { get; set; }
        public DateTime Created { get; set; }

        public GalleryImage()
        {
        }
        public GalleryImage(string f, FileStream fs, DateTime created)
        {
            file = f;
            Created = created;
            stream = fs;
        }
    }

Sorry, aaber_cc's løsning med en reference er den samme.

Var vidst for sent ud :-(


Mvh.
Lasse

Er det fuldt størrelses billeder du viser i din listbox?

Hvis det er små billeder, kan du gemme en thumnail i memory, og ha en reference til det originale billede, i tilfælde det fulde billede skal bruges senere.

Jeg takke for jeres svar...

Jeg endte med at gemme kun filepath i galleryList og indlæse det på ny fra (GalleryImage)listboxBilder.SelectedItem. Men når jeg så disband'er det forrige billedet inden jeg indlæser et nyt giver det ingen overhead.

Jeg vurderer I begge to gav mig den pegepind jeg behøvede, så smid lige et svar aaberg_cc... Så får I halvdelen hver.

svar