Question

我正在进行5张牌扑克游戏，但在调试器中运行程序时我一直收到错误。

我通过我的调试器运行了8次这样的代码，并且我有5次错误 - 没有更改任何代码。什么可能导致它有时不正确地读取字符串？请帮我搞清楚。

必要的代码：

-Initialization

char *suit[4] = {"Hearts", "Diamonds", "Clubs", "Spades"};
char *face[13] = {"Ace", "Two", "Three", "Four", "Five", "Six", "Seven", 
"Eight", "Nine", "Ten", "Jack", "Queen", "King"};
Card p1_hand[5] = {{0,0,0}};

- 主要

中的功能调用

print_hand(face, suit, p1_hand);

- 头文件中的函数// Card struct

typedef struct card
{
int card_number;
int face_index;
int suit_index;
} Card;

void print_hand (char *face[4], char *suit[13], Card p1_hand[5]);

- 功能本身

void print_hand (char *face[], char *suit[], Card p1_hand[5])
{
int i = 0, j = 0, k = 0;

printf("You're hand:\n");
for(i = 0; i < 5; i++)
{
    j = p1_hand[i].face_index;
    k = p1_hand[i].suit_index;
    printf("%d: %5s of %-8s\n", (i+1), face[j], suit[k]);
}
}

我在第一个printf之前放置了一个调试器停止点，有时我从中得到它

face 0x0040fb5c {0x0000000a Error reading characters of string.}

但有时我得到这个......根本没有改变代码

face 0x002afe40 {0x013a787c "Ace"}（这是正确的）

请帮我解决这个问题！这让我发疯了。

修改 * 我的代码 *

的main.c

#include "poker.h"

int main (void)
{
//initialize arrays
char *suit[4] = {"Hearts", "Diamonds", "Clubs", "Spades"};
char *face[13] = {"Ace", "Two", "Three", "Four", "Five", "Six", "Seven", "Eight", 
"Nine", "Ten", "Jack", "Queen", "King"};
int deck[4][13] = {0};
int num_face_player_1[13] = {0}, num_face_dealer[13] = {0};
int num_suit_player_1[4] = {0}, num_suit_dealer[4] = {0};

//Players' hands
Card p1_hand[5] = {{0,0,0}}, dealer_hand[5] = {{0,0,0}};

//Other
int count = 0;
int menu = 0, choice_menu = 0;
int invalid = 0;

srand ((unsigned) time (NULL));

//MENU
do //menu loop
{
    menu = print_menu ();
    choice_menu = 4; //so its not 0;
    choice_menu = menu_choice (menu);
} while (choice_menu ==0);
if (choice_menu == 2) //closes the game
{
    return 0;
}

system ("cls");
shuffle (deck);
deal (deck, p1_hand);

do //checks to make sure the dealer doesnt have any of the same cards
{
    deal (deck, dealer_hand);
    invalid = 0;
    for(count = 0; count < 5; count++)
    {
        if ((dealer_hand[count].card_number == p1_hand[count].card_number) && (dealer_hand[count].face_index == p1_hand[count].face_index) && (dealer_hand[count].suit_index == p1_hand[count].suit_index))
        {
            invalid = 1;
        }
    }
} while (invalid == 1);

for(count = 0; count < 4; count++)
{
    printf("%s\n", *suit[count]);
}
for(count = 0; count < 13; count++)
{
    printf("%s\n", *face[count]);
}
print_hand(face, suit, p1_hand);

//these populate the number of facecards / suits arrays in order to see what hand you got
pop_num_faces(num_face_player_1, p1_hand);
pop_num_faces(num_face_dealer, dealer_hand);
pop_num_suit(num_suit_player_1, p1_hand);
pop_num_suit(num_suit_dealer, dealer_hand);

return 0;
}

poker.h

#ifndef POKER_H
#define POKER_H

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

typedef struct card
{
    int card_number;
    int face_index;
    int suit_index;
} Card;

int print_menu (void);
int menu_choice (int menu_choice);
void press_any_key (void);
void shuffle (int wDeck[4][13]);
void deal(const int wDeck[][13], Card p_hand[5]);
void pop_num_faces (int num_faces[], Card hand[5]);
void pop_num_suit (int num_suit[4], Card hand[5]);
void print_hand (char *face[4], char *suit[13], Card p1_hand[5]);
void check_hand(Card p1_hand[5], int num_suit[], int num_faces[]);

