Ad:

Java Discussion Thread
Unsolved
Views: 5162

Nov 5th, 2004

Need Advice on for loops with vowels

Expand Post »

Hi I have written this program to find out how many vowels are in a sentance.
What i need to do now is is capitalise the vowels and put a star next to the capital showing how many times that vowel was used in the sentance.

so that the output is like this
A: ***** (if there were 5 A's)
E: *** (if ther were 3 E's)
I: ***
O: ****
U: *

any help and advice woud be much appreciated

// filename VowelCounter.java
//created Fri 5th nov 2004
// counts vowels in a sentance

import java.io.BufferedReader;
import java.io.InputStreamReader;
import java.io.IOException;

public class VowelCounter

{
public static void main(final String[] pArgs) throws IOException
{

final InputStreamReader tInputStreamReader = new InputStreamReader(System.in);
final BufferedReader tKeyboard = new BufferedReader(tInputStreamReader);

System.out.println(" this will count how many vowels are in a sentance");
System.out.println(" Please enter a Short sentance");
final String tVowel = tKeyboard.readLine();
final int tVowelCount = tVowel.length();
//begin for loop
for (int tVowelNumber =0; tVowelNumber<tVowelCount; tVowelNumber++)
{
final char tVow = tVowel.charAt(tVowelNumber);
final char tLowerVow = Character.toLowerCase(tVow);

// start of if statement
if (tLowerVow=='a' || tLowerVow=='e' || tLowerVow=='i' || tLowerVow=='o' || tLowerVow=='u')
{
System.out.print(tVow);
System.out.println();
}
// end if statement

}
//end for loop
System.out.println(" These are the vowels that are in the sentance");

}
}

Similar Threads

displaying data from a sorted array in C++
need advice on nested loops. in PHP
Loops in Python in Python
Nested For Loops in C++

grifflyn

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

17 posts
since Sep 2004

Permalink

Nov 5th, 2004

Re: Need Advice on for loops with vowels

The simple solution is to keep a counter for each vowel. When you encounter that vowel, increment the counter. Then at the end you can do something like this:

Java Syntax (Toggle Plain Text)
public void print_histogram ( char vowel, int frequency ) {
  System.out.print ( vowel + ": " );
  while ( --frequency >= 0 )
    System.out.print ( "*" );
  System.out.println();
}
public void print_histogram ( char vowel, int frequency ) {
  System.out.print ( vowel + ": " );
  while ( --frequency >= 0 )
    System.out.print ( "*" );
  System.out.println();
}

Call print_histogram for each vowel with the counter as the second argument, and you're sorted.

Narue

Administrator

Reputation Points: 6442

Solved Threads: 1393

Bad Cop

Offline

11,807 posts
since Sep 2004

Permalink

Nov 6th, 2004

Re: Need Advice on for loops with vowels

Hi Narue,
Thanks for the reply can you give me a quick explanation on what "public void print_histogram" does. We havent been shown this in our lectures and i arent sure how it works. I am finding all this Java stuff overwhelming along with the rest of the subjects i am doing.
Is there and way that i can do the program without using "Public histogram" and just using "for", "while" and "do" loops with "charAt"

public void print_histogram ( char vowel, int frequency )
{
System.out.print ( vowel + ": " );
while ( --frequency >= 0 )
System.out.print ( "*" );
System.out.println();
}

grifflyn

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

17 posts
since Sep 2004

Permalink

Nov 6th, 2004

Re: Need Advice on for loops with vowels

>can you give me a quick explanation on what "public void print_histogram" does
It prints a histogram, like the one you wanted:

Java Syntax (Toggle Plain Text)
print_histogram ( 'A', 5 );
print_histogram ( 'E', 3 );
print_histogram ( 'I', 3 );
print_histogram ( 'O', 4 );
print_histogram ( 'U', 1 );
print_histogram ( 'A', 5 );
print_histogram ( 'E', 3 );
print_histogram ( 'I', 3 );
print_histogram ( 'O', 4 );
print_histogram ( 'U', 1 );

Would result in the following:

Java Syntax (Toggle Plain Text)

A: *****
E: ***
I: *** 
O: ****
U: *

My choice of putting the operations inside of a method is irrelevant, you can put everything in one big main method if you want:

Java Syntax (Toggle Plain Text)
/*
  Description: Simple vowel histogram program
  Author:      Julienne (Narue) Guth
  Creation:    20041106
*/
package scratchpad;
 
import java.util.*;
 
public class Main {
  public static void main ( String[] args ) {
    final String vowels = "AEIOU";
 
    Scanner in = new Scanner ( System.in );
    int[] frequency = new int[vowels.length()];
    String line;
 
    System.out.print ( "Enter a line: " );
    line = (in.nextLine()).toUpperCase();
 
    for ( int i = 0; i < line.length(); i++ ) {
      int match = vowels.indexOf ( line.charAt ( i ) );
 
      if ( match != -1 )
        ++frequency[match];
    }
 
    for ( int i = 0; i < vowels.length(); i++ ) {
      System.out.print ( vowels.charAt ( i ) + ": " );
      for ( int j = 0; j < frequency[i]; j++ )
        System.out.print ( "*" );
      System.out.println();
    }
  }
}
/*
  Description: Simple vowel histogram program
  Author:      Julienne (Narue) Guth
  Creation:    20041106
*/
package scratchpad;

import java.util.*;

public class Main {
  public static void main ( String[] args ) {
    final String vowels = "AEIOU";
    
    Scanner in = new Scanner ( System.in );
    int[] frequency = new int[vowels.length()];
    String line;
    
    System.out.print ( "Enter a line: " );
    line = (in.nextLine()).toUpperCase();
    
    for ( int i = 0; i < line.length(); i++ ) {
      int match = vowels.indexOf ( line.charAt ( i ) );
      
      if ( match != -1 )
        ++frequency[match];
    }
    
    for ( int i = 0; i < vowels.length(); i++ ) {
      System.out.print ( vowels.charAt ( i ) + ": " );
      for ( int j = 0; j < frequency[i]; j++ )
        System.out.print ( "*" );
      System.out.println();
    }
  }
}

Narue

Administrator

Reputation Points: 6442

Solved Threads: 1393

Bad Cop

Offline

11,807 posts
since Sep 2004

Permalink

Nov 6th, 2004

Re: Need Advice on for loops with vowels

Hi Narue,
Thanks for the help it still don't make sense to me and when i tried to put the "public void print_histogram" line in i am now getting the following error msg

"C:\Documents and Settings\Administrator\My Documents\A School Documents\Programming\week8\VowelCounter.java:14: ';' expected
public void print_histogram
^
1 error"
The code you used isn't anything like what we have been taught as of yet. I am just hoping that the language will sink in eventualy. Like within the next 3 weeks as i have my first exam then LOL
I doubt that i can blag it if i am stuggling with simple things like histograms

thanks for your help anyway i guess i need to read my book a bit more

grifflyn

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

17 posts
since Sep 2004

Permalink

Nov 6th, 2004

Re: Need Advice on for loops with vowels

>i am now getting the following error msg
Then you're not using the function correctly.

>The code you used isn't anything like what we have been taught as of yet.
Well, since you haven't mentioned your level of experience I have to make some assumptions or dumb down my code so much as to risk insulting you. Here, this doesn't use Strings or arrays, but it's significantly larger even when the histogram method is used:

Java Syntax (Toggle Plain Text)
/*
  Description: Simple vowel histogram program
  Author:      Julienne (Narue) Guth
  Creation:    20041106
*/
package scratchpad;
 
import java.io.*;
 
public class Main {
  public static void print_histogram ( final char ch, int freq ) {
    System.out.print ( ch + ": " );
    for ( int i = 0; i < freq; i++ )
      System.out.print ( "*" );
    System.out.println();
  }
 
  public static void main ( String[] args ) throws IOException {
    int ca, ce, ci, co, cu;
    char ch;
 
    ca = ce = ci = co = cu = 0;
    while ( ( ch = (char)System.in.read() ) != -1 && ch != '\n' ) {
      ch = Character.toUpperCase ( ch );
 
      if ( ch == 'A' )
        ++ca;
      else if ( ch == 'E' )
        ++ce;
      else if ( ch == 'I' )
        ++ci;
      else if ( ch == 'O' )
        ++co;
      else if ( ch == 'U' )
        ++cu;
    }
 
    print_histogram ( 'A', ca );
    print_histogram ( 'E', ce );
    print_histogram ( 'I', ci );
    print_histogram ( 'O', co );
    print_histogram ( 'U', cu );
  }
}
/*
  Description: Simple vowel histogram program
  Author:      Julienne (Narue) Guth
  Creation:    20041106
*/
package scratchpad;

import java.io.*;

public class Main {
  public static void print_histogram ( final char ch, int freq ) {
    System.out.print ( ch + ": " );
    for ( int i = 0; i < freq; i++ )
      System.out.print ( "*" );
    System.out.println();
  }
  
  public static void main ( String[] args ) throws IOException {
    int ca, ce, ci, co, cu;
    char ch;
    
    ca = ce = ci = co = cu = 0;
    while ( ( ch = (char)System.in.read() ) != -1 && ch != '\n' ) {
      ch = Character.toUpperCase ( ch );
      
      if ( ch == 'A' )
        ++ca;
      else if ( ch == 'E' )
        ++ce;
      else if ( ch == 'I' )
        ++ci;
      else if ( ch == 'O' )
        ++co;
      else if ( ch == 'U' )
        ++cu;
    }
    
    print_histogram ( 'A', ca );
    print_histogram ( 'E', ce );
    print_histogram ( 'I', ci );
    print_histogram ( 'O', co );
    print_histogram ( 'U', cu );
  }
}

Narue

Administrator

Reputation Points: 6442

Solved Threads: 1393

Bad Cop

Offline

11,807 posts
since Sep 2004

Permalink

Apr 27th, 2007

Whats wrong with this program

publicclass DebugTwo4
// This program calculates tuition bills
{
publicstaticvoid main(String args[])
{
int myCredits = 15;
double yourCredits = 16.5;
double rate = 75.84;
tuitionBill(myCredits,rate);
tuitionBill(yourCredits,rate);
}
publicstaticvoid tuitionBill(c,r)
{
System.out.println("Total due " + (r*c));
}
publicstaticvoid tuitionBill(c, r)
{
System.out.println("Total due " + (r*c));
}
}

nikkibaby1979

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

1 posts
since Apr 2007

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in Java Forum Timeline: Help Needed

Next Thread in Java Forum Timeline: Problem With Hibernate.cfg.xml File

DaniWeb Message

Cancel Changes

User Name
Password