具有数组键C ++的二进制映射

时间:2018-10-29 14:08:40

标签: c++ data-structures

这是我在黑客级别上提出的sherlockAndAnagrams解决方案。目的是找到字符串中匹配子字符串字谜的数量。例如,“ abba”将具有字谜[[a],“ a”],[“ ab”,“ ba”],[“ abb”,“ bba”],[“ b”,“ b”]。

为解决这个问题,我试图创建一个二进制映射以使用数组作为键。但是,我编写的比较器函数无法捕获每个测试用例。对于我之前给出的示例(“ abba”),它将查找所有匹配项,但不匹配[“ b”,“ b”]。

关于比较器功能,我是否不了解某些内容?我相信它满足严格的弱排序。

struct cmpByArray {
    bool operator()(const array<unsigned int, 26>& a, const array<unsigned int, 26>& b) const {
        for(size_t i=0; i<a.size(); i++)
        {
            if(a[i] < b[i])
                return true;
        }
        return false;
    }
};


int sherlockAndAnagrams(string s) {
    int matches = 0;
    map< array<unsigned int, 26>, int, cmpByArray> m1;
    for(size_t i=0; i<s.length(); i++)
    {
        cout << s[i] << std::endl;
        for(size_t j=i; j<s.length(); j++)
        {
            cout << '-';
            array<unsigned int, 26> arr = {0};
            for(size_t k=i; k<=j; k++)
            {
                cout << s[k]; 
                arr[s[k]-'a']++;
            }
            cout << endl;
            if( m1.find(arr) != m1.end())
            {
                matches++;
                cout << "match: " << endl;
            } 
            else
            {
                m1[arr]++;
            }
        }
    }
    return matches;
}

1 个答案:

答案 0 :(得分:4)

您拥有UB,因为您的比较器不满足严格的弱订购要求。

你有

    if(a[i] < b[i])
            return true;

但是您还需要比较的另一端

    if(a[i] > b[i])
            return false;

考虑案例{1, 2} < {0, 3}。 如果没有第二次检查,您将返回true