Computer science assignment. (strings)

[clook]

i need a make a program that prompts the user to enter a string, and for it to identify the uppercase characters and replace each uppercase char with the corresponding lowercase character. it also needs to count the character

here's what i have so far:

#include<stdio.h>
#include<string.h>
main()
{

char
		letter[20];
	  
int uppercount,i;

	uppercount=0;
  		
	printf("Please enter characters:\n");
	gets(letter);

	if(letter[i]>= 'A' && letter[i] <='Z')      
    uppercount++;
	{
		letter[i] = letter[i] - 'A'+ 'a';
   
 }
	printf("%c\n", letter);
	   printf("The number of uppercase letters entered: %d", uppercount);
}

doesn't seem to work as i would like.. what changes should i make?

 

[interested_learner]

Initialize your variables and be aware the the letter in the printf will attempt to print out the address of the array as a character (you need to dereference your array by adding a and put it inside a loop since %c will print (I believe) out one character at a time). You also need some kind of loop to count the uppercase letters.

 

[piercas]

Hello, thank you for sharing your code with me. It seems like you are on the right track, but there are a few changes that you can make to improve your program. Here are some suggestions:

1. Instead of using the "gets" function, which is known to be unsafe, you can use "fgets" to get input from the user. This function allows you to specify the maximum number of characters to be read, which can help prevent buffer overflow.

2. The "uppercount" variable should be initialized outside of the "if" statement, otherwise it will only count the uppercase letters in the first character entered.

3. In order to replace the uppercase characters with lowercase ones, you can use the "tolower" function from the <ctype.h> library. This function takes in a character as an argument and returns its lowercase equivalent.

4. You can use a "for" loop to iterate through the string and check each character for uppercase letters. This way, you can count the number of uppercase letters and replace them in one go.

Here is an updated version of your code with these changes:

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

int main() {

char letter[20];
int uppercount = 0;

printf("Please enter characters:\n");
fgets(letter, 20, stdin);

for (int i = 0; i < strlen(letter); i++) {
if (isupper(letter)) { // checks if character is uppercase
uppercount++; // increments count
letter = tolower(letter); // replaces with lowercase equivalent
}
}

printf("The modified string is: %s\n", letter);
printf("The number of uppercase letters entered: %d\n", uppercount);

return 0;
}

I hope this helps! Keep up the good work on your computer science assignment.