Kopiowanie wartości z jednej tablicy jednowymiarowej do drugiej

Question

Cześć,

Mam za zadanie skopiować z danej tablicy liczb całkowitych tab1 do nowej tablicy tab2 wyłącznie wartości dodatnie. Obie tablice mają być jednowymiarowe o rozmiarze równym 10. Elementy pierwszej tablicy mają być zadeklarowane, więc zapisałam tak:

int[] tab1 = { -5, -4, -3, -2, -1, 0, 1, 2, 3};
int[] tab2 = new int[10];

Dalej próbowałam metody Array.Copy ale nie działa. Teraz wpisałam pętlę for,

for (int i = 0; i < tab1.Length; i++)
{
 tab2[i] = tab1[i];
}

ale nadal nie wiem jak zadeklarować to, żeby w tab2 były tylko wartości z tab1 ale większe od 0.

Mogę prosić o naprowadzenie jak ruszyć z tym dalej?

Dziękuję :)

 

Comments

Dodać instrukcję warunkową if

---

wpisałam:

for (int i =1; i < tab1.Length; i++)
            {
                if (i > 0)
                {
                    Console.WriteLine(tab2[i]);
                }
            }

ale wychodzą same 0 :(

 

---

W warunku pętli for sprawdzasz tab1, a później starasz się wypisać tab2. Zmieniłbym to na jakąś jednolitą wersję. Poza tym w warunku if powinno być np. 

if (tab2[i] > 0)

zamiast 

if (i > 0)

 

Answer 1

Obie tablice mają być jednowymiarowe o rozmiarze równym 10

przedstawiona przez Ciebie tablica zawiera 9 elementów (czy tak ma być pomimo powyższej treści?)

//              1   2   3   4   5  6  7  8  9      
int[] tab1 = { -5, -4, -3, -2, -1, 0, 1, 2, 3 };

for (int i =1; i < tab1.Length; i++)
{
    if (i > 0)
    {
        Console.WriteLine(tab2[i]);
    }
 }

musisz sprawdzać wartość przypisaną w tablicy pod podanym indeksem

for (int i=0, j=0; i<tab1.Length; i++)
{
    if (tab1[i] > 0)
    {
        tab[j++] = tab1[i];
    }
 }

 

Comments

Dodałam 10 wartość, teraz jest:

int[] tab1 = { -5, -4, -3, -2, -1, 0, 1, 2, 3, 4};

i dopisałam sugerowaną pętlę, ale nadal nic nie wychodzi. Visual podkreśla część:

 int[] tab1 = { -5, -4, -3, -2, -1, 0, 1, 2, 3, 4};
            int[] tab2 = new int[10];
            for (int i =1; j=0; i<tab1.Length; i++)
            {
                if (tab1[i] > 0)
                {
                    tab2[j++] = tab1[i];
                
                }
            }

Jednak nie mam pojęcia jak to zrobić .. :(

---

Pętla działa, sprawdź: on-line.

Zera się pojawiają mimo wszystko w drugiej tablicy, bo gdy deklarujesz tablicę

int[] tab2 = new int[10];

z podaną wielkością, "automatycznie" do wszystkich jej elementów (indeksów) domyślnie jest wstawiana wartość 0 (w przypadku tablicy zawierającej intiger-y)

możesz pominąć te elementy tablicy, które mają wartość przypisaną 0

[ on-line ]

    for (int i=0; i<tab2.Length; i++)
    {
        if (tab2[i] == 0) continue;
        Console.Write(tab2[i] + " ");
    }

ale przy dużych tablicach to może okazać się nieefektywne.

---

Faktycznie, nie wpisałam wcześniej tej drugiej pętli for

Dziękuję za pomoc!

---

Jako ciekawostkę, bez użycia pętli

[ on-line ]  (użycie System.Linq)

    int[] tab1 = { -5, -4, -3, -2, -1, 0, 1, 2, 3, 4 };
    int[] tab2 = new int[10];

    tab2 = tab1.Where(e => e > 0).ToArray();
    Console.WriteLine(String.Join(" ", tab2));

[ on-line ]  (użycie Array.FindAll)

    tab2 = Array.FindAll(tab1, e => e > 0).ToArray();
    Console.WriteLine(String.Join(" ", tab2));

 

---

najlepiej posortować i zrobić odcięcie od pierwszej liczby dodatniej, w ten sposób ograniczamy się tylko do O(n) operacji + 1 operacja więc złożoność znacznie lepsza niż kwadratowa lub logarytmiczna.
Np malejąco i pozbywać się ogonka, lub normalnie jak wspomniałem z odcięciem ^^ też opcja.