Thread: Problem with do loop in for in my structure in structure program

it doesn't give me syntax error but compile error... i notice it's from my for loop "specifically the do to input a name " i want to know how i can fix this problem in my do if anyone can help me..

Code:

#include <stdio.h>
// This program is a structure which in structure task im doing in class
//THe problem i see in this program lies which in the do loop
//this program suppose to record the names of people and their date, and time..


   /*structure of date*/
     struct date
	{
		int year;
		int month;
		int day; 
     };
	/*structure of time*/
    struct time
	{  int hour;
	   int minute;
	   int second;
    };
    /*structure of the records*/
    struct record
    { char Names[30];
    };
    
    /*structures with structures inside of it*/
    struct info
    { struct date record1[30];
      struct time record2[30];
      struct record people[30];
    };

      int main(void)
  {
      struct info event;
      int temp,num,i,x=0;
      char letter;
	  printf("How many people would u wish to log?\n");
                       scanf("%d",&num);
             if( num > 30){
                  printf("ERROR\n");
                  printf("How many people would u wish to log?\n");
                       scanf("%d",&num); }
             for( i = 0; i < num; ++ i) 
             {

                /* the do is my primary problem in this program*/
                 do
                   {    
                        printf("Enter  name\n");
                        letter = getchar();
                        event.people[i].Names[x] = letter;
                        ++x;
                   }while(letter != '\n');
                        
                        event.people[i].Names[x-1] = '\0';
                        
                 printf("Enter year\n");
                         scanf("%d",&temp);
                 
                 event.record1[i].year = temp;
                 
                 printf("Enter month\n");
                         scanf("%d",&temp);
                 
                 event.record1[i].month = temp;
                 
                 printf("Enter day\n");
                          scanf("%d",&temp);
                 
                 
                 event.record1[i].day = temp;
            
            
             printf("------------------------------\n");
            
            
             printf("NOW ENTER THE hour!!!!!\n");
                         scanf("%d",&temp);
                 event.record2[i].hour = temp;
             
             
             
             printf("ENTER MINUTES!!!!!!\n");
                           scanf("%d",&temp);
             
              event.record2[i].minute = temp;
             
             
             printf("ENTER SECONDS!!!!\n");
                          
                           scanf("%d",&temp);
             
             event.record2[i].second = temp; 
			 
       printf("%s",event.people[i].Names);
     
    }
            
            getchar();
}

it doesn't print out the words i input from getchar, and instead and prints out the printf("Enter world"); and also it print's enter world a certain nubmer of types when u enter a number after it askes u to log....

Originally Posted by Darkinyuasha1

it doesn't print out the words i input from getchar, and instead and prints out the printf("Enter world"); and also it print's enter world a certain nubmer of types when u enter a number after it askes u to log....

getchar() functionn returns an int. Better change the letter to int type. If u wanted to read a string why dont u use fgets function rather than looping around manking it messy

Code:

char str[10];

fgets(str,strlen(str),stdin);  // this would read the string from the key board and store that in the str

ssharish2005