调试器停止工作

Question

我的程序需要允许用户输入员工姓名和年度总销售额。当用户完成将员工添加到阵列中时，程序应确定哪些员工的销售额最高，哪些员工的销售量最低。然后，它应该打印出两个数字之间的差异。

在我下面的代码中，我有一个保存用户输入的年销售额的totalPay类(它包括以前赋值中的其他变量和方法，这里没有使用)。salesPerson类保存雇员的姓名和totalPay对象，其中包括他们的年销售额。(我意识到这太复杂了，但我正在修改我之前的作业，而不是从头开始。)

当我运行这段代码时，它允许我输入姓名和销售额，但是当我输入“yes或no”来添加另一个雇员时，它崩溃了，并告诉我在第58行有一个NullPointerException，在代码中有记录。

我运行了调试器（没有任何断点），它只是停在代码中提到的第46行。它没有给出错误消息，只是没有更新该变量和我对调试器置灰的“进入”按钮，我不能再单击它们了。（如果相关的话，我正在使用NetBeans。）

任何想法将不胜感激！

编辑：这是输出和错误消息。

名字？美国队长

投入年销售额：80

是否添加其他员工？是或否

不

commission.Commission.main（Commission.java:58）线程"main"java.lang.NullPointerException异常

package commission;
//Commicaion calulator


import java.util.Scanner;


public class Commission 
{
    public static void main(String args []) 
     {   
     salesPerson[] emps = new salesPerson[10]; //Employee Array
     String cont = "yes";
     String n="";
     double s=0;
     int i=0;
     salesPerson high = new salesPerson();
     salesPerson low = new salesPerson();

      // scanner object for input
        Scanner keyboard = new Scanner(System.in);

      //Enter in employee name 
     while (cont == "yes"){
        System.out.print("Name? ");
        n = keyboard.nextLine();
        emps[i] = new salesPerson();
        emps[i].setName(n);

         //Loop of yes or no entering more employees
        //If yes add another name if no continue with total Commision
        //Enter in the sales amount of commistion
        System.out.print("Input annual sales: ");  
        s=keyboard.nextDouble();
        emps[i].pay.annual = s;

        System.out.println("Add another employee? yes or no ");
        keyboard.nextLine();
        cont = keyboard.next(); //Line 46: Debugger stops here.

        if (cont =="yes")
            i++;
        if (i==9){
            System.out.println("You have reached the maximum number of employees.");
            cont = "no";
        }
     }

     i=0;
     for (i=0; i<emps.length; i++){
         if (emps[i].pay.annual > high.pay.annual) //Line 58: It claims the error is here.
             high = emps[i];

         if (emps[i].pay.annual < low.pay.annual)
             low = emps[i];
     }

     double diff = high.pay.annual - low.pay.annual;
     System.out.println("Employee "+low.getName()+" needs to earn "+diff+" more to match Employee "+high.getName());

    // Output table for composation with increments of $5000
//        int tempAnnual =(int) pay.annual;
//        for (i=tempAnnual; i<= pay.annual; i+=5000)
//            System.out.println(i+"   "+ pay.getReward(i));   
    }


  public static class totalPay
  {

      double salary=50000.0; //Yearly earned 50000 yr fixed income

      double bonusRate1=.05; //bounus commission rate of 5% per sale

      double commission; //Commission earned after a sale

      double annual; //Sales inputted

      double reward; // Yearly pay with bonus

      double bonusRate2= bonusRate1 + 1.15 ; // Sales target starts at 80%

    public double getReward(double annual)
    {
       double rate;
       if (annual < 80000) 
        rate=0;
       else if ((annual >= 80000) || (annual < 100000 ))
        rate=bonusRate1;
       else 
         rate=bonusRate2;

      commission = annual * rate;

      reward=salary + commission; 

      return reward;
    }

  }

  public static class salesPerson
      {
    String name; //Employee Name
    totalPay pay = new totalPay();

    public void setName(String n) //Name
    {      
     name=n;         
      }
    public String getName()
    {
        return name;
    }
      }

  }

Answer 1

调试器已停止，因为它在从扫描仪输入时被阻止。这在文件中有规定：

查找并返回此扫描程序中的下一个完整令牌。完整标记的前面和后面是与分隔符模式匹配的输入。此方法可能会在等待输入扫描时阻塞，即使之前对 hasNext（） 的调用返回 true。

除此之外，您很幸运地输入了该代码块。您不正确地比较了字符串，因此乍一看，除非在某些特殊情况下，否则您不会进入该循环。这也是发生NPE的原因；您在错误的伪装下初始化数组的元素（==with a String），因此：

• 您可能永远不会初始化任何东西。
• 您只能初始化第一件事（如果（cont==“yes”））

我只介绍了几个高点，但在大多数情况下，阻塞 IO 是调试器停止的原因。一旦你开始使用 .equals，其他错误可能会变得更容易看到，但我鼓励你与同学、导师或助教一起进行面对面的代码审查。这里散布着很多关于代码的误解，这将使以后更难调试或修复。

Answer 2

它停止调试器，因为它等待您使用键盘输入。如果键入输入并按enter键，调试器将从此处继续。

顺便说一下，您应该仔细阅读java的命名约定和编码最佳实践

Answer 3

创建此最大大小为10的数组：

salesPerson[] emps = new salesPerson[10];

但仅为每个输入的SalesPerson对象创建和分配一个对象引用。由于您只输入1个名称，因此数组中只有第1个条目有效，因此剩余的9个为空。然后您尝试遍历整个数组（emps.length为10）：

  for (i=0; i<emps.length; i++){
         if (emps[i].pay.annual > high.pay.annual) 

当索引第一个空引用时，这会导致NPE。您需要将循环更改为：

  int numEntered = i;  //last increment

      for (i=0; i< numEnetered; i++){
             if (emps[i].pay.annual > high.pay.annual)