如果您能为我的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!
}