Zmiana tabeli jednowymiarowej na dwuwymiarową

Question

Jako zadanie miałem zmodyfikować program tworzący tabele jednowymiarową na dwuwymiarową. Muszę użyć funkcji i struktur. Niestety gdy wpiszę dane dla tab[1][0] i kliknę enter wywala mi cały program.

Oto kod:

#include <iostream>
using namespace std;


struct tablica
{
    int **dane;
    int rozmiar;
    int rozmiar1;
};

void inicjuj(tablica &tab, int wysokosc, int szerokosc)
{
    tab.rozmiar = wysokosc;
    tab.rozmiar1 = szerokosc;
    int **dane = new int *[tab.rozmiar];
    for(int i=0; i<tab.rozmiar; i++)
    {
        dane[i] = new int[tab.rozmiar1];
    }
}
void wypisz(const tablica &tab)
{
    if(tab.rozmiar < 1 && tab.rozmiar1 < 1)
    {
        cout << "Pusta tablica";
    }
    else
    {
        for (int i=0; i<tab.rozmiar; i++)
        {
            for (int j=0; j<tab.rozmiar1; j++)
            {
                cout << tab.dane[i][j] << " ";
        }
	}
    }
    cout << endl;
}

void wczytaj(tablica &tab)
{
    if(tab.rozmiar < 1 && tab.rozmiar1 < 1)
    {
        cout << "Pusta tablica";
    }
    else
    {
        for(int i=0; i<tab.rozmiar; i++)
        {
            for(int j=0; j<tab.rozmiar1; j++)
            {
                //cout<<"["<<i<<"]["<<j<<"]"<<endl;
                cout << "Podaj wartosc nr ["<< i+1  << "][" << j+1 << "]" << ": ";
                cin >> tab.dane[i][j];
            }
        }
    }
}

void zwolnij(tablica &tab)
{
    delete []tab.dane;
    tab.dane = NULL;
    tab.rozmiar = 0;
    tab.rozmiar1 = 0;
}

int main()
{
    tablica mojaTablica1;

    inicjuj(mojaTablica1, 2, 2);

    wczytaj(mojaTablica1);

    wypisz(mojaTablica1);

    zwolnij(mojaTablica1);

    wypisz(mojaTablica1);
}

 

Comments

tab.rozmiar = wysokosc;
tab.rozmiar1 = szerokosc;

Doskonałe nazewnictwo pól struktury. Tak jakby nie można było ich nazwać odpowiednio wysokosc i szerokosc.

void zwolnij(tablica &tab)
{
    delete []tab.dane;
    tab.dane = NULL;
    tab.rozmiar = 0;
    tab.rozmiar1 = 0;
}

To nie jest prawidłowe usuwanie. Tak powinno być:

void zwolnij(tablica &tab)
{
    for (int i = 0; i < tab.rozmiar; ++i) {
        delete[] tab.dane[i];
    }
        
    delete []tab.dane;
    tab.dane = NULL;
    tab.rozmiar = 0;
    tab.rozmiar1 = 0;
}

 

Answer 1

Problem polega na tym, że w linii 16, w funkcji inicjuj, deklarujesz nowy wskaźnik dane, zamiast przypisać zaalokowaną pamięć do tab.dane.

Pamiętaj, żeby zmienić też linię 19 i tam odwoływać się również do tab.dane, zamiast do dane.