实现评估简单数学表达式的函数。 （C ++）

Question

我正在尝试编写一个评估简单数学表达式的函数（只有四个操作）。我使用堆栈和向量来做到这一点。但堆栈操作并不像我预期的那样。我找不到原因。我愿意接受不同的解决方案。

该函数应该采用这样的字符串：

"5 * 44 + 3 / 2 * 4 - 12"

并将结果返回为double。

#include <iostream>
#include <vector>
#include <stack>
#include <string>
#include <cstdlib>

using namespace std;

vector<string> split(const std::string& str, char delim = ' ')
{
    vector<string> elements;
    stringstream ss(str);
    string token;
    while (getline(ss, token, delim)) {
        elements.push_back(token);
    }

    return elements;
}


double evaluate(string operation)
{
    vector<string> values = split(operation, ' ');
    stack<string> result_stack;
    double result = 0;

    for(unsigned int i = 0; i < values.size(); i++){
        if(values[i] == "*"){
            double mini_result = stod(result_stack.top()) * stod(values[i+1]);
            result_stack.pop();
            i++;
            result_stack.push(to_string(mini_result));
        }
        else if(values[i] == "/"){
            double mini_result = stod(result_stack.top()) / stod(values[i+1]);
            result_stack.pop();
            i++;
            result_stack.push(to_string(mini_result));
        }
        else{
            result_stack.push(values[i]);
        }
    }

    for(unsigned int i = 0; i<result_stack.size(); i++){
        if(result_stack.top() == "-"){
            result_stack.pop();
            result = stod(result_stack.top()) - result;
            result_stack.pop();
        }
        else if(result_stack.top() == "+"){
            result_stack.pop();
            result += stod(result_stack.top());
            result_stack.pop();
        }
        else{
            result += stod(result_stack.top());
            result_stack.pop();
        }
    }

    return result;

}




int main()
{

    cout<<evaluate("5 * 44 + 3 / 2 * 4 - 12");
}

在第二个for循环之前，result_stack中的值对于此示例应该是这样的。 “12 | - | 6 | + | 220”。返回值应为214。

但在第二个for循环之前，堆栈只包含“12 | - | 6”值。 “+”和“220”值不存在。一些额外的流行音乐出现，我不指望。

stack content should be like this for this example

Answer 1

您的减法操作的操作数被交换。这是因为当您处理+和-操作的堆栈时，您将从右到左解析方程式。

以你的例子，当在堆栈中找到-时，result是12。你弹出-，然后减去6，留下6结果，它应该是-6。

你应该用

result = stod(result_stack.top()) - result;

减法。

对于无效方程，代码中也没有错误检查。

Answer 2

这里回答：answer of the question

完成的代码在这里：

#include <iostream>
#include <vector>
#include <sstream>
#include <string>
#include <stack>

using namespace std;

string trim(const string& str)
{
    size_t first = str.find_first_not_of(' ');
    if (string::npos == first)
    {
        return str;
    }
    size_t last = str.find_last_not_of(' ');
    return str.substr(first, (last - first + 1));
}

vector<string> split(const std::string& str, char delim = ' ')
{
    vector<string> elements;
    stringstream ss(str);
    string token;
    while (getline(ss, token, delim)) {
        elements.push_back(token);
    }

    return elements;
}

double evaluate(string operation)
{
    vector<string> values = split(operation, ' ');
    vector<string> result_vector;
    double result = 0;

    for(unsigned int i = 0; i < values.size(); i++){
        if(values[i] == "*"){
            double mini_result = stod(result_vector.back()) * stod(values[i+1]);
            result_vector.pop_back();
            i++;
            result_vector.push_back(to_string(mini_result));
        }
        else if(values[i] == "/"){
            double mini_result = stod(result_vector.back()) / stod(values[i+1]);
            result_vector.pop_back();
            i++;
            result_vector.push_back(to_string(mini_result));
        }
        else{
            result_vector.push_back(values[i]);
        }
    }

    auto iterator = result_vector.begin();
    while(iterator != result_vector.end()){
        if(*iterator == "-"){
            iterator++;
            result -= stod(*iterator);
        }
        else if(*iterator == "+"){
            iterator++;
            result += stod(*iterator);
        }
        else{
            result += stod(*iterator);
        }
        iterator++;
    }
    return result;
}


int main()
{
    cout<<evaluate("5 * 44 + 3 / 2 * 4 - 12");
}