Возвращаемый указатель из функции C

Question

В моей программе C эта функция будет обрабатывать всю работу по открытию определенного файла, а затем возвращать указатель файла, поэтому основные или другие функции могут читать содержимое с помощью fp, но до сих пор я не смогли заставить это работать.

Я просто изучаю язык, поэтому возможно, что я делаю что-то очень плохое.

int open_text_file(char text_file_name[]) 
{ 
    FILE *fp; 

    if((fp = fopen(text_file_name, "r")) != 0) 
    { 
      return fp; 
    } 

    else 
    { 
      printf("Cannot open file \"%s\"\n", text_file_name); 
    } 
} 

Answer 1

В первой строке, у вас есть

int open_text_file(char text_file_name[]) 

Объявляет тип возвращаемого как int То, что вы должны иметь это

FILE * open_text_file(char text_file_name[]) 

Как хорошо, в вашей "еще" case, вы должны вернуть что-то, чтобы указать сообщение об ошибке вызывающему.

return NULL 

является подходящим выбором. Убедитесь, что вы проверяете возвращаемое значение, когда вы его вызываете.

Answer 2

FILE* open_text_file(); должен быть прототипом.

int не является FILE*.

Answer 3

Функция немного бессмысленна, поскольку все, что она делает, это то, что делает fopen(), а также сообщение об ошибке. Это не очень хороший дизайн, так как ветка ошибки также должна возвращать указатель (предположительно NULL), который затем должен быть снова проверен в вызывающем коде. Лучше было бы просто сказать:

FILE * fp = fopen(somefile, "r"); 
if (fp == NULL) { 
    fprintf(stderr, "Cannot open %s\n", somefile); 
    exit(1); // or whatever you need to do to handle the error 
}