Thread: loop

Hello,
For my Java class, I need to prompt the user for sales amounts, and break on a negative number. so far I have this

		while (true) {
 
			for (int count = 0; count < salesAmounts.length; count++) {
				System.out.print("Please enter sales amount (negative to end): ");
				salesAmounts[count] = askSalesAmount.nextInt();
 
				if (salesAmounts[count] <= 0)
					;
			}
			for (int showArr: salesAmounts) {	
				System.out.print(showArr);

this works, but it always prints a last "Please enter sales amount (negative to end):" statement. I can't figure out how to make it stop when the negative number is input.
also, I'm trying to show the numbers in the array. But the System.out.print doesn't show anything. Is there a good way to show the array with the numbers grouped together in a range. For example 0 through 10, 11 through 20 - and so on.

Well, if you fill the array up, until count < salesAmount.length, it should then exit the first for loop and move to the second, printing each item in the array. As for exiting the for loop when a number <= 0 is entered, enter a break statement in the if clause?

I'm not sure what you mean about grouping the numbers together, they're already in an array. If you want 1-10 on one line, and 11-20 on the next, I guess you could make a couple of new arrays (one for numbers 1-10, another for numbers 11-20, etc), and then loop through your original array, comparing each number and adding it to the relavent array. Using System.out.print as you're doing will print them all in a line, with no spaces or line breaks.

Thank you for the quick replies.
I thought since
for (int showArr: salesAmounts) {
System.out.print(showArr);
was outside of the brackets for the while statement, that it was outside of the loop. Should I put a break statement after the while, so it would look like this
[code]
while (true) {

for (int count = 0; count < salesAmounts.length; count++) {
System.out.print("Please enter sales amount (negative to end): ");
salesAmounts[count] = askSalesAmount.nextInt();

if (salesAmounts[count] <= 0)
;
}
break;
for (int showArr: salesAmounts) {
System.out.print(showArr);
[\code]

thank you jim829

dboxall123 to clarify "I'm not sure what you mean about grouping the numbers together, "
my output is supposed to show
amounts occurances
sales amount 100 - 200 4
sales amount 300 - 400 2

and so on to show how many time the sales amount for the week was 100 to 200, 300 to 400, and so on.
So, like you suggested, I would want to create an array for each group of numbers? So array1 would have 100 through 200, array2 300 through 400, and so on?

Ah, I didn't realise that the second for loop was outside of the while loop, I was a bit confused by it.

my output is supposed to show
amounts occurances
sales amount 100 - 200 4
sales amount 300 - 400 2

That's even easier, just create some integer variables, all set to zero. Then loop through your array, incrementing each one as required.

sorry, I'm very new to Java, and I'm not sure I'm explaining myself well. So this is the requirement.
"Use a single-subscripted array to solve the following problem: A company pays its salespeople on a commission basis. The salespeople receive $200 per week plus 9% of their gross sales for that week. For example, a salesperson who grosses $5000 in sales in a week receives $200 plus 9% of $5000 or a total of $650. Write an application (using an array of counters) that determines how many of the salespeople earned salaries in each of the following ranges (assume that each salesperson’s salary is truncated to an integer amount):
a) $200-$299
b) $300-$399
and so on. So how would I know what to increment by? I won't know what entered into the array.

So you don't have to do any of the math, right? You just have an array of integers? So, for example,

int twoHundredTo299 = 0;
int threeHundredTo399 = 0;
 
for (int wage : wages) {
    if ((wage >= 200) && (wage <= 299)) {
         twoHundredTo299 ++;
    } else if ((wage >= 300) && (wage <= 399)) {
        threeHundredTo399 ++;
    } 
}

--- Update ---

Oh hang on, I've just seen the bit about using an array of counters. So, i guess you still do it like the above example, but instead of incrementing the variables, you do something like this:

for (int wage : wages) {
    if ((wage >= 200) && (wage <= 299)) {
         salaryRanges[0] ++;
    } else if ((wage >= 300) && (wage <= 399)) {
        salaryRanges[1] ++;
    } 
}

dboxall123, I really appreciate your help, but I'm still not getting the output correctly. I'm sure it's my fault because I'm new to Java.
So in your example, is salaryRanges[0] an array you use just for getting the total of the 200 to 299 range?
The code I have so far is
[code]
import java.util.Arrays;
import java.util.Scanner;

public class Test1 {

public static void main(String[] args) {

Scanner askSalesAmount = new Scanner(System.in);
int[] salesAmounts = new int[10];

while (true) {

for (int count = 0; count < salesAmounts.length; count++) {
System.out.print("Please enter sales amount (negative to end): ");
salesAmounts[count] = askSalesAmount.nextInt();

if (salesAmounts[count] <= 0)
break;
}
}
for (int showArr: salesAmounts) {
System.out.print(showArr);
}
}
}
[\code]

I've used salesAmounts for my array to get the input from the user, but from what you're saying, I need to add an array for each range I'm searching? Is that correct?

Right, so I think that the assignment is asking you to create an array to hold the count of each salary range, so you would need to create a second array, let's call it amountsPaid. The size of amountsPaid will be the amount of ranges that there are (200-299, 300-399, 400-499, etc). Then you would loop through your salesAmounts array, checking each item. If the item is between 200-299, increment the first item (amountsPaid[0]) by one, if it's between 300-399, increment the second item by one. I think that's what it means by 'an array of counters'.

okay, sorry to be a pest, but in your example,

for (int wage : wages) {
if ((wage >= 200) && (wage <= 299)) {
salaryRanges[0] ++;
} else if ((wage >= 300) && (wage <= 399)) {
salaryRanges[1] ++;
}
}

where do you get wage and wages? is that the array that the user is inputting the data to? so in my code salesAmount is the array that is getting input from the user. And I guess I'm making an array for the 200 to 299 range. So If I call that twoHundredTo299, then according to your example, would I set it up like this

for (twoHundredTo299 : twoHundredTo299s ) {
if ((salesAmounts >=200 && ( salesAmounts<=299)) {
twoHundredTo299 [0] ++;

I'm doing something wrong because Eclipse has errors.

Wage and wages were obviously just examples. And no, that wasn't what I meant. The exercise asks for an 'array of counters'. So, I think you need to set up an array, and each item of the array is set to how many times each possible wage range occurs. Do as Jim said, post your whole code. Also post the entire exercise.