为什么变量没有初始化？

Question

/** A salesclerk gets 3% commission for sales up to $5,000, 
4% for sales from $5,000 to $10,000, 
5% for sales from $10,000 to $15,000, and 
6% for sales over $15,000. 
Enter a sales figure and print the total pay. (dependent)*/

import java.util.Scanner;

public class Commission{
    public static void main(String[] args){

        double salesFigure, commission;
      
        Scanner scanner=new Scanner(System.in);
     
        System.out.println("Enter the total sales amount:");
        salesFigure =scanner.nextDouble();
     
        if(salesFigure > 5000) 
           commission= salesFigure*.03;
        if(salesFigure > 5000 && salesFigure <=1000) 
           commission= salesFigure*.04;
        if(salesFigure > 10000 && salesFigure<=15000) 
           commission= salesFigure*.05;
        if(salesFigure > 15000) 
           commission= salesFigure*.06;

        System.out.println("With a sales amount of $"+salesFigure+", $"+commission+" is the amount of commission earned.");

   }
}

/** For testing purposes use 
sales amounts     which should yield commission amounts of
4,000       120
5,500       220
12,000         600
20,000         1,200 */

对不起，我是新手，但我不知道为什么我会收到错误，提示可能未在以下行初始化佣金：

 System.out.println("With a sales amount of $"+salesFigure+", $"+commission+" is the amount of commission earned.");

Answer 1

如果您的 salesFigure 低于 5000，您的佣金将不会被初始化，我认为您在编码时犯了一个错误，应该是这样的：

import java.util.Scanner;
   public class Commission
   {
      public static void main(String[] args)
      {
      double salesFigure;
      double commission;
      
      Scanner scanner=new Scanner(System.in);
     
      System.out.println("Enter the total sales amount:");
      salesFigure =scanner.nextDouble();
     
      // this should fix the problem > --> <
      if(salesFigure < 5000) commission= salesFigure*.03;
      if(salesFigure > 5000 && salesFigure <=1000) commission= salesFigure*.04;
      if(salesFigure > 10000 && salesFigure<=15000) commission= salesFigure*.05;
      if(salesFigure > 15000) commission= salesFigure*.06;
      System.out.println("With a sales amount of $"+salesFigure+", $"+commission+" is the amount of commission earned.");

   }
}

那应该可以解决问题。 也是小吹毛求疵，但最好在单独的行上声明变量，这样它更具可读性。

Answer 2

为什么变量没有初始化？

那是因为你在声明它时没有初始化它：

double salesFigure, commission;

此外，当您尝试为commission赋值时，所有可能的commission赋值都需要通过给定的条件。

    if(salesFigure > 5000) 
       commission= salesFigure*.03;
    if(salesFigure > 5000 && salesFigure <=1000) 
       commission= salesFigure*.04;
    if(salesFigure > 10000 && salesFigure<=15000) 
       commission= salesFigure*.05;
    if(salesFigure > 15000) 
       commission= salesFigure*.06;

因此，如果您的编译器告诉您，则有可能如果所有条件都失败，您的commission变量将保持未初始化状态。

---

要解决此问题，您可以使用一些默认值对其进行初始化：

double commission = 0.0d;

其次，检查你的条件。 确保它们不是一些永远不会成立的矛盾条件。

Answer 3

一个简单的解决方案是按如下方式反转检查：

    if (salesFigure > 15000)
       commission= salesFigure*.06;
    else if (salesFigure > 10000)
       commission= salesFigure*.05;
    else if (salesFigure > 5000) 
       commission= salesFigure*.04;
    else
       commission= salesFigure*.03;

这样你就不需要初始化佣金，因为它总是会收到一个值（我强烈建议在使用大量 if / else if 语句时总是使用大括号 {} ）。

如果您为百分比使用额外的变量，您可以编写更简洁的代码：

 double percentage;
 if (salesFigure > 15000) {
     percentage = 6;
 } else if (salesFigure > 10000) {
     percentage = 5;
 } else if (salesFigure > 5000) {
     percentage = 4;
 } else {
     percentage = 3;
 }
 commission = salesFigure * (percentage / 100.0);