Thread: Java for loops to count vowels/consonants usinf the logic of the main

Norm, I entered a println inside my for loop for vowels just after the if statement about 'a'. I ended the loop just after 'a' and put in the println. A sea of red appeared but I continued with the launch. A notice appeared saying that, " A handler conflict occurred. This may disable some commands." All is not well with my debugger now. In the past, I have not used the debugger with logic problems.

I have reread my textbook a 100 time concerning if statements. They are true or false. If I enter one vowel how can this machine state that I have entered one vowel and two consonants? In the consomnant loop I made if statements stating that the vowels were not to be counted. How can the vowel loop create two consonants out of thin air? Yes I know that I am missing something even though I have reread my texts concerning for loops and if statements. I will not give up and I will figure it out somehow and you will be the first to know. Thanks for your assistance.

aussiemcgr, I have changed my program as follows:
[code = java]
import java.util.Scanner;
public class NewFoothill
{

public static void main(String[] args) throws Exception
{
Scanner input = new Scanner(System.in);

int count = 0;

System.out.println(" Enter the String ");
String s1 = input.nextLine();

s1 = s1.toUpperCase();
System.out.println("=====RESULT=====" + s1);
s1 = s1.toLowerCase();
System.out.println("=====RESULT=====" + s1);
System.out.println(" String s1 ");

for (int i = 0; i < s1.length(); i++)
{
char c = s1.charAt(i);

if ( c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u')
{
count++;
continue;
}
}
System.out.println(" There are " + " " + count + " " + " vowels. ");

for (int i = 0; i < s1.length(); i++)
{
char c = s1.charAt(i);

if ( c != 'a' || c !='e' || c != 'i' || c != 'o' || c != 'u')
{
count++;
continue;
}

if ( c == 'w' || c == 'l' || c == 'm' || c == 't' || c == 'f' || c == 'h')
{
count++;
}
}
System.out.println(" There are " + " " + count + " " + " consonants. ");
}
}
[/code]

You noted that I should not reset my count variable to 0 after I finish counting vowels. I believe you are refering to the control variable of my for loop that controls the count of consonants. However, I am not sure if you are talking about the "initial action", loop-continuation-condition" or the "action-after-each-iteration". If you are talking about the "initial action" my text says that it can be a list of zero or more comma-separated variable declaration statements or assignment expressions. That is where I get lost. I tried leaving the initial action out but that did not work. I do not know the correct way to change it. Should I put a number there? If so what number? Right now it is

 (int i = 0;

I have felt that my problem was in the control variable but I do not know how to correct it. I really do need your knowledge concerning this problem. To keep this test short I did not enter all the 21 consonants. I only entered the consonants within the test sentence: Welcome to Foothill. However I did enter all of the vowels into the loops. Again, let me thank you for your kind assistance. I have tried to figure this problem out on my own with the help of my text books, but I know I am missing something.

Hi Norm, I changed my if statement in the for loop concerning consonants to read as follows:

if (( c != 'a' ) && ( c != 'e' ) && ( c != 'i' ) && ( c != 'o' ) && ( c != 'u )
{
   count++;
   continue;
}

console output

Enter the String
A
=====RESULT=====A
=====RESULT=====a
   String s1
   There are 1 vowels.
   There are 1 consonants.

console output

Enter the String
Welcome to Foothill.
=====RESULT=====WELCOME TO FOOTHILL.
=====RESULT=====welcome to foothill.
   String s1
   There are 7 vowels.
   There are 20 consonants.

Norm, progress has been made. In the past when I entered a vowel the output was always: 1 vowel and 2 consonants. Now I get 1 vowel and 1 consonant. The correct answer should be 1 vowel and 0 consonants.

aussiemcgr in a post noted above that you do not reset the "count variable" to 0 after you finish counting the vowels. I do not understand how to do that correctly. Is that one of my problems? Thanks again for your continued help and support.

You can try this
if ( c == 'a' || c == 'e' || c == 'i' || c == 'o' || c == 'u' ) {
// This checks to see if there is a vowel. For each letter checked in the loop if any of these are true than the current letter is a vowel.
// Add the code you need every time you hit a vowel.
}
else {
// If the if statement fails then the letter is obviously a constant. We dont need to check it. Just add your code to handle each constant
// in the else statement. Every time the if statement fails it will jump right to this else statement. And if the if statement is true it will skip this
// else statement.
}

Originally Posted by willie lee

I did what you suggested and all worked well concerning the proper counting of vowels and consonants. However, when I entered the target sentence (Welcome to Foothill.) the count of vowels was correct, but the count of consonants was not. My loop counted 13 consonants. There should be only 10. I checked the output for each word separately and the count was correct. Why does my target sentence not work? I will fiddle with my program again tomorrow. Thanks for your help. I am getting very close to making my program work correctly.

The problem is that the way jcrosby10 suggested spaces will be counted as consonants. If I were you I would first remove the spaces (or any other non-letter characters) from the entered String and then count the vowels. After that the only thing you need to do is subtract the number of vowels from the String's length (which after the removal will be the number of letters only). To remove the spaces (or any other character) from the String you can use the String's replaceAll method. If you google it you will find many examples of how it works.
Hope I was clear.

Norm, I do not understand how I should implement a separate counter for each thing I am counting. Could you please give me one example. In the meantime, I implemented aussiemcgr's suggestion. All worked well, but the consonant count was still wrong concerning the target sentence (Welcome to Foothill.). The consonant count was 13 whereas it should have been 10. I am getting very close to making this program work correctly. I no longer get the 1 vowel and 2 consonants count which was driving me crazy. Thanks again for all the support.

Norm, thanks for your example. My program is counting white space and punct. I did the following:
[code = java]
System.out.println(" Enter the String. ");
String s1 = input.nextLine();

s1 = toUpperCase();
System.out.println("=====RESULT=====" + s1);
s1 = to LowerCase();
System.out.println("=====RESULT=====" + s1);
s1 = s1.replaceAll("\\p{Punct} + ", " ");
System.out.println("=====RESULT=====" + s1);
System.out.println(" String s1 ");
[/code]
Output
[code = java]
Enter the String
Welcome to Foothill.
=====RESULT=====WELCOME TO FOOTHILL.
=====RESULT=====welcome to foothill.
=====RESULT=====welcome to foothill.
String s1
There are 7 vowels.
There are 13 consonants.
[/code]
The replaceAll method, as I used it, does not get rid of the white space and punct. as I wanted. What did I do wrong and how can I correct it.

Just try using && instead of || in the second for loop.I think that should solve a major part of the problem.