Може ли някой да ми каже какво не е наред с сортирането ми чрез сливане?

Може ли някой да ми каже какво не е наред с моята реализация на mergesort по-долу? От часове се чеша по главата..

void merge(int arr[], int low, int mid, int high)
{
    int i = 0;
    int j = 0;
    int k = 0;

    int buf1[1024];
    int buf1Length = mid - low + 1;

    int buf2[1024];
    int buf2Length = high - (mid + 1) + 1;

    for (i = low; i <= mid; i++)
    {
        buf1[j++] = arr[i];
    }

    for (i = mid + 1; i <= high; i++)
    {
        buf2[k++] = arr[i];
    }

    i = 0;
    j = 0;
    k = 0;

    while (j < buf1Length && k < buf2Length)
    {
        if (buf1[j] <= buf2[k])
        {
            arr[i++] = buf1[j++];
        }
        else
        {
            arr[i++] = buf2[k++];
        }
    }

    while (j < buf1Length)
        arr[i++] = buf1[j++];

    while (k < buf2Length)
        arr[i++] = buf2[k++];
}

void mergeSort(int arr[], int low, int high)
{
    int mid;

    if (low < high)
    {
        mid = (low + high) / 2;
        mergeSort(arr, low, mid);
        mergeSort(arr, mid + 1, high);
        merge(arr, low, mid, high);
    }
}

int main()
{
    int i;
    int arr[] = {6, 7, 2, 4, 9, 8};
    mergeSort(arr, 0, 5);

    printf("\n\n results: \n");
    for (i = 0; i < 6; i++)
    {
       printf("%d ", arr[i]);
    }
    printf("\n\n\n");

    return 0;
}

c mergesort
person Robert Hill    schedule 14.02.2011    source източник
comment
Ако това е домашно, моля, уверете се, че сте готови за това.   -  person Adam Robinson    schedule 14.02.2011

Отговори (3)


arrow_upward
0
arrow_downward

Сигурни ли сте, че искате да правите това:

i = 0;
j = 0;
k = 0;

?

На пръв поглед мисля, че трябва да бъде:

i = low;
j = 0;
k = 0;
person Mikel    schedule 14.02.2011
comment
О ****! Как го пропуснах! Благодаря ти! - person Robert Hill; 14.02.2011
comment
Добре дошли. Моля, не забравяйте да приемете отговора, ако работи. - person Mikel; 14.02.2011

arrow_upward
0
arrow_downward

Mergesort трябва да обедини сортирани подпоследователности:

a = 1,3,5
b = 2,4,6
c = merge(a, b) // C = 1,2,3,4,5,6

Не сортирате никаква подпоследователност.

person vz0    schedule 14.02.2011

arrow_upward
0
arrow_downward

Проблемът без съмнение е във вашата merge() функция.

Бих си представил, че функцията за сливане ще върви по следния начин (всички грешки тук са оставени на читателя :X ... това не е тествано и не е стандартно и трябва да се използва само за концептуални цели! ):

/**
 * let mid be the last element of the first array
 * making mid + 1 the first element of the second
 */
merge(int[] arr, int low, int mid, int high) {

  int[] buff;
  int lowpos;
  int highpos;
  int i;

  buff = new int[high - low + 1];
  lowpos = low;
  highpos = mid + 1;

  // merge the two subarrays into the buffer
  for(i = 0; i < (high - low + 1); i++) {
    if(lowpos > mid)  buff[i] = arr[highpos++];

    else if(highpos > high) buff[i] = arr[lowpos++];

    else if(arr[lowpos] < arr[highpos] buff[i] = arr[lowpos++];

    else buff[i] = arr[highpos++];
  }

  // copy the buffer back into the main array
  for(i = 0; i < (high - low + 1) i++) arr[i+low] = buff[i];
}
person corsiKa    schedule 14.02.2011