Why is the grid not lining up properly?

Question

relevant code:
     private static final int SCREEN_WIDTH = 640;
     private static final int SCREEN_HEIGHT = 640;
     private final int[][] a;
     private final int width;
     private final int height;
     private final int boxSizeX;
     private final int boxSizeY;
     private final int startX;
     private final int startY;
     private final char[]answer;
     private boolean userAnswer;
public project3main(int[][] array, int boxsize, char[]answer, boolean userAnswer) {
          this.a=array;//the array representing the walls
          this.width = a[0].length;
          this.height = a.length;
          this.answer = answer;
          this.setPreferredSize(new Dimension(SCREEN_WIDTH,SCREEN_HEIGHT));
          this.boxSizeX = (SCREEN_WIDTH)/(width+2);
          this.boxSizeY = (SCREEN_HEIGHT)/(height+2);
          this.startX = boxSizeX/2;
          this.startY = boxSizeY/2;
          this.userAnswer = userAnswer;
     public void paintComponent(Graphics g) {
          //draw the ceiling
               this.a[height-1][this.a[height-1].length-1] = SOUTH_WALL;
               g.drawLine(this.boxSizeX, this.boxSizeY, (SCREEN_WIDTH-(this.boxSizeX)), this.boxSizeY);     
               g.drawLine(this.boxSizeX, this.boxSizeY+this.boxSizeY, this.boxSizeX, (SCREEN_HEIGHT-(this.boxSizeY)));     
               int currY=this.boxSizeY;
               int currX=this.boxSizeX;
               for(int i=0;i<this.height;i++)
                    currY = (this.boxSizeY)*(i+1);
                    for(int j=0;j<this.width;j++)
                         currX = (this.boxSizeX)*(j+1);
                         if(a[i][j] == EAST_WALL)
                              g.drawLine(currX+this.boxSizeX, currY, currX+this.boxSizeX, currY+this.boxSizeY);
                         else if(a[i][j] == SOUTH_EAST_WALL)
                              g.drawLine(currX+this.boxSizeX, currY, currX+this.boxSizeX, currY+this.boxSizeY);
                              g.drawLine(currX, currY+this.boxSizeY, currX+this.boxSizeX, currY+this.boxSizeY);
                         else if(a[i][j] == SOUTH_WALL)
                              g.drawLine(currX, currY+this.boxSizeY, currX+this.boxSizeX, currY+this.boxSizeY);
               }basically, this takes in a matrix which is filled with integers that represent if each node has a south, east, or a combination of the two.
It creates a standard box-size relative to the width of the screen and the height of the screen (set at 640 for each)
It then draws the ceiling and the left wall.
However, the ceiling and the left wall do not line up. That is, the outside wall does not occur at the rightmost point of the ceilling.
Can anyone see why just by looking at the code?
further code furnished upon request
Edited by: thenextbesthang on May 4, 2008 10:03 AM

Answer 1

thenextbesthang wrote:
how do you post a jframe?? You just make a minimal JFrame and post the code:
import java.awt.Dimension;
import java.awt.Graphics;
import javax.swing.JFrame;
import javax.swing.JPanel;
public class FooFrame
    private static final int EAST_WALL = 0; // TODO change as appropriate
    private static final int SOUTH_WALL = 0; // TODO change as appropriate
    private static final int SOUTH_EAST_WALL = 0; // TODO change as appropriate
    private static int SCREEN_WIDTH = 640;
    private static int SCREEN_HEIGHT = 640;
    private int[][] a =
        // TODO put in reasonable numbers here.
    private int width;
    private int height;
    private int boxSizeX;
    private int boxSizeY;
    private int startX;
    private int startY;
    //private char[] answer;
    //private boolean userAnswer;
    public FooFrame()
        width = a[0].length;
        height = a.length;
        //this.answer = answer;
        //this.setPreferredSize(new Dimension(SCREEN_WIDTH, SCREEN_HEIGHT));
        boxSizeX = (SCREEN_WIDTH) / (width + 2);
        boxSizeY = (SCREEN_HEIGHT) / (height + 2);
        startX = boxSizeX / 2;
        startY = boxSizeY / 2;
        //userAnswer = userAnswer;
    public JPanel createMyPanel()
        JPanel panel = new JPanel()
            @Override
            protected void paintComponent(Graphics g)
                super.paintComponent(g);
                myPaint(g);
        panel.setPreferredSize(new Dimension(SCREEN_WIDTH, SCREEN_HEIGHT));
        return panel;
    private void myPaint(Graphics g)
        a[height - 1][a[height - 1].length - 1] = SOUTH_WALL;
        g.drawLine(boxSizeX, boxSizeY,
            (SCREEN_WIDTH - (boxSizeX)), boxSizeY);
        g.drawLine(boxSizeX, boxSizeY + boxSizeY, boxSizeX,
            (SCREEN_HEIGHT - (boxSizeY)));
        int currY = boxSizeY;
        int currX = boxSizeX;
        for (int i = 0; i < height; i++)
            currY = (boxSizeY) * (i + 1);
            for (int j = 0; j < width; j++)
                currX = (boxSizeX) * (j + 1);
                if (a[i][j] == EAST_WALL)
                    g.drawLine(currX + boxSizeX, currY, currX
                        + boxSizeX, currY + boxSizeY);
                else if (a[i][j] == SOUTH_EAST_WALL)
                    g.drawLine(currX + boxSizeX, currY, currX
                        + boxSizeX, currY + boxSizeY);
                    g.drawLine(currX, currY + boxSizeY, currX
                        + boxSizeX, currY + boxSizeY);
                else if (a[i][j] == SOUTH_WALL)
                    g.drawLine(currX, currY + boxSizeY, currX
                        + boxSizeX, currY + boxSizeY);
    public static void main(String[] args)
        JFrame frame = new JFrame("Foo Frame");
        frame.getContentPane().add(new FooFrame().createMyPanel());
        frame.pack();
        frame.setVisible(true);
}