[maria_isabel]

I neeed a program to perform the following tasks:
• Read in a c-string from the user.
• Reverse the string (e.g. “ABC” would become “CBA”).
• Make all of the characters uppercase.
• Remove all the vowels from the string
• Print out the new c-string.
See the screenshot below as a guide:

Please enter a string of at most 30 characters:
The quick brown fox...
Here is your vowel-free string!
...XF NWRB KCQ HT
Press any key to continue . . .

the program i have does not work properly but it compiles and run and i dont know whats wrong with it

[JackOfAllTrades]

Dream.In.Code has a policy by which we prefer to see a good faith effort on your part before providing source code for homework assignments. Please post the code you have written in an effort to resolve the problem, and our members would be happy to provide some guidance. Be sure to include a description of any errors you are encountering as well.

Please post like this:

Thank you for helping us helping you.

[Drekey]

Show us the code Maria

[maria_isabel]

#include <iostream>
#include <cstdlib>

using namespace std;


int main()
{//count will hold the number of characters in the character array
 //number will mark the end of the characters after the vowels are removed
 //myString is the string(character array) that the user enters
 //targetString is storage for the uppercase version
 //noVowel will store the vowel eliminated string 
 int i,count, number;  
 char myString[31], targetString[31], noVowel[31];
 
 cout<<"Please enter a string of at most 30 characters: "<<endl;
 cin.get(myString,31);
 //first need to find the length of the string
  i=2;
 while (myString[i]!='\0')
	 i++;	
 count=i;
 targetString[count]='\0';
 //loop to uppercase and reverse the string
 for (i=count-1;i>=0;i--)
	 targetString[count]=toupper(myString[i]);
 number=0;
 //strip all the vowels out
 for(i=0;i<=count;i++){
	 if(targetString[i]!='A' && targetString[i]!='E' && targetString[i]=='I' 
			&& targetString[i]!='O'&&targetString[i]!='U') {
	   noVowel[number]= targetString[i];												   
	   number++;													
	 }//end if
 }//end for
 //null terminate the string
 noVowel[number]='\0';
 
 cout<<"Here is your vowel-free string!";
 cout<<endl;
 cout<<targetString[i]<<endl;
 system("PAUSE");
 return 0;
}

This post has been edited by maria_isabel: 20 January 2009 - 11:12 AM

[Drekey]

what is it printing?

cause i see you're printing just one element from the targetstring. dunno if that's the problem.

[dan_ram]

i was quite confused but i've made some changes:

#include <iostream>
#include <cstdlib>
using namespace std;
int main()
{
	int i, count, number;
	char myString[31], targetString[31], noVowel[31];
	cout << "Please enter a string of at most 30 characters: " << endl;
	cin.get(myString, 31);
	/* //first need to find the length of the string
	  i=2;
	 while (myString[i]!='\0')
		 i++;
	 count=i;
	 targetString[count]='\0';*/
	count = strlen(myString);
	targetString[count+1]='\0';
	for (i = count;i >= 0;i--,count--)
		targetString[count] = toupper(myString[i]);
	number = 0;
	for (i = 0;i <= strlen(myString);i++)
	{
		if (targetString[i] != 'A' && targetString[i] != 'E' && targetString[i] != 'I'
				&& targetString[i] != 'O' && targetString[i] != 'U')
		{
			noVowel[number] = targetString[i];
			number++;
		}
	}
	noVowel[number] = '\0';
	cout << "Here is your vowel-free string:";
	cout << endl;
	cout << noVowel << endl;
	system("PAUSE");
	return 0;
}

the changes:
you can use strlen to find the length of string,
in the loop to reverse and uppercase, you had not reduced the value of count,
mostly, you gave targetString[i]=='i'
while outputting, to get the whole string just say cout<<noVowel and not the with "i"