Tem vários erros aí, sem falar a organização e nomenclatura ruim e o uso de coisas que não são necessárias.
Já no código principal se não reservar o espaço para o terminador da string vai corromper memória, e o pior que pode acontecer é funcionar, como acontece em vários casos, e vai achar que está certo.
Para pedir dados use scanf()
ou fgets()
, jamais gets()
. Leia Como ler do stdin em C?.
Para verificar o fim da string veja se encontrou o terminador. Leia Sempre que vou varrer um string em C devo usar o "strlen()", certo?.
Esse tam
não faz o menor sentido, parece um código aleatório que foi colocado ali.
Não corrigi o erro de não converter certo toda faixa de caracteres que pode ser digitado porque este é um erro conceitual, assim como a função diz que vai converter algo e acaba imprimindo esse algo também.
Mudei o retorno da função para compilar corretamente já que não há retorno. Se fosse para ter um retorno teria que codificar isso e provavelmente deveria tirar o printf()
, mas não faz sentido retornar se a conversão é feita inplace no original.
#include <stdio.h>
void convmaiuscula(char texto[11]) {
for (int i = 0; texto[i] != '\0'; i++) texto[i] -= 32;
printf("A palavra em maiusculas %s", texto);
}
int main() {
char palavra[11];
printf("Digite uma palavra: ");
scanf("%s10", palavra);
convmaiuscula(palavra);
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.