如何使用“if”语句

Question

我试图在此代码中插入“if”语句，但它不能很好地工作。其固定工资和奖励（37.28 * 1.32）将给予94,261.02一次。低于94,261.02只是一个普通的37.28佣金。所有iny“int”行用红色加下划线显示！分数。所以我试图找出问题：

 System.out.println("Enter your annual sales"); 
     String annual = input.nextLine(); 

     int salary = 7550281; 
     int commission = 38_28; 
     int compensation = Integer.parseInt(annual) * commission + salary; 
     System.out.println("compensation is: "+compensation); 

     if (Integer.parseInt(annual) < 92416_02) { 
      int salary = 7550281; 
      int commission = 37_28 * 1_32; 
      int compensation = Integer.parseInt(annual) * commission + salary; 
      System.out.println("compensation is: "+compensation); 

     } else if (Integer.parseInt(annual) > 92416_02){ 
     int salary = 7550281; 
     int commission = 38_28; 
     int compensation = Integer.parseInt(annual) * commission + salary; 
     System.out.println("compensation is: "+compensation); 
     } 

谢谢。

Answer 1

使用花车和.代替_，如：float commission = 38.28;

Answer 2

我修改你的代码，因为你没有提供完整的代码块，我通过我的想象做的代码段。我建议你使用BigDecimal类来进行更精确的计算。 '

import java.util.Scanner;   
public class stack_overflow { 
    public static void main(String args[]){ 
     System.out.println("Enter your annual sales"); 
     Scanner input = new Scanner(System.in); 
     String annual = input.nextLine(); 
     double salary = 7550281; 
     double commission = 38.28; 
     double compensation = Double.parseDouble(annual) * commission + salary; 
     System.out.println("compensation is: "+compensation); 

     if (Double.parseDouble(annual) < 92416.02) { 
      salary = 7550281; 
      commission = 37.28 * 1.32; 
      compensation = Double.parseDouble(annual) * commission + salary; 
      System.out.println("compensation is: "+compensation); 

     } else if (Integer.parseInt(annual) > 92416.02){ 
      salary = 7550281; 
      commission = 38.28; 
      compensation = Integer.parseInt(annual) * commission + salary; 
      System.out.println("compensation is: "+compensation); 
    } 
} 

Answer 3

很多问题将归结为您正在使用的Java版本。

目前，让我们假设你正在使用的Java 7和38_28是一个有效的声明，则在每次if块

// Declared here... 
int salary = 7550281; 
int commission = 38_28; 
int compensation = Integer.parseInt(annual) * commission + salary; 
if (Integer.parseInt(annual) < 92416_02) { 
    // Redeclared here... 
    int salary = 7550281; 
    int commission = 37_28 * 1_32; 
    int compensation = Integer.parseInt(annual) * commission + salary; 
} else if (Integer.parseInt(annual) > 92416_02) { 
    // Redeclared here... 
    int salary = 7550281; 
    int commission = 38_28; 
    int compensation = Integer.parseInt(annual) * commission + salary; 
} 

这不是必需的范围内重新声明的变量。你只需要一次声明它们，例如...

int salary = 7550281; 
int commission = 38_28; 
int compensation = Integer.parseInt(annual) * commission + salary; 
if (Integer.parseInt(annual) < 92416_02) { 
    salary = 7550281; 
    commission = 37_28 * 1_32; 
    compensation = Integer.parseInt(annual) * commission + salary; 
} else if (Integer.parseInt(annual) > 92416_02) { 
    salary = 7550281; 
    commission = 38_28; 
    compensation = Integer.parseInt(annual) * commission + salary; 
} 

我想你也将使用long在int，以防止任何可能的溢出会更安全

尼特挑

你也一再地转换annual值。虽然它没有什么问题，但它确实会使代码混乱并使其难以阅读。它会建议将其转换一次，简单地重新使用所得到的值，例如...

int annualAmount = Integer.parseInt(annual); 
if (annualAmount < 92416_02) { 
    //... 
    compensation = annualAmount * commission + salary; 
} else if (annualAmount > 92416_02) { 
    //... 
    compensation = annualAmount * commission + salary; 
} 

Answer 4

你的代码具有以下问题：

1. 局部变量重复（工资，报酬和佣金被宣布两次）。如果要将值分配给已存在的变量，则不应在变量名称前面指定类型（此处为int）。
2. 您的乘法无效。 37_28 * 1_32给出492096。下划线根本不重要。你可能不得不将结果除以100来赋予它逻辑意义。
3. 当年份恰好是92416_02时，您的代码无法处理这种情况。删除else if子句并初始化您的commission，或者仅使用else而不遵循if。另外，由于有很多共同的线，你可以将它们从块中移出。

还要注意，用户必须输入他们的年度乘以100，因为parseInt不会识别下划线。否则，此代码可能可以做你想做的事情： System.out.println（“输入你的年销售额”）; 字符串年度=输入。nextLine（）;

int salary = 7550281; 
    int commission = 38_28; 
    if (Integer.parseInt(annual) < 92416_02) { 
     commission = 37_28 * 1_32/100; 
    } 
    int compensation = Integer.parseInt(annual) * commission/100 + salary; 
    System.out.println("compensation is: "+compensation); 

P.S.不要听那些建议使用花车或双打进行计算的人 - 这是一个糟糕的，容易出错的练习，因为计算错误会随着时间的推移而积累。使用int，long，BigInteger或BigDecimal（使用String构造函数）