具有动态分配的 char 数组的 strcpy 函数

Question

我面临着使这段代码工作的问题：

char **createCharArray() {
    char **charArray = new char*[PARAM_COUNT];
    for (int i = 0; i < PARAM_COUNT; ++i) {
        charArray[i] = new char[MAXSIZE];
    }
    return charArray;
}
 
void deleteCharArray(char **charArray) {
    for (int i = 0; i < PARAM_COUNT; ++i) {
        delete[] charArray[i];
    }
    delete[] charArray;
}    

int main(){
    char ** test = createCharArray(); 
    char *asd = new char[MAXSIZE];
    cin >> asd;

    for (int i = 0; i < PARAM_COUNT; ++i) {
        strcpy_s(test[i], asd);
    }

    for (int i = 0; i < PARAM_COUNT; ++i) {
        cout << i << " " << test[i] << endl;
    }

    deleteCharArray(test);
    return 0;
}

如何将该字符串复制到 char 数组中，我在哪里弄错了？

编辑：正如评论中的 Igor Tandetnik 和 user17732522 以及下面的回复中 Joseph Larson 的回答，这是通过向 strcpy_s 函数添加 buffer 参数来解决的，使其总共有 3 个参数。

Answer 1

有几件事我觉得很麻烦。首先，您已经看到人们说您应该使用 std::string 而不是 char 数组，这是事实。但是新程序员应该了解整个语言，因此了解如何使用 char 数组很有价值。

因此，让我们忽略 C++ 字符串并查看您的代码：

char ** test = createCharArray(); // array of pointers to char arrays
char *asd = new char[MAXSIZE];
cin >> asd;

for (int i = 0; i < PARAM_COUNT; ++i) {
    strcpy_s(test[i], asd);
}

for (int i = 0; i < PARAM_COUNT; ++i) {
    cout << i << " " << test[i] << endl;
}

deleteCharArray(test);
return 0;

让我们从这里开始。我们不知道 createCharArray() 的作用。它是否做了它应该做的一切？它不仅应该创建一个 char 指针数组，而且按照您使用它的方式，它还需要创建它们各自指向的缓冲区。所以它可能看起来像这样：

char ** createCharArray() {
   char ** array = new char *[PARAM_COUNT];
   for (int index = 0; index < PARAM_COUNT; ++index) {
        array[index] = new char[MAXSIZE];
   }
   return array;
}

如果你的看起来非常不同，你可能有问题。

之后，让我们看一下：

for (int i = 0; i < PARAM_COUNT; ++i) {
    strcpy_s(test[i], asd);
}

正如其他人所说，这个版本的 strcpy_s 采用三个参数：
strcpy_s(测试[i], asd, MAXSIZE);

请注意，您多次将同一字符串复制到位。我想知道您的代码是否真的应该这样做：

for (int i = 0; i < PARAM_COUNT; ++i) {
    cin >> asd;
    strcpy_s(test[i], asd, MAXSIZE);
}

最后，删除方法需要删除各个指针，然后删除指针数组。