#endif

poker.c

#include "poker.h"

//prints menu
int print_menu (void)


//decides what to do with the menu choice
int menu_choice (int menu_choice)
//Shuffles the deck
void shuffle (int wDeck[4][13])
{
int row = 0;
int column = 0;
int card = 0;

for (card = 1; card <= 52; card++)
{
    do
    {
        row = rand () % 4;
        column = rand () % 13;
    } while (wDeck[row][column] != 0);

    wDeck[row][column] = card;
}
}

//deals the deck to the player
void deal(const int wDeck[][13], Card p_hand[5])
{
int row = 0;
int column = 0;
int card = 0;
int index = 0;

for(card = 1; card < 52; card++)
{
    for (row = 0; row <4; row++)
    {
        for (column = 0; column < 13; column++)
        {
            if(wDeck[row][column] == card)
            {
                p_hand[index].card_number = card;
                p_hand[index].face_index = column;
                p_hand[index].suit_index = row;
            }
        }   
    }
    index++;
}
}

//populates num faces
void pop_num_faces (int num_faces[], Card hand[5])
{
int i = 0;

//program keeps populating num_faces with random numbers, so i have to  re-initialize here
    num_faces[0] = 0;
    num_faces[1] = 0;
    num_faces[2] = 0;
    num_faces[3] = 0;
    num_faces[4] = 0;
    num_faces[5] = 0;
    num_faces[6] = 0;
    num_faces[7] = 0;
    num_faces[8] = 0;
    num_faces[9] = 0;
    num_faces[10] = 0;
    num_faces[11] = 0;
    num_faces[12] = 0;

for(i = 0; i < 5; i++)
{
    switch(hand[i].face_index)
    {
    case 0: num_faces[0] ++;
        break;
    case 1: num_faces[1] ++;
        break;
    case 2: num_faces[2] ++;
        break;
    case 3: num_faces[3] ++;
        break;
    case 4: num_faces[4] ++;
        break;
    case 5: num_faces[5] ++;
        break;
    case 6: num_faces[6] ++;
        break;
    case 7: num_faces[7] ++;
        break;
    case 8: num_faces[8] ++;
        break;
    case 9: num_faces[9] ++;
        break;
    case 10: num_faces[10] ++;
        break;
    case 11: num_faces[11] ++;
        break;
    case 12: num_faces[12] ++;
        break;
    }
}
}

//populates num_suit
void pop_num_suit (int num_suit[4], Card hand[5])
{
int i = 0;
num_suit[0] = 0;
num_suit[1] = 0;
num_suit[2] = 0;
num_suit[3] = 0;

for(i = 0; i < 5; i++)
{
    switch(hand[i].suit_index)
    {
    case 0: num_suit[0] ++;
        break;
    case 1: num_suit[1] ++;
        break;
    case 2: num_suit[2] ++;
        break;
    case 3: num_suit[3] ++;
        break;
    }
}
}

void print_hand (char *face[4], char *suit[13], Card p1_hand[5])
{
printf("You're hand:\n");
for(i = 0; i < 5; i++)
{
    j = p1_hand[i].face_index;
    k = p1_hand[i].suit_index;
    printf("--DEBUG--: %d %d %d\n", p1_hand[i].card_number, j, k);
    printf("%d: %5s of %-8s\n", (i+1), face[j], suit[k]);
}
}

Answer 1

问题出在deal()函数中。这是它的调试版本：

void deal(int wDeck[][13], Card p_hand[5])
{
    int row = 0;
    int column = 0;
    int card = 0;
    int index = 0;

    for (card = 1; card < 52; card++)
    {
        for (row = 0; row < 4; row++)
        {
            for (column = 0; column < 13; column++)
            {
                if (wDeck[row][column] == card)
                {
                    p_hand[index].card_number = card;
                    p_hand[index].face_index = column;
                    p_hand[index].suit_index = row;
                    printf("Assign %d\n", index);
                }
            }
        }
        index++;
        printf("Index: %d\n", index);
    }
}

运行时，index从1到52;可惜p_hand数组只有5个元素！ Assign多次打印。你有阵列索引可怕失控。

顺便说一下，你太喜欢开关了。这样：

