2016-11-01 4 views
-6

Я пытаюсь получить следующий код для работы сортировки слияния с векторами вместо массивов стилей C, и мне трудно понять, почему он сбой сегментации. Может ли кто-нибудь помочь мне понять проблему здесь?Передача вектора по ссылке: Ошибка сегментации

#include <bits/stdc++.h> 
using namespace std; 

void merge(vector<int>& a, int l, int m, int r) { 
    vector<int> L; 
    vector<int> R; 

    for (int i = 0; i <= m; i++) 
     L.push_back(a[i]); 
    for (int i = m+1; i <= r; i++) 
     R.push_back(a[i]); 

    int i = 0, j = 0; // Initial index of first and second subarray 
    int k = l; // Initial index of merged subarray 
    while (i < L.size() && j < R.size()) { 
     if (L[i] <= R[j]) { 
      a[k] = L[i]; 
      i++; 
     } 
     else { 
      a[k] = R[j]; 
      j++; 
     } 
     k++; 
    } 

    // Filling leftovers 
    while (i < L.size()) { 
     a[k] = L[i]; 
     k++; 
     i++; 
    } 
    while (j < R.size()) { 
     a[k] = R[j]; 
     k++; 
     j++; 
    } 
} 

void merge_sort(vector<int>& a, int l, int r) { 
    if (l < r) { 
     int m = l + (r-l)/2; // Avoids integer overflow. 
     merge_sort(a, l, m); 
     merge_sort(a, m+1, r); 
     merge(a, l, m, r); 
    } 
} 

int main() 
{ 
    vector<int> a = {2, 4, 1, 5, 3, 9}; 
    int size = a.size();  
    merge_sort(a, 0, size-1); 
    for (int i = 0; i < size; i++) 
     cout << a[i] << ' '; 
    cout << endl; 
    return 0; 
} 
+2

Правильный инструмент для решения таких проблем - ваш отладчик. Перед тем, как просить о переполнении стека, вы должны пропустить свой код по очереди *. Для получения дополнительной информации, пожалуйста, прочтите [Как отлаживать небольшие программы (Эрик Липперт)] (https://ericlippert.com/2014/03/05/how-to-debug-small-programs/). Как минимум, вы должны \ [изменить] ваш вопрос, чтобы включить пример [Минимальный, полный и проверенный] (http://stackoverflow.com/help/mcve), который воспроизводит вашу проблему, а также замечания, сделанные вами в отладчик. –

+0

'L' слишком много - вы помещаете в него элементы' m + 1', а не 'm-l'. В результате 'k' может выйти из конца' a'. В результате ваша программа проявляет неопределенное поведение, за счет переполнения буфера. –

+0

ππάντα ῥεῖ Я уже пробовал то, что вы предлагаете, но поскольку я не мог пройти через это, я подозревал, что мне не хватает подробного указания на использование векторов, которые могут быть проблемой здесь. Я добавляю сообщения valgrind к вопросу через минуту. Извините за отсутствие их. Спасибо что подметил это. – Duh

ответ

0

Вам нужно перейти от l к m при заполнении вектора L:

void merge(vector<int>& a, int l, int m, int r) { 
    vector<int> L; 
    vector<int> R; 

    for (int i = l; i <= m; i++) 
     L.push_back(a[i]); 
    ... 
} 

В противном случае L содержит дополнительные элементы за пределами диапазона интересов. Затем индекс k отходит от конца вектора a при итерации для размера L.

+0

Спасибо, что указали это. Не могу поверить, что я этого не замечал. – Duh

+0

Вам также не нужно писать цикл для этого: 'vector L (a.begin() + l, a.begin() + m + 1);' – PaulMcKenzie

Смежные вопросы