运行时间的比较 - 类似的代码运行速度慢4倍

时间:2013-09-21 02:29:50

标签: c++ performance backtracking

我正在解决回文分区问题,该问题需要返回给定字符串(http://oj.leetcode.com/problems/palindrome-partitioning/)的所有回文分区。我找到了两个解决这个问题的方法。我很难理解为什么这两种方法的运行时间存在巨大差异:

(假设bool isPal()是一个给定的函数,它在O(长度)时间内判断给定的字符串是否是回文)

第一个解决方案:(使用回溯):

void find(string s, int start, vector<string> &r, vector<vector<string> > &res){
    if (start >= s.size()){
        res.push_back(r);
    }else{
    for (int i=start;i<s.size();i++){            
        if (isPal(s.substr(start, i-start+1))){
            r.push_back(s.substr(start,i-start+1));
            find(s,i+1,r,res);        
            r.pop_back();
        }  
    }
    }  
}

vector<vector<string>> partition(string s) {
    vector<vector<string> > res;
    vector<string> r;
    find(s,0,r,res);
    return res;    
}

第二个解决方案:

vector<vector<string> > partition(string s) {
    vector<vector<string> > answer;

    if(s.size() == 0)
        return answer;

    // if s is Palindrome, push to answer
    if(isPal(s))
        answer.push_back(vector<string>(1,s));

    if(s.size() == 1)
        return answer;

    for(int i=1; i<s.size(); ++i) {
        string s1 = s.substr(0, i);
        string s2 = s.substr(i, s.size()-i);

        if(isPal(s1)) {
            // get sub_answers of partition(s[i:])
            vector<vector<string> > sub_answer = partition(s2);

            // add s1 to the begin of sub_answers
            for(int j=0; j<sub_answer.size(); ++j) {
                sub_answer[j].insert(sub_answer[j].begin(), s1);
                answer.push_back(sub_answer[j]);
            }
        }
    }
    return answer;
}

尽管两种解决方案都具有相同的基本思想,但第一种解决方案在 108ms 中运行,而第二种解决方案在 472ms 时间内运行。这是因为理论上的低效率(可能是第二个解决方案多次解决了一个问题),还是因为实现效率低下(某些C ++概念)? 请指出第二种解决方案效率低下的原因。

注意:两种解决方案都是正确的,并给出相同的结果。

1 个答案:

答案 0 :(得分:1)

我假设两个实现都返回相同的结果? 除此之外,还有一些值得评论的内容:

sub_answer[j].insert(sub_answer[j].begin(), s1);

这会强制sub_answer[j]的所有内容向后移动,以便为s1提供空间。在添加其余内容之前,最好将s1添加到矢量。

answer.push_back(sub_answer[j]);
现在再次复制

sub_answer[j]。由于您不再需要sub_answer[j],您可以使用std::move来交换所有权而不是复制:

answer.push_back(std::move(sub_answer[j]));