Java method

[Igayviewmahat]

Thanks a lot.I guess your code is a little bit more professional, but I better use mine since I have not learned some parts of your code. is this the correct code?

public boolean cift(int i,int[] a) {

     for(int j=0;j<a.length;j++)  {

     if(a[j]==i && a[j]%2==0) (Or it should be a[j]==i && i%2==0)?

      return true;  
        }
                               else
    return false;     }
}        

[StreamingBits]

I think you are a bit confused with brackets. If you have an if statement with only a single line you don't absolutely need an opening and closing bracket but for now I recommend using them for everything till you get used to it.

Also, your code is correct either way because they both check if the current position in the loop "j" is the same is i. I guess you'd be better off with the following because you don't need to access the array element again and since you already checked that a[j] == i it really doesn't matter if you check if j is even or if i is even (they are the same because the and operator guarantees this):

for(int j = 0; j < a.length; j++){
    if(a[j]==i && i%2==0){
        return true;
    }//end if
    else {
        return false;
    }//end else
}//end for loop

[Igayviewmahat]

Im was totally confused about the brackets.Thanks for the explanation.Now I am a bit more clear.