Is this Correct?

[marniel647]

im using a subprogram is this the correct usage of void..

#include <conio.h>
#include <stdio.h>
#include <string.h>

void getEmpDetails(char *empID){
  FILE *empt;
  char employeeID[50];
  char empname[50];
  char salLevel[50];
  char *comString;
  while (!feof(empt)){
   fgets (comString,sizeof(comString),empt);
   if (strstr(comString,rString)){
     printf ("Emp Name: %s\nEmp ID: %s\nEmp Sal Lvl: %s",empname,employeeID,salLevel);
     break;
   } else {
    printf("Emp Not Found");
    return 1;
   }
  }
}

main(){
 clrscr();
 char empID[50];
 printf("Enter Emp ID: ")
 scanf("%s",empID);
 getEmpDetails(empID);
 getch();
 return 0;
}

[ascii]

a void function cant return a value, so no. you must delete the

return 1

and it will work

[Zhuge]

Rather, change it to simply return; if you actually want to return from the function (which I assume you do).

[ascii]

well its good practice to add that, but otherwise it will return automatically at the end of the function.

[marniel647]

so when i enter emp id in the main..

it can get by the void..?

[ankitsingh12]

The function is not returning the emp id/details. Your function is just printing the details, which it will continue to do irrespective of the return type. If the return type is void then:
return;
is a good practice.