0

I am making a program that acts as the net of a 2x2x2 Rubik's Cube. In this method, I plan to make an if statement for each of the 12 possible moves (up clockwise, up counterclockwise, down clockwise, etc.), but my code for turning up clockwise is not working. "move" is a string that is designated in the main class and "cube" and "temp" are both 3d char arrays containing the current state of the Rubik's Cube. "getTurn" is a method that rotates the face clockwise and takes "cube" twice (once for the cube that it will be editing and once for a temp variable that does indeed work), the int value of the face being turned, and a boolean value for which way it is being turned. It is working as intended, but when I try to create the movement of the sides of the Rubik's Cube, the temp variable for "cube", called "temp" is somehow altered as the code goes, creating incorrect outputs. I believe there is something wrong with the way that my temp is set up, because it is not behaving properly. Thank you for helping me out!

public char[][][] getMove(char[][][] cube, char[][][] temp, String move) {

    //0 = u
    //1 = d
    //2 = l
    //3 = r
    //4 = f
    //5 = b

    if(move.equals("u")) {
        cube = getTurn(cube, cube, 0, true);

        cube[4][0][0] = temp[3][0][0]; cube[4][0][1] = temp[3][0][1];
        cube[2][0][0] = temp[4][0][0]; cube[2][0][1] = temp[4][0][1];
        cube[5][0][0] = temp[2][0][0]; cube[5][0][1] = temp[2][0][1];
        cube[3][0][0] = temp[5][0][0]; cube[3][0][1] = temp[5][0][1];

EDIT:

I just got it working by removing the 3d array "temp" and replacing it with eight temp ints:

public char[][][] getMove(char[][][] cube, String move) {

    //0 = u
    //1 = d
    //2 = l
    //3 = r
    //4 = f
    //5 = b

    if(move.equals("u")) {
        cube = getTurn(cube, cube, 0, true);

        char temp1 = cube[3][0][0];
        char temp2 = cube[3][0][1];
        char temp3 = cube[4][0][0];
        char temp4 = cube[4][0][1];
        char temp5 = cube[2][0][0];
        char temp6 = cube[2][0][1];
        char temp7 = cube[5][0][0];
        char temp8 = cube[5][0][1];
        cube[4][0][0] = temp1; cube[4][0][1] = temp2;
        cube[2][0][0] = temp3; cube[2][0][1] = temp4;
        cube[5][0][0] = temp5; cube[5][0][1] = temp6;
        cube[3][0][0] = temp7; cube[3][0][1] = temp8;

If anyone knows why my previous version was not working, I would love to know.

David Bass
  • 1
  • 1
  • Based on the provided code it is hard to tell what's wrong with it – MaxZoom Sep 12 '16 at 21:00
  • @MaxZoom "cube" and "temp" are both copies of the array "cube" from the main method, which at the start of the program is initialized and printed with cube[0][0][0] = 'w', cube[0][0][1] = 'w', and so on with the first index of the array being 0-5 for faces of the Rubik's Cube, the second two indices signifying which of the 4 squares on each face, and the chars representing colors. – David Bass Sep 12 '16 at 21:02
  • if temp is copy of cube then might be they are referencing to same address that's why its altering temp array as well.. how you have created temp is it just temp = cube? – RRR_J Sep 13 '16 at 07:54

0 Answers0