Help on C problems...

[enjoyincubus]

Hi there

i have a problem in C which i cannot complete...can you guys help?


Write a program that asks the user the ff: data

full name
address
age
birthday

and store each record in an array, prints the contents of the array in the order of occurence. The program must have at least two functions

input_data()
print_data()

dont use global variables

Sample output of the problem:


Enter Record # 1
Full name: John Doe
Address: Alabama
Age: 20
Birthday: June 4

Do you want to enter another record? (Y/N)

if the user will press Y it will go back into inputing the data

Enter Record # 2
Full name: White Smith
Address: Vancouver
Age: 30
Birthday: December 15

but instead of Record # 1 it will change to Record # 2 and so on

if the user will Press N then it will show the inputted data from the user:

Enter Record # 1
Full name: John Doe
Address: Alabama
Age: 20
Birthday: June 4

Enter Record # 2
Full name: White Smith
Address: Vancouver
Age: 30
Birthday: December 15

one last problem is to put this program in a file


Help is very much appreaciated!

[enjoyincubus]

pls...i really need this

[enjoyincubus]

heres my code...but theres a lot of errors that i dont uderstand

 #include <stdio.h> 
#include <conio.h>
 
typedef struct {
 
char lname [50];
char fname [50];
char mi;
char address[50];
char birthday[50];
char course[50];
int age;
int year;
}personal;
 
typedef personal per_rec[50];
personal input_data();
void print_data(per_rec[], int ctr);
 
void main(void)
{
personal B; per_rec x[];
int ctr=0;
char choice;
clrscr();
 
do
 
{
 
x[ctr]=input_data();
printf("Enter record? Y/N");
scanf("%c", &choice);
ctr++;
 
}
 
while(choice != "N")
    print_data(B[], ctr);
}getch();
 
}
 
personal input_data();
{
 
printf("last name"); gets(B.lname);
printf("first name"); gets(B.fname);
printf("middle initial")
scanf("%c", &B.mi);
printf("address"); gets(B.address);
printf("age");
scanf("%d", &B.age);
printf("birthday"); gets(B.birthday);
printf("course"); gets(B.course);
printf("year");
scanf("%d", &B.year);
return(B);
 
}
 
void print_data(per_rec x[], int ctr)
{
 
int b = 0;
 
while (b <= ctr)
 
{
 
printf(" ", B[b]);
 
b++;
 
}
 
}
 

[Banfa]

My comments embedded in bold

#include <stdio.h>
#include <conio.h>
 
typedef struct {
    char lname [50];
    char fname [50];
    char mi;
    char address[50];
    char birthday[50];
    char course[50];
    int age;
    int year;
}personal;
 
typedef personal per_rec[50];
 /* Personnaly I find it a little confusing typedef arrays of 
   something, it also leaves your program set to a fix number 
   of records. */
 
personal input_data();
void print_data(per_rec[], int ctr);
 /* per_rec is already an array, I do not think you want an 
   array of per_rec but an pointer to a variable of type per_rec 
   or per_rec * for the first parameter. Of course per_rec[] actually 
   declares a pointer so this is about how the programmer is 
   interpreting what he sees*/
 
void main(void)
 /* main should be declared:
       int main( int argv, char *argp[] )
   or something with similarly typed arguments.  It must return int. */
{
    personal B; per_rec x[];
 /* Put separate declarations on separate lines, per_rec x[]; is of
   no use .  Even if your compiler will declare zero length arrays you 
   never allocate any memory to this array.  Anyway per_rec is already 
   an array type, did you want an array of arrays or did you want
       per_rec x;
   which is an array of personel.*/
    int ctr=0;
    char choice;
    clrscr();
 /* My system does not support this function, it is a 
   non-standard library function. */
 
    do
 
    {
 
 /* The order of this code assumes that the user will always 
   enter at least 1 record */
        x[ctr]=input_data();
        printf("Enter record? Y/N");
 /* "Enter another record?" would be a more appropriate question. */
        scanf("%c", &choice);
        ctr++;
 
    }
 
    while(choice != "N")
 /* Choice is a char "N" is an array of char at 'N' and a '\0', use 'N' if you want the character constant N.  Also you are missing a ; at the end of the while statement.  Finally what if the user enters 'n' or what if they enter 'j' or 'H'? */
    print_data(B[], ctr);
 /* B is not an array declared personal B; and it is never 
   initialised, I suspect you mean per_rec*/
}getch();
 /* The } at the beginning of this line is an error. */
}
 
