Проверка ошибок в скобках для преобразования инфикса в постфикс Java

Я работаю над программой для преобразования инфикса в постфикс, и у меня все работает, за исключением того, что я не могу понять, где поставить проверку на ошибку для отсутствующей левой скобки. В основном пользователь вводит строку, и программа переходит к этому классу и преобразует его, но я хочу убедиться, что они ввели правильное количество скобок. Я пробовал несколько мест, но продолжаю придумывать EmptyStackExceptions.

import java.util.*;

public class PostfixConversion {


   public static boolean precedence(char first, char second)
   {
      int v1 = 0, v2 = 0;
      //find value for first
      if(first == '-' || first == '+'){
         v1 = 1;
      }else if(first == '*' || first == '/'){
         v1 = 2;    
      }//end if

      //find value for second
      if(second == '-' || second == '+'){
         v2 = 1;
      }else if(second == '*' || second == '/'){
         v2 = 2;    
      }//end if

     if(v1 < v2){
        return false;
     }//end if

     return true;
  }//end precedence method

 //converts infix expression into postfix expression
 public static String convertToPostfix(String infixExp)
  {
     String postFix = "The Postfix Expression is: ";
     Stack<Character> stack = new Stack<Character>();
     char character = ' ';

     for(int i = 0; i < infixExp.length(); i++)
     {
         character = infixExp.charAt(i);

         //determine if character is an operator
         if(character == '*' || character == '-' || character == '/' || character == '+')
         {
             while(!stack.empty() && precedence(stack.peek(), character)){
                 postFix += stack.pop();
             }//end while
             stack.push(character);
         }
         else if(character == '(') //check for left parenthesis
         {
             stack.push(character);
         }
         else if (character == ')') 
         {
             while(!stack.peek().equals('(') && !stack.isEmpty()){ //add characters until left parenthesis
                 postFix += stack.pop();
             }//end while

             if(!stack.isEmpty() && stack.peek().equals('(')){
                 stack.pop(); // pop/remove left parenthesis
             }
         }
         else
         {
             postFix += character;
         }//end if
     }//end for
     while(!stack.empty()) //add the remaining elements of stack to postfix expression
     {
         if(stack.peek().equals('('))
         {
             postFix = "There is no matching right parenthesis.";
             return postFix;
         }
         postFix += stack.pop();
     }
         return postFix;
 }//end convertToPostfix
}

person Anthony Pittelli    schedule 19.04.2013    source источник
comment
Приведите пример, на котором вы получаете EmptyStackExceptions   -  person Andremoniy    schedule 19.04.2013
comment
@Andremoniy Err, выражение с отсутствующей левой скобкой?   -  person user207421    schedule 27.02.2017


Ответы (1)


Прежде всего, вы должны изменить свой цикл while на эту форму:

while (!stack.empty() && precedence(stack.peek(), character)) {
    postFix += stack.pop();
}

т.е. изменить порядок выражений в while проверке: stack.empty() проверка должна быть первой.

Вторым исправлением будет добавление сообщения об ошибке "There is no matching left parenthesis." здесь:

if (!stack.isEmpty() && stack.peek().equals('(')) {
   stack.pop(); // pop/remove left parenthesis
} else {
   postFix = "There is no matching left parenthesis.";
   return postFix;
}
person Andremoniy    schedule 19.04.2013