2011-08-01 8 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

あなたの 'dataset'、' your output'と 'expected output'を提供してください。 – hari

+0

'l'はひどい識別子です。多くの文脈では、ほとんどの1文字の識別子は悪いですが、どの文脈でも 'l'は単なるひどい恐ろしいものです!変数の名前を変更することを検討してください(lowとhigh、leftとright、llimとrlim、...) – pmg

答えて

5

修正このライン:swap(a, rand() % (r - l + 1), r);

は次のようになります。どのようにしてswap(a, l + rand() % (r - l + 1), r);

+1

ありがとう、それが解決します!そして、最初は私は 'l'は' 1'だと思った... – Fox

関連する問題