Thread: problem printing out structure after loading

hi,
I am reading from a file which has 6 records(for now) and loading them into a structure. but when i try to print it back, it only prints the last record 6 times. it prints all 6 records right after loading it. but printing the whole structure later fails. i know theres a prob but am unable to figure out. any help will be appreciated. thanx in advance. here are some snippets of my code...

2 structures...

Code:

typedef struct Cars{
	int id;
	char make[SIZE];
	char model[SIZE];
	char color[SIZE];
	int year;
	long price;
	CAR_TYPE type;
}Car;

typedef struct CarInventory{
	Car **pList;
	int maxSize;
	int curSize;
}CarInv;

memory allocation
cInv.pList = (Car **)calloc(cInv.maxSize, sizeof(Car *));(in main)

Code:

BOOL bReallocateMemory(CarInv *cInv, Car *aCar)
{
	if(cInv->maxSize <= cInv->curSize )
	{
		cInv->maxSize = (cInv->maxSize >= 1) ? cInv->maxSize * 2 : 1;
		printf("Reallocating array size to %d", cInv->maxSize);
		cInv->pList = (Car **)realloc(cInv->pList, cInv->maxSize * sizeof(Car *));

		if(cInv->pList == NULL)
		{
			printf("realloc failed\n");
			return FALSE;
		}
	}
	cInv->pList[cInv->curSize] = (Car *)malloc(sizeof(Car));
	if(cInv->pList[cInv->curSize] == NULL)
	{
		printf("malloc failed\n");
		return FALSE;
	}
	cInv->pList[cInv->curSize] = aCar;
	printf(" id is cInv->pList[%d]->%d", cInv->curSize, cInv->pList[cInv->curSize]->id);
	printf(" make is cInv->pList[%d]->%s", cInv->curSize, cInv->pList[cInv->curSize]->make);
	(cInv->curSize)++;

	return TRUE;
}

printing in another function

Code:

for(i = 0; i < cInv->curSize; i++)
	{
		printf("\n%2d %16s %16s %16s %6d %8ld %6s",
				cInv->pList[i]->id,
				cInv->pList[i]->make,
				cInv->pList[i]->model,
				cInv->pList[i]->color,
				cInv->pList[i]->year,
				cInv->pList[i]->price,
				carType[cInv->pList[i]->type]
				);
	}

i tried tht too.......but doesn't work..... i don't know wht else to do. I have been trying to figure it out...but no such luck. thanks anyway :-)

i printed out maxSize & it was 0. so i did a malloc. is this right?

Code:

cInv.pList = (Car **)malloc(sizeof(Car *));
if(cInv.pList == NULL)
{
	printf("malloc failed\n");
	return 3;
}

but still the same problem persists.

initially curSize is 0, but i keep incrementing it when user wants to add a record

pList is a pointer to a pointer to a structure & i keep mallocing the array as well as each element in the array to store a record. theres no limit to how many records i can add

as to the statement below,

Code:

cInv->pList = (Car **)realloc(cInv->pList, cInv->maxSize * sizeof(Car *));

how can i declare a temporary var as pList is of type Car **?

i still havn't leant about linked lists.......sorry!

yes u r right. i don't even know anything abt linked list. gotta do it this way.... & i can add records too now. it writes to file, loads to memory...... but the only thing which it isn't doing is printing from memory......i donno whts wrong......... sorry i shld say printing properly, coz it prints the last record as many times as there are records :-(

no. aCar is just an object..... Car aCar. do u want me to post the whole code?

yes i do & this is it....

Code:

void vPrintList(const CarInv *cInv)
{
	int i = 0;
	int index = 0;
	char carType[6][10] = {"", "SEDAN", "COUPE", "SPORT", "SUV", "TRUCK"};

	for(i = 0; i < cInv->curSize; i++)
	{
		printf("\n%2d %16s %16s %16s %6d %8ld %6s",
				cInv->pList[i]->id,
				cInv->pList[i]->make,
				cInv->pList[i]->model,
				cInv->pList[i]->color,
				cInv->pList[i]->year,
				cInv->pList[i]->price,
				carType[cInv->pList[i]->type]
				);
	}
	return;
}