Remove zero's (empty spaces) from arrays

Question

I've written a console application (.NET 5.0) in C# in Visual Studio which prints out all the even and odd numbers you give as input.

It works as intended but there is a problem that I will encounter in the future when making similar applications.

Whenever a number isn't odd the respective place of that array (Number_odd) will have a zero added to it. How do I stop that from happening? I currently filtered out all the zero's by not printing any zero.

The output currently looks like "odd: 5 9 7 1 3" with all the zero's filtered. Without filtering the output looks like "odd: 0 5 9 7 1 3 0 0 0"

If I (for instance) say that "0" is an odd number I cannot print it because it gets filtered. How do I fix that?

using System;

namespace ConsoleApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            int[] numbers = Array.ConvertAll(Console.ReadLine().Split(" "), Convert.ToInt32);
            int uitkomst = 0;
            int[] numbers_odd = new int[numbers.Length];
            int[] numbers_even = new int[numbers.Length];

            for (int repeat = 0; repeat < numbers.Length; repeat++)
            {
                uitkomst = numbers[repeat] % 2;
                //Console.WriteLine(uitkomst);

                if(uitkomst == 1) // ODD
                {
                    numbers_odd[repeat] = numbers[repeat];
                    //Console.WriteLine(numbers_odd[repeat]);
                }

                if (uitkomst == 0) // Even
                {
                    numbers_even[repeat] = numbers[repeat];
                    //Console.WriteLine(numbers_even[repeat]);
                }
            }

            Console.Write("even: ");
            foreach (int item in numbers_even)
            {
                if(item == 0)
                {

                }
                else
                {
                    Console.Write(item + " ");
                }
            }
            Console.WriteLine(" ");

            Console.Write("odd: ");
            foreach (int item in numbers_odd)
            {
                if (item == 0)
                {

                }
                else
                {
                    Console.Write(item + " ");
                }
            }

        }
    }
}

First recommendation: Use lists instead of arrays, that way you can add items to it without worrying about how many elements you need in the list. Then only add the numbers you need, and ignore all the zeros. — DavidG
– DavidG, Commented Nov 10, 2021 at 13:26
an int [] is initialized with zeros. You actually only sets some indexes, not all, that's why you have zeroes at some places — Cid
– Cid, Commented Nov 10, 2021 at 13:29

Jamiec · Accepted Answer · 2021-11-10 13:29:33Z

4

Make these two lines

int[] numbers_odd = new int[numbers.Length];
int[] numbers_even = new int[numbers.Length];

use List<int> instead

var numbers_odd = new List<int>();
var numbers_even = new List<int>();

and then Add values to the correct one

if(uitkomst == 1) // ODD
{
    numbers_odd.Add(numbers[repeat]);
    //Console.WriteLine(numbers_odd[repeat]);
 }

 if (uitkomst == 0) // Even
 {
     numbers_even.Add(numbers[repeat]);
     //Console.WriteLine(numbers_even[repeat]);
 }

Now your lists only contain the odd or even numbers and no zeros

Console.Write("even: ");
foreach (int item in numbers_even)
{
     Console.Write(item + " ");
}

answered Nov 10, 2021 at 13:29

Jamiec

136k15 gold badges142 silver badges202 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

Durga Prasad Over a year ago

Isn't a list<int> also a fixed array under the hood?

Jamiec Over a year ago

@DurgaPrasad yes, with the advantage that it will grow itself as needed saving you from having to do so manually.

Trevor Over a year ago

@DurgaPrasad referencesource.microsoft.com/#mscorlib/system/collections/… check out the source :)

Durga Prasad Over a year ago

Agreed. My concern was if OP uses list as mentioned in answer and iterates over capacity instead of count with a for loop isn't the problem still present?

Jamiec Over a year ago

@DurgaPrasad no, a for loop will iterate the actual items of the list not its underlying array.

Marc Gravell · Accepted Answer · 2021-11-10 13:30:41Z

2

It isn't that a zero is being added; an array is a fixed size, and is initialized with all default values - zero in this case. You then loop over numbers.Length and only write to one of the two arrays - leaving the other array zero at that position.

Perhaps consider using a list instead of an array for the two parts, and .Add(...) as you choose.

answered Nov 10, 2021 at 13:30

Marc Gravell

1.1m273 gold badges2.6k silver badges3k bronze badges

1 Comment

Mical - Over a year ago

Thanks, I don't know what lists are but I will look into it. Thank you all for your answers!

Mach · Accepted Answer · 2021-11-10 13:53:47Z

1

The answer posted is already a good solution. Just wanted to add that, If you are learning C# and working with collections (arrays, lists etc.) consider Linq as one of useful tools.

Here is how you can leverage them in your example.

var numbers = Array.ConvertAll(Console.ReadLine().Split(" "), Convert.ToInt32);

var numbers_odd = numbers.Where(n => (n % 2)==1);

var numbers_even = numbers.Where(n =>(n % 2) == 0);

answered Nov 10, 2021 at 13:53

Mach

563 bronze badges

1 Comment

Mical - Over a year ago

Im currently learning c#. so im trying to find multiple possibilities but thank you for the recommendation :)

Collectives™ on Stack Overflow

Remove zero's (empty spaces) from arrays

3 Answers 3

5 Comments

1 Comment

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

5 Comments

1 Comment

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related