Dynamiczne dwuwymiarowe tablice

Question

#include <iostream>
using namespace std;
int main()
{
    int n=0, m=0;
    const int x=2;
    int ** tablica= new int*[x];
        cout<<"Podaj m: ";
        cin>>m;
        cout<<"Podaj n: ";
        cin>>n;
    tablica[0]=new int[m];
    tablica[1]=new int[n];
    for(int i=0;i<m;i++)
    {
        for(int j=0;j<n;j++)
        {
            cout<<"Tablica["<<i<<"]["<<j<<"]=";
            cin>>tablica[i][j];
        }
    }
    for(int i=0;i<m;i++)
    {
        for(int j=0;j<n;j++)
        {
            cout<<tablica[i][j]<<"\t";
        }
        cout<<endl;
    }
    delete [] tablica[0];
    delete [] tablica[1];
    delete [] tablica;
    return 0;
}

Witam. Mój program w założeniu miał wypisywać macierz, a wartości w niej zawarte użytkownik może wpisać. Dlaczego w kodzie nie moge wpisać wartości do macierzy większej niz 2x2? Przy założeniu m=2 i n=2 program działa, ale gdy zwiększam liczbę wierszy, tzn. m=3, n=3 to niestety program się wysypuje, a ma to miejsce gdy program przechodzi do 3 wiersza. Co dziwne, gdy wartość m=2 a n=5 lub n=10 to spokojnie program działa. Mógłby mnie ktoś poratować podpowiedzią co jest nie tak w tym kodzie? Pozdrawiam i dziękuje z góry :)

Answer 1

 Zrób tak:

#include <iostream>
using namespace std;
int main()
{
    int n=0, m=0;

        cout<<"Podaj m: ";
        cin>>m;
        cout<<"Podaj n: ";
        cin>>n;

    int ** tablica = new int*[m]; 

    for(int i=0; i<m; i++)
    {
        tablica[i] = new int[n];
        for(int j=0; j<n; j++)
        {
            cout<<"Tablica["<<i<<"]["<<j<<"]=";
            cin>>tablica[i][j];
        }
    }
    for(int i=0; i<m; i++)
    {
        for(int j=0;j<n;j++)
        {
            cout<<tablica[i][j]<<"\t";
        }
        cout<<endl;
    }
    for(int i=0; i<m; i++) delete [] tablica[i];
    delete [] tablica;
    return 0;
}

Jest to poprawne tworzenie dwuwymiarowej tablicy.

Comments

Zapomniałeś poprawić kodu dotyczącego czyszczenia pamięci. Przy m = 1 program wycrashuje, a przy m>2 będziesz miał wyciek pamięci

---

Czyszczenie pamięci poprawiłem w momencie gdy napisałeś swoją odpowiedź.

Answer 2

W tym kodzie tworzysz dwie niezależne tablice jednowymiarowe i umieszczasz je w dwuelementowej tablicy.
 

Aby utworzyć macierz musisz utworzyć m-elementową tablicę tablic, a następnie w każdej komórce tej tablicy utworzyć tablicę n-elementową.
 

tablica = new int*[m];
for (int i=0; i<m; i++)
   tablica[i] = new int[n];

Nie zapomnij zwolnić całej pamięci po zakończeniu zabawy z macierzą