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

Merge sort logic failure in allocating memory.

本文关键字：内存 失败 分配 排序 合并      更新时间：2023-10-16

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

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())){

它大约在列表排序成功进行一半时，直到编译器吐出：

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

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

在 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);
}
我没有

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

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

没有 -1 的 &playlist[playlist.size（）] 我认为是您问题的根源，但我没有尝试过。

相关文章：