Question

我写了一个rpn，带有一个struktogram。

最新问题：现在它无法正常工作。

如果输入字符串是“5 +（（1 + 2）* 4） - 3”

我的输出是： 5 1 2 + 4 * 3 - +

我必须得到这个结果： 5 1 2 + 4 * + 3 -

编辑了来源

* 这是原来的问题，但帮助了我，现在修复了原来的错误：* ，

在调试时循环或 int i = 12，c值为0 \ 0或其他 并且这个值被添加到输出（名称：公式）字符串作为'（'括号。我不知道为什么。最后' - 操作符号，不添加到（或不查看）输出字符串（公式）的末尾我错误地将这个问题归咎于'（'。我用其他字符串输入值尝试了程序，但总是把'（'放到我的字符串中，我不知道为什么......我看到它与支架的数量无关。总是只有一个' （'添加到我的字符串...... *）是的，用英语LengyelFormula = rpn（匈牙利语）*

static void Main(string[] args)
    {
        String str = "5 + ( ( 1 + 2 ) *  4 ) −3";
        String result=LengyelFormaKonvertalas(str);
        Console.WriteLine(result.ToString());
        Console.ReadLine();
    }

    static String LengyelFormaKonvertalas(String input) // this is the rpn method
    {
       Stack stack = new Stack();
       String str = input.Replace(" ",string.Empty);
       StringBuilder formula = new StringBuilder();
       for (int i = 0; i < str.Length; i++)
       {
           char x=str[i];
           if (x == '(')
               stack.Push(x);
           else if (IsOperandus(x)) // is it operand
           {
               formula.Append(x);
           }
           else if (IsOperator(x))  // is it operation
           {
               if (stack.Count>0 && (char)stack.Peek()!='(' && Prior(x)<=Prior((char)stack.Peek()) )
               {
                   char y = (char)stack.Pop();
                   formula.Append(y);
               }
               if (stack.Count > 0 && (char)stack.Peek() != '(' && Prior(x) < Prior((char)stack.Peek()))
               {
                   char y = (char)stack.Pop();
                   formula.Append(y);
               }
               stack.Push(x);
           }
           else
           {
              char y=(char)stack.Pop();
              if (y!='(')
              {
                  formula.Append(y);
              }
           }
       }
       while (stack.Count>0)
       {
           char c = (char)stack.Pop();
           formula.Append(c);
       }
       return formula.ToString();
    }

    static bool IsOperator(char c)
    {
        return (c=='-'|| c=='+' || c=='*' || c=='/');
    }
    static bool IsOperandus(char c)
    {
        return (c>='0' && c<='9' || c=='.');
    }
    static int Prior(char c)
    {
        switch (c)
        {
            case '=':
                return 1;
            case '+':
                return 2;
            case '-':
                return 2;
            case '*':
                return 3;
            case '/':
                return 3;
            case '^':
                return 4;
            default:
                throw new ArgumentException("Rossz paraméter");                                          
        }
    }
}

Answer 1

using System;
using System.Collections.Generic;
using System.Text;

class Sample {
    static void Main(string[] args){
        String str = "5 + ( ( 1 + 2 ) *  4 ) -3";
        String result=LengyelFormaKonvertalas(str);
        Console.WriteLine(result);
        Console.ReadLine();
    }

    static String LengyelFormaKonvertalas(String input){
       Stack<char> stack = new Stack<char>();
       String str = input.Replace(" ", string.Empty);
       StringBuilder formula = new StringBuilder();
       for (int i = 0; i < str.Length; i++){
           char x=str[i];
           if (x == '(')
               stack.Push(x);
           else if (x == ')'){
               while(stack.Count>0 && stack.Peek() != '(')
                   formula.Append(stack.Pop());
               stack.Pop();
           } else if (IsOperandus(x)){
               formula.Append(x);
           } else if (IsOperator(x)) {
               while(stack.Count>0 && stack.Peek() != '(' && Prior(x)<=Prior(stack.Peek()) )
                   formula.Append(stack.Pop());
               stack.Push(x);
           }
           else {
              char y= stack.Pop();
              if (y!='(') 
                  formula.Append(y);
           }
       }
       while (stack.Count>0) {
           formula.Append(stack.Pop());
       }
       return formula.ToString();
    }

    static bool IsOperator(char c){
        return (c=='-'|| c=='+' || c=='*' || c=='/');
    }
    static bool IsOperandus(char c){
        return (c>='0' && c<='9' || c=='.');
    }
    static int Prior(char c){
        switch (c){
            case '=':
                return 1;
            case '+':
                return 2;
            case '-':
                return 2;
            case '*':
                return 3;
            case '/':
                return 3;
            case '^':
                return 4;
            default:
                throw new ArgumentException("Rossz parameter");                                          
        }
    }
}

Answer 2

在IsOperator中，你检查c ==' - ' 但在字符串中，你写-3。
- 与 - 是不同的人物我不知道波兰语的东西，所以也许我错过了一些东西，但这就是为什么没有' - '运算符被打印，它没有通过IsOperator检查并进入else子句，它不会将它添加到公式中。

Answer 3

当你得到)时，你应该弹出所有操作符并将它们添加到你的公式中，直到你到达(，然后弹出'（''。

当您获得运算符时，只应弹出堆栈并将此运算符添加到公式中，如果其优先级大于或等于x的优先级。您的第二次检查是多余的，因为它已被第一次检查覆盖。

作为一般规则：使用1+2+3，1+2-3，1*2+3和1+2*3之类的简单输入来尝试您的程序，看看您是否得到了正确的结果。像这样系统地测试应该可以帮助您更快地发现错误。

反向抛光符号C＃无法正常工作

3 个答案: