我的avg方法有什么问题？[副本]

Question

是的，我知道这里有很多方法。这是任务的一部分。在这段代码中，除了输入相等的数字之外，一切都正常工作

public static void main(String[] args) {
    //Use Main Method for gathering input
    float input = 1;
    // Declare variable for sum
    float theSum = 0;
    // Declare variable for average
    float average = 0;
    // Declare variable for counting the number of user inputs
    int counter = 0;
    /* Initialize the while loop using an input of 0 as a sentinel value
     * to exit the loop*/
    while (input != 0) {
        if (input!=0){
            counter++;
        }
        input = Float.parseFloat(
                JOptionPane.showInputDialog(
                null, "Please enter a number.  Enter 0 to quit: "));
        // Invoke sum method and pass input and summation to sum method


        theSum = (sum(input, theSum));
        if (theSum > 100)
        {
            JOptionPane.showMessageDialog(null, "The sum of your numbers "
                    + "are greater than 100!");
            break;
        }
    }
        // Invoke display method and pass summation, average, and counter variables to it
                average = (avg(theSum, counter));    
                display(theSum, average, counter);
    }


public static float sum(float num1, float sum) {
    //Add the user's input number to the sum variable
    sum += num1;
    //Return value of sum variable as new summation variable
    return sum;
}

public static float avg(float num1, float num2) {
    //Declare and initialize variable for average
    //Calculate average
    float average = num1 / num2;
    //Return value of average variable
    return average;
}

public static void display(float sum, float average, int counter) {

    /* I am subtracting 1 from variable counter so as not to include the sentinel value
     * of 0 that the user had to enter to exit the input loop in the overall count*/

    // Display the count, sum, and average to the user
    if (sum > 100) {
        JOptionPane.showMessageDialog(null, "Count = " + (counter) + ", Sum = " + sum + ", Average = " + average);
    }
    if (sum <= 100) {
        JOptionPane.showMessageDialog(null, "Count = " + (counter - 1) + ", Sum = " + sum + ", Average = " + average);
    }

}

}

Answer 1

您的计数器比必要的大 1。除以（计数器 - 1）可以修复它。

Answer 2

问题在于您退出 while 循环的方式，如@chrylis所述。因此，如果总和

if (sum <= 100) {
        JOptionPane.showMessageDialog(null, "Count = " + (counter - 1) + ", Sum = " + sum + ", Average = " + average);
    }

正如您在示例中看到的：“如果我输入 8，10，19 和 0 退出输出是计数 3 和 37 平均值 9.25”

这是因为计数器值是4（所以avg将是37/4=9.25），但在显示结果时您将计数器减去1，因此您得到计数为3。

do-while循环将解决这个问题，因为最后会检查条件，因此对于两个< code >,循环将以相同的方式退出

< code>do-while循环如下所示:

do{

//here goes your code

}while (input != 0);

Answer 3

原因是，根据总和的不同，您会以不同的方式退出< code>while循环。如果总和小于100，即使当您输入数字< code>0来“退出”时，您仍然要多经历一次循环。老实说，整个循环需要彻底重构；一个<代码> do...while循环更容易阅读和调试。