c++ - 我是否正确创建了此字符串数组？ C++

指导是“创建一个可创建50,000个字符串数组的函数-重复“Hi”，“Hey”，“Hello”，“Whats up” 12,500次。从该函数返回字符串数组。给该函数适当的值名称和返回类型。”

这是我编写的代码，可以编译，但我不确定所有数字是否正确。

string* greetingArray()
{
    string greetings[50000];
    for(int i = 0; i < 50000; i++)
    {
        greetings[i++] = "Hi";
        greetings[i++] = "Hey";
        greetings[i++] = "Hello";
        greetings[i++] = "What's up";
    }

    for (int i = 0; i < 50000; i++)
    {
        cout << greetings[i] << endl;
    }
    return 0;
}

int main()
{
    greetingArray();
    return 0;
}

我在一个简单的main中调用了它，这是调用函数的一种好方法吗？
我在第一个for循环中的代码为“i

最佳答案

在您的代码中，当您说

for(int i = 0; i < 50000; i++)
{
    greetings[i++] = "Hi";
    greetings[i++] = "Hey";
    greetings[i++] = "Hello";
    greetings[i++] = "What's up";
}

您在每个for循环之后错误地递增i，因此数组中的每个第5个索引都未初始化。

这是一个贯穿过程:

i = 0，greetings [0] =“Hi”，i ++的计算结果为1

i = 1，greetings [1] =“嘿”，i ++的计算结果为2

i = 2，greetings [2] =“Hello”，i ++的值为3

i = 3，greetings [3] =“怎么了”，i ++的计算结果为4

i = 4，for循环结束，i ++求值为5

i = 5，问候语[5] =“Hi”

此时，greetings[4]未初始化，因此访问它是 undefined 的。

您可以将代码更改为:

for(int i = 0; i < 50000; i++)
{
    greetings[i++] = "Hi";
    greetings[i++] = "Hey";
    greetings[i++] = "Hello";
    greetings[i] = "What's up";
}

但这不是做事的非常安全的方法。我们很幸运，因为50k除以4。

编辑

为了后代，我想我应该向您指出其他答案，这些答案提到您实际上需要从函数而不是0返回数组。所以return greetings;在最后。

更好的解决方案是使用std::vector:

#define AMOUNT 50000 //change this number for whatever count of strings you need
std::vector<std::string> greetings;
while(greetings.size() < AMOUNT){
   greetings.push_back("Hi");
   greetings.push_back("Hey");
   greetings.push_back("Hello");
   greetings.push_back("What's up");
}

//print the vector's contents
for (std::vector<string>::iterator strIter = greetings.begin(); strIter != greetings.end(); ++strIter){
   cout << *strIter << endl;
}