deletion in array .. wrong with code

Question

when i code this no error appears to me but when compile it >> give me error >> what is the wrong with that ??

public static void main (String args []){
    int arr[] = {20 , 30 ,40 , 50 , 60 ,200 };

    int searchKey = 20 ; 


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

        if (arr[i]== searchKey){
            System.out.println(searchKey + "found");
            for(int z = i ; z < arr.length ;z++){
                arr[z] = arr[z+1];
            }
            for(int k=0 ; k<arr.length ; k++)
                System.out.print(arr[k] + " ");

            break;
        }
        else if (i ==  (arr.length-1) ) {
             System.out.print(searchKey+  " not found");
             break;
        }
    }

}

There is no compiler error just an IndexOutOfBoundException at runtime. — SomeJavaGuy, Jan 05 '16 at 08:38
it give me logic error but in this line : arr[z] = arr[z+1]; — ÕMãř Ghazy, Jan 05 '16 at 08:39
Possible duplicate of [What causes a java.lang.ArrayIndexOutOfBoundsException and how do I prevent it?](http://stackoverflow.com/questions/5554734/what-causes-a-java-lang-arrayindexoutofboundsexception-and-how-do-i-prevent-it) — YoungHobbit, Jan 05 '16 at 08:40
You can't delete an array element, If you want to delete an element, then it's better to use ArrayList instead of the array..... This way definitely you will get an ArrayIndexOutOfBoundsException — ManKeer, Jan 05 '16 at 08:47

Abdelhak · Accepted Answer · 2016-01-05T08:48:07.883

0

With this for loop you go in java.lang.ArrayIndexOutOfBoundsException. Replace

for(int z = i ; z < arr.length ;z++)

With arr.length-1 instead arr.length:

for(int z = i ; z < arr.length-1 ;z++)

edited Jan 05 '16 at 08:48

answered Jan 05 '16 at 08:39

Abdelhak

8,299
4
22
36

score 0 · Answer 2 · edited Jan 05 '16 at 08:40

0

I can see that you would get an out of bounds exception with

for(int z = i ; z < arr.length ;z++) {
    arr[z] = arr[z+1];
}

try reducing the looping by

for(int z = i ; z < arr.length -1 ;z++)

edited Jan 05 '16 at 08:40

Maroun

94,125
30
188
241

answered Jan 05 '16 at 08:39

Scary Wombat

44,617
6
35
64

deletion in array .. wrong with code

2 Answers2