試圖評估一個表達式

Question

我正在研究一個RPN計算器，並認爲我幾乎在那裏，除了它返回的表達式不是解決方案。

def evaluate(expression) 
    expression = expression.split 

    operators = expression.select { |v| v =~ /\W/} 
    operands = expression.select { |v| v =~ /\d/} 

    new_expression = operands.zip(operators) 
    eval = new_expression.join 
end 

這應返回-7：

puts evaluate('5 8 + 4 - 5 *') 
#=> 5+8-4*5 

Answer 1

eval = new_expression.join 

這就要求new_expression.join並將結果保存在一個名爲eval一個局部變量。因爲你永遠不使用局部變量，你可能也只是採寫：

new_expression.join 

如果這是你有意者來電Ruby的eval法的new_expression.join作爲其參數的結果，你應該刪除賦值運算符：

eval new_expression.join 

這就是說如果您只評估受信任的輸入，使用eval來評估表達式只是一個好主意。

Answer 2

您正在創建一個名爲eval一個局部變量，但你真的想打電話給Kernel#eval與表達作爲參數，像這樣：

eval operands.zip(operators).join 

Answer 3

這不正是你寫什麼，我。即它給變量eval提供了字符串。解決方案：

def evaluate(expression) 
    expression = expression.split 

    operators = expression.select { |v| v =~ /\W/} 
    operands = expression.select { |v| v =~ /\d/} 

    new_expression = operands.zip(operators) 
    eval new_expression.join 
end 

puts evaluate('5 8 + 4 - 5 *')