Thread: not looping properly, please help.

program is functional, but for only one user entry. I can't figure out why its not looping properly. Does anyone see anything that I am not seeing? Any and all help is greatly appreciated.

import  java.util.Scanner;
public class GradeGrouper
{
	public static void main (String [] args)
	{
		int score;
		char grade;
		int countA;
		int countB;
		int countC;
		int countD;
		int countF;
		System.out.println ("The program will tally the number of Letter Grades, as well as");
		System.out.println (" the total quantity of grades entered.");
		System.out.println ("Enter a series of non-negative integers to be compiled.");
		System.out.println ("Enter a negative number when complete.");
		Scanner keyboard = new Scanner (System.in);
		score = keyboard.nextInt ();
		do
		{			
			countA = 0;
			countB = 0;
			countC = 0;
			countD = 0;
			countF = 0;
				if (score >= 90)
				{grade = 'A';
					countA++;}
				else if (score >= 80)
				{grade = 'B';
					countB++;}
				else if (score >= 70)
				{grade = 'C';
					countC++;}
				else if (score >= 60)
				{grade = 'D';
					countD++;}
				else
				{grade = 'F';
					countF++;}
		}
		while (score <= 0);	
			System.out.println ("The total quantity of integers entered is ");
			System.out.println (+ (countA + countB + countC + countD + countF));
			System.out.println ("The number of A's entered is " + countA);
			System.out.println ("The number of B's entered is " + countB);
			System.out.println ("The number of C's entered is " + countC);
			System.out.println ("The number of D's entered is " + countD);
			System.out.println ("The number of F's entered is " + countF);
 
 
	}
}

Look at your do-while loop condition:

do
		{
 
                ...............
 
		}
		while (score <= 0);

I believe the loop only loops while the condition is true and if it's false, then the loop ends. Think about what the condition is stating. Also, I noticed that the score input is outside of the loop, so I think it might only do one loop regardless.

WOW, did I really miss that?!? I'll test that in a little bit, the code is on a different confuser.

Originally Posted by KevinWorkman

What do you expect it to do? What does it do instead?

It is supposed to loop through the grade assignment / grade tally 'if - else/if' loop provided the user enters a non-negative integer. It was (is until I change it) only allowing for one user entry because of what KevinWorkman pointed out, my condition says (while score is less than 0). that is definitely causing a problem. Might not be the only one, but its a big one. I'll post up a functional code if that fixes it.

okay, fixing the condition allowed it to stay in the loop for however many entries the user wants, and it exits on the negative integer, but something is wrong with the loop. More specifically, the operators in the loop. For some reason, no matter how many entries the program sees, it only registers one, and assigns the correct letter grade and adjusts the correct grade tally, but it ignores all the other entries. any insight? code below...

import  java.util.Scanner;
public class GradeGrouper
{
	public static void main (String [] args)
	{
		int score;
		char grade;
		int countA;
		int countB;
		int countC;
		int countD;
		int countF;
		System.out.println ("The program will tally the number of Letter Grades, as well as");
		System.out.println ("the total quantity of grades entered.");
		System.out.println ("Enter a series of non-negative integers to be compiled.");
		System.out.println ("Enter a negative number when complete.");
		Scanner keyboard = new Scanner (System.in);
		do
		{	
			score = keyboard.nextInt ();		
			countA = 0;
			countB = 0;
			countC = 0;
			countD = 0;
			countF = 0;
			while (score >= 0);
			{
				if (score >= 90)
				{grade = 'A';
					countA++;}
				else if (score >= 80)
				{grade = 'B';
					countB++;}
				else if (score >= 70)
				{grade = 'C';
					countC++;}
				else if (score >= 60)
				{grade = 'D';
					countD++;}
				else
				{grade = 'F';
					countF++;}
			}
		}
			System.out.println ("The total quantity of integers entered is ");
			System.out.println (+ (countA + countB + countC + countD + countF));
			System.out.println ("The number of A's entered is " + countA);
			System.out.println ("The number of B's entered is " + countB);
			System.out.println ("The number of C's entered is " + countC);
			System.out.println ("The number of D's entered is " + countD);
			System.out.println ("The number of F's entered is " + countF);
 
	}
}

You must study doWhile loop. Is your program compiling?

yes its compiling. It'll run continuously in the loop, until the negative integer is entered, but its only registering an processing the first entry. That is what is confusing me, it seems that the loop is running correctly, but something inside is wrong?

unfortunately, "studying" the do while loop isn't really an option. I don't have a textbook. I am pretty sure that one would help, but... Let me be clear, I am NOT looking for the answer, but knowledge about a doWhile loop that will help me figure it out.

Alright, I'll work on that, give me a bit. Thank you.