如果我不给它提供命令行参数，为什么我的程序会崩溃？ [关闭]

Question

我制作了一个简单的测试程序来玩转 C++11 线程。

#include <iostream>
#include <stdlib.h>
#include <thread>
using namespace std;

void tee(int civ)
{
    for(int loop=0; loop<19; loop++, civ++)
    {
        civ = civ%19;
        cout << loop << "\t" << civ << endl;
        this_thread::sleep_for(chrono::milliseconds(300));
    }
}

void koot()
{
    while(true)
    {
        cout << ":) ";
        this_thread::sleep_for(chrono::milliseconds(300));
    }
}

int main(int argc, char *argv[])
{
    thread saie(tee, atoi(argv[1])),
        kamaa(koot);
    saie.join();
    kamaa.join();

    return 0;
}

只要我提供命令行参数，它就可以正常工作，但如果我不提供，它就会崩溃。 如何解决？ 我尝试检查参数计数，如果它们存在，则无济于事。

编辑：我必须添加这一行：

if(argc < 2) return 1;

Answer 1

因为您正在访问它而崩溃

argv[1]

如果有的话，它将保存一个命令行参数（程序名除外）。你应该检查argc是否大于1。为什么大于1？因为第一个命令行参数是程序本身的名称。所以argc 总是大于0。索引从0 开始。所以如果argc == 1，只有argv[0]是有效的。

#include <iostream>
int main(int argc, char* argv[])
{
  // no need to check argc for argv[0]
  std::cout << argc << " " << argv[0] << "\n";
}

Answer 2

argv[1] 为 null，导致对 atoi() 的调用崩溃。请注意，C++ 中的数组索引是从零开始的！

Answer 3

因为你没有检查argc > 1 并且你尝试访问argv[1]

解决这个问题的方法是首先检查argc > 1，然后访问argv[1]。

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

    if(argc > 1){
    thread saie(tee, atoi(argv[1])),
        kamaa(koot);
    saie.join();
    kamaa.join();
   }
    return 0;
}

要记住的要点：

1. argc 默认为 1。这是保存编号的参数计数器。传递给程序的参数。默认情况下，它为 1，因为程序获取可执行文件的名称（和路径）

2. argv 保存以 NULL 结尾的字符数组（或字符串）的数组。 argv[0] 将始终包含可执行文件的名称。

3. 永远不要假设用户总是会输入参数。并且在将来访问argv 或任何数组时始终进行边界检查。

Answer 4

试试下面的代码：

int main(int argc, char *argv[])
{
    if(argc < 2) {   cout<<"No command line arguments found\n Aborting!\n"; return 1;}
    else         {   thread saie(tee, atoi(argv[1])),kamaa(koot);}
    saie.join();
    kamaa.join();

    return 0;
}

您正在尝试访问不存在的命令行参数 argv[1]。检查命令行参数是否存在总是更好。