由于比较功能导致的分割错误

Question

我尝试解决spoj的问题。 http://www.spoj.com/problems/BUSYMAN/

尽管我能够解决它，但我遇到了一个非常奇怪的错误。 我试图了解其原因，但是失败了。 我有两个代码。

////////////////////////////////////////////////// /

#include<iostream>
#include<vector>
#include<algorithm>

using namespace std;

class activity
{
    public:
    int start,end;
};

bool comp(activity p, activity q)
{
    if(p.end<q.end)return true;
    if(p.end==q.end&&p.start<=q.start)return true;
    return false;
}

int main()
{
    int t;
    cin>>t;
    vector<activity> v;
    for(int i=0;i<t;i++)
    {
        int n;
        cin>>n;

        v.resize(n);
        for(int j=0;j<n;j++)cin>>v[j].start>>v[j].end;
        sort(v.begin(),v.end(),comp);
        int ans=0,currend=0;
        for(int j=0;j<n;j++)
        {
            if(v[j].start>=currend){ans++;currend=v[j].end;
        }

    }
    cout<<ans<<endl;
    }
}

/////////////////////////////////////////////

#include<iostream>
#include<vector>
#include<algorithm>

using namespace std;

class activity
{
    public:
    int start,end;
};

bool comp(activity p, activity q)
{
    if(p.end<q.end)return true;
    if(p.end==q.end&&p.start>=q.start)return true;
    return false;
}

int main()
{
    int t;
    cin>>t;
    int n;
    vector<activity> v;
    for(int i=0;i<t;i++)
    {
        cin>>n;
        v.resize(n);
        for(int j=0;j<n;j++)
            cin>>v[j].start>>v[j].end;
        sort(v.begin(),v.end(),comp);
        int ans=0,currend=0;
        for(int j=0;j<n;j++)
        {
            if(v[j].start>=currend)
            {
                ans++;currend=v[j].end;
            }
        }
        cout<<ans<<endl;
    }
}

//////////////////////////////

我的问题是，第一个在spoj上给出了分段错误，而第二个没有。 两者之间的唯一区别是比较功能。 我只是碰巧以两种相似的不同方式定义了比较函数的第二条语句。 但这在第一种情况下给了我分割错误，但在第二种情况下却没有。

在此处输入图片说明

在上面的两个图像中，有两个具有各自提交ID的代码，在第三幅图像中，一个显示段错误，而另一个则显示错误。 您也可以在我的spoj个人资料中使用提交ID进行验证。

Answer 1

因为bool comp(activity p, activity q)不符合Compare要求，请参见std::sort

应该是这样的：

bool comp(const activity& p, const activity& q)
{
    return p.end < q.end || (p.end ==q.end && p.start < q.start);
}

要么

struct comp {
    bool operator()(const activity& p, const activity& q) const
    {
        return p.end < q.end || (p.end ==q.end && p.start < q.start);
    }
};

要么

struct comp {
    bool operator()(const activity& p, const activity& q) const
    {
        return std::tie(p.end, p.start) < std::tie(q.end, q.start);
    }
};

Answer 2

C ++的规则是std::sort的比较器必须给出严格的弱排序 。

因此，比较器必须为相等的元素返回false 。 但是这个测试：

if(p.end<q.end)return true;
if(p.end==q.end&&p.start<=q.start)return true;
return false;

如果元素相等，则返回true ，因此这是一个无效的比较器。

第二次尝试：

if(p.end<q.end)return true;
if(p.end==q.end&&p.start>=q.start)return true;
return false;

这具有相同的问题，也导致不确定的行为。

当代码具有未定义的行为时，您无法从观察到的行为中推断出任何东西，这仅仅是您获得什么行为的机会（也许取决于有关编译器特定选择的排序算法的某些细节）。

在第一个比较器中将<=更改为<将产生一个有效的比较器，其排序顺序为结束和开始递增。