Thread: Problem with the loop ..... ;(

Hi java masters,

Problem with the loop where I'm not able to trace it in memory - looking for some advice/hint.

Program prompt the user to enter a line of text and then it iwll count how many occurrences there are of the letters a,e,i,o, and u regardless of case.

****** problem is that I'm getting the right answer when I will remove //A and leave only a++ and by leaving as SOP(a); , when I add SOP(a+A); than I'm getting weird results. I can't trace the memory and can't find jgrasp option to show the memory so I created this loop with trials and errors.

This is what I have so far:

import java.util.Scanner;
   public class test
   {
      public static void main(String [] args)
      {
         Scanner keyboard = new Scanner (System.in);
 
         String input;
 
         int a,e,i,o,u,A,E,I,O,U;
         int countOne, countTwo, countThree, countFour, countFive, numVawels, size; //a, e, i, o, u;
 
         System.out.print("Enter a line of text: ");
         input = keyboard.nextLine();
 
         //size = input.length();
 
         a = 0;
         A = 0;
         e = 0;
      	E = 0;
      	i = 0;
      	I = 0;
      	o = 0;
      	O = 0;
      	u = 0;
      	U = 0;
 
         countOne = 0;
         countTwo = 0;
      	countThree = 0;
      	countFour = 0;
      	countFive = 0;
 
         while(countOne<input.length())
         {
         	if(input.charAt(countOne)=='a'||input.charAt(countOne)=='A')//||input.charAt(count)=='e'||input.charAt(count)=='i'||input.charAt(count)=='o'||input.charAt(count)=='u')
 
               a ++;
               A ++;
            	countOne ++;
         }
 
         System.out.println(a+A);
 
      	while(countTwo<input.length())
         {
         	if(input.charAt(countTwo)=='e'||input.charAt(countTwo)=='E')//||input.charAt(count)=='e'||input.charAt(count)=='i'||input.charAt(count)=='o'||input.charAt(count)=='u')
 
               e ++;
               E ++;
            	countTwo ++;
         }
 
         System.out.println(e);

I'm sorry for that, the input should look like this:

Enter a line of text: aeiou AEIOU

A: 2
E: 2
I: 2
O: 2
U: 2

From your code, a and A have the same purpose. Both of them are to counter the letters of 'a' and 'A'. So you need only one of them and can delete another one.

Enter a line of text: aAe
5
1

my loop:

while(countOne<input.length())
         {
         	if(input.charAt(countOne)=='a'||input.charAt(countOne)=='A')//||input.charAt(count)=='e'||input.charAt(count)=='i'||input.charAt(count)=='o'||input.charAt(count)=='u')
 
               a ++;
               A ++;
            	countOne ++;
         }
 
         System.out.println(a+A);
 
      	while(countTwo<input.length())
         {
         	if(input.charAt(countTwo)=='e'||input.charAt(countTwo)=='E')//||input.charAt(count)=='e'||input.charAt(count)=='i'||input.charAt(count)=='o'||input.charAt(count)=='u')
 
               e ++;
               E ++;
            	countTwo ++;
         }
 
         System.out.println(e);

**NOTE

when I'm going to remove //E ++; and //A++; output appears to be correct BUT I don't get it why? since SOP states System.out.println(e); AND NOT e+A where I need to get both lower and upper case letters... ~confused

Enter a line of text: aAeee
2
3

Hi, I'm new to java as well, so somebody correct me if I'm wrong

I would use a switch/case statement to simplify this code.

Another hint, use keyboard.toLowerCase()

Originally Posted by luck999

From your code, a and A have the same purpose. Both of them are to counter the letters of 'a' and 'A'. So you need only one of them and can delete another one.

Lucky,

remove but from where from int? or while?

Originally Posted by blackscholes

Another hint, use keyboard.toLowerCase()

Hi,
thx for reply I don't know what that function does, my prof didn't go over this. What does it do? where do you put it?

Originally Posted by luck999

From your code, a and A have the same purpose. Both of them are to counter the letters of 'a' and 'A'. So you need only one of them and can delete another one.

Luck so you mean a and A it is the same for java? so I should remove the counter A++; just leave a++ but what about declaration a=0, A=0; leave it right?

It turns all characters in a string to lower case : D

Its a method of the String class, so put it after input.keyboard..

A and a are definitely not the same....

keyboard.toLowerCase() is a method of the String class that takes all the letters in a String and makes them lower case. It would help you a lot in your program because you won't have to worry about capital vowels, they would all be the same capitalization.