Thread: Image looping issue

Hello everyone

I have been trying to loop an image using the for loop within graphics 2D which consists of 17 images on the x axis and 17 images (across) on the y axis (down), creating a 'L' shape. When I compile and run the program however it prints 17 images on the y axis and none on the x. I have no idea what fault I have made as whatever I do it seems to ignore the x axis (or at least the way i interpret it to be printed.) How do I make it print along the x-axis? What exactly am I doing wrong? Here is my code:

  private void drawMaze(Graphics2D g){
    	Graphics2D g2 = (Graphics2D)g;
 
    	    int width = 17; //number of images drawn along the x-axis (none)
            int height = 17; //number of images drawn along the y axis
            int xOffset = 21; //width of image
            int yOffset = 21; //height of image
            int currentX = 10; //initial x coordinate of image
            int currentY = 10; //initial y coordinate of image
            for(int i = 0; i < width; i++) {
                 for(int j = 0; j < height; j++) {
                       g2.drawImage(cobblestone.getImage(), currentX, currentY,this);
                       currentY += yOffset;
                 }
                 currentX += xOffset;
                 currentY = 0;

Any help would greatly be appreciated, thanks!

The nested loops look ok. The ending } for the outer loop is missing. Is there any code is missing?

Add a call to print() that prints out the x and then all the y values on each column for each image that is drawn to show where the images are being drawn. Add a println() after the end of the inner loop.

how would I program it to just print the outer edges?

Control the values of x and y to place the images where you want them drawn.

You should be able to use loops. There are two rows and two columns

Draw the grid of images. Write down the x,y value for each of the 4 sides and look at how the x,y values change for each of the 4 areas you are putting images into.

The easiest solution looks like 3 to 4 loops

There are 4 areas to put the images in: two rows and two columns
To put images in a row, change the x value, leave the y value unchanged.
To put images in a column, change the y value, leave the x value unchanged.

The loops could be doubled up to do two rows (or two columns) at a time.

when I set the value of Current X within my loop to 540, it only printed out one image in my second column

Can you post the code did that?
Try debugging the code by adding printlns that print the values of x and y where the images are being drawn.

What prints out when you add the println to print the x,y locations for each of the images you are drawing?

I assume that the x,y locations are the problem you are having, so if you print them out we can see where the problem is.

Please post the print outs that shows the x and y values of where the image is being draw and the code that generated the output.

For testing you should reduce the width and height to a smaller value like 5 or 7 vs 17.

Look at the code to get the names of the variables. You are probably correct with their names.

Please post the print outs that shows the x and y values of where the image is being draw and the code that generated the output.

Where is the print out of the values of the x and y locations where you are drawing the images?

I don't see any println statements in the code you posted. Add some println statements to print out the values of where each image is being drawn.

To get debug output so you can see what the code is doing, print the values of the variables using:

System.out.println("var=" + var + ", anotherVar=" 
              + anotherVar + ", andMore=" + andMore);

replace var, anotherVar and andMore with the names of the variables in your program

Originally Posted by Norm

To get debug output so you can see what the code is doing, print the values of the variables using:

System.out.println("var=" + var + ", anotherVar=" 
              + andMore);

I am a bit confused as to what is going on here, what are all the '=' and quotes at different places for?

The = are part of the String to be printed. The above would print out something like this:
var=234, anotherVar=34, andMore=333


The "s define Strings. The +s concatenate all the Strings into a single String which is printed.