Question

我正在编写一个程序，它从命令行读取“flags”以及程序名称。我希望程序读取几个标志（-i，-c和-n），并在调用一个或多个标志时执行不同的函数。

这是我开始编写的代码：

  printf("Argv 0: %s\n", argv[0]);
  printf("Argv 1: %s\n", argv[1]);
  printf("Argv 2: %s\n", argv[2]);

  for (int i = 1; i <= argc + 1; i++) {
    if (strcmp("-i", argv[i]) == 0) {
      printf("%s\n", "found -i"); 
    }
    else{
      printf("%s\n", "did not find -i");
    }

  }

只试用一个标志（-i），但我希望它同时读取一个或多个标志并调用相应的函数。

执行程序时：

./program-name test -i
Argv 0: test
Argv 1: -i
Argv 2: (null)
found -i
Segmentation fault

Answer 1

为简化起见，argv[]是命令行参数的计数，参数保存在0中。由于C数组具有基于for (int i = 1; i <= argc + 1; i++)的索引，因此在您的代码中，您需要更改

for (int i = 1; i < argc;  i++)

到

C11

限制对有效参数列表的访问。

为此添加一些引用，引用argc，章节§5.1.2.2.1，程序启动（强调我的）

如果argv[0]的值大于零，则argv[0][0]指向的字符串代表程序名称; argc如果是，则为空字符程序名称不能从主机环境中获得。如果argv[1]的值是大于一， argv[argc-1]通过"ignore_dirs": ["tmp"]指向的字符串代表程序参数。

Answer 2

你超越了argv的界限; for循环的条件不正确。它应该是i < argc。

另外，为什么重新发明轮子？查看getopt()。

Answer 3

不要重做经过验证的旧代码。它浪费时间并重复努力。作为一个长时间的C编码器，我总是（当我记得的时候）在我尝试从头开始之前先去查看代码库。我首先重新调整用途然后创建。我足够聪明，知道有很多聪明的程序员在我面前。

使用免费且有效的getopt（）和getopt_long（）函数为您解析命令行参数。如果你谷歌搜索＆＃34; getopt example＆＃34;在代码库中有很多例子。 but here is the GNU C tutorial

#include <ctype.h>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>

int
main (int argc, char **argv)
{
  int aflag = 0;
  int bflag = 0;
  char *cvalue = NULL;
  int index;
  int c;

  opterr = 0;

  while ((c = getopt (argc, argv, "abc:")) != -1)
    switch (c)
      {
      case 'a':
        aflag = 1;
        break;
      case 'b':
        bflag = 1;
        break;
      case 'c':
        cvalue = optarg;
        break;
      case '?':
        if (optopt == 'c')
          fprintf (stderr, "Option -%c requires an argument.\n", optopt);
        else if (isprint (optopt))
          fprintf (stderr, "Unknown option `-%c'.\n", optopt);
        else
          fprintf (stderr,
                   "Unknown option character `\\x%x'.\n",
                   optopt);
        return 1;
      default:
        abort ();
      }

  printf ("aflag = %d, bflag = %d, cvalue = %s\n",
          aflag, bflag, cvalue);

  for (index = optind; index < argc; index++)
    printf ("Non-option argument %s\n", argv[index]);
  return 0;
}

Answer 4

使用您的代码和循环，这一行是错误的：

(strcmp("-i", argv[i])

当你将argv [i]传递给strcmp时，你应该确保它不是NULL。已知该函数以NULL指针作为参数进行segfault。您应该更好地测试argv [i]！= NULL或将i＆lt; = argc放在循环中而不是i＆lt; = argc + 1，其中包含argv字符串数组中的NULL指针。

迭代argv []会产生分段错误

4 个答案: