c - 检查目录时出现指针问题

Question

我正在制作一个函数(一个更大项目的一部分)，它检查 C 中是否存在文件。这是代码:

#include <stdio.h>
#include <sys/types.h>
#include <dirent.h>
int file_exist(file_path, file_name)
{
  DIR *dp;
  FILE *fc;
  struct dirent *ep;

 dp = opendir(file_path);
 fc = fopen(file_name, "r");
  if(dp = NULL)
    error_escape("Opening path");
  else
    chdir(file_path);
  if(fc = NULL)
  {
    error_escape("Opening file");
    return(-1);
  }
  else
    return(0);
}

但是，当我尝试编译它时，我得到了这个结果:

file_exist.c: In function ‘file_exist’:
file_exist.c:10:15: warning: passing argument 1 of ‘opendir’ makes pointer from integer without a cast
  dp = opendir(file_path);
               ^
In file included from file_exist.c:3:0:
/usr/include/dirent.h:134:13: note: expected ‘const char *’ but argument is of type ‘int’
 extern DIR *opendir (const char *__name) __nonnull ((1));
             ^
file_exist.c:11:13: warning: passing argument 1 of ‘fopen’ makes pointer from integer without a cast
  fc = fopen(file_name, "r");
             ^
In file included from file_exist.c:1:0:
/usr/include/stdio.h:272:14: note: expected ‘const char * __restrict__’ but argument is of type ‘int’
 extern FILE *fopen (const char *__restrict __filename,
              ^

我已经在很多地方寻找解决方案，并尝试了许多不同的方法来查看是否可以修复它，但我就是不知道该怎么做。预先感谢您提供的所有帮助。

Answer 1

file_exist.c: In function ‘file_exist’:
file_exist.c:10:15: warning: passing argument 1 of ‘opendir’ makes pointer from integer without a cast
  dp = opendir(file_path);
               ^

这个错误是因为你的函数声明为

int file_exist(file_path, file_name)

由于未声明参数的类型，因此假定它们是 int。您必须将这一行替换为

int file_exist(char *file_path, char *file_name)

但是，正如 David van rijn 指出的那样，问题在于您在做什么

if(dp = NULL)

并将 dp 的值替换为 NULL 而不是检查它。你需要写

if(dp == NULL)

此外，您的代码中存在逻辑错误(在这些修复之后):

int file_exist(char *file_path, char *file_name)
{
  DIR *dp;
  FILE *fc;
  struct dirent *ep;

  dp = opendir(file_path);
  fc = fopen(file_name, "r");
  if(dp == NULL)
    error_escape("Opening path");
  else
    chdir(file_path);
  if(fc == NULL)
  {
    error_escape("Opening file");
    return(-1);
  }
  else
    return(0);
}

因为您在转到正确的目录之前打开文件。您需要在 fopen 之前执行 chdir:

int file_exist(char *file_path, char *file_name)
{
  DIR *dp;
  FILE *fc;
  struct dirent *ep;

  dp = opendir(file_path);
  if(dp == NULL)
    error_escape("Opening path");
  else
    chdir(file_path);

  fc = fopen(file_name, "r");
  if(fc == NULL)
  {
    error_escape("Opening file");
    return(-1);
  }
  else
    return(0);
}

最后，您的代码正在泄漏文件描述符，因为您没有关闭正在打开的目录和文件。你需要做类似的事情

int file_exist(char *file_path, char *file_name)
{
  DIR *dp;
  FILE *fc;
  struct dirent *ep;

  dp = opendir(file_path);
  if(dp == NULL)
    error_escape("Opening path");
  else
    chdir(file_path);
  closedir(dp);

  fc = fopen(file_name, "r");
  if(fc == NULL)
  {
    error_escape("Opening file");
    return(-1);
  }
  else {
    close(fc);
    return(0);
  }
}

一个稍微好一点的方法是:

int file_exist(char *file_path, char *file_name)
{
  int dir;
  FILE *fc;

  dir = chdir(file_path);
  if (dir == -1) {
    error_escape("Opening path");
    return -1;
  }

  fc = fopen(file_name, "r");
  if(fc == NULL)
  {
    error_escape("Opening file");
    return(-1);
  }
  else {
    close(fc);
    return(0);
  }
}