方法总是返回零答案

【问题标题】：Method always returns zero方法总是返回零
【发布时间】：2014-02-28 18:54:03
【问题描述】：

我对 java 还是很陌生，我正在尝试创建一个程序来询问用户 2 个随机频率之间的差异。一切正常，除了当我尝试获得 2 个频率的差异时，答案始终为 0。如何让它显示实际差异？这是我编写的用于创建音调并计算差异的类：

public class Quiz{

    private PitchPlay one = new PitchPlay();
    private PitchPlay two = new PitchPlay();

    private int frequencyOne;
    private int frequencyTwo;

    private int dif = new Integer(Math.abs(frequencyTwo - frequencyOne));

public int run(){
    frequencyOne = new Integer((int)(Math.random() * 5000 + 50));
    frequencyTwo = new Integer((int)(Math.random() * 5000 + 50));
    return this.dif;
}

public int freqDif(){
    return this.dif;
}

public void playQuiz(){
    one.play(one.note(frequencyOne, 2, 15));//note(frequency, duration, volume)
    two.play(two.note(frequencyTwo, 2, 15));
}
}

这里是使用测验类的类：

public class Action implements ActionListener{
Quiz one = new Quiz();

public void actionPerformed (ActionEvent e){
        if(e.getSource() == playSoundButton)
        {
            if(answerResponse.getText().compareTo("Correct!")  == 0 || answerResponse.getText().compareTo("Play and Listen...") == 0)
            {
                one.run();
                one.playQuiz();
            }
            else
            {
                one.playQuiz();
            }
        }
        if(e.getSource() == submitButton)
        {   
            String responseText = new String(responseField.getText());
            if(responseText !=null && !"".equals(responseText)){
                try{
                    Integer responseNumber = Integer.parseInt(responseText);

            if(responseNumber == one.freqDif())
            {
                    answerResponse.setText("Correct!");
                    answerResponse.setVisible(true);
            }
            else
            {
                answerResponse.setText("Wrong Answer. The difference is " + one.freqDif() + " hertz.");
                answerResponse.setVisible(true);
            }
                }catch(NumberFormatException f){
                    f.printStackTrace(System.out);
                }
            }



        }

    }
}

【问题讨论】：

标签： java return return-value

【解决方案1】：

private int dif = new Integer(Math.abs(frequencyTwo - frequencyOne));

这是在创建变量时计算的，而不是在您使用变量时计算的。您需要在分配变量后进行计算。

【讨论】：

【解决方案2】：

您的代码有几个问题：

正如其他人提到的那样，这条线
```
private int dif = new Integer(Math.abs(frequencyTwo - frequencyOne));
```
创建一个变量并分配一个从其他两个变量计算的值。使用创建时其他变量的值。每次这两个变量的值发生变化时，您都需要重新计算。这不会为您自动发生。
new Integer(...) 创建一个 Integer 对象。当您随后将此对象分配给int 变量时，该值将从对象中取出。这会为您的代码增加一些不必要的内存和运行时间。相反，您应该直接分配给您的变量。例如，
```
frequencyOne = (int)(Math.random() * 5000 + 50);
```

【讨论】：