C 中文件存在的问题

Question

这是我的代码，用于检查文件是否存在：

#include<stdio.h>
#include<zlib.h>
#include<unistd.h>
#include<string.h>


int main(int argc, char *argv[])
{
   char *path=NULL;
   FILE *file = NULL;
   char *fileSeparator = "/";
   size_t size=100;
   int index ;
   printf("\nArgument count is = %d", argc);

   if (argc <= 1)
   {
      printf("\nUsage: ./output filename1 filename2 ...");
      printf("\n The program will display human readable information about the PNG file provided");
   }
   else if (argc > 1)
   {
      for (index = 1; index < argc;index++)
      {
            path = getcwd(path, size);
            strcat(path, fileSeparator);
            printf("\n File name entered is = %s", argv[index]);
            strcat(path,argv[index]);
            printf("\n The complete path of the file name is = %s", path);
            if (access(path, F_OK) != -1)
            {
                  printf("File does exist");
            }
            else
            {
                  printf("File does not exist");
            }
            path=NULL;
      }
   }
   return 0;
}

运行命令 ./output test.txt test2.txt 输出为：

$ ./output test.txt test2.txt

Argument count is = 3
 File name entered is = test.txt
 The complete path of the file name is = /home/welcomeuser/test.txt
 File does not exist
 File name entered is = test2.txt
 The complete path of the file name is = /home/welcomeuser/test2.txt
 File does not exist

现在 test.txt 确实存在于系统上：

$ ls
assignment.c  output.exe  output.exe.stackdump  test.txt

但 test.txt 显示为不存在的文件。

请帮助我理解这里的问题。另外，请随时发布任何改进代码/避免错误的建议。

问候， 黑暗者

Answer 1

仅仅因为对 access() 的调用失败并不意味着该文件不存在。呼叫可能会因其他原因而失败。

使用 printf("error:%s\n", strerror(errno)); 打印出错误消息的文本。

此外，您仍然错误地附加到从 getcwd 收到的“路径”，就像您在上一个问题中一样。即使它没有崩溃，它仍然不正确，并且可能会给您带来问题......甚至可能是您现在遇到的问题。

getcwd() 为您的路径分配一个缓冲区，但该缓冲区的大小仅适合路径。您正在附加到该缓冲区，超出末尾。这很糟糕，你不能这样做。它会导致问题，有时甚至会崩溃。您需要暂停并了解这个 getcwd 函数是如何工作的以及如何正确使用它。

Answer 2

我强烈建议通过 malloc() 分配足够的空间来存储路径和fpathconf()（提示，路径最大）。

分配和组装它的非标准方法是 asprintf()。

只要确保在不再需要时释放生成的路径，并检查由于用户拼写错误而可能失败的每个调用是否失败。

如果使用 malloc()，请始终检查是否失败（结果为 NULL）。

祝你的任务顺利:)