Zadanie SPOJ - Kalkulator 2

Question

Siemka, ostatnio próbowałem zrobić zadanie ze SPOJ o nazwie Kalkulator 2, zadanie proste ale gdy zadowolony oddałem sędziemu moje zadanie chamsko odrzucił je i dostałem komunikat o błędzie wykonania a w nim link: https://en.wikipedia.org/wiki/Segmentation_fault . A tu oto część mojego kodu: 

#include <iostream>

using namespace std;

int a, b, za[9];
string zn;

int main()
{
    while (cin>>zn>>a>>b)
    {
    if (zn=="z")
    {
    za[a]=b;
    }
    else if (zn=="+")
    {
    cout<<za[a]+za[b]<<endl;
    }

(Reszta to jest to samo co ten else if ale pozamieniałem symbole matematyczne).

Może ktoś jest bardziej into informatyka i wytłumaczy mi na czym polega mój błąd. Byłbym bardzo wdzięczny dla tego rycerza kodowania który pomoże mi rozwikłać problem ;)

Comments

oto część mojego kodu

W tej części, poza kiepskim formatowaniem, nie ma błędu, który mógłby powodować SF. Pokaż resztę kodu.

 

pr0tip: użyj switcha, a zamiast std::stringa użyj po prostu zmiennej typu char.

---

#include <iostream>

using namespace std;

int a, b, za[9];
string zn;

int main()
{
    while (cin>>zn>>a>>b)
    {
    if (zn=="z")
    {
    za[a]=b;
    }
    else if (zn=="+")
    {
    cout<<za[a]+za[b]<<endl;
    }
    else if (zn=="-")
    {
    cout<<za[a]-za[b]<<endl;
    }
    else if (zn=="*")
    {
    cout<<za[a]*za[b]<<endl;
    }
    else if (zn=="/")
    {
    cout<<za[a]/za[b]<<endl;
    }
    else if (zn=="%")
    {
    cout<<za[a]%za[b]<<endl;
    }
    }
    return 0;
}

A oto i kod.

Answer 1

W odpowiedzi na komentarz.

Błędu w kodzie nie masz, ale kod nie spełnia założeń zadania (wcześniej go nie czytałem). Rejestr, według tego, co przeczytałem, powinien mieć 10 pozycji, a u Ciebie, nie wiedzieć czemu, jest 9. To może powodować błędy związane z dostępem do pamięci.

 

Tu masz kod, który powinien działać:

int main()
{
	int a, b, result, reg[10];
	char zn;

	while (std::cin >> zn >> a >> b) {
		switch(zn) {
		case 'z': reg[a] = b; continue;
		case '+': result = reg[a] + reg[b]; break;
		case '-': result = reg[a] - reg[b]; break;
		case '*': result = reg[a] * reg[b]; break;
		case '/': result = reg[a] / reg[b]; break;
		case '%': result = reg[a] % reg[b]; break;
		default: continue;
		}

		std::cout << result << '\n';
	}
	
	return 0;
}