找到两个向量之间的交点

时间:2013-11-20 15:38:18

标签: c++ algorithm vector intersection

我做得非常好,事实上我找到了交叉点,并认为我有正确的代码。唯一的问题是它似乎没有打印出最后一个值。

所以如果我有两套:

9 12 7 8 1 19 11 2 14

15 10 8 2 5 16 14 7 19 0 11 3 13 18 9 17 1 12

我的代码将产生以下输出:

1
2
7
8
9
11
12
14

但是这些集合的正确交集应该是:

1
2
7
8
9
11
12
14
19

所以,我的代码没有打印出最后一个值,我找不到原因。

void findIntersection(vector<int> A, vector<int> B)
{
    vector<int> intersection;
    int n1 = A.size();
    int n2 = B.size();
    int i = 0, j =0;
    while(i <= n1 && j <= n2)
    {
        if(A[i] > B[j])
        {
            j++;
        }
        else if( B[j] > A[i])
        {
            i++;
        }
        else
        {
            intersection.push_back(A[i]);
            i++;
            j++;
        }
    }

    for(unsigned int i = 0; i <= intersection.size(); i++)
    {
        cout << intersection[i] << endl;
    }
}

void slowintersect(vector<vector<int> > v)
{
    vector<int> vec;
    vector<int> c;
    int store_0;

    int row = 0;
    for(size_t j =0; j < v.at(row).size(); j++)
    {
        store_0 = v[row][j];
        c.push_back(store_0);
    }

    for(size_t i = 0; i < v.size(); i++)
    {
        for(size_t k = 0; k < v.at(i).size(); k++)
        {
            vec.push_back(k);
        }
    }

    findIntersection(c, vec);
}

3 个答案:

答案 0 :(得分:7)

#include <algorithm>
#include <set>

set<int> intersect;
set_intersection(s1.begin(),s1.end(),s2.begin(),s2.end(), // sorted!
                  std::inserter(intersect,intersect.begin()));

example

如果您不想std::algorithm,则代码中会出现多个错误。即:

for(unsigned int i = 0; i <= intersection.size(); i++)
                           ^
                        should be <

这是一个重要的错误:

int i = 0, j =0;
while(i <= n1 && j <= n2)
{        ^          ^
         <          <
    if(A[i] > B[j])
    {
        j++;

如果A [0]大于每个B元素,你将跳过所有n-1个A元素,并结束循环

答案 1 :(得分:3)

我假设您不想使用C ++标准算法。

一些事情。
1.只有在最初对两个向量进行排序时,您的算法才有效。是吗?
2.你不应该访问vector [vector.size()]元素,因为它超出范围。

我的第二点意味着:

while(i <= n1 && j <= n2)

将此更改为

while(i < n1 && j < n2)

并更改以下内容

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

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

此外,最重要的错误!!!!

for(size_t k = 0; k < v.at(i).size(); k++)
            {
                vec.push_back(k);

            }

将其更改为:

for(size_t k = 0; k < v[i].size(); k++)
            {
                vec.push_back(v[i][k]);

            }

答案 2 :(得分:1)

你的while合作被打破了。您必须继续,直到两个索引结束。目前,当其中一个索引到达结束时,您将停止。

解决问题后,您还必须确保不要将i增加到n1以外; jn2相同。

在开发过程中,您应该使用vector.at(i)代替vector[i]来启用安全检查。