Uniwersalna tablica [x] [y] - potrzebuję pomocy.

pytanie zadane 23 stycznia 2018 w C i C++ przez Szczurk3y Nowicjusz (200 p.)
edycja 23 stycznia 2018

Witam, potrzebuję pomocy - potrzebuję do mojego programu, który piszę, zrobić uniwersalną tablicę 2 wymiarową. Nie wiem co nie działa i nie wiem dlaczego, po podaniu wartości dla kolumny program się zawiesza i wyłącza. Z góry dziękuję za wyjaśnienie problemu :)

#include <iostream>
#include <Windows.h>
using namespace std;
int wiersz, kolumny;
int wspolzednaXweza, wspolzednaYweza;
int wspolzednaXjedzenia, wspolzednaYjedzenia;
int x,y;
int main() {
    cout << "Podaj dla wiersz: " << endl;
    cin >> wiersz;
    cout << "Podaj dla kolumny: " << endl;
    cin >> kolumny;
    char plansza [x] [y]; // gdy deklaruje char plansza  [x][y] na początku wyskakuje [Error] 'plansza' was not declared in this scope.
    for (int x=wiersz; x>=0; x--) {
        for (int y=kolumny; y>=0; y--) {
            plansza [x] [y] = 'p';  // p = pole puste
        }
    }
}

komentarz 23 stycznia 2018 przez niezalogowany

Na przyszłość: kod wrzucamy w przeznaczony do tego bloczek {...} code

komentarz 23 stycznia 2018 przez Szczurk3y Nowicjusz (200 p.)

okey

komentarz 24 stycznia 2018 przez j23 Mędrzec (195,220 p.)

Po co ten <windows.h>?

komentarz 24 stycznia 2018 przez Szczurk3y Nowicjusz (200 p.)

Usunąłem część kodu, gdzie potrzebowałem tej biblioteki. Zapomniałem jej samej usunąć :P Oczywiście, w tym pokazanym kodzie nic nie robi ;)

2 odpowiedzi

odpowiedź 23 stycznia 2018 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
wybrane 26 stycznia 2018 przez Szczurk3y

odpowiedź 23 stycznia 2018 przez niezalogowany

Z tego co się orientuję, to ten zapis nie jest do końca legalny wg standardu. W pełni poprawnie powinieneś zaalokować pamięć dynamicznie:

// <-wczytane x,y
char **plansza = new char*[x];
for(int i=0; i<x; ++i) {
  plansza[i] = new char[y];
}

// robisz z nią co chcesz
...

// usuwasz z pamięci
for(int i=0; i<x; ++i) {
  delete[] plansza[i];
}
delete[] plansza;