Question

我试图在optarg上使用atoi，但它也可能是任何东西。我一直试图弄清楚为什么我的getopt_long不起作用。当我输入我的switch语句时，optarg被设置为null并且它保持这种状态。我检查了我的冒号，他们是正确的。这是我的代码。

static struct option long_options[] = 
    {
        {"algorithm", required_argument, 0, 'a'},
        {"reverse", no_argument, 0, 'r'},
        {"key", required_argument, 0, 'k'},
        {"output", required_argument, 0, 'o'},
        {"help", no_argument, 0, 'h'},
        {"version", no_argument, 0, 'V'},
        {0, 0, 0, 0}
    };

int option_index = 0;
int c;
//Getopt to get the correct options from the command line.
while ((c = getopt_long(argc, argv, "a:rk:o:hV", long_options, 
        &option_index)) != -1)
{
    bool endOption = false;

    if (endOption) break;

    switch (c) 
    {
        case 0:
        {
            endOption = true;
            break;
        }
        case 'a':
        {
            if (optarg == "insertion") algorithm = 0;
            break;
        }

        case 'r':
        {
            reverseFlag = true;
            break;
        }
        case 'k':
        {
            while (optarg != " ")
            {
                if (optarg == ",")
                {
                    optarg++;
                }
                else 
                {
                    sortOrder.push_back(atoi(optarg)); //error here
                    optarg++;
                }
            }
        }
        case 'o':
        {
            fileFlag = true;
            break;
        }
        case 'h':
        case 'V':
        default:
        {
            cerr<<"You have entered an incorrect flag, do it better"<<endl;
            break;
        }
    }
}

//更多东西//

我尝试过使用双冒号以及其他所有东西。

Answer 1

您无法比较这样的字符串：

 if (optarg == "insertion") algorithm = 0;

您需要使用strcmp()或等效内容，如下所示：

if (strcmp(optarg, "insertion") == 0) algorithm = 0;

可能还有其他问题，但会立即浮现在脑海中。

修改optarg做optarg++可能不是一个好主意。这是函数给你的信息;改变它是狡猾的。您可以获取指针的副本然后递增它，逐步通过它指向的字符串。当然不能保证-k的论证中会有空格，这可能是你麻烦的根源，也可能是误用字符指针比较而不是使用strcmp()或等价物。

break之后，您似乎错过了case 'k':。

循环体内的bool endOption是奇数：在每次迭代时都设置为false;它仍处于false时进行测试，因此break未执行;当true返回getopt_long()时，它会设置为0;但是在下一次迭代之前重置该值。您应该在主while ((c = getopt_long(...)) != -1)循环之外声明此变量。然后它会表现得很明智（尽管如果在再次调用getopt_long()之前设置它就应该终止循环 - 所以它的条件可能应该在主while循环的底部。

将-k处理中的循环测试更改为while (strcmp(optarg, "") != 0)（关键区别是两个双引号之间缺少空格），然后我得到了理智的行为。请注意我如何检测选项，以便我可以看到正在执行的内容。当我将比较运行到包含空格的字符串时，在循环终止之前，我得到了一个奇特的环境转储。这就是导致核心转储的事情。

#include <getopt.h>
#include <vector>
#include <iostream>
#include <cstdlib>
using namespace std;

static struct option long_options[] = 
{
    {"algorithm", required_argument, 0, 'a'},
    {"reverse", no_argument, 0, 'r'},
    {"key", required_argument, 0, 'k'},
    {"output", required_argument, 0, 'o'},
    {"help", no_argument, 0, 'h'},
    {"version", no_argument, 0, 'V'},
    {0, 0, 0, 0}
};

int main(int argc, char **argv)
{
    vector<int> sortOrder;
    int option_index = 0;
    int c;
    bool reverseFlag = false;
    int algorithm = 1;
    bool fileFlag = false;

    //Getopt to get the correct options from the command line.
    while ((c = getopt_long(argc, argv, "a:rk:o:hV", long_options, &option_index)) != -1)
    {
        bool endOption = false;

        if (endOption) break;

        switch (c) 
        {
            case 0:
                    cerr << "Found 0 value\n";
                    endOption = true;
                    break;
            case 'a':
                    cerr << "Found -a option: " << optarg << endl;
                    if (strcmp(optarg, "insertion") == 0)
                        algorithm = 0;
                    break;
            case 'r':
                    cerr << "Found -r option\n";
                    reverseFlag = true;
                    break;
            case 'k':
                    cerr << "Found -k option: " << optarg << endl;
                    while (strcmp(optarg, "") != 0)
                    {
                        if (strcmp(optarg, ",") == 0)
                            optarg++;
                        else 
                        {
                            cerr << "pushback <<" << optarg << ">>\n";
                            sortOrder.push_back(atoi(optarg)); //error here
                            optarg++;
                        }
                    }
                    break;
            case 'o':
                    cerr << "Found -o option: " << optarg << endl;
                    fileFlag = true;
                    break;
            case 'h':
            case 'V':
            default:
                    cerr << "You have entered an incorrect flag, do it better" << endl;
                    break;
        }
    }
}

当它运行时：

./getopt -k 2,1

我得到了输出：

Found -k option: 2,1
pushback <<2,1>>
pushback <<,1>>
pushback <<1>>

您可能希望更改比较：

if (strcmp(optarg, ",") == 0)

为：

if (*optarg == ',')

您甚至可以合理地将循环条件更改为：

while (*optarg != '\0')

因此完全避免strcmp()。顺便说一句，我不确定<cstring>中有哪个标题，但其中一个标题是。

为了记录，我正在测试运行MacOS X 10.7.2（Lion）的Mac Mini并使用提供的G ++编译器i686-apple-darwin11-llvm-g++-4.2 (GCC) 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2335.15.00)。

optarg设置为null

1 个答案: