2016-01-20 159 views
2

我正在做一個隨機隨機洗牌10次(12)的項目,但它沒有與我的代碼一起工作。無法隨機洗牌

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

void show(int[],int); 

void shuffle(int[],int,int); 

int main (void) 
{ 
    int karten[]={1,2,3,4,5,6,7,8,9,10}; 
    int n = sizeof(karten)/sizeof(int); 
    //printf("%d",n); 
    int s=12; 

    srand(time(NULL)); 
    printf("Karten vor dem Mischen: \n"); 
    show(karten,n); 
    shuffle(karten,n,s); 
    printf("Karten nach dem Mischen:\n"); 
    show(karten,n); 
    return 0; 
} 
void show(int karten[],int n) 
{ 
    for(int i=0;i<n;i++) 
    { 
     printf("%d,",karten[i]); 
    } 
    printf("\n"); 
} 
void shuffle(int karten[],int n,int s) 
{ 
    int i=0; 
    int d=0; 
    int v[]={(int)malloc(sizeof(karten))}; 
    for(int z=0;z<=s;z++) 
    { 
     i=rand()%10; 
     d=rand()%10; 
     //printf("%d-->%d\n",i,d); 
     v[i]=karten[i]; 
     v[d]=karten[d]; 
     karten[d]=v[i]; 
     karten[i]=v[d]; 

    } 
    printf("Es wurden %d Vertauschungen gemacht\n",s); 
    free(v); 
} 

的錯誤是

(((process returned 255))) 

和程序崩潰。

回答

2

sizeof(karten)不給你陣列的長度,見sizeof array clarification。只是改寫這樣的代碼(不需要另一個數組):

void shuffle(int karten[],int n,int s) 
{ 
    int i=0, vi; 
    int d=0, vd; 
    for(int z=0;z<=s;z++) 
    { 
     i=rand()%10; 
     d=rand()%10; 
     vi=karten[i]; 
     vd=karten[d]; 
     karten[d]=vi; 
     karten[i]=vd; 
    } 
    printf("Es wurden %d Vertauschungen gemacht\n",s); 
} 
+0

我想你應該解釋爲什麼*。畢竟main中的sizeof(karten)'是有效的,因此可能會讓人困惑。 – user694733

3

void shuffle(int karten[],int n,int s)相當於
void shuffle(int *karten,int n,int s)
所以int v[]={(int)malloc(sizeof(karten))};分配的karten
只是其尺寸是一個指向一個int,所以4字節(在大多數系統上)。
因此,您正試圖訪問未分配的內存。