Jak skrócić tego ifa?

Question

class Program
    {
        static void Main(string[] args)
        {

            int[] a;
            int[] b;

            a = new int[] { -2, -1, 0, 1, 4 };
            b = new int[] { -3, -2, -1, 1, 2, 3 };
            for (int i = 0; i < a.Length; i++)
            {
                if (b[0] != a[i] && b[1] != a[i] && b[2] != a[i] && b[3] != a[i] && b[4] != a[i])
                {
                    Console.Write(a[i] + " ");
                }
                else if (a[i] == 0)

                {
                    Console.WriteLine("empty");
                    break;
                }
            }
        }

Domyślam się że jest potrzebna druga pętla, ale nie umiem tego zrobić :c

Answer 1

A może z użyciem Linq ? Pamiętaj żeby na początku pliku dodać:

using System.Linq;

static void Main()
{
    var a = new int[] { -2, -1, 0, 1, 4 };
    var b = new int[] { -3, -2, -1, 1, 2, 3 };

    for (int i = 0; i < a.Length; i++)
    {
        if (!b.Contains(a[i]))
        {
            Console.Write(a[i] + " ");
        }
        else if (a[i] == 0)
        {
            Console.WriteLine("empty");
            break;
        }
    }
}

W zasadzie to z użyciem Linq przy typach prostych możesz użyć metody Except żeby pozbyć się powtarzających się elementów

static void Main()
{
    var a = new int[] { -2, -1, 0, 1, 4 };
    var b = new int[] { -3, -2, -1, 1, 2, 3 };

    int[] c = a.Except(b).ToArray();

    foreach (var i in c)
    {
        Console.Write(i + " ");
    }
}

Tutaj jeszcze sposób bez Linq, z użyciem drugiej pętli

static void Main(string[] args)
{
    var a = new int[] { -2, -1, 0, 1, 4 };
    var b = new int[] { -3, -2, -1, 1, 2, 3 };

    for (int i = 0; i < a.Length; i++)
    {
        bool isInArrayA = false;

        for (int j = 0; j < b.Length; j++)
        {
            if (a[i] == b[j])
            {
                isInArrayA = true;
                break;
            }
        }

        if (!isInArrayA)
        {
            Console.Write(a[i] + " ");
        } 
        else if(a[i] == 0)
        {
            Console.WriteLine("empty");
            break;
        }
    }
}

 

Comments

Dziękuję <3

---

int[] c = a.Except(b).ToArray();
 
foreach (var i in c)
{
    Console.Write(i + " ");
}

Po co ten ToArray skoro tylko raz po tym iterujesz?

---

BTW,  

foreach (var i in c)
{
    Console.Write(i + " ");
}

// na

Console.Write(String.Join(" ", c));

 

---

@marcin99b, Masz rację, nie jest tutaj w ogóle potrzebny. Zrobiłem to wyłącznie po to żeby typ zgadzał się ze zmiennymi a oraz b. Oczywiście, najlepiej operować na interfejsie IEnumerable 

---

Oczywiście, najlepiej operować na interfejsie IEnumerable 

 nie najlepiej, bo jak sie nie wie co sie robi, to bardzo łatwo można nim rozwalić wydajność 

ale w tym przypadku, jak i tak już masz to ienumerable i chcesz po nim przelecieć tylko raz... to nie widze sensu żeby robić to, a później dodatkowo przelatywać po tych samych danych (po tablicy)

lepiej zrobić mniej akcji niż więcej, jeśli efekt będzie taki sam

Answer 2

Druga pętla w pętli for i jeden warunek sprawdzający.

Comments

A jak wyglądałby ten warunek, albo wypisywanie?

Próbowałam zrobić tak:

 for (int i = 0; i < a.Length; i++)
            {
                for (int j = 0; j < b.Length; j++)
                {
                    if (a[i] != b[j])
                    {
                        Console.WriteLine(a[i]);
                    }
                    else if (a[i] == 0)

                    {
                        Console.WriteLine("empty");
                        break;
                    }
                }

Ale to jest bezsensu, bo pętla wypisuje mi po kolei wszystkie porównania, a nie te liczby, które nie występują w drugiej tablicy :c