O código tem alguns problemas. O principal é usar o operador ==
para comparar strings. Em C uma string é apenas uma sequência de caracteres e precisa de uma função própria para comparar todos eles e retornar se é maior, menor ou igual. Essa função é a strncmp()
. Na verdade qualquer linguagem precisa fazer assim, mas algumas escondem isso no operador, mas isso é outro assunto e não vou entrar em detalhes.
Outro problema desse código é tentar retornar uma variável local. Isso não é possível já que o conteúdo no stack poderá não estar disponível mais. Tem compilador que deixa fazer, o que é ruim já que há grande possibilidade de corrupção de memória. No compilador que eu usei, nem compila.
Então a solução é passar um buffer para a função com memória pré alocada. Aí quem precisa do texto aloca como quiser e libera se necessário. Só é necessário se usar o malloc()
. Em alguns casos é possível usar um array no lugar do ponteiro e alocação dinâmica.
Até poderia fazer a alocação dentro da função e retornar esse ponteiro, mas isso costuma ser ruim porque a programador pode esquecer que ele precisa fazer a liberação. Fica uma coisa assimétrica, a função aloca e outra função cuida da liberação.
Eu fiz retornar o próprio buffer para que a função possa ser usada como expressão também, mas se isso nunca fosse necessário (pouco provável em código real), poderia retornar nada, já que a passagem do buffer é feito por um ponteiro, portanto é uma referência ao real objeto.
Note que eu deixei a função de forma simplificada. Essa função não é segura, nada garante que foi alocado a quantidade de memória suficiente para caber a substring. No momento nada impede do final ser menor que o início, o que seria um erro. Seria legal melhorá-la para tratar essas coisas.
Entendo que isso seja um exercício, mas o uso de memcpy()
é mais adequado nesta situação do que fazer uma laço por conta própria.
Tinha um outro problema somando o final e início da string. Agora não tem mais, mas se isso ainda estivesse no código, estaria errado, é o oposto, deveria ser uma diferença.
Evite usar tipo não sinalizado sem necessidade.
Tem pequenas organizações de código que eu fiz também.
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
char *substring(char *str, char *buf, int start, int end) {
for (int i = start; i < end; i++)
buf[i] = str[i];
return buf;
}
int main() {
char *texto = malloc(6);
if (strncmp(substring("Hello World", texto, 0, 5), "Hello", 5) == 0)
printf("yep");
free(texto);
return 0;
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Em C++ tudo seria diferente porque ele possui um tipo string "nativo".