Thread: 'request for member 'track' in something not a structure or untion

Trying to scanf to a struct, but when compiling, I am given: 'request for member 'track' in something not a structure or union.

I have looked around on the internet and one suggestion was to replace the '.' with '->', which did not help. I have a feeling it is to do with where and how I have placed the struct, as example codes seem to place them in a different 'main'. Also, I am unsure if I should be entering an array number into the 'scanf' if there is an 'if statement' specifically for array entry.

Thank you! I hope what I have asked makes sense.

Here is the code, excluding the majority of the switch statement:

Code:

#include <stdio.h> 
#include <stdlib.h> 
#include <unistd.h>
#include <windows.h>//for changing colour

struct song //creates a struct called song
{   
    char track[20];
    char trackname[20]; 
    char artist[20];
    char genre[20];
};

int main(void)

{
    system("color f5");//gives the window a white background (f) and purple text (5)
    
    
    int menu, option, playchoice; 
    char song[20];
    
    printf("Make a selection\n\n");
    
    printf("1=Play\n");
    printf("2=Music\n");
    printf("3=Manage\n");
    printf("4=Four\n");
    printf("5=Quit\n");
    scanf("%d", &menu );
    
    switch( menu ) 
    
    {
            

        case 3: 
            

    int qtyadd;
    
    printf("Number of tracks to add: \n\n");
    scanf("%i", &qtyadd);  
    
    { 

             printf("Filepath for track to add: \n\n");
             scanf("%s", song[1].track]; //references to the first record and its first variable
             printf("Name of track: \n\n");
             scanf("%s", songs[1].trackname);
             printf("Artist: \n\n");
             scanf("%s", songs[1].artist);
             printf("Genre: \n\n");
             scanf("%s", songs[1].genre);
} 
for(i=0;i<qtyadd && i<99;i++) 


struct song songs[99]; 
int i; 
for(i=0;i<qtyadd && i<99;i++) ;
            
            
        break;

    }

    		
system("pause"); 
}

Code:

            scanf("%s", songs[1].track]; //references to the first record and its first variable

But you have to define the array of song before you use it.
Kurt

Thank you, ZuK. The code stopped that problem. It now says: syntax error before ']' token.

I am assuming that that is because you said I would have to define the array of song before I use it, which I am unsure about, but I changed the struct to:

Code:

struct songs[99] //creates a struct called song
{   
    char track[20];
    char trackname[20]; 
    char artist[20];
    char genre[20];
};

and it says that there is a syntax error before '[' token, referencing to the first line ('struct songs [99]')

You need to define the struct before you can instantiate it. When you say "struct songs {" you're still defining it. You can start declaring instances right after the definition is over.

If you want to declare an array of structs, you have a couple of ways of doing it:

Code:

struct song //creates a struct called song
{   
    char track[20];
    char trackname[20]; 
    char artist[20];
    char genre[20];
} info[99];

or

Code:

struct song //creates a struct called song
{   
    char track[20];
    char trackname[20]; 
    char artist[20];
    char genre[20];
} ;

int main(void)
{
      struct song info[99];
      .....

}

Either way, you have to declare it before using it, as Zuk suggested

your original code was nearly ok
just move

Code:

struct song songs[99];

up
Kurt

Thanks all to your replies. I tried all these, but could not sort the problem out -probably because I don't have much programming experience and didn't do it properly.

However, the code on its own works (without the switch statement), but the switch also works on its own, so it's to do with the way I lay it out when putting it into the switch.

Finally got the code to work!

For some reason I had made a duplicate of a 'for()' and I'm not sure what else was done to fix it. A friend fixed it and he did it too quickly!

Thank you all for your time though!