这个循环的正确类型是什么？

Question

其中 RoomCentreNodeVectors 是整数向量的向量。 roomKills 是一个整数向量，而 level.Nodes() 返回一个 Node 对象的向量。 Node 对象的 Room() 函数返回和 int。

for (std::vector<std::vector<int>>::size_type i = 0; i < level.RoomCentreNodeVectors().size(); i++)
{
    RoomKills.push_back(level.Nodes()[level.RoomCentreNodeVectors()[i][0]].Room());
}

或

for (std::vector<int>::size_type i = 0; i < level.RoomCentreNodeVectors().size(); i++)
{
    RoomKills.push_back(level.Nodes()[level.RoomCentreNodeVectors()[i][0]].Room());
}

Answer 1

简短的回答是您应该使用被索引的向量类型的 size_type。如果 RoomCentreNodeVectors() 返回 int 向量的向量，则您的第一个循环正确指定了类型。它很可能等于 size_t。

剩下的只是回答你的问题，如果你愿意，你可以忽略它。

首先，typedef 将有助于使其更易于阅读。例如

typedef std::vector<Node> NodeVec;
typedef std::vector<int> IntVec;
typedef std::vector<IntVec> IntVecVec;
for (IntVecVec::size_type i = 0; ...

除非 RoomCentreNodeVectors() 的值在此循环期间发生变化，否则我肯定会保存返回值，这样您就不必每次都调用该函数。如果函数返回一个值而不是引用类型，则尤其如此，因为您将在每次循环中复制该向量两次。而且，正如 R. Martinho Fernandes 所提到的，您也可以使用迭代器，并避免整个 size_type 问题：

const IntVecVec& nodeVecs = RoomCentreNodeVectors();
IntVecVec::const_iterator end = nodeVecs .end();
const NodeVec& nodes = level.Nodes(); // if Nodes() doesn't change during loop
for (IntVecVec::const_iterator iter = nodeVecs .begin(); iter != end; ++iter)
{
    const IntVec& vec = *iter;
    int j = vec[0];
    int room = nodes[j].Room();
    RoomKills.push_back(room);
}

Answer 2

两者都是正确的，因为size_type 的实际类型不依赖于模板特化。它通常与size_t 相同，我建议您使用它以获得更好的可读性：

for (size_t i = 0; i < level.RoomCentreNodeVectors().size(); i++)
{
    RoomKills.push_back(level.Nodes()[level.RoomCentreNodeVectors()[i][0]].Room());
}

如果我在代码中看到你的 sn-p，我会重构它。

Answer 3

我会将此代码更改为使用迭代器...然后您可以使用 C++11 auto 关键字执行以下操作：

for(auto iter = begin(level.RoomCentreNodeVectors); iter != end(level.RoomCentreNodeVectors); iter++)
{
    RoomKills.push_back(level.Nodes()[(*iter)[0]].Room());
}