Thread: problems with structures and linked lists

Im trying to create a structure that has a pointer to a list and allow someone to enter 5 grades. Display that and then enter one more followed by another reprint. Just keep getting an error saying "expecting struct Grade"

any help would be great i have been stuck for a day and a half on this.

Code:

#include <stdio.h>

struct Grade
{
	int value;
	struct Grade *nextaddr;
};

int main () 
{
	struct Grade number1, number2, number3, number4, number5, *first;
	void display (struct Grade *);
	int num1, num2, num3, num4, num5, num6;

	printf("Enter first grade: ");
	scanf("%d", &num1);
	printf("\nEnter second grade: ");
	scanf("%d", &num2);
	printf("\nEnter third grade: ");
	scanf("%d", &num3);
	printf("\nEnter fourth grade: ");
	scanf("%d", &num4);
	printf("\nEnter fifth grade: ");
	scanf("%d", &num5);

	number1.value = num1;
	number2.value = num2;
	number3.value= num3;
	number4.value = num4;
	number5.value = num5;


	first = &number1;
	number1.nextaddr = &number2;
	number2.nextaddr = &number3;
	number3.nextaddr = &number4;
	number4.nextaddr = &number5;
	number5.nextaddr = NULL;
	
	printf("\nTHE GRADES ENTERED ARE:\n");
	
	display(first);	
	
	struct Grade number1, number2, number3, number4, number5, number6, *first;
	
	printf("\nEnter on additional grade: ");
	scanf("%d", &num6);

	
	
	number1.value = num1;
	number2.value = num2;
	number3.value= num3;
	number4.value = num4;
	number5.value = num5;
	number6.value = num6;

	first = &number1;
	number1.nextaddr = &number2;
	number2.nextaddr = &number3;
	number3.nextaddr = &number4;
	number4.nextaddr = &number5;
	number5.nextaddr = &number6;
	number6.nextaddr = NULL;
	
	printf("\nTHE GRADES ENTERED ARE:\n");
	
	display(first);
	
	return 0;
}

void display(struct Grade *contents)
{
	while (contents != NULL)
	{
		printf("%s\n", contents->value);
		contents = contents->nextaddr;
	}
}

Code:

struct Grade number1, number2, number3, number4, number5, number6, *first;

why are you trying to redeclare these variables?

For the large apart, this code looks fine, except for your repeat of code.. what's that about?

i don't know i guess i was failing conservative... it just seemed like it needed to be there.

- remove the 2nd long list of 'struct Grade' declarations
- include in the 1st long list of 'struct Grade' declarations variable named 'number6'
- change the printf statement in the while loop to print a decimal number, not a string

awesome... it works great! I have sucha hard time finding the little things sometimes, but im getting better. Thanks

Code:

void display (struct Grade *);

take this line out of the function. It doesn't go in functions.

should i put it at the top before int main?

yes would be best i think to leave it outside any function, like a global variable