这是为什么会崩溃？ - C

Question

我很接近完成这一点，至少我希望如此，因为如果我不是，我会去疯了。我已经设法摆脱了所有的语法和错误，摆脱编译器，我开始希望我会终于完成，然后...它崩溃...

给一点点假设要打印出'甲板'中的五个值 - 然后再次随机地将其打印出来（以显示它被洗牌）

我很亲密。我可以品尝它！

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 
#define SIZE 52 

enum faces{Ace = 0, Jack = 10, Queen, King}; 
char * facecheck(int d); 
void shuffle(int deck[]); 
void draw(int deck[SIZE]); 
void cards(int hand); 
int main() 
{ 
    int deck[SIZE], i, n; 
    char suits[4][9] = 
    { 
     "Hearts", 
     "Diamonds", 
     "Clubs", 
     "Spades"}; 


    srand(time(NULL)) ; 

    for(i = 0; i<SIZE; i++) 
    { 
     deck[i] = i; 
     i++; 
    }; 

    draw(deck); 
    shuffle(deck); 
    draw(deck); 




    return 0; 
} 

char * facecheck(int d) 
{ 
    static char * face[] = 
    { 
     "Ace", 
     "Jack", 
     "Queen", 
     "King" }; 

    if(d == Ace) 
     return face[0]; 
    else 
    { 
     if(d == Jack) 
      return face[1]; 
     else 
     { 
      if(d == Queen) 
       return face[2]; 
      else 
      { 
       if(d == King) 
        return face[3]; 
      } 
     } 
    } 
} 



void shuffle(int deck[]) 
{ 
    int i, j, temp; 

    for(i = 0; i < SIZE; i++) 
    { 
      j = rand() % SIZE; 
      temp = deck[i]; 
      deck[i] = deck[j]; 
      deck[j] = temp; 
      } 
    printf("The deck has been shuffled \n"); 
} 

void draw(int deck[SIZE]) 
{ 
    int i; 
    int hand[i]; 

    for(i = 0; i < 5; i++) 
    { 
      cards(hand[i]); 
      putchar('\n'); 
      } 
} 

void cards(hand) 
{ 
    int i; 

     char suits[4][9] = 
    { 
     "Hearts", 
     "Diamonds", 
     "Clubs", 
     "Spades"}; 

     for(i=0; i<SIZE; i++) 
    {  
     if(i%13 == 0 || i%13 == 10 || i%13 == 11 || i%13 == 12) 
      printf("%s ", facecheck(i%13)); 
     else printf("%d ", i%13+1); 
     printf("of %s \n", suits[i/13]); 
    } 

} 

Answer 1

它可能崩溃，因为在这里：

for(i = 0; i<SIZE; i++) 
{ 
    deck[i] = i; 
    i++; 
}; 

你增加我的两倍。 删除i++行：

for(i = 0; i<SIZE; i++) 
{ 
    deck[i] = i; 
}; 

而且，在这里：

void draw(int deck[SIZE]) 
{ 
    int i; 
    int hand[i]; 

行int hand[i]可能会失败，因为我不是一个常数（它不是也初始化）。

Answer 2

一些注意事项：

void draw(int deck[SIZE]) 
{ 
    int i; 
    int hand[i]; 

    for(i = 0; i < 5; i++) 
    { 
     cards(hand[i]); 
     putchar('\n'); 
    } 
} 

i默认情况下不进行初始化，并且将包含之前，那么你创建这个任意长度的数组无论是在内存地址 - 你的意思是这样做呢？

然后你将手[0]，手[1]，...，手[4]传递给纸牌，即使它们未定义。

这可能会导致崩溃。

也许你想是这样的：

void draw(int deck[SIZE]) 
{ 
    int numCards = 5; 
    int i; 
    int hand[numCards]; 
    int card; 
    for(i = 0; i < numCards; i++) 
    { 
     card = deck[i];  /* Draw a card */ 
     hand[i] = card;  /* Store it in a "hand" array (even though you dont use it) */ 
     cards(card);  /* Display info about the card */ 
    } 
} 

---

在这里，您打印出每张牌的整数值会是什么，不管你的甲板上的排序。

void cards(hand) 
{ 
    int i; 

    char suits[4][9] = 
    { 
     "Hearts", 
     "Diamonds", 
     "Clubs", 
     "Spades"}; 

    for(i=0; i<SIZE; i++) 
    {  
     if(i%13 == 0 || i%13 == 10 || i%13 == 11 || i%13 == 12) 
      printf("%s ", facecheck(i%13)); 
     else 
      printf("%d ", i%13+1); 
     printf("of %s \n", suits[i/13]); 
    } 

} 

我不认为你的意思是这样做。我想你的意思是这样

void cards(int card) 
{ 
    char suits[4][9] = 
    { 
     "Hearts", 
     "Diamonds", 
     "Clubs", 
     "Spades"}; 


    if(card%13 == 0 || card%13 == 10 || card%13 == 11 || card%13 == 12) 
     printf("%s ", facecheck(card%13)); 
    else 
     printf("%d ", card%13+1); 
    printf("of %s \n", suits[card/13]); 
} 

既然你已经调用每个卡，没有理由遍历任何关于此功能。

Answer 3

for(i = 0; i<SIZE; i++) 
{ 
    deck[i] = i; 
    i++; 
}; 

for循环中分号的错误？

for(i = 0; i<SIZE; i++) 
{ 
    deck[i] = i; 
    i++; //one of the answer stated to remove this sentence 
}