personal input_data();
{
 /* You use but never declare the variable B in this function. */
 
 /* Gets is an unsafe function as it has no limit on the amount 
   of data it writed to the supplied buffer and can easily cause 
   buffer overflow which can crash the program.  I suggest you use
       fgets( <PointerToBuffer>, <BufferLength>, stdin );
   instead which protects against buffer overflow. */
 
 /* Your printf statements leave no space after the text so the input is squashed together with the prompt line.  Also printf with no newline character is not garuntted to output the data without an fflush(stdin). */
 
 /* Only 1 line of code per line of file. */
 
    printf("last name"); gets(B.lname);
 
 /* Putting all that together this should be 
 
       printf("last name : "); 
       fflush(stdin);
       fgets( B.lname, sizeof B.lname, stdin );
*/
 
printf("first name"); gets(B.fname);
 /* Smae here */
printf("middle initial")
scanf("%c", &B.mi);
 /* scanf input is fraught with trouble, you can use fgets here. */
    printf("address"); gets(B.address);
    printf("age");
    scanf("%d", &B.age);
 /* scanf input is fraught with trouble, you could use fgets on a 
   temporary buffer and use strtol to convert it to an integer.*/
    printf("birthday"); gets(B.birthday);
    printf("course"); gets(B.course);
    printf("year");
    scanf("%d", &B.year);
    return(B);
 /* Returning a structure of ~260 bytes on the stack is not a 
   good idea.  Perhaps you could pass a pointer to a structure to 
   be filled in to this function rather than returning a structure. */
}
 
void print_data(per_rec x[], int ctr)
 /* See comment about per_rec x[] at start of file. This function 
   actually makes no attempt to print anything except spaces. */
{
 
    int b = 0;
 
    while (b <= ctr)
 /* ctrl is the number of filled in records, the indexes in the array 
   have the range 0 - <numberOfRecords>-1.  The condition 
   b <= ctrl will mean that you access 1 record passed the filled in 
   data.  You you have filled in all 50 records this will be an out of 
   bounds memory access. 
 
   you could use a for loop here
 
    for( b=0; b<ctr; b++)
 
    that is how you have implemented your while loop. */
    {
 
        printf(" ", B[b]);
 /* The is no variable B declared in this function, I suspect you 
   want to be referencing x */
        b++;
 
    }
 
}

[enjoyincubus]

i have figured it out

but my only problem is Record #, it wont increase.

ex:

Record #1 <-------- this is the problem how do i increament this?..what is the code?
Name: Ed
Age: 30
...
...

Record #2
Name: ....
Age: .....

i cant seem to make it work

pls help

here is my program btw:

 #include <stdio.h> 
#include <conio.h>
#include <string.h>
#include <ctype.h>
 
typedef struct
{
char sur[20];
char first[20];
char mid[20];
}name;
 
typedef struct
{
name fname;
char address[40];
char age[10];
char bday[20];
char course[20];
char ylevel[10];
}data;
 
void input_data (data a[], int);
void print_data (data a[], int);
 
void main (void)
{
data record[100];
int y=0,cnt=0, x=0;
char ans ='a';
 
while(x<100)
{
if(toupper(ans)!='N')
{
input_data(record,x);
flushall();
clrscr();
printf("There are %d vacant entries \n",100-(cnt-1));
printf("Fill another Record?\t");
ans = getchar();
cnt++;
}
x++;
}
 
while (y<cnt)
{
print_data (record,y);
y+=1;
getch();
}}
 
void input_data (data rec[], int x)
{
clrscr();
printf("\nFullname: \n");
printf("Surname:\t");
flushall();
gets(rec[x].fname.sur);
printf("First Name:\t");
gets(rec[x].fname.first);
printf("Middle Name:\t");
gets(rec[x].fname.mid);
printf("Address:\n");
gets(rec[x].address);
printf("Age:\n");
gets(rec[x].age);
printf("Birthday:\n");
flushall();
gets(rec[x].bday);
printf("Course:\n");
gets(rec[x].course);
printf("Year Level:\t");
gets(rec[x].ylevel);
 
}
 
 
void print_data(data rec[], int x)
{
printf("Fullname: %s %s %s \n",rec[x].fname.sur, rec[x].fname.first,
        rec[x].fname.mid);
printf("Address: %s\n", rec[x].address);
printf("Age: %s\n", rec[x].age);
printf("Birthday: %s\n", rec[x].bday);
printf("Course: %s\n", rec[x].course);
printf("Year level: %s\n", rec[x].ylevel);
 
}
 

[D_C]

In print_data(), something like

printf("Record #%s \n", x);

should be the first line.

[enjoyincubus]

thanks a lot i forgot about that one,

change %s to %d since its integers...

thanks again!