I'm fairly new to C and I've read a few chapters of a C book I have and now I have to make an assignment but I am all confused hope someone can help me out.
I have to read 2 strings from user (char arrays) input with a max length of 100 characters and convert them to capital letters and print them out with a newline \n after each word.
Until now I have this:
int main() {
char chars[100];
int i = 0;
char str1;
char str2;
int j = 0;
scanf("\n %c", str1);
scanf("\n %c", str2);
while (str1[i] != '\0') {
chars[i] = str1[i];
toupper(chars[i]);
printf(chars[i]);
i++;
}
while (str2[j] != '\0') {
chars[j] = str2[j];
toupper(chars[j]);
printf(chars[j]);
j++;
}
return 0;
}
after it takes the 2 inputs from user, it says stops running and says run failed.
This is your immediate problem:
scanf("\n %c", str1);
This format string tells scanf to read some amount of whitespace followed by a single character, which is to be stored in str1.
@H2CO3 is right: scanf is almost never the right answer for reading input from the user. It performs no bounds checking and is too finicky about the format of user input; one unexpected character will completely confuse it. You are very strongly encouraged to use fgets instead if at all possible:
fgets(str1, 100, stdin);
If you are not allowed to do this, you should ask your professor why not. Seriously.
If you absolutely must use scanf to read input here, you can do it the way @user2479209 described in their answer.
There is another problem with your program, which is that toupper(chars[i]) will not change the value stored in chars[i]. You have to assign the result of toupper back to the array explicitly, e.g.:
chars[i] = toupper(chars[i]);