Turning 2 arrays into a two-dimensional array in java

Question

Hi I am trying to take two arrays and turn them into one 2 dimensional array. However, I keep getting an error:

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 2
 at test5.sum(test5.java:12)
 at test5.main(test5.java:38)

Here is my code:

public class test5 {
    int[][] final23;

    public int[][] sum(int[] x, int[] y) {
        final23 = new int[2][x.length];
        for (int i = 0; i < final23[i].length; i++) {

            final23[1][i] = x[i];
            final23[2][i] = y[i];
        }
        return final23;
    }

    public void print() {
        for (int i = 0; i < final23[i].length; i++) {
            for (int j = 0; j < final23[i].length; j++) {

                System.out.print(final23[i][j] + " ");
            }
        }
    }

    public static void main(String[] args) {
        int l[] = { 7, 7, 3 };
        int k[] = { 4, 6, 2 };
        test5 X = new test5();

        X.sum(k, l);
        X.print();
    }
}

I am not really sure what the problem is. Sorry if the question is dumb, I am new to coding!

luiscubal · Accepted Answer · 2010-09-10 22:09:44Z

5

The problem is:

final23 [2][i] = y[i];

Java arrays always start at 0. So final23 only has [0] and [1].

Any array with n elements can go from 0 to n-1.

answered Sep 10, 2010 at 22:09

luiscubal

25.2k9 gold badges61 silver badges84 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Carlos Over a year ago

As well as arrays in most of the common programming languages. At least in those derived from C.

Carlos · Accepted Answer · 2010-09-10 22:25:22Z

3

There is also a second problem with your program. You have this loop in both sum and print methods:

for (int i = 0; i < final23[i].length; i++)

In sum method it should be

for (int i = 0; i < final23[0].length; i++)

And in print method

for (int i = 0; i < final23.length; i++)

Otherwise you'll get ArrayIndexOutOfBoundsException again.

Note that the program works correctly only if both input arrays have the same length. This might be ok for your purposes, but keep that in mind.

edited Sep 10, 2010 at 22:25

answered Sep 10, 2010 at 22:19

Carlos

2,51326 silver badges29 bronze badges

Comments

Dave McClelland · Accepted Answer · 2010-09-10 22:10:59Z

1

Try

for (int i = 0; i < final23[i].length; i++)
 {

  final23 [0][i] = x[i];
  final23 [1][i] = y[i];
 }

Remember, all arrays are 0 based, even n-dimensional ones.

answered Sep 10, 2010 at 22:10

Dave McClelland

3,4231 gold badge31 silver badges45 bronze badges

Collectives™ on Stack Overflow

Turning 2 arrays into a two-dimensional array in java

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related