【问题标题】:Getopt returns -1 yet it has valid dataGetopt 返回 -1 但它有有效数据
【发布时间】:2021-10-10 16:11:05
【问题描述】:

我在 GDB 中运行我的程序以确定 getopt 为何返回 -1,我可以看到它显然具有有效数据。我没有从程序的命令行获取 argv/argc,我是自己生成的,但它显然仍然是有效数据。

(相关)代码:

    char* argv[4];                                                                                                                         
    printf("calling create_argv\n");                                                                                                       
    int argc = create_argv(argstring, argv, 4);                                                                                            
    int opt;                                                                                                                               
    int myBuf = 0, total_mbufs = 0;                                                                                                        
    bufs_list* bufToUse = einf->bufs_head;                                                                                                                                                                                                                                        
    while ((opt = getopt(argc, argv, "f:t:")) != -1){                                                                                              
    printf("opt is %s\n", opt);                                                                                                            
    switch(opt){                                                                                                                                   
        case 'f':                                                                                                                                      
        fd = fopen(optarg, "r");                                                                                                               
        break;                                                                                                                         
        case 't':                                                                                                                                      
        fsize = atoi(optarg);                                                                                                                  
        break;                                                                                                                         
        default: break;                                                                                                                
    }                                                                                                                              
}    

create_argv:(根据我对 GDB 的了解)

int create_argv(char* string, char* argv[], int nargs){                                                                                                                                                                                                                                                                                                                                                                                                                                                 
    int idx = 0;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                    
    char* s;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        
    char* p;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        
    printf("in create_argv\n");                                                                                                                                                                                                                                                                                                                                                                                                                                                                     
    for (s = string, p = string; *s != '\0' && *s != '\n'; s++){                                                                                                                                                                                                                                                                                                                                                                                                                                            
        printf("%c\n", *s);                                                                                                                                                                                                                                                                                                                                                                                                                                                                             
        if (*s == ' '){                                                                                                                                                                                                                                                                                                                                                                                                                                                                                         
            *s = '\0';                                                                                                                                                                                                                                                                                                                                                                                                                                                                                      
            argv[idx++] = p;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                
            printf("%s\n", p);                                                                                                                                                                                                                                                                                                                                                                                                                                                                              
            p = ++s;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        
            if (idx == nargs) return idx; //bail out if we can't fit any more                                                                                                                                                                                                                                                                                                                                                                                                                       
        }                                                                                                                                                                                                                                                                                                                                                                                                                                                                                       
    }                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               
    *s = '\0';                                                                                                                                                                                                                                                                                                                                                                                                                                                                                      
    argv[idx++] = p;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                
    printf("%s\n", p);                                                                                                                                                                                                                                                                                                                                                                                                                                                                              
    p = ++s;                                                                                                                                                                                                                                                                                                                                                                                                                                                                                        
    return idx; //equivalent to argc
}                                                                                                                           

所以 gdb 说一切都处于完美的工作状态:

(gdb) p opt
$2 = -1
(gdb) p argc
$3 = 2
(gdb) p argv
$4 = {0x7a9935 "-f", 0x7a9938 "file.txt", 0x2f76006f666e692f <Add

(argv[2] 无效,但没关系,因为 argc 只有 2)。

那么 getopt 有什么问题呢?它显然有一个有效的 argc,一个有效的 argv,但它拒绝承认它得到的是有效数据。

编辑:输出显示 while 循环永远不会运行 -

calling create_argv
in create_argv
-
f

-f 
i
l
e
.
t
x
t
file.txt
no file loaded (did you specify -f filename?). Refusing to load an empty file

【问题讨论】:

  • 它看起来不像getopt 在错误时返回-1,而是在它到达参数末尾时返回。这是man pagegetopt。您可以将程序的输出添加到问题中吗?
  • @GuyMarino 输出添加 - while 循环永远不会运行,尽管那里有参数
  • 关于 printf 的好点。我不认为这会导致这个问题,但记住这是一件好事
  • 好的,我不确定这是否能解决问题,但是尝试一个带有否定条件的 do-while 循环并打印optoptoptarg 的值。它们可能包含您要查找的内容。如果您不想重写代码,只需在循环运行后在调试器中检查它们即可。
  • optarg 和 optopt 在循环(失败的第一次迭代)运行后立即为 0

标签: c getopt


【解决方案1】:

将其放在这里以备后用,但getopt 期望argv 的第一个值是程序名称,并在不解析值的情况下对其进行修整。这意味着如果您自己生成 argv 值,则需要在前面添加一个虚拟程序名称。

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2014-04-13
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2020-02-22
    • 2023-03-14
    • 1970-01-01
    相关资源
    最近更新 更多