Are these locations all at the same x value, all at the same y value, or in any way relative to each other?
If not, then you should create an array of locations ( PVectors) containing the locations.
If so, then you don't need your x and y variables; you can just scale i to fit your needs.

For example, what I have in mind:

1. void PlaceImages() {
2.   int randomImage = (int)random(1, 23);
3.   PImage loadedImage;
4.   String imagePath = "shape/1.png";
5.   loadedImage = loadImage(imagePath);
6.   for (int i = 0; i<=5; i++) {
7.     image(loadedImage, 90 + (loadedImage.width + 10) * i, 90);
8.   }
9. }

Thanks for your assistance.Their positions are not related as they ( the Images) would all be scattered  within the bounds of a rect() whose x1,y1,x2,y2 are all known. Tried Using the PVector Class from the Processing Reference, Please is this what you meant?

1. for (int i = 0; i<=5; i++) {
2.     image(loadedImage, x1.y, x2.y);
             x2.add(x1);
3.   }

1. PVector x1;
2. PVector x2;
3.    x1 = new PVector(40, 20);
     x2 = new PVector(25, 50);
   

I was thinking that you would have an array of PVectors and then, in the for loop, reference the PVectors to display the images. Like so:

1. PVector[] locations = new PVector[5]; //Initialize these PVectors in setup()
   
2. 
   
3. ...
   
4. 
   
5. void PlaceImages() {
6.   int randomImage = (int)random(1, 23);
7.   PImage loadedImage;
8.   String imagePath = "shape/1.png";
9.   loadedImage = loadImage(imagePath);
10.   for (int i = 0; i < images.lengthj; i ++) {
11.     image(loadedImage, locations[i].x, locations[i].y);
12.   }
13. }

Thanks once again,

1. ...
2. PVector[] locations = new PVector[5];// from setup()
   
3. for (int i = 0; i<=5; i++) {
4.          image(loadedImage, locations[i].x, locations[i].x);
5.      }

Did you initialize the PVectors in setup() properly?
Also, you might change that <= sign to a < sign.

Yea, i think i did,
PVector[] locations;

1. void setup(){
2. locations = new PVector[5];
3. }

then the PlaceImages() function that contains the for... loop is called in draw()

Also, Using <, still gives me the same error.

1.  for (int i = 1; i<5; i++) {
2.          image(loadedImage, locations[i].x, locations[i].x);
3.  }

   

Thanks once again for your time

Thank You So Much for your time. Been able to have it work as expected.

don't load the image in the draw loop. the image is the same every time and it only needs to be done once, so do it in setup(). use a global PImage.