Problem with arrays

**coorscollector** · January 21st, 2011, 02:00 AM

I am a beginner and my teacher assigned us a problem where we are given an array named arr. We are supposed to create a new array that only has the odd values from arr. This is my code:

public int [] youMakeMeOdd(int [] arr)
{
  int[] ans = new int[arr.length];
  for (int i = 0; i<arr.length; i++)
  {
      if(arr[i] % 2 != 0)
      {
            ans[i] = arr[i];       
      }
  }
  return ans;
}

However, say for example that arr had only even integers to begin with. My new array will be filled with 0's.
But it is supposed to not have the zeros.

Say arr was {2,4,5,6}. My new array should be {5}. Instead it is{0,0,5,0}. I need a way to get rid of the zeros. Any help would be appreciated.

**JavaHater** · January 21st, 2011, 02:25 AM

you can make a count of how many odd integers there are first. After that you can initialize the ans array with that count size. Then go through the array again and store the odd number. Or you can use ArrayList if your teacher doesn't forbid.

**coorscollector** · January 21st, 2011, 02:38 AM

ArrayLists are forbidden so that's a no go. This might sound a little dumb, but how do I count the odd integers?

**DanBrown** · January 21st, 2011, 03:50 AM

First i would like to tell problem with your code ,see all the elements of the array are initalized with value o
when you define it.
Now you are adding lement whenever any odd no found so it could be in any index position within your array so

ans[i] = arr[i];

this line is storing odd no at original index as in the input array.

public int [] youMakeMeOdd(int [] arr)
{
      int[] ans = new int[arr.length];
     int k=0;
 
      for (int i = 0; i<arr.length; i++)
     {
             if(arr[i] % 2 != 0)
            {
                    ans[k] = arr[i]; 
                    k++;
 
            }
      }
return ans;
}

Say arr was {2,4,5,6}. My new array should be {5}. Instead it is{0,0,5,0}. I need a way to get rid of the zeros. Any help would be appreciated.

In the modified code above
Output for this input will be {5,0,0,0}
now you can display elements upto first zero found.

for(int i=0;i<ans.length;i++){
System.out.println(ans[i]);
if(ans[i] == 0)
break;
}

Thread: Problem with arrays

LinkBack

Thread Tools

Display

Problem with arrays

Related threads:

Re: Problem with arrays

Re: Problem with arrays

Re: Problem with arrays

Similar Threads

2d arrays help

arrays

2 D arrays problem

Arrays

2d Arrays