Thread: assigning value to a char array in a structure

Hello, I am a first year student. I use gcc/netbeans to work with c. I have made a global structure then am trying to create another of that type that is an array.
i am trying to fill the array->charname[] inside of a switch statement. I have tried all the reference material available to me and am lost and tied. I have totaly swithced directions and am considering using a function instead of switch, but fear I am over complicating a simple task. I hope this isn't rude, please forgive,
here is my noodle mess:

Code:

/* 9B
* File: main.c
* Author: Clay
*
* Created on November 13, 2011, 7:29 PM
*/
#include <stdio.h>
#include <stdlib.h>
#define MNAME 30
#define MSTUD 50
#define MGRADE 4
struct NameGrade // Global structure NameGrade
   {
     char name[MNAME];
     int grade;
     struct NameGrade *nextstud;
   };
int main() 
  { //Variables 
     char choice, entkey, cont; 
     int tempG, i, ct;
     char tempN[MNAME]; 
     struct NameGrade Student[MSTUD];
     typedef struct Student STU; 
//ASK user for input
      printf("Enter your desired action below:\n"
     "A to create a linked list of names and grades.\n"
     "B to add a new student to the list.\n"
     "C to modify a student in the list\n"
     "D to delete a student from the list.\n"
     "E to display the list.\n"
     "F to EXIT this program.\n");
     scanf("%c", &choice);//USER enters choice from list
//switch to execute USERS choice  temp disabled till I fix STU.name 
//switch (choice) 
//{Create an initial linked list of students and grades 
// case 'A':
// case 'a': 
       if(choice == "a"||"A")  //this IF is temporary till I figure
//out why it isn't taking name
      while(cont != 'n' || 'N');
           {
               i = 0;
               printf("Enter Name: "); 
               scanf("%s", &tempN);
               Student->name[i] = tempN[i];
               printf("Enter a grade for %s.\n", Student->name[i]); 
               scanf("%d", &tempG);
               printf("Continue entering names and grades?\n"
               "Y for yes\n"
               "N for no\n");
               scanf("%c", &cont); 
               i++;
            }
/* break;           I have stopped here obviously***************

//Insert a new student into the list
case 'B':
case 'b': 
//
break;
//Modify an existing student in the list
case 'C':
case 'c': 
//
break;
//delete an existing student from linked list
case 'D':
case 'd': 
//
break; 
//Display the linked list done?
case 'E':
case 'e': 

//ct = 0;used to count list entries
for(i = 0; STU[i] != '\0'; i++)
//ct++;
printf(STU[i]);


break; 
//EXIT from the program
case 'F':
case 'f': 
// exit;
break;
*/
return(0);
}

Wow, huge difference. Thank you DRK. I did the changes and its working, except when it is supposed to scanf for cont, it jumps to enter name followed by grade. If I enter n for grade it exits succesfully. Any ideas?

Never mind... I moved the printf and scanf for cont after the i++ and it works great. Thank you VERY much once again.

Ok, I am trying to print the list now and can't figure out malloc. I believe I need to use malloc for Student however Student is not a ptr. This is what I have so far:

Code:

#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#define MNAME 30
#define MSTUD 50
#define MGRADE 4
#define TRUE 1
#define FALSE 0
int EXIT = FALSE;
struct NameGrade  // Global structure NameGrade
      {
       char name[MNAME];
       int grade;
       struct NameGrade *nextstud;
      };
      
int main()
{
          //Variables        
         char choice, cont = '\0';    
         int tempG, i, ct;
         char tempN[MNAME];
  
struct NameGrade Student[MSTUD];                     //here is my trouble
Student[] = (int *) malloc(MSTUD * sizeof(int));
if (Student == (int *) NULL)
    {
    printf("Failed to allocate Student structure array\n");
    exit(1);
    }

Ok, I am trying to print the list now and can't figure out malloc. I believe I need to use malloc for Student however Student is not a ptr. This is what I have so far:

Code:

#include <string.h>
#include <stdio.h>
#include <stdlib.h>
#define MNAME 30
#define MSTUD 50
#define MGRADE 4
#define TRUE 1
#define FALSE 0
int EXIT = FALSE;
struct NameGrade  // Global structure NameGrade
      {
       char name[MNAME];
       int grade;
       struct NameGrade *nextstud;
      };
      
int main()
{
          //Variables        
         char choice, cont = '\0';    
         int tempG, i, ct;
         char tempN[MNAME];
  
struct NameGrade Student[MSTUD];                     //here is my trouble
Student[] = (int *) malloc(MSTUD * sizeof(int));
if (Student == (int *) NULL)
    {
    printf("Failed to allocate Student structure array\n");
    exit(1);
    }

You don't need, and actually cannot use malloc unless you want to allocate memory to a pointer. But here you do :

Code:

struct NameGrade Student[MSTUD];

Which means that you declare a statically allocated array of struct NameGrade, named Student with MSTUB size, and the memory is already allocated. And if you would use malloc, here is what you would do :

Code:

struct NameGrade *student = NULL;
student = (struct NameGrade*)malloc(MSTUD * sizeof(struct NameGrade) );

Note that I replaced your (int*) and (int) in the malloc, because the first one is to cast the return of the "malloc" (which even if not necessary is often clearer), and the second to give the size of the memory to allocate (here MSTUD times the size of the your structure).

But I think here you want to do a linked list, so you should allocate the elements one by one in a loop and link them to each other.

Thank you laserlight for these information!
I used this way because I think it is easier to understand the usage of malloc like that because the code tells you better what it is doing like : "I want a struct NameGrade pointer that will point to MSTUD struct NameGrade". At least that's how I learned it and I remember that I had a hard time with pointers, and that kind of stuff were helping.

And I didn't know for the first part, but I remember having some "implicit declaration of malloc" warnings with everything working fine, is it because my compiler was smart enough to find it by itself?

So for dynamic MSTUD should be a variable. Just declare it as 'int MSTUD' and do not initialize it. I will give this a whirl. This is begining to get confusing and very exciting. Thank you LaserLight and Tibo-88.

It worked great. Now that I created the list I am going to try to delete from the list and modify existing elements in the list.