Problema al inicializar cadenas con punteros

[DickGumshoe]

Hola.

Estoy intentando inicializar una cadena mediante punteros, para reservar el espacio exacto.

Tengo esto, pero al comprobar si copia lo que pongo, me salen solo caracteres raros...

Código C:

void inicializarCadenas(char *cad1, char *cad2, int n)
{
     char c;
     int j=0;
     
     cad1 = (char*)malloc(sizeof(char));
     
     while((c = getchar()) != '\n')
     {
        cad1[j] = c;
        j++;
        cad1 = (char*)realloc(cad1, (j+1) * sizeof(char));
     }
        
     cad1[j] = '\0';
     
}
int main()
{
    char *cadena1, *cadena2;
    int n,i;
    
    inicializarCadenas(cadena1, cadena2, n);
    
    puts(cadena1);
    
    system("pause");
    return 0;
}
 

Muchas gracias!

Saludos.

[DickGumshoe]

Siento el doble post.

He intentado esto:

Código C:

char* inicializarCadenas(char *cad)
{
     char c;
     int j=0;
     
     cad = (char*)malloc(sizeof(char));
     
     while((c = getchar()) != '\n')
     {
        cad[j] = c;
        j++;
        cad = (char*)realloc(cad, (j+1) * sizeof(char));
     }
        
     cad[j] = '\0';
     
     return cad;
}
int main()
{
    char *cadena1, *cadena2;
    int n,i;
    
    cadena1 = inicializarCadenas(cadena1);
    puts(cadena1);
    
    
    
    system("pause");
    return 0;
}
 

Y ahora sí sale.

¿Por qué es necesario devolver la cadena? Creía que si no devolvía nada (void) me hacía los cambios de todos los parámetros automáticamente...

Gracias.

[alexg88]

Pues es muy sencillo, dentro de la función inicializarCadenas, estás asignando al puntero cad nuevas direcciones de memoria, pero ese cambio de dirección no se ve fuera de la función.

Para que pudieses hacerlo así tendrías que pasarle un doble puntero:

Código C:

void inicializarCadenas(char **cad)
{
     char c;
     int j=0;
     
     *cad = (char*)malloc(sizeof(char));
     
     while((c = getchar()) != '\n')
     {
        (*cad)[j] = c;
        j++;
        *cad = (char*)realloc(*cad, (j+1) * sizeof(char));
     }
        
     (*cad)[j] = '\0';
}
 
int main()
{
    char *cadena1, *cadena2;
    int n,i;
    
    inicializarCadenas(&cadena1);
    puts(cadena1);
    
    
    
    system("pause");
    return 0;
}
 

[DickGumshoe]

Muchísimas gracias. Ya he entendido por qué no se hacían los cambios.
Saludos.