Asignacion dinamica de memoria

[Carlos2005]

Hola,

con el código en C que pongo a continuación intento conseguir que se lea un vector del que previamente no se sabe el tamaño, y una vez leido lo imprima en pantalla. Uso malloc y realloc para ir asignando memoria conforme se van añadiendo elementos al vector. El problema es que si el vector es de uno o dos elementos me los imprime bien, pero si tiene tres o más elementos, curiosamente solo imprime bien el segundo elemento (indice 1) , el primero lo imprime a cero y el resto va dando valores extrños (siempre los mismos), ¿Por qué ocurre esto? Muchas gracias. Saludos.

El código es el siguiente:

#include <stdio.h>

void leer_vector(int *vect, int *n); //Carga un vector de tamaño indeterminado
void imprime_vector(int *vect, int n); //Imprime un vector de tamaño n

int main()
{
int norigen; //Número de elementos del vector origen

int *vectororigen;
vectororigen = NULL;
int tamano = 1*sizeof(int);
vectororigen = (int *) malloc( tamano );

leer_vector(vectororigen, &norigen);
imprime_vector(vectororigen, norigen);

return 0;
}

void leer_vector(int *vect, int *n)
{
int i=0;
char respuesta;
int nuevo_tamano=0;

do
{
fflush(stdin);
printf("\n¿Elemento %d del vector origen?", i+1);
scanf("%d", &vect[i]);
fflush(stdin);
printf("\n¿Otro elemento?(S/N)");
scanf("%c", &respuesta);
if (respuesta=='s' || respuesta=='S')
{
nuevo_tamano = sizeof(int)*(i+2);
vect = (int *) realloc (vect, nuevo_tamano );
i++;
}
*n = i+1;

}while(respuesta=='s' || respuesta=='S');
}


void imprime_vector(int *vect, int n)
{
int i;

for(i=0; i<n; i++)
printf("\t%d", vect[i] );
printf("\n");
}

[_Lucifer_]

Yo lo probé con 6 números y me funcionó bien, el detalle es que esos "scanf("%c", &respuesta);" nunca me han gustado, siempre he creído que son una fuente de problemas.

Saludos

Agrego: Que no se te olvide liberar la memoria antes de que termine el programa, esta no es la causa de la falla, pero es un detalle importante.

[Mephisto]

Por que no lo debuggeas para ver que no se esten encimando las localidades de memoria que se asignan en el realloc con algun otra variable del prog, esto pudiera ser una causa de que se te pierdan los valores... Mas considerando que a Lucifer si le funciono el código pudiera ser por lo que te menciono...

[fightmx]

Saludos, el paso de parámetros por valor o por referencia aplica también a los punteros, en este caso el valor de 'vectororigen' no es modificado por la función leer_vector puesto que está siendo pasado por valor, un puntero doble resuelve el problema.

Otra cosa, evita el uso de fflush(stdin). (Buscando un poco en la red encuentras la explicación).

[_Lucifer_]

Iniciado por fightmx

...
el paso de parámetros por valor o por referencia aplica también a los punteros, en este caso el valor de 'vectororigen' no es modificado por la función leer_vector puesto que está siendo pasado por valor, un puntero doble resuelve el problema.
...

Si eso es verdad, ¿cómo me funcionó a mi?

Saludos

[fightmx]

Buena pregunta Lucifer, aparentemente todo funciona como se espera, aunque muy en el fondo este caso es algo particular, cada vez que se invoca realloc estamos reasignando nuestro bloque de memoria, este nuevo bloque puede o no coincidir con la posición del bloque que teniamos previamente reservado, esto en realidad como tal no lo controlamos. Ahora, lo que esta sucediendo aquí es que el bloque de memoria que estamos reasignado sigue conservando la misma posición inicial en memoria que el bloque de origen y esto ha permitido que el puntero "vectororigen" siga apuntando al mismo, lo cual no significa que este bien.

Un simple cambio en el código puede dejar ver el error, ya que realloc reubicará el bloque en memoria:

Código:

void leer_vector(int *vect, int *n)
{
	int i=0;
	char respuesta;
	int nuevo_tamano=0;
	int *ptr = malloc(sizeof *ptr);

	do
	{
		fflush(stdin);
		printf("\n¿Elemento %d del vector origen?", i+1);
		scanf("%d", &vect[i]);
		fflush(stdin);
		printf("\n¿Otro elemento?(S/N)");
		scanf("%c", &respuesta);
		if (respuesta=='s' || respuesta=='S')
		{
			nuevo_tamano = sizeof(int)*(i+2);
			vect = (int *) realloc (vect, nuevo_tamano );
			i++;
		}
		*n = i+1;

	}while(respuesta=='s' || respuesta=='S');
}

Otra cosa Carlos2005 cada vez que utilices malloc o realloc incluye stdlib.h.

Un Saludo.