Duda malloc, reserva dinámica de memoria

[rodrigoemece]

Acabo de empezar con la reserva dinámica de memoria y no logro entenderlo del todo, ¿podríais echarme una mano?

El programa debe duplicar las letras de una palabra introducida y devolver el número de letras de la original. Todo ello desde una función (Duplicar()). La función también debe reservar memoria dinámica para la nueva palabra.

¿Debo reservar memoria para la primera palabra también? ¿La reserva de memoria la debo de hacer en la función o en el main? Tal como tengo el programa ahora no devuelve la palabra duplicada a no ser que lo ponga así:
(Quitando free(palabra2) y escribiendo *palabra2=(char *)malloc(*nl*sizeof(char)+1); en vez de palabra2=(char *)malloc(*nl*sizeof(char)+1);)

Código C:

void Duplicar(char *palabra, char *palabra2, int *nl)
{
    int i=0;
    int j=0;
    
    while(palabra[i]!='\0') i++;        
    *nl=i;
    i=0;
    *palabra2=(char *)malloc(*nl*sizeof(char)+1);   
    while(palabra[i]!='\0')
    {
        palabra2[j]=palabra[i];
        j++;
        palabra2[j]=palabra[i];
        i++;
        j++;
    }   
    palabra2[j]='\0';    
}
 

Así que intuyo que el problema está en malloc y como lo utilizo, muchas gracias por vuestra ayuda.

Código C:

#include <stdio.h>
#define LNX 9999
void Duplicar(char *, char *, int *);
 
main()
{
    char word[LNX],word2[LNX];
    int letras=0;
    printf("Introduza a palabra a duplicar: ");
    gets(word);
    Duplicar(word,word2,&letras);
    printf("\nA palabra duplicada e: %s",word2);
    printf("\nO numero de letras da palabra orixinal e: %d",letras);
   return 0;
    
}
 
void Duplicar(char *palabra, char *palabra2, int *nl)
{
    int i=0;
    int j=0;
    
    while(palabra[i]!='\0') i++;        
    *nl=i;
    i=0;
    palabra2=(char *)malloc(*nl*sizeof(char)+1);   
    while(palabra[i]!='\0')
    {
        palabra2[j]=palabra[i];
        j++;
        palabra2[j]=palabra[i];
        i++;
        j++;
    }   
    palabra2[j]='\0';
    
    free(palabra2);
 
}
 

[ZeKi]

Primeramente, es obvio que si haces el free(palabra2); no va a funcionar puesto que estás liberando el puntero que guarda la palabra duplicada (recordemos que lo que le estamos pasando a la función es un puntero, no una copia) por lo que todo lo que hagas con ese puntero dentro de la función se mantendrá a la salida.

Luego el tema de *palabra2 en la reserva de memoria. Tienes que usar el asterisco en la asignación del malloc debido a que de esta forma le indicas al compilador el tamaño que tiene lo que está apuntando el puntero, si no lo usas estarías trabajando con la propia variable puntero y no con lo que apunta.

[gonzo13]

Hola,

imagino que nl es el tamaño de palabra, por lo que puedes evitar pasarlo por referencia.

No tiene ningún sentido usar la función malloc (también podrías usar calloc) dentro del bucle, ya que en cada iteración estarías solicitando un nuevo espacio de memoria y dejando "perdida" la que solicitaste en la iteración anterior (es decir, que estás provocando "lacks" o "fugas" de memoria. Algo nada deseable). Tan importante es controlar la solicitud de memoria como la posterior liberación de la misma.

Código C:

#include <stdlib.h>
 
void *calloc(size_t nmemb, size_t size);
void *malloc(size_t size);
void free(void *ptr);
 

Un saludo,
gonzo

p.d.: recuerda que el tamaño de palabra2 es el doble que el de palabra

[rodrigoemece]

Muchas gracias a todos