Basically, I'm trying to write a program that displays a new image (from a bank of around 100) on mouse-click. (I mean, the program does other stuff, but this is the issue I'm having trouble with specifically). Right now, I have a program that will run for about three trials, and then it gives me an OutOfMemory error (I tried increasing the memory available, which helped somewhat, but it still crashes). All of this leads me to believe there must be a better way.

Right now, I have all the images in my data folder, and all the file names in a text document. My program loads the text file (called names), sets n at 0, and then loads the image names[n]. Then, on mouseclick, it adds 1 to n (and I put in a part where n resets to zero when it runs out of images). 

I'm not sure I even explained that well, so let me just drop in the relevant parts of the code here:

2. PImage img;
3. int n = 0;
4. 
   
5. void setup()
6. {    
7.   size(800,800);
8. }
9. 
   
10. void draw()
11. {
12.   background(255);
13.   String names[] = loadStrings("imgfiles.txt");
14.   img = loadImage(names[n]);
15.   image(img, 0, 0);
16. }
17. 
    
18. void mouseClicked() 
19. {   
20.   n = n+1;
21.   if (n > 133) {
22.     n = 0;
23.   }
24.   
25.   String message = "N is now:" + n;
26.   println(message);
27. }

Can anyone show me how to make this not crash? I'm absolutely willing to believe that I've done this in the most roundabout way possible. There must be a better way!

(also, if it matters, imgfiles.txt is literally just filenames separated by line breaks.)

Thank you so much!

Clara

UPDATE: I solved it! Never mind!

I put noloop() in the setup part, and then told the program to redraw() on mouse clicked. If anyone else is having similar struggles, this worked for me!!