2011-08-01 113 views
1

這就是我所擁有的,它不起作用。無法弄清楚爲什麼......這個問題可能在quick_sort函數中。幫助實現快速排序

#include <stdio.h> 
#include <stdlib.h> 

void quick_sort(int * a, int l, int r); 
void swap(int * a, int i, int j); 

int main(void) 
{ 
    #define N 10 
    int a[N], i; 
    for (i = 0; i < N; ++i) { 
    a[i] = rand(); 
    printf("%d\n", a[i]); 
    } 
    quick_sort(a, 0, N - 1); 
    for (i = 0; i < N; ++i) 
    printf("%d\n", a[i]); 
    return 0; 
    #undef N 
} 

void quick_sort(int * a, int l, int r) 
{ 
    int i, j; 
    if (l < r) { 
    swap(a, rand() % (r - l + 1), r); 
    i = l; 
    for (j = l; j < r; ++j) 
     if (a[j] < a[r]) { 
     swap(a, i, j); 
     ++i; 
     } 
    swap(a, i, r); 
    quick_sort(a, l, i - 1); 
    quick_sort(a, i + 1, r); 
    } 
} 

void swap(int * a, int i, int j) 
{ 
    int k; 
    k = a[i]; 
    a[i] = a[j]; 
    a[j] = k; 
} 
+0

以何種方式對不對工作...? – razlebe

+1

請提供你的'數據集','你的輸出'和'預期輸出'。 – hari

+0

'l'是一個可怕的標識符。在許多情況下,大多數單字母標識符都很糟糕,但是在任何情況下,「l」都非常可怕!考慮重命名變量('低'和'高';'左'和'右','llim'和'rlim',...) – pmg

回答

5

修復此行:swap(a, rand() % (r - l + 1), r);

應該是:swap(a, l + rand() % (r - l + 1), r);

+1

謝謝,解決了!起初我以爲'l'是'1'...... – Fox