C# System.OutOfMemoryException was unhandled

Question

i use foreach to read all the image from a folder

        string[] filePaths = Directory.GetFiles(Workspace.InputFolder, "*.*");
        foreach (string imageFile in filePaths)
        {
            // Some Process here, the output are correct, just after output 
               the error happen
        }

But it come out error

System.OutOfMemoryException was unhandled
  Message=Out of memory.
  Source=System.Drawing

Is the problem cause by the foreach loop keep looping after the process finish? What should i do for releasing the memory? Thanks.

It depends greatly on what you are doing with each filename. If you are opening a file, make sure and Dispose() the objects when you are done. — drdwilcox
– drdwilcox, Commented Nov 14, 2011 at 19:53
You need to post more code, specifically what you're doing inside the foreach loop. My gut feeling is that you're creating disposable object instances and not disposing of them correctly, but it's not possible to know for sure unless you post some more code. — matt
– matt, Commented Nov 14, 2011 at 19:55
i use the image to do OCR that convert the image into digital data. — KX.T
– KX.T, Commented Nov 14, 2011 at 19:55

Reed Copsey · Accepted Answer · 2011-11-14 19:56:26Z

6

Given your exception, it looks like you're working with objects in the System.Drawing namespace.

If you are opening and manipulating an image in your foreach loop, for example, make sure that you call Dispose() to release the images resources as soon as you're done with them. You can, alternatively, wrap this in a using statement, ie:

    foreach (string imageFile in filePaths)
    {
        using (var image = Image.FromFile(imageFile)
        {
            // Use the image...
        } // Image will get disposed correctly here, now.
    }

Be aware that it's not just images that are potentially the problem, but any resource that implements IDisposable. Many of the classes in System.Drawing are disposable - make sure that you either access them as above (via using) or call Dispose() on them when done.

answered Nov 14, 2011 at 19:56

Reed Copsey

567k80 gold badges1.2k silver badges1.4k bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

KX.T Over a year ago

i did try your solution but same error message come out, how should i call the Dispose() function?

Reed Copsey Over a year ago

@Xuan You'd need to call the Dispose() method on all objects that include it, once you've finished using them, inside of your foreach loop. This is going to include any images, but also potentially your OCR classes, etc.

KX.T Over a year ago

@ReedCopsey Thanks for helping, i solve the problem by change the . to specific image file(*.tif).

matt Over a year ago

Xuan, simply specifying "*.tif" isn't going to fix the root issue. The problem at hand is that you almost certainly are working with Disposable types, and aren't properly disposing of the instances when you're done processing them.

Collectives™ on Stack Overflow

C# System.OutOfMemoryException was unhandled

1 Answer 1

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related