不知道为什么我在 C 中洗牌时会出现分割错误

Question

我知道这与我的指针变量有关，我的目标是将西装的颜色实现到我给定的预先存在的代码中。所以我在甲板结构下创建了一个字符指针。它将正确打印出所有内容，直到我开始洗牌，在这种情况下，我会出现分段错误。

 #include <stdio.h>
 #include <time.h> //time function
 #include <stdlib.h> //random number generator functions
 #include <string.h> //need for Linux
 #define MAX 9
 #define MAX_CARDS 52
 #define MAX_RANKS 13
 #define MAX_SUITS 4
 #define COLS 2 //number of columns to display in output
//structure definition
struct card{
char *rank;
char suit[MAX];
char *color;
};
typedef struct card Card;
//array of pointers to strings for ranks
char *ranks[MAX_RANKS] = {"Ace", "Two", "Three", "Four", "Five", "Six",     "Seven",
"Eight", "Nine", "Ten", "Jack", "Queen", "King"};
char *color[2] = {"(black)", "(red)"};
//two-dimensional array of strings for suits
char suits[MAX_SUITS][MAX] = {"Clubs", "Diamonds", "Hearts", "Spades"};
//[0][1] [0][2][0][3]
void initialize(Card []);
void shuffle(Card []);
void display(const Card[]);
int main(){
char newline = 'n'; //to repeat while loop
//declare an array of 52 cards
Card deck[MAX_CARDS] = {"","",""};
initialize(deck);
deck[0].color = "(black)" ;
deck[1].color = "(red)";
printf("Display an ordered deck of cards:n");
display(deck);
while('n' == newline){
    printf("nshuffling deck ... nn");
    shuffle(deck);
    display(deck);
    printf("nnWould you like to shuffle again?nIf so, press        "Enter" key. If not, enter any other char. ");
    newline = getchar();
}
    return 0;
}
/*
 initialize the deck of cards to string values
 deck: an array of structure cards
 */
void initialize(Card deck[]){
    int i = 0;
    for(i=0;i<MAX_CARDS;i++){
        deck[i].rank = ranks[i%MAX_RANKS];
        strncpy(deck[i].suit, suits[i/MAX_RANKS], MAX);
    }
}
/*
 use the pseudo-random number generator to shuffle the cards
 deck: an array of structure cards
 */
void shuffle(Card deck[]){
    int swapper = 0; //index of card to be swapped
    int i = 0; //counter
    Card temp = {"", ""}; //temp holding place for swap
    srand(time(NULL)); //seed the random numbers with current time
    for(i=0;i<MAX_CARDS;i++){
        //generate a pseudo-random number from 0 to 51
        swapper = rand() % MAX_CARDS;
        //swap current card with da swapper
        temp = deck[i];
        deck[i] = deck[swapper];
        deck[swapper] = temp;
    }
}
/*
 display the deck of cards
 deck: an array of structure cards 
 */
void display(const Card deck[]){
    int i = 0;
    for(i=0;i<MAX_CARDS;i++)
    {
        printf("%5s of %-10s", deck[i].rank, deck[i].suit);//%5s makes "of" line up -10 makes cols.
        if (strcmp(deck[i].suit, "Clubs") == 0 || strcmp(deck[i].suit, "Spades") == 0)
        {
            printf("     %5s", deck[0].color);
        }
         else
        {
            printf("      %5s", deck[1].color);
        }
        //put in a newline every %x loops
        if((COLS-1) == (i%COLS)){
            printf("n");
        }
    }

}

Answer 1

这是因为您的卡片具有您实际上不使用的color，除了这里：

deck[0].color = "(black)" ;
deck[1].color = "(red)";

现在，当你洗牌deck时，你很可能会在那里得到另一张卡，它的.color仍然是空的，你的代码崩溃了printf("...%s...", deck[0].color);

解决方案当然是a(完全删除color成员，b(然后将打印代码修改为：

    if (strcmp(deck[i].suit, "Clubs") == 0 || strcmp(deck[i].suit, "Spades") == 0) {
        printf("     (black)");
    }
    else {
        printf("     (red)  ");
    }

---

但是，我个人会使用枚举来表示等级和花色，例如

enum Suit {
    HEARTS = 0, 
    SPADES, 
    DIAMONDS, 
    CLUBS
};
struct Card {
    enum Suit suit;
    ...
};

和印刷

char *suit_to_string[4] = {"Hearts", "Spades", "Diamonds", "Clubs"};
...
printf("%s", suit_to_string[deck[i].suit]);