Using foreach to get the even numbers out of array c#

user_12985783 0
Viewed 105

I have this homework assignment where I have array called myArr with values 4, 9, 2, 5, 1, 4, 8, 3, 3 and I have to extract even numbers out of there and place them in myOtherArray and then printing them with foreach. I tried doing the code but I kinda got stuck since I know it won't work.

Here is my attempt:

int[] myArr = { 4, 9, 2, 5, 1, 4, 8, 3, 3 };

int num = 0;

int[] myOtherArr = new int[num];

for (int i = 0; i < myArr.Length; i++)
{
    if (myArr[i] % 2 == 0)
    {
        num++;
    
    }
}

Num is the counter to count how many even numbers are there, but right now I don't know how to save them into myOtherArray because I get the error that Index was outside the bounds of the array.

for (int i = 0; i < myArr.Length; i++)
{
     if (myArr[i] % 2 == 0)
     {
          myArr[i] = myOtherArr[i];

     }
}

Right now I don't know how to do this. Please help I want to be ready for my exam comming soon.

Thank you all in forward for your answers.

c#
4 Answers

you can also try using Array.FindAll() built-in method to achieve this , instead of looping construct. Please try below code.

var myArr = new int[] { 4, 9, 2, 5, 1, 4, 8, 3, 3 };
var myOtherArr = Array.FindAll(myArr, v => v%2 == 0);

Thank you.

user_67951250

The problem is here:

int[] myOtherArr = new int[num]; // you initialize the array to 0 length.

Try this:

int[] myArr = { 4, 9, 2, 5, 1, 4, 8, 3, 3 };

int num = 0;



for (int i = 0; i < myArr.Length; i++)
{
    if (myArr[i] % 2 == 0)
    {
        num++;
    
    }
}

int[] myOtherArr = new int[num];
int j = 0;
for (int i = 0; i < myArr.Length; i++)
{
     if (myArr[i] % 2 == 0)
     {
          myOtherArr[j] = myArr[I];
          j++;
     }
}
user_121966640

I did it this way because you don't have to figure out the length of the array.

int[] myArr = { 4, 9, 2, 5, 1, 4, 8, 3, 3 };
int[] myOtherArr;

List<int> cache = new List<int>();

//goes through each number in this array and stores the ones for that round in "num".
//If the number is "% 2 == 0" then it will be added to a list.
foreach (int num in myArr)
{
    if (num % 2 == 0)
    {
        cache.Add(num);
    }
}


//the list is converted and stored in your array
myOtherArr = cache.ToArray();

//all numbers are displayed
foreach (int num in myOtherArr)
{
    Console.WriteLine(num);
}

but you can also write it like this (I like to do this, for example).

int[] myArr = { 4, 9, 2, 5, 1, 4, 8, 3, 3 };
int[] myOtherArr;

List<int> cache = new List<int>();

//goes through each number in this array and stores the ones for that round in "num".
//If the number is "% 2 == 0" then it will be added to a list.
foreach (int num in myArr)
    if (num % 2 == 0)
        cache.Add(num);

//the list is converted and stored in your array
myOtherArr = cache.ToArray();

//all numbers are displayed
foreach (int num in myOtherArr)
    Console.WriteLine(num);
user_171584720

I would try something like this

    int[] myArr = { 4, 9, 2, 5, 1, 4, 8, 3, 3 };
    
    var i=0;
    foreach (var item in myArr)
        if (item % 2 == 0) i++;
    
    var myOtherArr= new int[i];
    i=0;
    foreach (var item in myArr)
        if (item % 2 == 0) { myOtherArr[i] = item; i++; }
user_113922900
Related