[SOLUCIONADO] realloc array 3 dimensiones char

[luvac]

Me pueden hechar una mano,tengo una funcion que tiene como parametro un arreglo de tipo char de 2 dimensiones y tengo un arreglo de 3 dimensiones donde meto esta informacion pero necesito redimensionarla en 1 cada vez que entra,vamos que la primera vez [5][1],la segunda vez es [5][2] y asi susesivamente tengo este codigo

Código:

if (record_count==0) //primero verificamos esta variable para saber si hay que inicializar el arreglo
    {
        arr_number=argc;
        Col_name=azColName;
        rows=(char *** )calloc(argc, sizeof(char ** )) ; //se inicializa el array dependiendo de la variable argc
        for(int i = 0 ; i < argc; i++ )
        {
            rows[i] = (char ** ) calloc(1,sizeof(char * )) ;
        }

    }
    else{ //si ya se inicializo entonces solo reasignamos memoria

        for(int i = 0 ; i < argc; i++ )
        {
            rows[i] = (char ** ) realloc(rows[i],(record_count+1) * sizeof(char * )) ;
        }
    }

 for(i=0;i<argc;i++)
 {
 for(j=record_count; j<(record_count+1) ; j++ )
 {
   s=argv[i];
   rows[i][j]=argv[i];
 }
 }

el problema es que si lo hace, pero copia 2 veces la misma información y después 3 y así sucesivamente,y lo que tenia anteriormente lo borra y se va duplicando,alguien me hecha una mano.

[eferion]

Iniciado por luvac

Me pueden hechar una mano,tengo una funcion que tiene el problema es que si lo hace, pero copia 2 veces la misma información y después 3 y así sucesivamente,y lo que tenia anteriormente lo borra y se va duplicando,alguien me hecha una mano.

Código C++:

for(i=0;i<argc;i++)
 {
   for(j=record_count; j<(record_count+1) ; j++ )
   {
     s=argv[i];
     rows[i][j]=argv[i];
   }
 }
 

Ahí estás asignando argv[i] a cada elemento de rows[i], es decir, que si record_count vale 9, los elementos de rows[i][0] a rows[i][9] van a apuntar al mismo sitio: argv[i]

¿Es ese tu problema?

[luvac]

No entiendo?,argv[i] es un array de 2 dimensiones tipo char y cada vez que entra a la funcion argv trae nuevos datos,record_count se incrementa cada vez que sale entonces la primera vez vale 0 y va de 0 al 1,entonces solo agrega 1 elemento a la lista de row cuando termina incrementa en 1 y asi susesivamente creo que no puse el codigo completo lo anexo aqui:

Código:

 int Tadquery::callback(int argc, char **argv, char **azColName){
int i,j,h;
h=0;
char *s;

    if (record_count==0)
    {
        arr_number=argc;
        Col_name=azColName;
        rows=(char *** )calloc(argc, sizeof(char ** )) ;
        for(int i = 0 ; i < argc; i++ )
        {
            rows[i] = (char ** ) calloc(1,sizeof(char * )) ;
        }

    }
    else{

        for(int i = 0 ; i < argc; i++ )
        {
            rows[i] = (char ** ) realloc(rows[i],(record_count+1) * sizeof(char * )) ;
        }
    }

 for(i=0;i<argc;i++)
 {
   for(j=record_count; j<(record_count+1) ; j++ )
   {
     s=argv[i];//esto no lo tomes en cuenta solo lo hize para saber que datos traia
     rows[i][j]=argv[i];
   }
 }
system("cls");
 for(i=0;i<argc;i++)
 {
 for(j =0 ; j < (record_count+1) ; j++ )
 {
  printf("%s \n",rows[i][j]);
 }
 }

record_count++;
return 0;
}

Una disculpa

[eferion]

Código C++:

for(i=0;i<argc;i++)
 {
   for(j=record_count; j<(record_count+1) ; j++ )
   {
     rows[i][j]=argv[i];
   }
}
 

El bucle en cuestión... esto es lo que hace:

• supongo que "record_count" vale 1 y argc 3 (por ejemplo)
• i=0, j=0 =>rows[0][0] = argv[0]
• i=0, j=1 =>rows[0][1] = argv[0] <-- Nota que en [0,0] y [0,1] estás guardando lo mismo
• i=1, j=0 => rows[1][0] = argv[1]
• i=1, j=1 => rows[1][1] = argv[1] <-- Vuelves a duplicar valores
• i=2, j=0 => rows[2][0] = argv[2]
• i=2, j=1 => rows[2][1] = argv[2] <-- Vuelves a duplicar valores

Al final, en "rows" tendrás el siguiente contenido:

Código:

argv[0]    argv[0]
argv[1]    argv[1]
argv[2]    argv[2]

En la siguiente llamada al código, el susodicho bucle volvería a ejecutarse y la matriz "rows" quedaría tal que:

