3

所以，我在这里遇到了一个非常奇怪的问题。当我将值 user.salary 乘以 1.1 时，由于某种原因它变为 0！原来的 user.salary 变量很好，我确认它是 System.out.println 应该有的。

import javax.swing.JOptionPane;


class Employee{
    String firstname;
    String lastname;
    double salary;
    Employee (String firstname, String lastname, double Salary){
        this.firstname = firstname;
        this.lastname = lastname;
        this.salary = salary;
    }

public String getFirstname() {
    return firstname;
}

public void setFirstname(String firstname) {
    this.firstname = firstname;
}

public String getLastname() {
    return lastname;
}

public void setLastname(String lastname) {
    this.lastname = lastname;
}

public double getSalary() {
    return salary;
}

public void setSalary(double salary) {
    this.salary = salary;
}



}

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

String firstname = JOptionPane.showInputDialog("What is your first name?");
String lastname = JOptionPane.showInputDialog("What is your last name?");
String salarystring = JOptionPane.showInputDialog("What is your salary?");
Double salary = Double.parseDouble(salarystring);
Employee user = new Employee (firstname, lastname, salary);
String message = String.format("Hello, %s %s.", user.firstname, user.lastname);
JOptionPane.showMessageDialog(null, message);
double raise = (user.salary)*1.1;

JOptionPane.showMessageDialog(null, "Congratulations, you have received a raise! Your salary is now "+raise);
  } 
}
4

2 回答 2

9

您的构造函数有一个错字：

Employee (String firstname, String lastname, double Salary){

double salary应该是小写。因为它是这一行：

this.salary = salary;

什么都不做，因为它等同于this.salary = this.salary.

一旦您将参数更改为小写的薪水，它就会将您传递给构造函数的值分配给薪水字段。

于 2013-02-18T13:19:19.880 回答
5

您的构造函数中似乎存在问题：

Employee (String firstname, String lastname, double Salary){
    this.firstname = firstname;
    this.lastname = lastname;
    this.salary = salary; //You assign salary to itself, since the 
                          //parameter is Salary (with caps)
}

这种方式salary没有设置，并保持默认值0。剩下的问题只是数学。将行更改为此

 this.salary = Salary

将解决这个问题，我建议你使用Java的命名约定并以小写开头的参数命名，newSalary例如。

于 2013-02-18T13:19:38.817 回答