如果参数不是文件，则从标准输入中读取

Question

我有一个使用命令行参数的程序，检查它们是否为文件，是否为文件，然后执行某些操作，如果不是，则执行其他操作。

我的基本结构如下所示

for (i=1;i<argc-1;i++){
    if ((fp=open (argv[i],"r"))==NULL){
        #Do blah blah
    }
    else{
        #do blah blah
    }
}

但是由于某种原因，当我的论点不是文件时，它不是返回目录而是文件，而是返回错误信息。 我的if语句有问题吗？

Answer 1

如果有问题， open()系统调用将返回负值，因此fp技术上讲不会为NULL 。

另外，正如@Kninnug的评论中所述，open期望将int作为其第二个参数，因此您需要O_RDONLY而不是"r"

尝试改变一下

if((fp=open(argv[i], O_RDONLY))==-1)

或者，遵循@Kninnug的建议并使用fopen() 。 如果这样做，则将需要FILE *指针而不是int 。

Answer 2

您的文件名将在argv [1]中。 因此，您应该拥有for (i=1;i<argc;i++){而不是for (i=1;i<argc-1;i++){

Answer 3

更改

if ((fp=open (argv[i],"r"))==NULL)

至

if((fp=open(argv[i],"r"))==-1)

Answer 4

检查参数是否为文件的路径并尝试打开文件是两件事。 fopen()失败的原因有很多，但是参数是合法的文件名。

要查看参数是否为文件，请使用stat() 。

#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h> 

int isfile(const char *path) {
  struct stat buf;
  int retval = stat(path, &buf);

  // A pedantic approach would also check the `errno`.
  if (retval) return 0;

  switch (buf.st_mode & S_IFMT) {
    case S_IFREG: return 1;
    // Maybe other cases could return 1 depending on OP's needs.
  }
  return 0;
}