Tem vários pequenos erros no seu programa:
O if
para mostrar o resultado está trocado:
if(chave==0)
printf("O caractere informado existe na string.\n");
else
printf("O caractere informado nao existe na string.\n");
O retorno que sai da função é 0
quando não existe, mas no if
o 0
escreve que "O caractere informado existe na string", e por isso deve trocar os dois printfs
.
A leitura do caratere a procurar não está correta:
printf("Informe um caractere: ");
scanf("%s",&chave);
Se é um caratere deve ser lido com %c
, caso contrário o programa vai tentar colocar o terminador \0
devido a ser string
e vai faze-lo num espaço de memória que não lhe pertence, criando um bug subtil, que pode surgir mais à frente.
Ao trocar para %c
vai lhe criar outro problema em relação à leitura anterior de scanf("%[^\n]s"
pois a quebra de linha em si não foi consumida. Para resolver os dois sem complicar pode fazer assim:
scanf(" %c",&chave);
Em que o espaço consome a quebra de linha anterior.
O for
para procura do caratere também não está correto:
for (comp; str[i] != chave; comp--)
{
if(str[i]==chave)
{
return 1;
}
str[i]--;
}
Aqui o inicio de (comp;
é redudante e não faz nada, logo pode remove-lo. Depois se utiliza o i
para aceder à posição e buscar a letra então o i
tem de aumentar e não o comp
diminuir. Para alem disso não está a considerar o caso em que não encontra a letra na condição de fim str[i] != chave
que lhe pode deixar percorrendo a memoria infinitamente.
Correto seria:
for (; i < comp; i++) {
if(str[i] == chave) {
return 1;
}
}
Que fica até mais simples.
Veja o seu código corrigido a funcionar no Ideone
Reinventar a roda
Eu sei que em C muitas vezes temos de facto que reinventar a roda, mas no seu caso está a faze-lo mais do que seria necessário, e a menos que seja para fins educativos deve de o evitar.
str_length
- Já existe uma função nativa para obter o tamanho da string
chamada strlen
,
Toda a lógica que o seu programa está a tentar fazer de achar a primeira ocorrência numa string já existe na função strstr
Para utilizar as duas funções que indiquei precisa de incluir <string.h>
, mas o seu programa fica muito mais curto e simples:
#include <stdio.h>
#include <string.h> //nova inclusão aqui
#define MAX 100
int main() {
char string[MAX];
printf("Digite uma string: ");
scanf("%[^\n]s",string);
int comp = strlen(string); //comprimento com strlen
printf("Comprimento de '%s': %d\n", string, comp);
printf("Informe um caractere: ");
char chave[2];
scanf("%s",chave); //leitura como string pois é necessário para o strstr
if(!strstr(string, chave)) //achar ocorrencia com strstr
printf("O caractere informado nao existe na string.\n");
else
printf("O caractere informado existe na string.\n");
return 0;
}
Veja também este exemplo no Ideone