Question

我正在做一个解决表达式的程序。我已经使它工作但是当代码通过分区符号＆＃34; /＆＃34;它出现在index out of bounds，我不知道如何解决它。可以问你很棒的java头脑，看看并给我提示做什么？我不希望答案只是提示，所以我可以学习。

谢谢！

我的课程的主要课程：

import java.util.ArrayList;
import java.util.Scanner;
import static java.lang.Integer.*;
import static java.lang.System.*;
public class ExpressionSolver
{
    int answer;
    String expString;
    ArrayList<String> exp;
    public ExpressionSolver(String s)
    {
        setExpression(s);
    }
    public void setExpression(String s)
    {
        expString = s;
        exp = new ArrayList<String>();
        for(String temp: s.split(" "))
        {
         exp.add(temp);
        }
    }
    public void solveExpression()
    { 
       int index1, index2, num1, num2;        
      while(exp.contains("*") || exp.contains("/"))
      {
           index1 = exp.indexOf("*");
           index2 = 0;
           if(index1 <= 0)
           {
             index1 = exp.indexOf("/");
             index2 = 1;
           }
         num1 = Integer.parseInt(exp.get(index1-1));
         num2 = Integer.parseInt(exp.get(index1+1));
        if(index2 == 0)
         {
            answer = num1 * num2;
         }
         else
         {
            answer = num1 / num2;
         }
         exp.remove(index1 - 1);
         exp.remove(index1 - 1);
         exp.set(index1 - 1, Integer.toString(answer));
      }
      while(exp.contains("+") || exp.contains("-"))
      {
           index1 = exp.indexOf("+");
           index2 = 0;
           if(index1 <= 0)
           {
             index1 = exp.indexOf("-");
          index2 = 1;
           }
         num1 = Integer.parseInt(exp.get(index1-1));
         num2 = Integer.parseInt(exp.get(index1+1));
        if(index2 == 0)
         {
            answer = num1 + num2;
         }
         else
         {
            answer = num1 - num2;
         }
         exp.remove(index1 - 1);
         exp.remove(index1 - 1);
         exp.set(index1 - 1, Integer.toString(answer));
      }
}
  public String toString()   
  {
   return expString + " = " + answer;
  }
}

我的跑步者班级

import java.util.ArrayList;
import java.util.Scanner;
import static java.lang.Integer.*;
import static java.lang.System.*;

public class ExpressionSolverRunner
{
    public static void main( String args[] )
    {  
      ExpressionSolver test = new ExpressionSolver("3 + 3");
      test.solveExpression();
      System.out.println(test);

      test.setExpression("3 * 5");
      test.solveExpression();
      System.out.println(test);

      test.setExpression("3 - 5");
      test.solveExpression();
      System.out.println(test);

      test.setExpression("3 / 5");
      test.solveExpression();
      System.out.println(test);
    }  

}

我的输出

----jGRASP exec: java ExpressionSolverRunner
3 + 3 = 6
3 * 5 = 15
3 - 5 = -2
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 99, Size: 1
    at java.util.ArrayList.rangeCheck(ArrayList.java:653)
    at java.util.ArrayList.set(ArrayList.java:444)
    at ExpressionSolver.solveExpression(ExpressionSolver.java:60)
    at ExpressionSolverRunner.main(ExpressionSolverRunner.java:29)

 ----jGRASP wedge2: exit code for process is 1.
 ----jGRASP: operation complete.

每当程序读取3 / 5时，它就会出现越界错误。答案应该是0。

Answer 1

关于您的代码的一些提示：

为了分析问题，如果代码完全反映了您在生成堆栈跟踪时使用的代码，那将会有所帮助。行号表示情况并非如此。
if(index1 <= 0)：不要这样：如果index1 == 0表示匹配（您测试的字符串的第一个字符）。我想你想要像if(index1 < 0)
这样的东西
说实话，我不明白为什么-测试没有抛出异常。也许您发布的代码与测试结果不一致？
你所谓的index2不是索引，而是逻辑上的布尔值。看起来你还在正确评估它，但如果你使用布尔值和不同的名称（例如boolean index1DidNotMatch或类似的东西），它会大大提高代码的可读性。

我们可以讨论更多的事情，但既然你想要一些tipps，我写的内容可能会帮助你解决问题。

越界错误索引99大小1

1 个答案: