Thread: New Programmer, new Error

Originally Posted by ASURugger

Does that still have to do with my linked list even though every time I take in a new string I creat a new struct and allocate the memory for it?

It has everything to do with the fact that you do nothing of the sort. Watch:

Code:

while (fgets(tempname, 40, p) != NULL) {
    tempstudent = malloc(sizeof(*tempstudent));
    strcpy(tempstudent->name, tempname);
    tempstudent->ave = 0.0f;
    tempstudent->next = head;
    head = tempstudent;
}

You have to malloc a node for every student. (This code adds the new student to the front of the line; you'll have to change it if you want the student at the back, or sorted, or whatever.)

I was looking at the wrong updated code. The updated code in the very first post can't possibly compile, as there's no way you can do a malloc to initalize a global object, so I don't know what code you are actually running.

Originally Posted by ASURugger

Code:

fgets(curr->name, 40, p);
                                curr->ave = 0.0;
		curr->next = (Student *)malloc(sizeof(Student));
		curr = curr->next;
		curr->next=NULL;
		count++;

That seems really similar to what youre doing, where am I wrong in this code here so that I don't do it again?

Though they look similar yet the two approaches are poles part as one adds nodes to the tail of the list while the other adds them to the head of the list.

I think I understand now. I was confused by the fact that if I assign head to element it means they are both pointing to the same address. I thought id be replacing a variable.

Thank you for your patience and help. Ill probably have more questions though... lol.

Code:

for (x = 0; x<count; x++)
{		printf("Enter the final average for the student: ");
		scanf_s("&#37;f", i);
		element->ave = i;
		element = element->next;
		
}

Ok there is a definite problem here as it produces an error saying cannot write to location 0x0000000.
Also count is not incrementing as I think it should, I will link my new while loop, mostly borrowed from tab.

Code:

while(!feof(pread))
{
	fgets(tempname, 128, pread);
	for(x = 0; x<128;x++)
	{
		if(tempname[x] == '\n')
		{
			tempname[x] = '\0';
			break;
		}
	}
	count++;
	strcpy(element->name, tempname);
	element->ave = i;
	element->next = head;
	head = element;
	
}

I added the for loop to add a null terminator to the string, i think that would work. count shows up as only 1. Should be 3 from my input file which has 3 names separated by '\n'.
Im assuming this is a pointer error in my linked list because i get the output "Enter student average" but when I input the double number the error occurs and the program stops working. Any suggestions?

But you don't malloc() any storage before copying to element->name.

Hmmm deja vu... I remember making similar corrections to your code last time but here goes...

Code:

while(fgets(tempname, 128, pread))
{
	element = malloc(sizeof(Student));
	if(element)
	{
		strtok(tempname, "\n"); // include <string.h>
		count++;
		strcpy(element->name, tempname);
		element->ave = i;
		element->next = head;
		head = element;
	}
}