I call my swap method in main, but it doesn't change anything. What am I doing wrong?
public static void main(String[] args){
int mainArr[] = new int[20];
for(int i = 0; i<mainArr.length; i++){
swapper(3, 14, mainArr);
System.out.print(i + mainArr[i] + " ");
}
}
public static void swapper (int a, int b, int[] mainArr){
int t = mainArr[a];
mainArr[a] = mainArr[b];
mainArr[b] = t;
}
My code yields
0, 1, 2, 3,...19
in normal ascending order, where I want it to swap the 4th and 15th element.
Move the method call: -
swapper(3, 14, mainArr);
outside your for loop. Since, if your loop runs even
number of times, it will not affect
the array.
Also, you need to initialize your array first, before actually swapping the elements. That you would need to do before invoking swapper
.
for(int i = 0; i<mainArr.length; i++){
mainArr[i] = i;
}
swapper(3, 14, mainArr);
for(int i = 0; i<mainArr.length; i++){
System.out.print(i + mainArr[i] + " ");
}
Writing the code as so:
int mainArr[] = new int[20];
for(int i =0; i <mainArr.length;i++)
{
mainArr[i]=i;
}
swapper(3,14,mainArr);
will resolve the issue. The problem was you happened to be calling swap an even number of times, so it had a total effect of nothing.
You are calling swapper the same number of times as there are elements in your array.
- If the array has a even length, nothing will change
- If the array has a odd length, it will change
public class swapInt
{
public static void main(String args[])
{
swap(new int[]{2,3,5,6,8},1,3);
}
public static void swap(int[]a,int i,int j)
{
int temp=a[i];
a[i]= a[j];
a[j]=temp;
for(int b=0;b<a.length;b++)
{
System.out.println(a[b]);
}
}
}