Thread: Struc Array

SOmeone helps me on this.. i got problem it said
read_data identifier not found....
search_data not found..

here my code

Code:

#include <stdio.h>
#include <conio.h> 
#include <stdlib.h>
struct stud
{
int id;
char name[15];
double gpa;
char fakulti;
};
void main()
{
    struct stud a[50];
    int n,reply,sid,option,i;    
    do
    {
        printf("\n 1. Insert records");
        printf("\n 2. Search record");
        printf("\n 3. exit");
        printf("\n\n Select proper option :");
        scanf("%d",&option);
        switch(option)
        {
        case 1 : // create
            printf("\nEnter Amount of data : ");
            scanf("%d",&n);
            read_data(a,n);
            break;        
        case 2 : // search
            printf("\nEnter ID : ");
            scanf("%d",&sid);
            reply = search_data(a,n,sid);
            if( reply == -1)
                printf("\n Not found");
            else
            {
                printf("\nID\tName\tGpa\tFakulti");
                printf("\n%d\t%s\t%.2f",a[reply].id,a[reply].name,a[reply].gpa,a[reply].fakulti);
            }
            break;
        case 3 : exit(0);
        } //swith
    }while(1);
} // main

int read_data(struct stud a[], int n)
{
    int i;
    printf("\nEnter %d records\n",n);
    for(i=0;i < n;i++)
    {
        printf("\nID : \t");
        scanf("%d",&a[i].id);
        printf("Name : \t");
        flushall();
        scanf("%s",a[i].name);
        printf("Marks : \t");
        scanf("%.2f",&a[i].gpa);
        printf("Fakulti : \t\n");
        flushall();
        scanf("%s",a[i].fakulti);

    } // for
     return 0;
} // read data

int search_data( struct stud a[], int n , int sid)
{
    int i;
    for(i=0;i < n;i++)
    {
        if( a[i].id == sid)
        return(i);
    }//for
    return(-1);
}

Try putting the prototype of these functions before main.

Code:

int read_data(struct stud a[], int n);
int search_data( struct stud a[], int n , int sid);

And never declare main as returning void, main has to return something.
Use int main instead.

i already changed as u said.. but when i run the program and insert input and once reach fakulti user input, it said struct_test.exe is stopped working.. sometime i having this kind of porblem.. btw i using visual

Originally Posted by AndiPersti

Code:

printf("\nID\tName\tGpa\tFakulti");
printf("\n%d\t%s\t%.2f",a[reply].id,a[reply].name,a[reply].gpa,a[reply].fakulti);

You are missing the format specifier for "fakulti".

Code:

scanf("%s",a[i].fakulti);

"fakulti" is a char not a string. You have to use %c (and take care about the dangling newline character)

Bye, Andreas

i already changed it.. but i still got problem.. once i enter input for fakulti, the prgram stopped working..

Show your new program.

Code:

#include <stdio.h>
#include <conio.h> 
#include <stdlib.h>
struct stud
{
int id;
char name[15];
double gpa;
char fakulti;
};
int read_data(struct stud a[], int n);
int search_data( struct stud a[], int n , int sid);
int main()
{
    struct stud a[50];
    int n,reply,sid,option;    
    do
    {
        printf("\n 1. Insert records");
        printf("\n 2. Search record");
        printf("\n 3. exit");
        printf("\n\n Select proper option :");
        scanf("%d",&option);
        switch(option)
        {
        case 1 : // create
            printf("\nEnter Amount of data : ");
            scanf("%d",&n);
            read_data(a,n);
            break;        
        case 2 : // search
            printf("\nEnter ID : ");
            scanf("%d",&sid);
            reply = search_data(a,n,sid);
            if( reply == -1)
                printf("\n Not found");
            else
            {
                printf("\nID\tName\tGpa\tFakulti");
                printf("\n%d\t%s\t%.2f\t%c",a[reply].id,a[reply].name,a[reply].gpa,a[reply].fakulti);
            }
            break;
        case 3 : exit(0);
        } //swith
    }while(1);
} // main

int read_data(struct stud a[], int n)
{
    int i;
    printf("\nEnter %d records\n",n);
    for(i=0;i < n;i++)
    {
        printf("\nID : \t");
        scanf("%d",&a[i].id);
        printf("Name : \t");
        flushall();
        scanf("%s",a[i].name);
        printf("Gpa : \t");
        scanf("%.2f",&a[i].gpa);
        printf("Fakulti : ");
        flushall();
        scanf("%c",a[i].fakulti);

    } // for
     return 0;
} // read data

int search_data( struct stud a[], int n , int sid)
{
    int i;
    for(i=0;i < n;i++)
    {
        if( a[i].id == sid)
        return(i);
    }//for
    return(-1);
}

The problem is with this line:

Code:

scanf("%c",a[i].fakulti);

Change it to:

Code:

scanf("%c", &(a[i].fakulti));

It's a char, not a pointer, so the address must be passed to scanf.

Also, I believe the dot in the specifier in this line is wrong:

Code:

scanf("%.2f",&a[i].gpa);

You should be getting a warning. What did you want to do here?

Code:

scanf("%.2f",&a[i].gpa);

i didnt get any error from this specifier.. i got another problem.. about the return value.. or maybe smth else.. not so sure once finishing enter the input its should display the option..

Thanx guys i already solved this problem.. thank for the help

Originally Posted by iMalc

You don't need those extra brackets in there.

Yes, it's just a bad habit of mine. But you are right, thanks for the fix.