[gdziemojimbuspiatka]

siemanko mireczki z #programowanie #cpp #informatyka
mam pytanie. potrzebuję utworzyć dynamiczną tablicę o trzech kolumnach.

A = new int* [n];

A[i] = new int[3];

potem chcę wczytać ze strumienia argumenty(datę), która jest w formacie yyyy-mm-dd
Więc zrobiłem prostą pętlę do wczytywania daty do tablicy:
for (int j = 0;j < n;j++) {
std::cin >> A[j][0] >> A[j][1] >> A[j][2];
A[j][1] *= -1;
A[j][2] *= -1;
}
i wywala mi błąd ze zdjęcia. No i nie wiem gdzie robię błąd. Nie wiem czy tak się w ogóle powinna tą tablicę deklarować. any ideas?

[SpinOff]

@gdziemojimbuspiatka: Co to jest n, i jak dokładnie deklarujesz tablicę? Pokaż cały kod.

[koostosh]

@gdziemojimbuspiatka: na moje oko źle tworzysz tablicę, ale trzebaby zobaczyć cały kod

[GandalfPl]

@gdziemojimbuspiatka: Pierwsza rzecz, ktora musisz zrobic, to przestawic ten szaj na jezyk angielski. Po to, zebys mogl sobie komunikat o bledzie wkleic w googla i sie dowiedziec, o co chodzi i jak to naprawic.

Poza tym, to nie ma prawa dzialac co napisales :D

[q.....n]

@gdziemojimbuspiatka: Włącz sobie adress sanitizer, powinien dawać o wiele bardziej szczegółowe informacje

[lonely_wolf_]

@gdziemojimbuspiatka:

CDCDCDCD Used by Microsoft's C/C++ debug malloc() function to mark uninitialized heap memory, usually returned from HeapAlloc()

źródło: https://en.wikipedia.org/wiki/Magic_number_%28programming%29

Zakładam, że wyszedłeś poza zasięg tablicy pierwszego wymiaru, ewentualnie drugiego.

[gdziemojimbuspiatka]

@SpinOff: @koostosh: @GandalfPl: @qarmin: @lonely_wolf_: źle tworzyłem tablicę. Przy każdym wywołaniu kolejnego wiersza powinienem wrzucić A[j] = new int[3]; I wtedy będzie działało, ale potem mam problemy przy funkcjach, więc olałem tablice dynamiczne i zrobiłem to po prostacku. Czyli tablicą statyczną. Jako, że w zadaniu mam informacje, że dat będzie max 100, to zrobiłem tablicę 100x3.
Zadanie polega na posortowaniu Z ilości dat. ze

[GandalfPl]

@gdziemojimbuspiatka: access violation (jak zgaduje) oznacza, ze znowu piszesz/czytasz niezainicjowana pamiec.

[Kaczus2B]

@gdziemojimbuspiatka: jak nie potrzebujesz takich tablic, to uzyj std::vector, mniejsze p-stwo pomylki.

[gdziemojimbuspiatka]

@GandalfPl: Nie widzę co tam może wyjść poza pamięć. ;/
@Kaczus2B: Nie ogarniam dobrze vectora, a zadanie mam do północy

[Kaczus2B]

@gdziemojimbuspiatka: w tej 24 linijce podejrzewam, ze tab[i][x] moze miec np wartosc ujemna? sprawdz.

[gdziemojimbuspiatka]

@Kaczus2B: No nie powinna mieć. to jest data, więc nie powinien miec nawet zerowych... A ponad zakres wpisanych dat nie wyjdzie bo ogranicza go rozmiar.

[Kaczus2B]

@gdziemojimbuspiatka: wypisz sobie, bo ja nie wiem co tam jest, to, ze Tobie wydaje sie, ze jest data, to jedno, ale wypisz sobie wartosc tab[i][x] jak i licz[tab[i][x]], zobacz czy nie wychodza poza zakres, a jak wychodza, zanalizuj czemu.

[koostosh]

@gdziemojimbuspiatka: cokolwiek próbujesz zrobić robisz to źle.

licz= new int[max] a potem robisz licz[tab[...]], oczywiście że wyjdzie poza zakres.
for w linijce 21 nie powinien być od zera.
gdyby nawet sortowanie działało, to sortowałbyś osobno lata osobno dni i osobno miesiące.

[gdziemojimbuspiatka]

@koostosh: fora poprawiłem, to co napisałeś niżej obliczę, a sortuje najpierw dni potem miesiace potem lata. i to stabilnie, więc dlaczego miałoby nie wyjść?

[koostosh]

@gdziemojimbuspiatka: a dobra, nie zauważyłem że robisz to od końca. Wciąż po posortowaniu nic nie robisz z wynikiem, wychodzisz z funkcji sortującej nie zmieniając tej tablicy

[gdziemojimbuspiatka]

@koostosh: już dodałem, ale teraz wywali mi błąd na 27 linijce na licz--; i nie wiem dlaczego, bo ustawiłem licz na max+1;
https://pastebin.com/tR4BPR70