Thread: Having trouble with a simple program......

I've been working on this program that should be simple for the last hour or so with no luck.

The question reads :

Write a function to read in a line of text (use getchar() routine) and display only the capital letters contained in the text. You can declare and use an array to store the text and a pointer to access the array.

What i've attemped so far is :

Code:

#include <stdio.h>
#include <ctype.h>


int main(){

char line_of_text[100], *charPtr;

charPtr = line_of_text[0];

while (charPtr = getchar());

if ((charPtr >= 'A') && (CharPtr =< 'Z'))

putchar();

However DevC++ is telling me that theres a problem with the charPtr = getchar() part. I'm also at a loss how to keep the loop running when it comes across a non capital letter , i think the way mine is itll simply stop the loop so i'm trying to think of something to put in the else part thatll cause it to go through the loop again. I'm sure this is a really easy problem for most people but i am utterly rubbish at programming and cant figure it out

The below line of code will infinitely waiting by accepting the characters.
This is wrong

Code:

while (charPtr = getchar());

Probably , you may be looking when the user hits an enter key then that is the end of the string.
[code]
while (charPtr = getchar())
{
line_of_text[i++] = charPtr;
if(charPtr == '\n')
break;
}
[code/]

Next Step you need to do is filter the Capital letters from the String.

Sorry for the Code Tags


Probably , you may be looking when the user hits an enter key then that is the end of the string.

Code:

while (charPtr = getchar())
{
line_of_text[i++] = charPtr;
if(charPtr == '\n')
break;
}

Next Step you need to do is filter the Capital letters from the String

This program is highly confusing me so i figured i'd try and get the basic operation of getchar and putchar down. I just threw this code out, is this the correct way to use them ?

Code:

#include <stdio.h>
#include <ctype.h>


int main(){



int numChar;

while ((numChar = getchar()) != '\n')
{
putchar(numChar);
}


return 0;

}

Whats confusing me is the bit at the end of the question that says "You can declare and use an array to store the text and a pointer to access the array" I gather from that that i need to input my line into an array, using a pointer to access it so i can filter out the capital letters, but when i try to use the getchar routine on the pointer it doesnt work, so i'm guessing theres another way.

Yeah the fact it said you could use an array and a pointer was what was really annoying me.

I've put in another line but i still think theres something else needed, cause im not sure what would happen if a non-capital letter came up. This is my current code

Code:

#include <stdio.h>
#include <ctype.h>


int main(){



int numChar;

while ((numChar = getchar()) != '\n')
{
if (isupper(numChar))
putchar(numChar);
}


return 0;

}

I think i need an else which will make the while loop start again, but i'm not sure if thats the right way to go about it. Thanks for your help.

Usually a question like that can be answered by compiling and running the code.
In this case if numChar is a small letter, the if is false and nothing will be put until the next iteration.

// look
int c;
while ((c = getchar()) > 0 && c != EOF) {
if (c >= 'A' && c <= 'Z')
putchar (c);
if (c == ' ' || c =='\t')
putchar (c);
}

/* - you read stdin with getchar into c variable (char c is enough for me but D.R. said int)
* - then check if you have reached the End Of File (quite lazy approach)
* - then you check okay is this really a capital letter? (it could be a more complicated
* function call when u grow up)
* - and then fine! print it out with putchar()
* (putchar does really require an argument as in put what?)
* - and for the dessert u might want to leave the formatting characters so
* u dont make new words (changing facts is really not a good idea);
*/