Program calculates postfix expressions #12
Conversation
| { | ||
| this->data = 0; | ||
| this->next = nullptr; | ||
| } |
|
|
||
| struct Element | ||
| { | ||
| Element(int data); |
| struct Element | ||
| { | ||
| Element(int data); | ||
| ~Element(); |
| int fromPostfixToResult(string const & str) | ||
| { | ||
| Stack s; | ||
| int sizeOfStr = (int)str.size(); |
There was a problem hiding this comment.
Зачем каст к инту, она ж возвращает size_t небось, можно было for (size_t i = 0; i < sizeOfStr; ++i)
| } | ||
| } | ||
| int result = s.peek(); | ||
| return result; |
There was a problem hiding this comment.
Можно просто return s.peek();, хотя как у Вас отлаживаться удобнее.
| // ���������� | ||
| ~Stack(); | ||
| // ����������� ����������� | ||
|
|
| // ���������� �������� � ������ | ||
| void push(int data); | ||
| // �������� �������� �� ������ | ||
| int pop(); |
There was a problem hiding this comment.
Может, лучше bool возвращать, раз оно только 0 или -1?
| // ���������� �� �������� ������� | ||
| int peek(); | ||
| // ��������� �������� �� ���� ������ | ||
| bool isEmpty(); |
There was a problem hiding this comment.
bool isEmpty() const;, оно не меняет состояние объекта
| // �������� �������� �� ������ | ||
| int pop(); | ||
| // ���������� �� �������� ������� | ||
| int peek(); |
There was a problem hiding this comment.
int peek() const;, оно не меняет состояние объекта
| cout << "result is " << fromPostfixToResult(str) << endl; | ||
|
|
||
|
|
||
| system("pause"); |
| int fromPostfixToResult(string const & str) | ||
| { | ||
| Stack s; | ||
| size_t sizeOfStr = str.size(); |
There was a problem hiding this comment.
const? Тут вообще много где можно const написать, но тут это было бы особенно полезно, потому что область видимости у sizeOfStr очень большая
| } | ||
| else if (!isOperator(str[i])) | ||
| { | ||
| if (s.peek() == -1) |
There was a problem hiding this comment.
Ой-ой. Ввожу 23-, мне говорят, что я неправ :) Вообще, для сигнализации об ошибке никогда-никогда нельзя использовать допустимые значения. Если множество допустимых значений покрывает весь тип, возвращаемый методом, то тип приходится расширять (например, возвращать пару), либо, что, как правило, более идеологически верно, использовать исключения.
I used stack for implementation