Kalkulator - spoj

Question

Mam problem z zadaniem ze spoj:

http://pl.spoj.com/problems/CALC/

Moje rozwiązanie:

https://pastebin.com/tKjk7bfN

 

Wyskakuje blad: błąd wykonania (SIGABRT). Wyniki testowane i zdają się być ok....Ale już widzę, że to chyba chodzi o problem ze wskaznikami. Zgadza się? Nie mam pojęcia jak to naprawić...Dopiero zaczynam nauke C++ i wydaje mi się, że nie do końca tak to się mogło robić....Ktoś, coś? ;)

ps. wybaczcie, że cały kod, ale nie jetem pewna gdzie leży błąd

Answer 1

Daruj sobie te wskaźniki i alokację pamięci. Wejście i wyjście w spoju są oddzielone, więc nie trzeba wpisywać wyników do tablicy tylko po to, żeby je potem wypisać.

Co do kodu:

wynik = new int[i + 1];
wynik = tab_wynikow;

To jest kompletnie bez sensu. W pierwszej linii przypisujesz do zmiennej wynik obszar pamięci na (i+1) intów tylko po to, by w kolejnej linijce tą zmienną nadpisać i utracić dostęp do zaalokowanej pamięci. Inna sprawa, że ta zmienna nie jest nigdzie w kodzie używana.

Sam błąd wynika zapewne z wyjścia poza zakres tablicy. Zmienna tab_wynikow ma miejsce na jednego inta, więc już przy drugim obrocie pętli wychodzisz poza zakres.

W linijce 45 z kolei źle zwalniasz pamięć.

Comments

Okej....Usunelam alokacje, dałam zwyczajne tab_wynikow[200] tak jak mowiles i zaakceptowane.

Tylko tak się zastanawiam...Jak by to trzeba było zmienić, żeby zrobić z alokacją pamięci?

---

Do tego w ogóle nie trzeba żadnych tablic. Rozwiązanie tego powinno wyglądać tak:

while (std::cin >> znak >> a >> b) {
   std::cout << WykonajOperacje(znak, a, b) << '\n';
}

Oczywiście funkcję WykonajOperacje należy samemu zaimplementować.

---

Rozumiem...Tylko zwyczajnie z ciekawości dla wiedzy jak używać wskaźników pytam ;)

---

http://cpp0x.pl/kursy/Kurs-C++/Poziom-5/Zarzadzanie-pamiecia-new-delete/581

---

Przeglądałam już x razy, ale coś widać nie rozumiem. No nic, poczytam więcej, dzięki :)