Average of numbers with sentinel value = 0

[Chago Bowers]

I am working on an exercise where I need to:

(Count positive and negative numbers and compute the average of numbers)
Write a program that reads an unspecified number of integers, determines how many
positive and negative values have been read, and computes the total and average of
the input values (not counting zeros). Your program ends with the input 0. Display
the average as a floating-point number.

I moved the different boolean statements around, but I'm not getting the sentinel value to end the run. It continues to let me add integers endlessly. The code I wrote is below:

package exerciseFourOne;
import java.util.Scanner;
public class AverageOfIntergers {

	public static void main(String[] args) {
		// TODO Auto-generated method stub

int positive = 0; // number of positive integers
int negative = 0; // number of negative integers
int sum = 0; // value of sum of integers

Scanner input = new Scanner (System.in);
System.out.println("Enter an integer, the input ends if it is 0:");

int number = input.nextInt();
while (number != 0){
	sum+= number;}
	if (number > 0){
	positive++;		
	if (number < 0)
	negative++;
	}
	if (number == 0) {
	System.out.println(" The number of positives is " + positive);
	System.out.println("The number of negatives is " + negative);
	System.out.println("The total is " + sum);
	double average = sum / (positive + negative); // average of integers
	System.out.println(" The average is " + average);
}

if (positive + negative == 0){
		System.out.println("No numbers are entered except 0");
	}	


	
	
}

	}

[Winston Gutkowski]

Chago Bowers wrote:I moved the different boolean statements around, but I'm not getting the sentinel value to end the run. It continues to let me add integers endlessly. The code I wrote is below:

Well the main thing I see is that you only ever input one number, and you do it outside the loop. So, given that, what do you think you need to change to get it to work? Your indenting is also bad, so I suspect you're not seeing that you have at least one mistake at line 17 (there are others as well).

Another thing you might want to think about is to separate the input process from the calculations completely. Right now, they're all jumbled up together, which makes your logic difficult to decipher.

HIH

Winston

PS: You're very close. You just need to think things through a bit more.