C++矢量字符串for循环push_back错误

Question

我觉得这是理所当然的，但由于某种原因我无法理解发生了什么。

当我运行这部分代码时，它将 URL 与数组中的字符串元素组合在一起，然后将其推送到字符串向量中，它成功推送了第一个 URL，但随后出现了某种内存泄漏？控制台无限循环乱码......

string anonlist[] = {"test1","test2","test3","test4","test5","test6","test7"};


for (int i=0; i<=7; i++)
{

    vector<string> nameurl; 
    nameurl.push_back("http://api.twitter.com/1/users/show.json?screen_name="+anonlist[i]);

    cout << nameurl[i] << endl;
}

Answer 1

数组中有 7 个项目，索引为 0 1 2 3 4 5 6。您的循环还将包括 i = 7 的情况，该情况超出了数组的范围。

另一个问题是您在循环内部声明向量，这意味着每次循环继续时都会创建一个新向量。您应该在循环之前声明它。

string anonlist[] = {"test1","test2","test3","test4","test5","test6","test7"};

vector<string> nameurl; 
for (int i=0; i < 7; i++)
{

    nameurl.push_back("http://api.twitter.com/1/users/show.json?screen_name="+anonlist[i]);

    cout << nameurl[i] << endl;
}

Answer 2

每次循环时都会创建一个新的 vector，插入一个元素，然后尝试访问索引 i 处的元素。当 i 为 0 时，第一次循环时索引 i 处只会有一个元素。

大概您希望将 nameurl 的声明移到循环之外，以便同一容器用于循环的所有迭代。

Answer 3

每次通过 for 循环都会创建一个新向量——该向量仅在 for 循环内具有块作用域。您应该将向量移到循环之外。

Answer 4

本回答的另一种方法是：

const string anonlist[] = {"test1", "test2", "test3", "test4", "test5", "test6", "test7"};
vector<string> nameurl(sizeof(anonlist) / sizeof(anonlist[0]), "http://api.twitter.com/1/users/show.json?screen_name="); 
for (size_t i = 0; i < nameurl.size(); ++i) {
    cout << (nameurl[i] += anonlist[i]) << endl;
}

这样，您可以只编辑非列表，而无需触及其余代码。并且，它构造了一个向量，其中所有元素都设置为基本 URI，这很好。