分配内存时合并排序逻辑失败。

Question

我正在研究合并排序的实现，它将按歌曲的长度对歌曲列表进行排序。我已经写了以下内容并且无法找出我的逻辑中的缺陷。这些函数不是返回排序列表，而是返回原始列表中间的一项列表。

vector<song> merge( vector<song> firstList, vector<song> secondList){
 vector<song> outPut;
 int i = 0; int n = 0;
 while( outPut.size() < (firstList.size() + secondList.size()-1 )){
    if( firstList[i].length < secondList[n].length){
        outPut.push_back( firstList[i]);
        i++;
    }else{
        outPut.push_back( secondList[n]);
        n++;
    }
 }
return outPut;
}
vector<song> mergeSortLength( vector<song> playlist){
 int scope = playlist.size()/2;
 vector<song> firstHalf( &playlist[0], &playlist[scope]);
 vector<song> secondHalf( &playlist[scope], &playlist[playlist.size()]);
 if ( firstHalf.size() != 1){
    firstHalf = mergeSortLength(firstHalf);
 }
 if ( secondHalf.size() != 1){
    secondHalf = mergeSortLength( secondHalf);
 }
 return merge( firstHalf, secondHalf);
}

如果我改变 while 循环条件

( outPut.size() < (firstList.size() + secondList.size() -1)){ 

到

( outPut.size() < (firstList.size() + secondList.size())){

列表排序成功到一半，直到编译器吐出：

playList(27898,0x7fff78b7b000) malloc: * mach_vm_map(size=7526769998340063232) 失败（错误代码=3） * 错误：无法分配区域 *** 在 malloc_error_break 中设置断点进行调试 libc++abi.dylib：以 std::bad_alloc 类型的未捕获异常终止：std::bad_alloc 中止陷阱：6

我非常感谢任何人的帮助。我的眼睛因为盯着这个而变得模糊。

Answer 1

在merge()中，代码不会检查是否已经到达向量的末尾，特别是如果i >= firstList.size()或n >= secondList.size()，在这种情况下另一个向量的其余部分将被复制。 while 语句的末尾不应有 -1。

VS 抱怨使用 &playlist[playlist.size()] 创建临时向量，因为下标超出范围。一个可选的方法是使用 begin() + index，它是一个迭代器：

    vector<song> firstHalf( playlist.begin()+0, playlist.begin()+scope);
    vector<song> secondHalf( playlist.begin()+scope, 
                 playlist.begin()+playlist.size());

已经过了一天，所以发布一个固定版本以防万一有人感兴趣。所有这些向量的创建和推回都会增加很多开销，最好进行一次分配，然后只使用迭代器或索引来处理子向量。

vector<song> merge( vector<song> firstList, vector<song> secondList){
    vector<song> outPut;
    size_t i = 0; size_t n = 0;
    while( outPut.size() < (firstList.size() + secondList.size()) ){
        if( firstList[i].length < secondList[n].length){
            outPut.push_back( firstList[i]);
            if(++i >= firstList.size()){
                do
                    outPut.push_back( secondList[n]);
                while(++n < secondList.size());
                break;
            }
        }else{
            outPut.push_back( secondList[n]);
            if(++n >= secondList.size()){
                do
                    outPut.push_back( firstList[i]);
                while(++i < firstList.size());
                break;
            }
        }
    }
    return outPut;
}

vector<song> mergeSortLength( vector<song> playlist){
    size_t scope = playlist.size()/2;
    vector<song> firstHalf( playlist.begin()+0, playlist.begin()+scope);
    vector<song> secondHalf( playlist.begin()+scope, playlist.begin()+playlist.size());
    if ( firstHalf.size() != 1){
        firstHalf = mergeSortLength(firstHalf);
    }
    if ( secondHalf.size() != 1){
        secondHalf = mergeSortLength( secondHalf);
    }
    return merge( firstHalf, secondHalf);
}

Answer 2

我没有看到很多关注，但也许你想要

vector<song> secondHalf( &playlist[scope+1], &playlist[playlist.size()-1]);

我认为没有 -1 的 &playlist[playlist.size()] 是你问题的根源， 但我没试过。