将 argv 复制到新数组中

Question

我遇到了以下代码的分段错误。 有人可以解释为什么吗？ 我希望能够将argv的内容复制到一个新的数组中，我称之为rArray。

#include <iostream>        
using namespace std;
int main( int argc, char **argv)
{
  char **rArray;
  int numRows = argc;
  cout << "You have " << argc << " arguments:" << endl << endl;
  cout << "ARGV ARRAY" << endl;
  for (int i = 0; i < argc; i++)
  { 
    cout << argv[i] << endl;
  }
  cout << endl << endl << "COPIED ARRAY" << endl;
  for(int i; i < numRows; i++)
  {
    for (int j = 0; j < argc; j++)
      {
        rArray[i][j] = argv[i][j];
      }
  }
  for (int i = 0; i < argc; i++)
  {
    cout << "Copied array at index " << i << "is equal to " << rArray[i] << endl;;
  }
  cin.get();
}

程序输出：

/a.out hello world
You have 3 arguments:
ARGV ARRAY
./a.out
hello
world

COPIED ARRAY
Segmentation fault: 11

为什么我会收到此错误？ 我该如何解决？

编辑：我得到了一个修复，将char **rArray更改为string rArray，并从那里动态分配大小。

Answer 1

其他人指出了代码的各种问题;如果您确实想复制argv，请使用std::string对象的std::vector：

#include <string>
#include <vector>
int main( int argc, char **argv ) {
    std::vector<std::string> args( argv, argv + argc );
}

Answer 2

您需要

为rArray分配内存，并且还需要初始化外部循环计数器i。

由于argv的内容是常量字符串，因此您只需复制指向它们的指针即可

rArray = new char*[argc+1];
for(int i=0; i <= argc; i++) {
    rArray[i] = argv[i];
}
// use rArray
delete [] rArray;

请注意，argv[argc]保证是NULL的。 我已经更新了循环以复制它（因此看起来不寻常i<=argc退出条件）

如果你真的想复制字符串的内容（正如minitech所建议的那样），代码会变得有点复杂：

rArray = new char*[argc+1];
for(int i=0; i < argc; i++) {
    int len = strlen(argv[i]) + 1;
    rArray[i] = new char[len];
    strcpy(rArray[i], argv[i]);
}
rArray[argc] = NULL;
// use rArray
for(int i=0; i < argc; i++) {
    delete [] rArray[i];
}
delete [] rArray;

Answer 3

一件事是你没有初始化i

for(int i; i < numRows; i++)
        ^-- !

第二件事是没有分配rArray

我建议使用 std::vector<std::string> ，并将所有参数复制到 vector，您无需担心分配/释放内存。