从vector中删除元素并将该元素添加到新的向量中c ++

Question

我有一个std::vector<int> numList ，它包含{9,6,3,3,2,1} 。 我想要的是在循环期间，一旦我们达到9 % 3 == 0 ，我想将9放入新的向量v1 ，将3放入v2 ，同时从原始列表中删除这些数字。 每次mod结果为0时，重复此过程。

我有这个但是崩溃了：

for(i = 0; i < listSize; i++)
{
    for(j = 0; j < listSize; j++)
    {
        if(i != j)
        {
            remainder = numList[i] % numList[j];

            if(numList[i] % numList[j] == 0)
            {
                //cout<< numList[i] << " " << numList[j]<<endl;
                v1.push_back(numList[i]);
                v2.push_back(numList[j]);

                numList.erase(numList.begin() + i);
                numList.erase(numList.begin() + j);
            }
        }
    }
}

Answer 1

您将必须放弃listSize并使用numList.size() ，这将为您提供当前大小。 您也可以删除整个remainder = numList[i] % numList[j]; ，而我们正在这样做。 我想您之后没有使用remainder ，请将其完全删除。

重要：

1. 应该先删除索引较大的元素，然后再删除较小的元素。
2. 您不应在发生擦除的周期中增加i和j －您不想跳过任何元素。
3. 元素中有1会与任何事物配对。 解决条件。

总而言之，这并不酷 ：

int i; // just absurd
// list of variable declarations that are not needed right now, or not needed at all

for(i = 0; i < listSize; i++)

这样更好：

for(int i = 0; i < numList.size(); i++)

Answer 2

我认为您需要以下内容

#include <iostream>
#include <iterator>
#include <vector>

int main()
{
    std::vector<int> v = { 9, 6, 3, 3, 2, 1 };
    std::vector<int> v1;
    std::vector<int> v2;

    for ( std::vector<int>::size_type i = 0; i < v.size(); )
    {
        std::vector<int>::size_type j = i + 1;
        while ( j < v.size() && v[i] % v[j] ) j++;
        if ( j != v.size() )
        {
            v1.push_back( v[i] );
            v2.push_back( v[j] );
            v.erase( std::next( v.begin(), j ) );    
            v.erase( std::next( v.begin(), i ) );    
        }
        else
        {
            i++;
        }
    }

    for ( int x : v1 ) std::cout << x << ' ';
    std::cout << std::endl;

    for ( int x : v2 ) std::cout << x << ' ';
    std::cout << std::endl;
}

程序输出为

9 6 2 
3 3 1 

考虑到内部循环中的索引不能小于匹配元素的外部循环中的索引。