2

我目前正在尝试从命令行获取带有 argv[] 的数字。换句话说,我试图从

./计算器 -q 2

我目前的设置是这样的:

#include <iostream>
using namespace std;

int check_q(char* argv[]){
    float q, qa;
    if(atoi(argv[1]) == q){
       qa = atof(argv[2]);
    }
    if(atoi(argv[3]) == q){
       qa = atof(argv[4]);
    }
    if(atoi(argv[5]) == q){
       qa = atof(argv[6]);
    }
    if(atoi(argv[7]) == q){
       qa = atof(argv[8]);
    }
return qa;
}

int main(int argc, char *argv[]){

    float qa = 0;
    check_q(argv);
    cout << qa << endl;

 return 0;}

关于我做错了什么的任何想法?

4

4 回答 4

2

您没有检查 的值argc以查看传递给程序的参数数量。如果您只传递两个参数,那么访问argv[3]将给出未定义的行为;所以你必须先检查参数的数量。

此外,如果您正在寻找具有 value 的参数,请与以下内容"-q"进行比较"-q"

if (std::string(argv[1]) == "-q")

您将其转换为数字并与未初始化的变量进行比较,这不会做任何有用的事情。

于 2013-03-04T01:48:13.777 回答
2

传下去:

#include <string>
#include <vector>
#include <iostream>
#include <algorithm>
#include <cassert>

float check_q(std::vector<std::string> const& args)
{
    int q = 42;

    for (auto it = args.begin(); it != args.end(); std::advance(it, 2))
    {
        if (std::stoi(*it) == q)
        {
            auto next = std::next(it);
            assert(next != args.end());

            return std::stof(*next);
        }
    }

    return 0;
}


int main(int argc, const char *argv[])
{
    const std::vector<std::string> args(argv, argv+argc);

    // pass it along
    check_q(args);
}
于 2013-03-04T01:54:55.323 回答
1

你在这里做错了很多事情:

#include <iostream>
using namespace std;

int check_q(char* argv[])
{
    float q, qa;           // you never assign `q` a value, so the following comparisons make no sense
    if(atoi(argv[1]) == q)   // you never check argc in main to determine if argv[whatever] is valid.  if the array isn't long enough, this will invoke undefined behavior.
    {
       qa = atof(argv[2]);  // you're assigning a value to `qa` declared in this function, leaving the one declared in main unchanged.  probably not what you intended
    }
    // and so on

    return qa;
}

int main(int argc, char *argv[])
{

    float qa = 0;
    check_q(argv);    // this function never changes the value of `qa` that's declared in main...
    cout << qa << endl;    // ... so this will always print 0

 return 0;

}

你可能想做更多的事情:

#include <iostream>
#include <string>
#include <vector>

float check_q(const std::vector<std::string>& args)
{
    if(args[1] == "-q")
    {
        return ::atof(args[2].c_str());
    }
    else
    {
        return 0.0f;   // or some other default
    }
}

int main(int argc, char *argv[])
{
    const std::vector<std::string> args(argv, argv+argc);

    if(args.size() >= 3) // argv[0] is usually the name of the executable
    {
        std::cout << check_q(argv) << std::endl;
    }
    else
    {
        std::cout << "not enough args" << std::endl;
    }
}

一旦你更有经验,你会想要使用像GNU getoptboost::program_options这样的库。

于 2013-03-04T01:52:29.737 回答
0

这段代码有很多东西没有多大意义,或者不安全或不优雅,我不确定您通过检查参数 1、3、5 和7,但这里至少有一个问题:

qainmain永远不会被更新,因为你没有给check_q它分配返回值。声明必须是:

float qa = 0;
qa = check_q(argv);

或者真的,只是:

float qa = check_q(argv);
于 2013-03-04T01:49:24.760 回答