[damienbudzik]

int x, y;
scanf("%d", &x); //liczba wierszy
scanf("%d", &y); //liczba kolumn
int tab[x][y];

Jakim cudem to u mnie działa? Nie powinienem użyć operatora new albo malloca?
#naukaprogramowania #jezykc #c++

[Marcin648]

Nie powinienem użyć operatora new albo malloca?

@damienbudzik: Powinieneś. Albo jak c++ std::vector.

[sorhu]

@damienbudzik
Co to znaczy "nie działa"?
Nie kompiluje się?
Nie daje wyniku?
Daje zły wynik?
Komputer pokopał Cię prądem? Stanął w płomieniach?

[damienbudzik]

@sorhu: Problem w tym, że działa, a wg mnie takie coś działać nie powinno.

[o.....j]

Bo "int x" przydziela zmiennej x konkretne miejsce na dysku, zmienna ta ma po prostu przypadkową wartość.

[pilot1123]

na dysku

@okpaj: w pamięci

Nie powinienem użyć operatora new albo malloca?

@damienbudzik: Powinieneś. Albo jak c++ std::vector.

@Marcin648: nie pisałem nic w cpp od wieków, ale przeciez malloca używa się do "inicjalizacji" wskaźników, a nie zmiennych automatycznych na stosie

[Marcin648]

@damienbudzik: A kompilator nie wywala jakiś ostrzeżeń? Domyślam się że kompilator domyśla się co chcesz zrobić, lub coś doszło w nowych standardach co mnie ominęło. @KrzaQ2 sprostuj proszę jak jest inaczej.

[T.....y]

@damienbudzik: pewnie dev cpp zgadlem?

[KrzaQ2]

@Marcin648: @damienbudzik: To jest poprawny kod C, tablica o dynamicznie ustalanej wielkości to VLA (Variable-Length Array). W C++ tego nie ma, ale domyślnie gcc (i chyba clang) kompilują w pseudostandardzie gnu++, który takie rzeczy dodaje. Jak do linii wywołania dodasz -std=c++14 to się nie skompiluje.

[pilot1123]

int tab[x][y];

@damienbudzik: aa, tu był problem, przepraszam za zamieszanie

[Malchos]

@KrzaQ2: @pilot1123: Szczerze mówiąc to ja nie do końca jeszcze rozumiem dlaczego powyższe działa i się kompiluje i byłbym wdzięczny za rozjaśnienie.
Biorę najpierw oryginalny kod OPa i kompiluję go jako program C:
gcc prog.c -Wall -o prog
Jedyny warning jaki dostaję dotyczy nieużywanej zmiennej tab. Errorów brak.
Skąd kompilator wie ile pamięci zarezerwować dla tablicy int tab[x][y] skoro jej rozmiar na etapie kompilacji nie jest znany?

Ten sam program

[KrzaQ2]

@Malchos: Co to za kompilator?

Kompilator widzi, że nie zna wielkości tablicy, więc generuje kod dla tablicy o tablicy o dynamicznej wielkości. sizeof takiej tablicy nie jest np. stałą kompilacji.

Jeśli chodzi o C++, to, heh bez -pedantic-errors gcc nie daje błędów :/ (a z samym -pedantic jest warning).

[KrzaQ2]

@Malchos: Może to Ci więcej pomoże: https://godbolt.org/g/19Xbos

[Malchos]

Co to za kompilator?

@KrzaQ2: gcc version 5.4.0 20160609 na Ubuntu.

Czyli jak rozumiem kompilator traktuje powyższy fragment jakoby tam zachodziła dynamiczna alokacja pamięci, ale w żaden sposób o tym nie poinformuje?
W przypadku g++ flaga -pedantic-errors faktycznie załatwia sprawę, ale gcc przynajmniej w mojej wersji dalej nie widzi problemu

[KrzaQ2]

@Malchos: tak jak pisałem, to jest w pełni poprawny standardowy C, więc gcc nie powinien narzekać (ew, może z flagą -std=c89).

Standard C nie definiuje jak ma być alokowane VLA, więc raczej spodziewałbym się (i tak pokazuje fragment z compiler explorera) że będzie to alokacja na stosie, tak jak byś użył alloca

[Malchos]

@KrzaQ2: Ok, już rozumiem, jakoś wcześniej umknęła mi instancja VLA i zasada ich działania, stąd mój brak zrozumienia tego fragmentu kodu. No i weszło to w C99 stąd dla flagi -std=c89 jest błąd tak jak wspomniałeś.
Dzięki za czas i wyjaśnienie ;)

[Patryk_Kow1]

Może coś się przyda
https://webinaria.proidea.pl/webinaria/c/