Question

我的问题有所不同，因为我不是在寻找range-3v解决方案。另外，我专门问如何用第二个循环解决该问题。我已经接受了我的问题下面的答案。因为我不需要第二个for循环，所以他们向我展示了如何将单个索引与1并用于奇数迭代。这解决了我的问题！

我正在编写一个函数，假设它的元素长度是偶数，它将采用向量；在我的函数中，我要从原始向量创建两个临时向量，其中它们的元素分别为{0,2,4,6,...}和{1,3,5,7,...}。然后添加对应的索引元素并将结果存储到结果向量中。

这是我的功能：

void sumElementPairsFromVector(const std::vector<int>& values, std::vector<int>& result)
{
    using It = std::vector<int>::const_iterator;
    std::vector<int> temp1, temp2;

    // First lets divide the original vector into two temp vectors
    for (It it1 = values.cbegin(); it1 != values.cend(); it1 += 2)
        temp1.push_back(*it1);

    for (It it2 = values.cbegin() + 1; it2 != values.cend() ; it2 += 2)
        temp2.push_back(*it2);

    // Add each corresponding vector and store that into our results.
    for (std::size_t i = 0; i < values.size() / 2; i++)
        result[i] = temp1[i] + temp2[i];
}

这是我的使用方式：

int main() 
{         
    std::vector<int> values{ 1,2,3,4,5,6 };
    for (auto i : values)
        std::cout << i << " ";
    std::cout << '\n';

    std::vector<int> results;

    sumElementPairsFromVector(values, results);
    for (auto i : results)
        std::cout << i << " ";
    std::cout << '\n';

    return 0;
}

预期输出应为：

1 2 3 4 5 6
3 7 11

该函数的以下代码行上的调试断言失败：

for (It it2 = values.cbegin() + 1; it2 != values.cend(); it2 += 2 )

我知道是什么原因导致了错误；在最后一次迭代后，它以2递增，并检查it2 != values.cend()是否超出向量的末尾。我该如何解决？

Answer 1

我知道是什么原因导致了错误；在它之后的最后一次迭代中以2递增，并检查it2 != values.cend()是否在运行超过向量的结尾。我该如何解决？

您不需要两个不同的循环来迭代向量values。

std::vector<int> temp1, temp2;
temp1.reserve(values.size() / 2); // reserve the memory
temp2.reserve(values.size() / 2);

for (std::size_t index = 0; index < values.size(); ++index)
{
    if (index & 1) temp2.emplace_back(values[index]); // odd index
    else temp1.emplace_back(values[index]);           // even index
}

第二，results当时没有分配任何内存

result[i] = temp1[i] + temp2[i];

因此 out of bound undefined behavior 。你应该

for (std::size_t i = 0; i < std::min(temp1.size(), temp2.size()); i++)
    result.emplace_back(temp1[i] + temp2[i]);

另一方面，如果目标是从连续对元素的总和中得到一个矢量，则temp1和temp2是多余的。 result可以简单地填充：

void sumElementPairsFromVector(const std::vector<int>& values, std::vector<int>& result)
{
    result.reserve(values.size() / 2);

    for (std::size_t index = 0; index < values.size() - 1; index += 2)
        result.emplace_back(values[index] + values[index+1]);
}

Answer 2

由于您不使用模板函数，因此您的函数仅使用向量，因此我认为不需要迭代器。

您可能会喜欢

fucntion cut(){

var num1=8.844

// should be return 8.84

var num2=8.847

// should be return 8.84
}

好吧，如果您确定... for (int i = 0; i < values.size(); i += 2) { result.push_back(values[i] + values[i + 1]); } ...始终是偶数。如果不是这种情况，您可以执行

之类的操作

values.size()

因为如果您不确定值的大小是否始终是偶数，则第一种方法将尝试访问第void sumElementPairsFromVector(const std::vector<int>& values, std::vector<int>& result) { for (int i = 1; i < values.size(); i += 2) { result.push_back(values[i] + values[i - 1]); } if (values.size() % 2) { // whatever you want to do with the last element of an uneven vector result.push_back(values[values.size() - 1]); } }个元素，其中n大于元素的数量，因此您可能会浪费值。

迭代器边界检查超过向量大小

2 个答案: