0

我是 JavaScript 的初学者。

此代码应该评估后缀表达式。但它不起作用。我认为代码非常合乎逻辑，我真的不知道它有什么问题。

如果有人可以帮助我，我将不胜感激！

<html>
<head>
</head>

<body>
<!-- starting javaScript -->
<script type = "text/javascript">
    var postStr = "123++"; //initial postfix expression     
    var postArr =  new Array();
    postArr = postStr.split(""); //string.split() turns string into array!

    for(var i=0; i<postArr.length; i++)
        document.write("value"+postArr[i]+"<br>");

    var stack=[];
    var result;
    var firstNum;
    var secNum;
    //var k;
    for(var i=0; i<postArr.length; i++)
    {
        if((postArr[i]!="^")||(postArr[i]!= "+")||(postArr[i]!= "-")||      (postArr[i]!= "*")||(postArr[i]!= "/"))
        {
            stack.push(postArr[i]);
            //document.write("length" + stack.length);
        }

        else if((postArr[i]=="^")||(postArr[i]== "+")||(postArr[i]== "-")||(postArr[i]== "*")||(postArr[i]== "/"))
        { 
            if(postArr[i]=='+')
            {
                firstNum=stack.pop();
                secNum=stack.pop();
                result = secNum + firstNum;
                stack.push("result = " + result);
            }
            else if(postArr[i]=='*')
            {
                firstNum=stack.pop();
                secNum=stack.pop();
                result = secNum * firstNum;
                stack.push("result = " + result);
            }
            else if(postArr[i]=='/')
            {
                firstNum=stack.pop();
                secNum=stack.pop();
                result = secNum / firstNum;
                stack.push("result = " + result);
            }
            else if(postArr[i]=='-')
            {
                firstNum=stack.pop();
                secNum=stack.pop();                                        
                                    result = secNum - firstNum;
                stack.push("result = " + result);
            } 
        }
    }
    var finalRes=stack.pop();
    document.write(finalRes); 
</script>
</body>
</html>
4

2 回答 2

3

有一些错误：

  1. 您的第一个条件应由and( &&) 运算符分隔，而不是or因为您使用的是否定条件

  2. 因为您将字符串转换为数字，所以不要忘记将字符转换为整数（使用类似的东西+postArr[i]）

  3. 您应该推送结果而不是带有注释（stack.push(result)而不是stack.push("result = " + result)）的字符串。

请参阅此 jsfiddle：http: //jsfiddle.net/scaillerie/GwDTM/2/。

于 2012-12-17T16:50:48.573 回答
1

我不确定这是否是唯一的问题，但此代码的条件是：

if((postArr[i]!="^")||(postArr[i]!= "+")||(postArr[i]!= "-")|| (postArr[i]!= "*")|| postArr[i]!= "/"))

将始终评估为 true，因为 postArr[i] 将始终不是选项之一。你可能想要这样的东西：

var ch = postArr[i];

if (! ((ch == "^") || (ch == "+") || (ch == "-")…

于 2012-12-17T16:43:35.193 回答