C# question to get a new array

Hi! I noticed a couple of issues in your code. Here are some suggestions to fix them:

1. The size of the new array should be the minimum of the two input arrays, since the common elements cannot exceed that size. You can find the minimum using Math.Min(one.Length, two.Length).

2. In the inner loop, you need to compare one[i] with two[k], not two[i]. This is because i is already used to index one, and using it to index two would not compare the elements pairwise.

3. The Console.WriteLine statement is incorrect, because you are trying to print an array instead of its elements. You can use a loop to print each element, or use the string.Join method to concatenate the elements with a delimiter.

Here is the corrected code:

int[] one = new int[] { 4, 5, 6, 7 };
int[] two = new int[] { 4, 5, 7, 8 };

static int[] twoArrays(int[] one, int[] two)
{
    int size = Math.Min(one.Length, two.Length);
    int[] newArray = new int[size];
    int x = 0;

    for (int i = 0; i < one.Length; i++)
    {
        for (int k = 0; k < two.Length; k++)
        {
            if (one[i] == two[k])
            {
                newArray[x] = one[i];
                x++;
                break; // no need to check further
            }
        }
    }

    return newArray;
}

int[] result = twoArrays(one, two);
Console.WriteLine(string.Join(", ", result));

This should output: 4, 5, 7