Not sure why I'm getting a NullPointerException when creating a Swing component

Posted by Alex on Stack Overflow See other posts from Stack Overflow or by Alex
Published on 2011-01-12T13:38:01Z Indexed on 2011/01/12 13:53 UTC
Read the original article Hit count: 233

Filed under:
|
|

The error occurs when creating the Box object.

    public void drawBoard(Board board){
    for(int row = 0; row < 8; row++){
        for(int col = 0; col < 8; col++){
            Box box = new Box(board.getSquare(col, row).getColour(), col, row);
            squarePanel[col][row].add(box);
        }
    }

Board is given from the Game constructor here (another class):

public Game() throws Throwable{
    View graphics = new View();
    board = new Board();
    board.setDefault();
    graphics.drawBoard(board);
}

The Board constructor looks like this:

public Board(){
    grid = new Square[COLUMNS][ROWS];
    for(int row = 0; row < 8; row++){
        for(int col = 0; col < 8; col++){
            grid[col][row] = new Square(this);
        }
    }
    for(int row = 0; row < 8; row++){
        for(int col = 0; col < 4; col++){
            int odd = 2*col + 1;
            int even = 2*col;
            getSquare(odd, row).setColour(Color.BLACK);
            getSquare(even, row).setColour(Color.WHITE);
        }
    }
}

And finally the Box class:

class Box extends JComponent{
Color boxColour;
int col, row;
public Box(Color boxColour, int col, int row){
    this.boxColour = boxColour;
    this.col = col;
    this.row = row;
    repaint();
}
public void paint(Graphics drawBox){
    drawBox.setColor(boxColour);
    drawBox.drawRect(50*col, 50*row, 50, 50);
    drawBox.fillRect(50*col, 50*row, 50, 50);
    }
}

So while looping through the array, it uses the two integers as coordinates to create the Box. The coordinates are referenced and then repaint() is run. The box also gets the colour, using the two integers, from the Square in the Board class. Since the colour is already set, before the drawBoard(board) method is run, that shouldn't be a problem, right?

Exception in thread "main" java.lang.NullPointerException
    at View.drawBoard(View.java:38)
    at Game.<init>(Game.java:21)
    at Game.main(Game.java:14)

The relevant part of Square

import java.awt.Color;

public class Square {

    private Piece piece;
    private Board board;
    private Color squareColour;

public Square(Board board){
    this.board = board;
}

public void setColour(Color squareColour){
    this.squareColour = squareColour;
}

public Color getColour(){
    return squareColour;
}

© Stack Overflow or respective owner

Related posts about java

Related posts about swing