PSNB Tech Blog

The block properties are defined in TileType.java. The first parameter for each piece is its base color, which the light and dark edges are calculated from.

BoardPanel.java defines two constants, MIN_COLOR and MAX_COLOR, that each color components should not exceed. The reason why is explained in the source code.

This comment has been removed by the author.

No problem! The properties for each piece can be found (and edited) in TileType.java.

The first parameter for each TileType is a Color. The constructor for Color is; Color(int red, int green, int blue). Each component can be between 0 and 255, though I clamped them to BoardPanel.COLOR_MIN and BoardPanel.COLOR_MAX to ensure the light and dark colors are calculated correctly.

For example, the base color for the "I" type is: Color(BoardPanel.COLOR_MIN, BoardPanel.COLOR_MAX, BoardPanel.COLOR_MAX), which is similar to Cyan.

If we wanted to make the "I" type green, we would change the color parameters to represent green Color(BoardPanel.MIN_COLOR, BoardPanel.MAX_COLOR, BoardPanel.MIN_COLOR).

If it helps, you can imagine MIN_COLOR as representing 0, and MAX_COLOR as representing 255.

Welcome back! Whichever method you prefer is fine by me, though I'm more likely to respond quickly here.

I assume that the confusion is stemming from calculating the array index, rather than what the method actually does. You can find a pretty decent explanation here: http://stackoverflow.com/questions/2151084/map-a-2d-array-onto-a-1d-array-c

can you please mail me your enhanced version. nasbils@hotmail.com. working on my final project and it could really help me

TileType.getBaseColor() returns a Color object, so you' need to compare it to another color, rather than a String.

Color color = ...; //Whatever color you want to compare.
if(tiles[row * COL_COUNT + col].getBaseColor().equals(color))

Also, I changed the board a while back to use a 2D array, which should make it a bit easier to use.

thanks for the reply.
im trying to add a restriction in the game. id like to make the line to be cleared only if they are the same color. i add a condition in method checkLine like below but theres error. can you show me the right way to do it? Its for education puposes. THX

for(int col = 1; col < COL_COUNT; col++) {
if(!isOccupied(col, line) && tiles[line * COL_COUNT + col].getBaseColor().equals(tiles[line * COL_COUNT + (col-1)].getBaseColor())) {
return false;
}
}

You're not checking the first column in the loop. You could do something like...

-------------------------
TileType tile = getTile(0, line); //Get the first tile on the line.
if(tile == null) {
return false; //First tile is empty
}
for(int i = 1; i < COL_COUNT; i++) {
if(getTile(i, line) != tile) {
return false;
}
}
return true;
-------------------------

By the way, you can compare an enum to another enum, so you don't need to compare the colors together.

Yeah, you can. But you'd need to make a few changes to the code to make it work.

You're going to need to load and store the different images when you initialize the game (before entering the main loop). You could probably store the images in a Map of some sort.

You'll also need to replace the rendering code to display the image rather than drawing each individual tile, which would be pretty easy to do.