Código:

argv[0]    argv[0]    argv[0]
argv[1]    argv[1]    argv[1]
argv[2]    argv[2]    argv[2]

Mi pregunta es si te refieres a este efecto cuando has dicho que te duplicaba el contenido.

[luvac]

Si a eso me referia,he cambiado esa parte a esto:

Código:

 
for(i=0;i<argc;i++)
 {
   rows[i][record_count]=argv[i];
 }

Pero lo sigue duplicando,¿sigue estando mal?

[eferion]

La pregunta no es si ese código está funcionando mal... la pregunta es ¿qué es lo que pretendes hacer?

[luvac]

tengo esta función "int Tadquery::callback(int argc, char **argv, char **azColName)" que recibe entre otros un arreglo de 2 dimensiones tipo char (argv) lo que pretendo es meter esa información en otro arreglo en este caso de 3 dimensiones ¿porque? bueno para tenerla almacenada y acomodada,así si el argc marca 5 bueno entonces [0][0],[1][0],[2][0] , [3][0] y [4][0] por ejemplo tiene el primer registro que mando, después si se vuelve a llamar entonces incrementamos y ahora [0][1],[1][1],[2][1],[3][1] y [4][1] tienen el segundo registro que mando y así sucesivamente,para después en otra función si el usuario requiere la información se puede buscar mas fácil. Saludos

[eferion]

Bueno, tal cual lo cuentas, tu código ya está haciendo eso.

Eso si, tienes que tener en cuenta que no estás copiando el contenido de los punteros sino que estás copiando los punteros en sí... eso quiere decir que un cambio en uno de los elementos de la matriz conlleva a que dicho cambio se va a reflejar en las otras "filas" de la matriz. Por ejemplo, un cambio en el contenido apuntado por [0][1] se reflejará también en [1][1], [2][1], [3][1], ...

Me explico:

Código C++:

int var = 4;
int* ptr1 = &var;
int* ptr2 = &var;
 
*ptr1 = 8;
 
printf( "%d %d %d", var, *ptr1, *ptr2 );
 

El ejemplo anterior va a imprimir "8 8 8"...

Por otro lado, dado que cada vez que incrementas el tamaño de la matriz la vuelves a rellenar de cero como si no tuviese valores previos, puedes cambiar tranquilamente el realloc por un malloc y un free... realloc tiene la mala costumbre de hacer una copia de la región vieja de memoria a la nueva... te puedes ahorrar ese tiempo si no lo necesitas.

[luvac]

Si creo que estoy copiando los punteros (me lleva!!!),¿como le hago para copiar el contenido del array?

[luvac]

Bueno tenias razon copiaba el puntero mas no la informacion,la maña que tengo por usar ides visuales,pero bueno el codigo quedo asi por si alguien le sirve:

Código:

if (record_count==0)
{
 rows=(char *** )calloc(1, sizeof(char ** )) ;
    for(int i = 0 ; i <1; i++ )
    {
        rows[i] = (char ** ) calloc(argc,sizeof(char * )) ;
        for ( int j = 0 ; j < argc ; j++ )
        {
            rows[i][j] = (char *) calloc(sizeof(argv[i]),sizeof(char));
        }
    }

}
else {
  if ((rows=(char *** )realloc(rows,(record_count+1)*sizeof(char ** )))==NULL)
        printf("Error");
    for(i =record_count ; i < record_count+1; i++ )
    {
       rows[i] = (char ** ) calloc(argc,sizeof(char * )) ;
        for (j = 0 ; j < argc ; j++ )
        {
           rows[i][j] = (char *) calloc(sizeof(argv[i]),sizeof(char));
        }
    }

}

Gracias eferion por hacerme ver mi error.

[eferion]

Dado que son strings, puedes usar la función strcpy para copiar los strings... ahora, ten presente que tendrás que reservar memoria para esos nuevos strings... el código ya se te empieza a complicar con esta configuración

Código C++:

if (record_count==0)
{
    arr_number=argc;
    Col_name=azColName;
    rows=(char *** )calloc(argc, sizeof(char ** )) ;
    for(int i = 0 ; i < argc; i++ )
    {
        rows[i] = (char ** ) calloc(1,sizeof(char * )) ;
        rows[i][0] = (char*) calloc( 20, sizeof(char));
    }
}
else
{
    for(int i = 0 ; i < argc; i++ )
    {
        rows[i] = (char ** ) realloc(rows[i],(record_count+1) * sizeof(char * )) ;
        rows[i][record_count] = (char*)calloc(20, sizeof(char));
    }
}
 

Más o menos quedaría así... esto suponiendo una longitud máxima de 19 caracteres para cada elemento de la tabla.

Y luego para almacenar los datos

Código C++:

for(i=0;i<argc;i++)
{
   strcpy( rows[i][record_count], argv[i]);
}
 

Espero no haber metido la pata en el código.