Out of Memory Error

Hello,
I am making a top 150 selling products report. It shows the stock number, the name of the product and then some sales info. I am using the following code to set a comment in the cell with the product name to show an image of the product. The code sets the image is based on a path that is defined in a different cell in the same row. It runs fine for half or more of the report, then it errors out saying it is out of memory. How can I fix this?

Here is my code:

Please Login or Register  to view this content.

Originally Posted by Arkadi

Just wondering about the image and whether that could be a problem.. I'd try commenting out the .Fill.UserPicture line, just to see if that is the problem. If the image is large and being put into the comments of many cells it could be an issue.

Thanks, I'll try that, although without that part, the code is kind of useless, because the only thing in the comment is the picture. If that does end up being the problem is there a way to optimized that? At the moment, its just grabbing them off the internet. Is there a way/would it be better to have the code save the images locally before getting them?

Gotcha, well that line is definitely the issue. Unfourtanately I can't reduce the size of the images, but they are all under 200KB, so they are not very big to start with.

I dont know much about Error Handling, but I added an 'On Error Resume Next' statement before the '.Fill.UserPicture' line. It seems to have fixed the problem, but is there some reason why using this would be bad?

Thanks so much for your help!

Here is my code:

Please Login or Register  to view this content.

It looks like it always errors out when i = 116, if I have it skip row 116 then it finishes fine.

The weird thing is that if I use 'On Error Resume Next', when it is finished row 116 has the picture it is supposed to have. So do all of the rows around 116.

Nevermind. It looks like the problem was that the row in question had an incorrect path. Fixing this fixed the problem without having to use error handling. It is a little strange that it the error was Out of Memory, but oh well.

Thanks again for your help.