Solamente viendo estas dos líneas ya se aprecian dos problemas en el código: pierde memoria.
char *str = (char *) malloc(sizeof(char)*2);
str = ft_strrev("Es");
La función no recibe el puntero str
en ningún momento y, sin embargo, si que lo modifica, luego la memoria reservada por malloc
se pierde irremediablemente.
str
no es más que un puntero, es decir, es una variable que en vez de almacenar números almacena direcciones de memoria. Lo que hace una asignación no es modificar la dirección almacenada en dicho puntero, con lo que el puntero pasará a apuntar a otra región de memoria diferente.
Si esto no sucediese (que la memoria no se perdiese) el programa tampoco funcionaría proque la reserva que has hecho es claramente insuficiente. Para almacenar 2 caracteres en un string necesitas que el mismo tenga capacidad para, al menos 3 caracteres. Nunca hay que olvidar que los strings terminan con el caracter nulo, luego dicho caracter, aunque no "se vea", sigue formando parte de la cadena y hay que reservarle un hueco.
Revisando la función ft_strrev
ya encontramos otro error grave:
char *ft_strrev(char *str)
{
char aux[20];
str = &(aux[0]);
return (str);
}
La función devuelve un puntero a un array que es local a la función. aux
solo existe mientras se está ejecutando ft_strrev
. Cuando la ejecución abandona la función, la memoria asociada a aux
se libera y puede ser utilizada con otros fines.
No puedes devolver la dirección de memoria de una variable local. NUNCA
Lo que tienes que hacer es reservar memoria dentro de ft_strrev
:
char * ft_strrev(char const* str)
{
char* to_return = (char*)malloc((strlen(str) + 1) * sizeof(char));
return to_return;
}
Ahora ya no se está devolviendo la dirección de memoria de una variable local (stack) sino que to_return
apunta al heap (memoria dinámica) del programa. La memoria dinámica tiene que ser gestionada por nosotros de forma manual, luego somos los que decidimos cuándo dicha memoria se libera y, por extensión, cuando no.
Invertir la cadena ahora no es más que un proceso iterativo: Se recorre str
y se va copiando cada caracter de dicha cadena en la posición que le corresponda en to_return
:
int length = strlen(str);
char* to_return = (char*)malloc(length + 1) * sizeof(char));
for (char* orig = str, * dest = to_return + length - 1; *orig; ++orig, --dest)
*dest = *orig;
to_return[length] = '\0';
return to_return;
count
está demás. El carácter nulo solo sirve para indicar que el string terminó. Por lo tanto no debes tenerlo en cuenta.