Thread: Image looping issue

The images for that print out would be on a row(y=115) going from left to right from 31 to 157.
What does the print out look like for when the images do not go where you want them to go?

No need to print values that don't change like Offset and length every time you print. Print them one time before the loop begins.

images printed are not where I want them to be.

That print out was for putting images on a row (y=115)
What did you want that code to do differently? Where were those images supposed to go? What should be the x and y values to put the images where you want them?

I wanted the first column to start at y = 10, x = 10 and print 7 times to end up at y = 115, x = 10.

If you want the y values to change (keeping the x value constant) then you will be drawing the images in a column.

What you posted was for the images to be in a row not a column. What does the code and printout look like for when you try to draw images in a column.

problem came when only one image was printed at the second column.

Can you show the printout of the x & y values and the code?

What part of the code works? Which of the four lines are drawn correctly: Left, top, right or bottom?

Which ones are you having trouble with?


Your println should be immediately following the call to drawImage so you see where EVERY image is being drawn. For example:

              // draw first column at constant x 
              for(int j = 0; j < height; j++) {
//                       g2.drawImage(cobblestone.getImage(), currentX, currentY,this);
                    System.out.println("currentX= " + currentX + ", currentY = " + currentY);
                    currentY += yOffset;
              }
              System.out.println(">>>> Next");
              // draw row at constant y  
              for(int k = 0; k < width; k++) {
//                	   g2.drawImage(cobblestone.getImage(), currentX, currentY,this);
                  System.out.println("currentX= " + currentX + ", currentY = " + currentY);
             	   currentX += xOffset;
              }

Pretty much all the code I posted you is working at the moment as it is only considering the left and bottom lines. I do not know how to do the top or right lines as each time I try it never works how I anticipate it to.

Here is the coordinates of every image:

currentX= 10, currentY = 52
currentX= 10, currentY = 73
currentX= 10, currentY = 94
currentX= 10, currentY = 115
currentX= 10, currentY = 136
>>>> Next Line
currentX= 10, currentY = 157
currentX= 31, currentY = 157
currentX= 52, currentY = 157
currentX= 73, currentY = 157
currentX= 94, currentY = 157
>>>> Restart
currentX= 10, currentY = 10
currentX= 10, currentY = 31
currentX= 10, currentY = 52
currentX= 10, currentY = 73
currentX= 10, currentY = 94
currentX= 10, currentY = 115
currentX= 10, currentY = 136
>>>> Next Line
currentX= 10, currentY = 157
currentX= 31, currentY = 157
currentX= 52, currentY = 157
currentX= 73, currentY = 157
currentX= 94, currentY = 157

I do not know how to do the top or right lines

For the top line:
What would be the y value for all the images? What would be the range of x values?

For the right line:
what would be the x value for all the images? What would be the range of y values?

If you can answer those questions, you should be able to write the for loops to draw the lines.

Glad you figured it out. Now on to the next problem.