Snippet of 1 Array to Check for Recurring Elements

Question

I was working on using only 1 array and a nested for loop to check for repeated elements and turn them to 0. I have an IndexBound problem and can't quite tell what's wrong.

Any help?

    int data[] = new int[20];
    for(int i = 0; i < data.length; i++) {
        data[i] = in.nextInt();
    }
    for (int i = 0; i < 18; i++) {
        for (int x = i + 1; x < 20; i++) {
            if (data[i] == data[x]) {
                data[x] = 0;
            }
        }
    }


Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 20
at Arrays.Prog415h.main(Prog415h.java:47)
if (data[i] == data[x]) {

score 2 · Accepted Answer · edited Mar 04 '17 at 23:21

2

Here, the increment of the nested for loop is i instead of x. This means that i is overrunning the array bounds for every iteration by both the outer and inner loops. To resolve the issue, change it to:

for (int x = i + 1; x < 20; x++)

edited Mar 04 '17 at 23:21

David C.

1,974
2
19
29

answered Mar 04 '17 at 20:10

Ganimj

61
3

Ok, so it was just a typo on my behalf. – Asker123 Mar 04 '17 at 20:12

Snippet of 1 Array to Check for Recurring Elements

1 Answers1