[C] Sumowanie zawartości kolumn tablicy dwuwymiarowej

Question

Trywialne zadania, lecz głowię się nad rozwiązaniem gdyż wyjście jest dosyć specyficzne :)

Do funkcji przekazuję wypełnioną tablicę dwuwymiarową, funkcja sumuje zawartość kolumn i wynik zapisuje w nowej tablicy jednowymiarowej.

// TAB_SIZE 2
// size - rozmiar tablicy, w tym przypadku TAB_SIZE

void sum_columns(int tab[][TAB_SIZE], int tab_sum[], int size)
{
	int i, j;

	for (i = 0; i < size; i++)
	{
		for (j = 0; j < size; j++)
		{
			tab_sum[i] += tab[i][j];
		}
	}
}

Takie dostaje wyjście:

--- wypisanie 2d ---
tab[0][0] = 4
tab[0][1] = 4

tab[1][0] = 2
tab[1][1] = 9

--- wypisanie 1d ---
tab[0] = 2280752
tab[1] = 1627419947

 

Czy funkcja jest dobrze napisana?

Answer 1

// sumuje kolumny i zapisuje w nowej tablicy jednowymiarowej
void sum_cols(int tab[][TAB_SIZE], int tab_sum[], int size)
{
	int i, j;

	for (j = 0; j < size; j++)
	{
		for (i = 0; i < size; i++)
		{
			tab_sum[j] += tab[i][j];
		}
	}
}

int col_tab[TAB_SIZE] = {0};

Nie zainicjalizowałem tablicy :)

Answer 2

A sumuje dobrze?

Bo to co wypisuje Ci pod tym: "--- wypisanie 1d ---" to są najprawdopodobniej adresy na pierwsze elementy tablic jednowymiarowych, które składają się na tablicę dwuwymiarową. Bo jak wiadomo tablica dwuwymiarowa to tablica tablic, więc wypisanie tylko jedno wymiaru daje adresy poszczególnych tablic tablicy.

Czyli, analogicznie jak masz:

int tablica[3] = { 1, 2, 3};
cout << tablica[0] << endl; // wypisuje wartość pierwszego elementu czyli 1
cout << *tablica << endl; // również wypisuje wartość pierwszego elementu, ponieważ tablice to są tak naprawdę wskaźniki
cout << tablica << ednl; // wypisuje adres pierwszego elementu
cout << *(tablica+1) << endl // wypisuje wartość drugiego elementu czyli 2
cout << (tablica+1) << endl; //adres drugiego elementu
cout << &tablica[1] << endl; // również adres drugiego elementu

Analogicznie działa to dla tablic wielowymiarowych.

Starałem się w miarę jasno pisać, jakby coś było nie jasne to pisz :)

Comments

Czysty C, więc zapis z C++ odpada (w tytule specjalnie dopisałem, że chodzi o C), ale mniejsza o to gdyż rozwiązałem problem.

Zawartość tablicy jednowymiarowej nie została zainicjalizowana zerami, stąd problem.

Answer 3

Funkcja jest prawie dobrze napisana, chociaż zapomniałeś o jednej bardzo ważnej rzeczy. Ile wynosi zawartość komórek tablicy wyjściowej przed rozpoczęciem iteracji?

Z innych spraw - ja bym odrzucił parametr wyjściowy na poczet zwracanej wartości, zaś typ parametru size zmienił na size_t... lub w ogóle zrezygnowałbym z gołych tablic, w zamian używając jakiegoś kontenera.

Comments

Zawartość tablicy jednowymiarowej nie została zainicjalizowana zerami, stąd problem. Wpadłem na to dokładnie wczoraj kilka chwil po tym jak położyłem się spać :)

O co chodzi z kontenerem? Wiesz, ja dopiero na bardzo wczesnym etapie programowania jestem :)

---

Poszukaj informacji o std::vector. To jeden z kontenerów, bardzo popularny. Mówiąc prostacko: taka wygodna tablica o zmiennej ilości elementów.