void pop_num_suit (int num_suit[4], Card hand[5])
{
    int i = 0;
    num_suit[0] = 0;
    num_suit[1] = 0;
    num_suit[2] = 0;
    num_suit[3] = 0;

    for(i = 0; i < 5; i++)
    {
        switch(hand[i].suit_index)
        {
        case 0: num_suit[0] ++;
            break;
        case 1: num_suit[1] ++;
            break;
        case 2: num_suit[2] ++;
            break;
        case 3: num_suit[3] ++;
            break;
        }
    }
}

应该是：

void pop_num_suit(int num_suit[4], Card hand[5])
{
    int i;
    for (i = 0; i < 4; i++)
        num_suit[i] = 0;
    for (i = 0; i < 5; i++)
        num_suit[hand[i].suit_index]++;
}

pop_num_faces()节省的空间更加引人注目。

Answer 2

评论有点大，因此CW答案。

void print_hand(char *face[], char *suit[], Card p1_hand[5])
{
    printf("Your hand:\n");
    for (int i = 0; i < 5; i++)
    {
        int j = p1_hand[i].face_index;
        int k = p1_hand[i].suit_index;
        printf("--DEBUG--: %d %d %d\n", p1_hand[i].card_number, j, k);
        printf("%d: %5s of %-8s\n", (i+1), face[j], suit[k]);
    }
}

这会在尝试使用该数据之前打印p1_hand的每个条目中的数据。这可能会告诉你一些正在发生的事情。您可以添加断言作为替代或添加：

        assert(j >= 0 && j < 13);
        assert(k >= 0 && j < 4);

SSCCE代码

这几乎是一个SSCCE--在print_hand()函数中，只有100多行代码在Unix上以与Windows崩溃相同的方式崩溃。

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

typedef struct card
{
    int card_number;
    int face_index;
    int suit_index;
} Card;

void shuffle(int wDeck[4][13]);
void deal(int wDeck[][13], Card p_hand[5]);
void print_hand(char *face[4], char *suit[13], Card p1_hand[5]);

static void print_deck(int deck[4][13]);

int main (void)
{
    char *suit[4] = {"Hearts", "Diamonds", "Clubs", "Spades"};
    char *face[13] = {"Ace", "Two", "Three", "Four", "Five", "Six", "Seven", "Eight",
        "Nine", "Ten", "Jack", "Queen", "King"};
    int deck[4][13] = { { 0 } };
    Card p1_hand[5] = {{0,0,0}};

    srand ((unsigned) time (NULL));

    printf("Shuffle\n");
    shuffle(deck);
    print_deck(deck);

    printf("Deal player\n");
    deal(deck, p1_hand);

    printf("Print player's hand\n");
    print_hand(face, suit, p1_hand);

    return 0;
}

static void print_deck(int deck[4][13])
{
    for (int x = 0; x < 4; x++)
    {
        for (int z = 0; z < 13; z++)
            printf(" %2d", deck[x][z]);
        putchar('\n');
    }
}

void shuffle(int wDeck[4][13])
{
    int row = 0;
    int column = 0;
    int card = 0;

    for (card = 1; card <= 52; card++)
    {
        do
        {
            row = rand () % 4;
            column = rand () % 13;
        } while (wDeck[row][column] != 0);

        wDeck[row][column] = card;
    }
}

void deal(int wDeck[][13], Card p_hand[5])
{
    int row = 0;
    int column = 0;
    int card = 0;
    int index = 0;

    for (card = 1; card < 52; card++)
    {
        for (row = 0; row <4; row++)
        {
            for (column = 0; column < 13; column++)
            {
                if (wDeck[row][column] == card)
                {
                    p_hand[index].card_number = card;
                    p_hand[index].face_index = column;
                    p_hand[index].suit_index = row;
                }
            }
        }
        index++;
    }
}

void print_hand(char *face[4], char *suit[13], Card p1_hand[5])
{
    int i = 0, j = 0, k = 0;

    printf("Your hand:\n");
    for (i = 0; i < 5; i++)
    {
        j = p1_hand[i].face_index;
        k = p1_hand[i].suit_index;
        printf("--DEBUG--: %d %d %d\n", p1_hand[i].card_number, j, k);
        printf("%d: %5s of %-8s\n", (i+1), face[j], suit[k]);
    }
}

好消息是shuffle()功能似乎没问题。

C帮助：MSVC调试器获取“读取字符串字符的错误”错误...但有时只是

2 个答案:

SSCCE代码