Kalkulator oparty na stosie

Question

https://pastebin.com/NNuc2Uin

Wybaczcie nazwy funkcji i lekkie pomieszanie w nazwach zmiennych.

Program działa. Zastosowałam stos aby przyswoić sobie w praktyce wiedzę o strukturach danych.

Zależy mi na ocenie przejrzystości oraz wydajności kodu. Kod jest napisany w podejściu proceduralnym.

Pozdrawiam!

Answer 1

Korzystasz z STL`a , metode:
 

void wyczysc() ...

Możesz zamienić na to, aby Twój globalny stos , zamieniał elementy z jakimś pustym lokalnym stosem np.
 

void wyczysc()
{
    std::stack<double> emptyStack;
    emptyStack.push(0);
    wyniki.swap( emptyStack );
}

Taka luźna propozycja :). emptyStack przestaje istnieć po wyjściu z funkcji.

Edit: W Twoim programie można dzielić przez zero. W funkcji podziel nie masz sprawdzenia.

Comments

Dziękuje ;) tak poza tym wszystko ok z kodem ?

---

Zawsze może być lepiej :D,

np. możesz się pokusić o zrobienie z tych funkcji jednej klasy, zamiana zwykłych funkcji na szablony, zrobić aby funkcje przyjmowały stałe referencje, a nie jak masz teraz przyjmowania zmiennej przez wartość. Można by wymieniać, a jak już pisałem "zawsze może być lepiej" :)