Thread: struct array not showing integer variable

Greetings everyone, I am a newbie to C and eager to learn.
Below is a snippet and reworded version of my problem, the scanf entry for my arrayone[] integer values are not being written to the text file. Everything else in the code is pretty much working except that bug. Is the snippet below sufficient for me to get some insight?

regards.

Code:

void loadgame(struct gamerecord arrayone[], int size)
{
    int i;


    gm = fopen("gameinformation.txt", "a");


    if (gm != NULL)
    {
        for (i = 0; i < size; ++i)
        {
            printf("\nEnter game name: ");
            scanf("%s", arrayone[i].title);
            printf("Enter The sequence number :");
            scanf("%d", arrayone[i].sequence);
            printf("Enter The list number :");
            scanf("%d", arrayone[i].list);
            
            
            //write items of data to the file
            fprintf(gm, "\n%s\t", arrayone[i].title);
            fprintf(gm, "%d\t", arrayone[i].sequence);
            fprintf(gm, "%d\t", arrayone[i].list);
            


        fclose(gm);
        }
    }

Can you provide input/output examples and what does your struct look like?

Originally Posted by jdodle

Can you provide input/output examples and what does your struct look like?

__________________________________________________ _______
This is an example of the output from the text file (from my actual project)

title first last 3733720 3733724 ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌ̈”Øðù8

Code:

struct gamerecord
{
    char title[50];
    int sequence[1];
    int list[3];
}arrayone[];

The previous post contains a few output not in the struct, (I changed the code to something simple, I dont want to post the full code online) I only have issues with the integer component that is giving me "3733720 3733724 ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌ̈”Øðù8" as values. I hope that is not an issue.

Originally Posted by Salem

You should pay attention to your use of printf/scanf formats.

$ gcc -Wall bar.c
bar.c: In function ‘loadgame’:
bar.c:31:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘int *’ [-Wformat]
bar.c:32:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘int *’ [-Wformat]

Why are sequence and list arrays of int, when you only read one int into each?

I made sequence and list into an array, in order to use arrayone[] as a for loop, without list and sequence being an array, how would I be able to use it in a for loop to capture only one entry? I would use a while loop if it works better.

the requirement of the project says I should use an array of structures. I thought everything should be an array. Forgive me if I am in error.

Originally Posted by Salem

You should pay attention to your use of printf/scanf formats.

$ gcc -Wall bar.c
bar.c: In function ‘loadgame’:
bar.c:31:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘int *’ [-Wformat]
bar.c:32:7: warning: format ‘%d’ expects argument of type ‘int’, but argument 3 has type ‘int *’ [-Wformat]

Why are sequence and list arrays of int, when you only read one int into each?

Why are sequence and list arrays of int, when you only read one int into each?

THANK YOU!!!!!!!!!!!!

I was using duplicated arrays. I removed the array from within my struct and it works!!!!

I am still getting garbage after the next entry but I will fix that

output looks like this now

"Title FIRST LAST 1 123 ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌ®Bfì@÷D"

I will figure out the "ÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌÌ®Bfì@÷D"... I hope

I realized I wasnt allocating enough space in my array to hold my string.

That resulted in my garbage output.

I allocated the space and now this part of my program runs as it should.

Now I shall move on to the other areas. Thank for prompting me to think, review and coming to a conclusion. That knowledge is invaluable.