如果您能为我的Java课堂作业提供帮助，我将不胜感激。 问题的提示是：

编写程序以读取非负整数列表，并显示最大整数，最小整数和所有整数的平均值。 用户通过输入负的前哨值来指示输入的结束，该负值在查找最大，最小和平均值时不使用。 平均值应为double类型的值，以便使用小数部分进行计算。

我的代码遇到的问题是，运行时，除非输入的第一个值为负数，否则循环不会结束，在这种情况下，它将返回：

输入的最大数量为：0

输入的最小数字为：0

输入的平均值为：NaN

请帮忙！ 谢谢。 -山姆

码：

package blah;
import java.util.Scanner;
public class blahblah
{
public static void main(String[] args)
{
Scanner keyboard = new Scanner(System.in);
System.out.println ("Please enter a list of positive integers.");
System.out.println ("Please enter a negative integer when finished.");
int in = 0;
int max = 0;
int min = 0;
int sum = 0;
int count = 0;
in = keyboard.nextInt();
while (in>=0)
{
if (in > max) {
in = max;
}
if (in < min) {
in = min;
}
sum += in;
count++;
if (in < 0) {
break;
}
}
System.out.println("The maximum number entered was:" + max);
System.out.println("The minimum number entered was:" + min);
System.out.println("The average of the numbers entered was:" + (double)sum/count);
}
}

一次循环-您永远不会更改in ...的值

另外，您还需要将in=max;更改为max = in;等。赋值会将右侧的值放入左侧的变量。

您需要在循环中再次读取nextInt：

while (in>=0)
{
if (in>max){
max=in;
}
if (in
min=in;
}
sum += in;
count++;
in = keyboard.nextInt();
//Check not needed here, handled by while loop
//if (in<0){
//     break;
//}
}

从注释中编辑：您的分配方向错误，因此您将输入设置为等于最小/最大，而不是将最小/最大设置为等于输入

这解决了我永无止境的循环！ 谢谢。 但是，现在我的程序返回的最大最小值和平均值均为0。在这里，我的逻辑问题是什么？

@SamPeezick因为您在in = min而不是min = in

您声明要读取的值不在while循环内，因此它仅读取第一个条目：

in = keyboard.nextInt ();
while (in>=0)
{
}
改成：
in = keyboard.nextInt ();
while (in>=0)
{
... stuff ...
in = keyboard.nextInt ();
}

将输入的读取移入循环内，并在负数处中断：

while (true) {
in = keyboard.nextInt();
if (in < 0) break;
// rest of loop
}

更好的方法是使用for循环，该循环很好地捆绑了所有与循环相关的逻辑：

for (int in = keyboard.nextInt(); in >= 0; in = keyboard.nextInt()) {
// your current loop code
}

分离出迭代代码可以清楚地知道什么代码是迭代代码，并使循环代码完全专用于程序的任务，从而使其更易于阅读和理解。

这也意味着您不需要声明int in，并且优良作法是尽可能减少变量的范围-在这种情况下，in仅存在于循环中，这是它的唯一位置使用/需要。

@jlordo是的-已经处理过，包括更好的替代恕我直言:)

为什么不while( ( in = keyboard.nextInt() ) >= 0 )？

@ PM77-1可以执行此操作，但是使用这种语法的做法不佳-CheckStyle插件认为这是一个问题。 有关使用for循环的信息，请参见我的编辑，该循环更清晰，更干净。

您将值放在相同的变量"中"

in = max;

应该是其他方式

max = in;

将您的代码更改为

in = keyboard.nextInt ();
while (in>=0){
if (in>max){
in=max;
}
if (in
in=min;
}
sum += in;
count++;
in = keyboard.nextInt ();
}

如您所见，我添加了in = keyboard.nextInt ();以允许从用户获取更多值

您必须重新读取用户的输入。代替：

in = keyboard.nextInt ();
while (in>=0) {
if (in>max){
in=max;
}
if (in
in=min;
}
sum += in;
count++;
if (in<0){
break;
}
}

使用：

in = keyboard.nextInt ();
while (in>=0) {
if (in>max){
in=max;
}
if (in
in=min;
}
sum += in;
count++;
// removed if, since loop checks it.
in = keyboard.nextInt (); // read on!
}