Runtime error in a method

I get runtime error in a method, I have tired for hours to figure it out but no positive progress so far.

Code:

public static String[][] produceKey () throws Exception { StringBuilder[][] key = new StringBuilder[theChars.length + 2][100]; char[] encryptChar = {'§', '¥', '$'}; //All the "a" characters will be replaced with one of the chars in encryptChar array. for (int q = 0; q < key.length; q++) { for (int w = 0; w < 100; w++) key[q][w] = new StringBuilder ("|aaaaaaaaaa|"); } //Time to generate the key code! for (int i = 0; i < key.length; i++) { FLoop: for (int j = 0; j < 100; j++) { for (int charPosition = 1; charPosition <= 10; charPosition++) { key[i][j].setCharAt (charPosition, encryptChar[(int) (3 * Math.random())]); } if (j == 0 && i == 0) continue FLoop; //There is nothing to compare with if i and j is zero //Here starts a check algoritm, to make sure there is no duplicated code for (int iC = 0; iC < i; iC++) { for (int jC = 0; jC < j; jC++) { if (key[i][j].equals (key[iC][jC])) { j--; continue FLoop; } } } } } //Convert key to a string array String[][] rkey = new String[key.length][100]; for (int n = 0; n < rkey.length; n++) { for (int m = 0; m < 100; m++) rkey[n][m] = new String (key[n][m].toString ()); } //Print the key into a file File fil = new File ("cryptKey.key"); PrintWriter fout = new PrintWriter (fil); for (int p = 0; p < rkey.length; p++) { for (int o = 0; o < 100; o++) fout.println (rkey[p][o].toString ()); } fout.close (); //DEBUGGING System.out.println (key[10][5]); System.out.println (key[theChars.length + 2][4]);//HERE IS WHERE THE PROGRAM CRASH return rkey; }

Code:

public static char[] theChars = {'!', '\'', '#', '¤', '%', '&', '/', '(', ')', '=', '?', '`', '1', '2', '3', '4', '5', '6', '7', '8', '9', '0', '+', '´', '‰', '½', '@', '£', '„', '€', '{', '[', ']', '}', '\\', 'q', 'w', 'e', 'r', 't', 'y', 'u', 'i', 'o', 'p', 'å', '¨', 'Q', 'W', 'E', 'R', 'T', 'Y', 'U', 'I', 'O', 'P', 'Å', '^', '~', 'a', 's', 'd', 'f', 'g', 'h', 'j', 'k', 'l', 'ö', 'ä', '\'', 'A', 'S', 'D', 'F', 'G', 'H', 'J', 'K', 'L', 'Ö', 'Ä', '*', '<', 'z', 'x', 'c', 'v', 'b', 'n', 'm', ',', '.', '-', '>', 'Z', 'X', 'C', 'V', 'B', 'N', 'M', ';', ':', '_', '…', 'µ', 'ª', '©', '¨', '¦', 'Œ', '¢', '¡', ' ', ' ', '«', '¬', '*', '®', '¯', '°', '±', '²', '³', '¶', '·', '¸', '¹', 'º', '»', '¼', '¾', '¿', 'À', 'Á', 'Â', 'Ã', 'Ä', 'Å', 'Æ', 'Ç', 'È', 'É', 'Ê', 'Ë', 'Ì', 'Í', 'Î', 'Ï', 'Ð', 'Ñ', 'Ò', 'Ó', 'Ô', 'Õ', 'Ö', '×', 'Ø', 'Ù', 'Ú', 'Û', 'Ü', 'Ý', 'Þ', 'ß', 'à', 'á', 'â', 'ã', 'ä', 'å', 'æ', 'ç', 'è', 'é', 'ê', 'ë', 'ì', 'í', 'î', 'ï', 'ð', 'ñ', 'ò', 'ó', 'ô', 'õ', 'ö', '÷', 'ø', 'ù', 'ú', 'û', 'ü', 'ý', 'þ', 'ÿ', '†', 'ƒ', '‡', 'Š', 'Ž', '×'};

And the exception
http://img703.imageshack.us/img703/1702/errorsa.png

theChars arrays length is 205.

key is declared thusly:

Code:

StringBuilder[][] key = new StringBuilder[theChars.length + 2][100];

and so it should be no surprise that it crashes with this:

Code:

System.out.println (key[theChars.length + 2][4]);

If you declare an array of length n, you can only use indices 0 to (n - 1). Please read a tutorial on arrays to learn more about this.

Don't post a screenshot. People can either not see it (like me, just a red cross) or it is too small and unreadable.

Copy and paste the full and exact error message and indicate on which line it occurs.

Quote:

Originally Posted by Junky

Don't post a screenshot. People can either not see it (like me, just a red cross) or it is too small and unreadable.

Copy and paste the full and exact error message and indicate on which line it occurs.

okey, next time I wont post image(how can you not see photobucket pictures? o_0).
Its an arrayindexoutofbounds execption.

Fubarable: I know arrays start with index 0. I was just confused.
Before I continue with this problem, I just spotted a new one.
Check the comment that say "Here starts a check algorithm, to make sure there is no duplicated code". That algorithm is not working(there is duplicates in the arrray!). Some help?

Quote:

Originally Posted by Pojahn_M

how can you not see photobucket pictures?

<whisper>
Using work computer. Certain links get blocked.
</whisper>

Code:

for (int i = 0; i < key.length; i++) { FLoop: for (int j = 0; j < 100; j++) { for (int charPosition = 1; charPosition <= 10; charPosition++) { key[i][j].setCharAt (charPosition, encryptChar[(int) (3 * Math.random())]); } if (j == 0 && i == 0) continue FLoop; //There is nothing to compare with if i and j is zero //Here starts a check algoritm, to make sure there is no duplicated code for (int iC = 0; iC < i; iC++) { for (int jC = 0; jC < j; jC++) { if (key[i][j].equals (key[iC][jC])) { j--; continue FLoop; } } } } }

If you find yourself writing that many nested loops and/or if statements then you need to rethink your design. It can only lead to confusion as to what the code is doing and you can easily introduce errors. A better approach is to break down what the code is supposed to do into smaller individual steps. Then you can move each step into its own method.

basically, what I want to do is comparing and 2d array with an other 2d array. It sounds easy, but I cant even think of how to do it.

It would be really simple if you first created a method which compares a 1 d array, then a method that uses the first to test the 2d array.

Code:

public boolean compareArray(int[] first, int[] second){ ... } public boolean compare2D(int[][] first, int[][] second){ for(each array in the 2d array) use compareArray }

As junky said, anytime you work with multi dimensional arrays, it can be made simpler by breaking things into steps(this works with many other things as well). Rather than working on one huge task, why not split it into a few(+) small tasks?

Quote:

Originally Posted by Pojahn_M

Fubarable: I know arrays start with index 0....

OK, well shut my mouth then.