11

これは、シャンティング ヤード アルゴリズムを使用した私の式パーサーです。1 つの状況を除いて、期待どおりに機能します。-2*3 のように単項マイナスを使用すると機能しません (アルゴリズムで何も見つからなかったので、そうすべきではないと思います)これを処理するには）これを修正できる簡単な方法はありますか？(これは () + - * / ^ のみが必要な単純なパーサーです) よろしく Pedram

#include <cctype>
#include <iostream>
#include <cstring>
#include <cstdlib>
#include <cmath>
using namespace std;
int olaviat (char c) {
   /*************
    **Operator precedence 
    *************/
  switch(c) {
       case '-' : case '+' :
           return 1 ;
       case '*' : case '/' :
           return 2 ;
       case '^' :
           return 3 ;
       default :
           return 0 ;
  }
}
double eval(char *exp) {
    /*************
    **Convert to reverse polish
    *************/
    char n [50] , o[50] ;
    static int nl = 0  , ol = 0 ;

    while (*exp) {
            while(isspace(*exp)) *exp++ ;
        if(*exp == '(') {
             o[ol++]  = *exp++ ;
           }
        else if (*exp == ')'){
            while(o[--ol]!='('){
                    n[nl++] = o[ol];
                    n[nl++] = ' ';
                  }
                  *exp++;
        }
        else if (isdigit(*exp)) {
          while (isdigit(*exp)) {
            n[nl++] = *exp++ ;
          }
        n[nl++] = ' ' ;
        }
        else if (strchr("+-*/^",*exp)){
            if(olaviat(*exp) > olaviat(o[ol-1])) {
               o[ol++]  = *exp++ ;


            }
            else {
                    if(olaviat(*exp) == olaviat(o[ol-1]) && olaviat(*exp)== 3) {
                      o[ol++]  = *exp++ ;
                    }else{
                n[nl++] = o[ol-1] ;
                n[nl++] = ' ' ;
                o[--ol] = '\0' ;

                    }
            }
        }

    }

for (int k = ol-1 ; k >= 0 ; k --){
    n[nl++] = o[k];
    n[nl++] = ' ' ;
}
/*******************************/
cout << "Reverse Polish" << endl ;
for (int i = 0 ; i < nl-1 ; i++){
        cout << n[i]  ;
    }
cout << endl ;
//n[nl+1] = '\0' ;
/*******************************
**Calculate Result
*******************************/
    double temp[50];
    char *e ;
    ol = 0;
   int  nol = 0 ;
    e=n ;
    int digitcount = 0;
    while (*e) {
            while (isspace(*e)) *e++;
        if (isdigit(*e)) {
          while (isdigit(*e)) {
             o[ol++] =*e++ ;
             digitcount++ ;
          }
        temp[nol++] = atof(o) ;
        for (int i = 0 ; i < digitcount ; i++)
            o[i]='\0' ;
        ol=0;
        digitcount = 0 ;
        }
        else if (strchr("+-*/^",*e)){
          // char opr ;
           double tempAns = 0;
           switch (*e) {
              case '+' :
                  tempAns = temp[nol-2] + temp [nol-1] ;
                  break ;
              case '-' :
                  tempAns = temp [nol-2] - temp [nol-1] ;
                  break;
              case '*' :
                  tempAns = temp [nol-2] * temp [nol-1] ;
                  break;
              case '/' :
                  tempAns = temp[nol-2] / temp[nol-1];
                  break ;
              case '^' :
                  tempAns = pow(temp[nol-2],temp [nol-1]);
                  break ;
              default :
                cout << "\n Unknown error" ;
                continue;
           }
           *e++ ;
           nol--;
           temp[nol-1] = tempAns ;
           temp[nol] = NULL ;
        }
        else {
            break ;
        }
    }
    double ans = temp[0];

  return ans ;
}

int main() {

char exp[100];
char c;
start :
    cin.get (exp , 99);
    cout << "\n\tANS= " << eval(exp)  ;
    cout << endl ;
    system("PAUSE");
    return 0;
}
4

2 に答える 2

12

上記のオプションは正しいですが、非常に面倒でバグが多くなります。ケースを考えてみましょう2*-(1+2)^-(2+5*-(2+4))。ご覧のとおり、多くのことを考慮する必要があります。また*-(、たとえば、が見つかった場合はいつでも*(0-(...、面倒な再帰関数でコード化される でそれを置き換えることを知っています。

最善の解決策ははるかに簡単です。-演算子を解析するときは、演算子が存在し、その演算子の前に別の演算子がある場合、左括弧が前にある場合、または入力の最初の文字である場合を考慮してください (これらの場合は、単項マイナスではなく単項マイナスであることを意味します)。バイナリより)。この場合、たとえばu（これは私の場合でした）別の文字に変更し、その優先順位を の優先順位と同じにし^ます。

また、数値リテラルの一部として扱うことには問題があります。などのケースを想像してください-2^4。Wolfram Alpha-16では、 ではなくになり16ます。

また、スタックの使用を検討してください。彼らはあなたの人生を楽にします。

私が何を意味したかを説明させてください。入力が与えられていると考えてください：

2 / - 7 + ( - 9 * 8 ) * 2 ^ - 9 - 5

私が提案した置換を行うと、次のようになります。

2 / u 7 + ( u 9 * 8 ) * 2 ^ u 9 - 5

ここで、オペレーターの優先順位スイッチを次のように変更する必要があります。

switch(c)
{
       case '-' : case '+' :
           return 1 ;
       case '*' : case '/' :
           return 2 ;
       case '^' : case 'u': //note the 'u' operator we added
           return 3 ;
       default :
           return 0 ;
}

もちろん、この単項演算子をサポートするには変更を加える必要があります。

于 2013-06-16T11:07:02.657 に答える
1

1 つのオプションは、最初の文字が '-' の場合、前に 0 を置くことです。- が ( の後にある場合にも、これを行う必要があります。

より優れたものは、単項マイナス演算子を実装するか、数値リテラルの一部として扱います。

于 2013-05-07T18:03:37.427 に答える