如何将几个不同的字符串分配给char *数组

Question

伙计们，我正在尝试输入以下几个不同的字符串：

cap
to
cat
card
two
too
up
boat
boot

变成一个char *数组，如下所示：

char* result[9]

在使用for循环将所有这些字符串分配给数组后，我发现数组中的所有元素都是相同的，即“ boot”。

我的代码和结果在这里：

#include<iostream>
#include<algorithm>
#include<cstring>
using namespace std;
int main()
{

    char* result[9];
    for(int counter=0;counter<9;counter++)
   {
        string temp;
        getline(cin,temp);
        result[counter]=(char*) temp.c_str();
        cout<<result[counter]<<endl;//correct
    }
cout<<endl;
cout<<endl;
for(int counter=0;counter<9;counter++)
    cout<<result[counter]<<endl;//false
}

这条线很好用，

 cout<<result[counter]<<endl;//correct

并打印所有不同的词，例如：

cap
to
cat
card
two
too
up
boat
boot

但不知何故，这条线

 cout<<result[counter]<<endl;//false 

它只打印“启动”九次。 我真的看不到这里的原因，希望你们能帮帮我，谢谢！

Answer 1

{
    string temp;
    getline(cin,temp);
    result[counter]=(char*) temp.c_str();   /* 1 */
    cout<<result[counter]<<endl;//correct
}  /* 2 */

在标记为1的行中，您将一个指向temp的基础字符串数据的指针存储在result[counter] 。 没有字符串内容被复制； 您只是将现有数据的地址存储在temp 。 （顺便说一下， .c_str()为(char*)是什么.c_str()已经返回了一个指向char的指针。）

在标记为2的行中， temp被销毁（这是该块中的局部变量）。 现在， result[counter]包含无效的指针：指向的对象不再存在。

在此之后尝试使用result的内容会result未定义的行为。

解决此问题的一种方法是使用字符串数组而不是指针数组：

string result[9];
for(int counter=0;counter<9;counter++)
{
    getline(cin,result[counter]);
    cout<<result[counter]<<'\n';
}

（我也删除了endl因为不需要在每个字符串之后显式刷新流。）

这也简化了代码，因为您不再需要temp变量：您只需将getline直接插入数组即可。

Answer 2

最好还是使用std::vector<string>

std::vector<std::string> result;
for(int counter=0;counter<9;counter++) {
  std::string temp;
  std::getline(std::cin, temp);
  result.push_back(temp);
}

如果要使用指针，则需要复制str.c_str（）的内容，因为它是一个临时变量，每次迭代后都会被销毁。

Answer 3

您的代码有问题，尤其是以下代码行：

result[counter]=(char*) temp.c_str();

由于temp存在于for循环中，并且c_str()返回的地址一旦temp被破坏将是无效的，因此您的代码将遭受UB的困扰。 该地址无效，因为c_str() ：

返回指向用作字符存储的基础数组的指针。

一旦temp超出范围，该基础数组就消失了，但是您现在可以在其范围之外对其进行引用！

你应该为每个在字符串分配新的内存result也许使用strdup如下所示：

#include<iostream>
#include<algorithm>
#include<cstring>
using namespace std;
int main()
{

    char* result[9];
    for(int counter=0;counter<9;counter++)
   {
        string temp;
        getline(cin,temp);
        result[counter]=strdup(temp.c_str()); //now it is correct!
        cout<<result[counter]<<endl;
    }
cout<<endl;
cout<<endl;
for(int counter=0;counter<9;counter++)
    cout<<result[counter]<<endl;
}

不要忘记strdup将分配需要手动释放的内存。

Answer 4

在以下循环中，您将声明临时字符串并一次又一次地使用它。 因此在循环的每次迭代中，都将编辑相同的指针。 因此，最后的值保持不变，其余值被覆盖。

for(int counter=0;counter<9;counter++)
   {
        string temp;
        getline(cin,temp);
        result[counter]=(char*) temp.c_str();
        cout<<result[counter]<<endl;//correct
    }

相反，将您的临时字符串声明为容量为9的字符串数组。

#include<iostream>
#include<algorithm>
#include<cstring>
using namespace std;
int main()
{

    char* result[9];
        string temp[9];
    for(int counter=0;counter<9;counter++)
   {
        getline(cin,temp[counter]);
        result[counter]=(char*) temp[counter].c_str();
        cout<<result[counter]<<endl;//correct
    }
cout<<endl;
cout<<endl;
for(int counter=0;counter<9;counter++)
    cout<<result[counter]<<endl;//false
}