0

具体来说，在 while 循环中有一些逻辑区域不允许程序正确流动。我已经完成了这个循环，它应该可以工作。我遇到的一些错误是当我输入“0”时它不会立即退出，换句话说，我必须按两次 0，这对我来说没有意义，除非我没有正确理解 while 循环。另一个错误是在我的 add() 方法中，我输入的内容只会告诉我输入的第一个数字。所以我相当肯定错误在我的循环中，但我看不到逻辑错误来自哪里。任何帮助将不胜感激，谢谢。

import javax.swing.JOptionPane;

public class RandyGilmanP2 {//Begin class


    public static void main(String[] args) {//Begin main
        JOptionPane.showMessageDialog(null, "Hello Welcome to Sum and Average"
                + "\n   of a Number Calculator Program" 
                + "\n                 By: Randy Gilman");
        //Declare variables
        float add = 0;//used to store the sum of the numbers inputed
        float numb = input();//used to store the value of Input() method
        float average;
        int count = 0;// Used as a counter variable
        //Loop that will be controlled by a sentenil value
        while (numb != 0) {//Begin for loop
            count += 1;
            //Call Input method    
            input();
            numb = input();
            //Method to find the sum of all the numbers inputed
            sum(add,numb); 
            add = sum(add,numb); 
            //Used to find the average of all the numbers entered (sum / count)
        }//End for loop
        avg(add,count);
        average = avg(add,count);//used to store the average of the numbers
        Results(count,add,average);
    }//End Main


   public static float input(){//Begin Method
         //Will keep gathering input from user until input is equal to 0
         String NumberString = JOptionPane.showInputDialog("Enter a floating point number"
            + "(The program ends when 0 is entered):");
             //Convert string to float
            float number = Float.parseFloat(NumberString);
            return number;


    }//End Method 

    public static float sum(float sum, float numb2){//Begin method
            //Add number to the previous number to compute total
            sum += numb2; 
            if (sum > 100)
                JOptionPane.showMessageDialog(null, "***WARNING***" + "\n            The sum of your numbers exceed 100");
            return sum;    
        }//End Method

        public static float avg(float num1, float num2){
            //Declare variables 
            float result;
            //Preform calculation of average
            result = num1 / num2;
            return result;

        }//End Method

        public static void Results(int counter, float addition, float aver){//Begin method
            //Declare variables
            JOptionPane.showMessageDialog(null,"The total amount of numbers you entered are: " + counter);
            JOptionPane.showMessageDialog(null,"The sum of the numbers you have entered is: " + addition);
            JOptionPane.showMessageDialog(null,"The average of the numbers you have entered is: " + aver);
        }//End Method

}//End Class
4

3 回答 3

4

这是我认为你不想要的东西：

        //Call Input method    
        input();
        numb = input();

做两次会导致两次要求一个数字 - 但只使用第二个......

另外，要非常小心浮点数和相等性。0 是一个很好的数字，但其他数字，尤其是分数就不是那么明显了……

除此之外，代码似乎还可以...

（而且您没有“添加”方法）

于 2013-10-05T19:25:18.057 回答
2

您要求输入，然后将其丢弃 

//in the initialization you do:...
float numb = input();//used to store the value of Input() method
//and in the loop you do it again, overwrite the first input
numb = input();

您应该只在序言中声明 numb 并将其余部分留给循环。

循环中的问题

查看您的循环，我看到以下问题：

while (numb != 0) {//Begin for loop
    count += 1;
    //Call Input method    
    input();                  <<-- 1. should be removed, input goes nowhere 
    numb = input();
    //Method to find the sum of all the numbers inputed
    sum(add,numb);            <<-- 2. should be removed, output goes nowhere
    add = sum(add,numb); 
    //Used to find the average of all the numbers entered (sum / count)
}//End for loop
avg(add,count);               <<-- 3. why do you keep repeating??
average = avg(add,count);//used to store the average of the numbers

虽然 java 允许您调用函数（返回一个值），就好像它们是过程一样（又不返回任何内容void），但只有在您对返回值不感兴趣时​​才应该这样做。
在这种情况下，您将调用该函数两次。
第一个调用要求用户输入然后丢弃结果，第二个调用再次询问用户然后存储答案。

因为您在循环中运行，所以很难区分重复与循环运行两次。
您永远不必调用函数两次，就像您在循环中所做的那样。

您不必预先宣布您的功能
只需说明您想要发生的事情，java 就会做到。

其他问题
这种情况下没有问题，因为numb是用户直接输入的。但一般来说，您永远不应该将浮点数与绝对值进行比较。
以下内容不保证为真。 

((a/b*b)-a == 0)

由于舍入错误，您可能会得到意外的结果，其中的结果((a/b*b)-a)将0.00000000001导致您的测试 (numb != 0) 失败。
因为您正在与用户输入进行比较，所以现在还可以，但是如果您正在检查计算的输出，请记住浮点计算是不精确的！

于 2013-10-05T19:30:15.337 回答
0

从我所看到的，你应该只在你的循环中调用一次“input()”。你叫它两次 - 即

          input(); 
          numb = input();

取出第一个“input();” 你应该很高兴。

您也不需要执行“sum(add,numb);” 两次，所以取出“sum(add,numb);” 行，只需使用“add = sum(add,numb);”。

于 2013-10-05T19:26:48.000 回答