c - 使用 malloc 进行指针分配出错-6ren

c - 使用 malloc 进行指针分配出错

转载作者：行者123 更新时间：2023-11-30 14:26:54

请随意复制粘贴并尝试一下，当单独的手在内存中分配了空间时，它会在函数handget中途停止工作

/* 
 * A program that is meant to classify five-card poker hands. 
 *Is still in progress
 */

#include<stdio.h>
#include<math.h>
#include<ctype.h>
#include<stdlib.h>
#include<assert.h>
#include<string.h>

#define SEED 7
#define SIZE 5

/*
 *
 */

typedef enum suits {
   clubs,
   diamonds,
   hearts,
   spades
} suits;

typedef enum values {
    zero,
    one, // not used, but ensures numerical values correspond
    two,
    three,
    four,
    five,
    six,
    seven,
    eight,
    nine,
    ten,
    jack,
    queen,
    king,
    ace
} values; 

typedef struct cards {
    suits suit;
    values value;
} cards;

int islegal(cards *hand, int nr_of_cards);
/* Given any number of cards, determines whether any duplicates
 * or false cards are present or not. If so, returns 0, otherwise 1. 
 */
int flop(cards **handPtr); 
/* Asks the user to input a hand of cards; 
 * returns the number of cards being input
 */
int *rawtoflop (cards *handPtr, int nr_of_cards);

int hander(cards **handPtr,int counter); 

int handget(cards **playerhands,cards *thishands, int handsize);

void printsuit(suits thissuit);
/* Prints the enum-type suits
 */

void printvalue(values thisvalue);
/* Prints the enum-type values
 */

void printhand(cards *hand, int nr_of_cards); 
/* Prints a hand of cards without further processing 
*/
int main(void) {
    cards *thishand, *playerhands;

    flop(&thishand);
    printhand(thishand,6);
    handget(&playerhands,thishand,6);

    return 0;
}

int islegal(cards *hand, int nr_of_cards) {
    int i, fulldeck[4][13]={0};
    int current_value, current_suit;
    cards *current_card = hand;
    int legal = 1; 

    for (i=0; i<nr_of_cards; i++) {
        current_value = (int) (*current_card).value;
        current_suit = (*current_card).suit;

        // if the current card has value zero, it is not a valid hand
        if (current_value==0) {
            legal = 0;
            break;
        }

        //check if the current card already appears, if yes invalid hand, if no, 
        //change the flag for that card in the full deck. 
        //Since (two) will correspond to fulldeck[.][0] there is a -2 offset.
        if ( (fulldeck[current_suit][current_value - 2]) > 0 ) {
            legal = 0;
            break;
        } else {
            fulldeck[current_suit][current_value - 2]++;
        }
        current_card++;
    }

    return legal;
}

int flop(cards **handPtr) {
    int i,*q=NULL,n=NULL;
    int j[5]={0,0,0,0,0};
    int k[5]={1,1,1,1,1},nr_of_cards = 5;//for more/less cards CHANGE HERE!
    char rawsuit, rawcard[4];

    // allocate the required amount of memory for your cards
    (*handPtr) = (cards *) malloc(nr_of_cards * sizeof(cards));

    n=memcmp(j,k,sizeof(j));
    while(n!=0) {
        printf("Please input the five cards on the table: ");
        q=rawtoflop(*handPtr,nr_of_cards);
        for (i=0;i<nr_of_cards;i++) {
            j[i]=*(q+i);
        }
        n=memcmp(j,k,sizeof(j));
    }

    free(&handPtr);
    return nr_of_cards;
}

int *rawtoflop (cards *handPtr, int nr_of_cards){
    int i,m[5],*mPtr;
    char rawsuit, rawcard[4];

    mPtr=m;
        // ask for the cards
    for (i=0; i<nr_of_cards; i++)/* do */{

        scanf("%3s", &rawcard);

        rawsuit = rawcard[0]; 

        if (rawcard[1]=='1') {
            if (rawcard[2]=='0') {
                (handPtr)[i].value = ten;
            } else {
                (handPtr)[i].value = zero;
            }
        } else if (rawcard[1]=='2') {
            (handPtr)[i].value = two;
        } else if (rawcard[1]=='3') {
            (handPtr)[i].value = three;
        } else if (rawcard[1]=='4') {
            (handPtr)[i].value = four;
        } else if (rawcard[1]=='5') {
            (handPtr)[i].value = five;
        } else if (rawcard[1]=='6') {
            (handPtr)[i].value = six;
        } else if (rawcard[1]=='7') {
            (handPtr)[i].value = seven;
        } else if (rawcard[1]=='8') {
            (handPtr)[i].value = eight;
        } else if (rawcard[1]=='9') {
            (handPtr)[i].value = nine;
        } else if (rawcard[1]=='J') {
            (handPtr)[i].value = jack;
        } else if (rawcard[1]=='Q') {
            (handPtr)[i].value = queen;
        } else if (rawcard[1]=='K') {
            (handPtr)[i].value = king;
        } else if (rawcard[1]=='A') {
            (handPtr)[i].value = ace;
        } else {
            (handPtr)[i].value = zero;
        }

        switch (rawsuit) {
            case 'h':
                (handPtr)[i].suit = hearts; 
                break;
            case 'd':
                (handPtr)[i].suit = diamonds;
                break;
            case 'c':
                (handPtr)[i].suit = clubs;
                break;
            case 's':
                (handPtr)[i].suit = spades;
                break;
            default:
                (handPtr)[i].value = zero;
        }

        m[i]=(islegal(handPtr,i+1));
    }
    return mPtr;
}


int hander(cards **handPtr,int counter) {
    int i, nr_of_cards = 2;
    char rawsuit, rawcard[4];

    if(counter==0){
        // allocate the required amount of memory for your cards
        (*handPtr) = (cards *) malloc(nr_of_cards * sizeof(cards));
    }
    // ask for the cards
    for (i=0; i<nr_of_cards; i++) do {

        scanf("%3s", &rawcard);

        rawsuit = rawcard[0]; 

        if (rawcard[1]=='1') {
            if (rawcard[2]=='0') {
                (*handPtr)[i].value = ten;
            } else {
                (*handPtr)[i].value = zero;
            }
        } else if (rawcard[1]=='2') {
            (*handPtr)[i].value = two;
        } else if (rawcard[1]=='3') {
            (*handPtr)[i].value = three;
        } else if (rawcard[1]=='4') {
            (*handPtr)[i].value = four;
        } else if (rawcard[1]=='5') {
            (*handPtr)[i].value = five;
        } else if (rawcard[1]=='6') {
            (*handPtr)[i].value = six;
        } else if (rawcard[1]=='7') {
            (*handPtr)[i].value = seven;
        } else if (rawcard[1]=='8') {
            (*handPtr)[i].value = eight;
        } else if (rawcard[1]=='9') {
            (*handPtr)[i].value = nine;
        } else if (rawcard[1]=='J') {
            (*handPtr)[i].value = jack;
        } else if (rawcard[1]=='Q') {
            (*handPtr)[i].value = queen;
        } else if (rawcard[1]=='K') {
            (*handPtr)[i].value = king;
        } else if (rawcard[1]=='A') {
            (*handPtr)[i].value = ace;
        } else {
            (*handPtr)[i].value = zero;
        }

        switch (rawsuit) {
            case 'h':
                (*handPtr)[i].suit = hearts; 
                break;
            case 'd':
                (*handPtr)[i].suit = diamonds;
                break;
            case 'c':
                (*handPtr)[i].suit = clubs;
                break;
            case 's':
                (*handPtr)[i].suit = spades;
                break;
            default:
                (*handPtr)[i].value = zero;
        }
    } while (!islegal(*handPtr, i+1));

    return nr_of_cards;
}

int handget(cards **playerhand,cards *thishands, int handsize) {

    cards *player=NULL,*individualhand=NULL;
    int nr_players=1;

    (*playerhand) = (cards *) malloc(7*sizeof(cards));

    memcpy(*playerhand,thishands,handsize*sizeof(cards));

    printf("Please enter the cards for player 1: ");

    hander(&player,0);
    memcpy(*playerhand+5,player,7*sizeof(cards));

        printf("1 we made it this far chaps!!!\n");
        individualhand =(cards *) malloc(7*sizeof(cards));//THIS IS WHERE IT ALL GOES WRONG!!
        printf("2 we made it this far chaps!!\n");

    return nr_players;
}

void printsuit(suits thissuit) {
    switch (thissuit) {
        case diamonds:
            printf("d");
            break;
        case clubs:
            printf("c");
            break;
        case hearts:
            printf("h");
            break;
        case spades:
            printf("s");
            break;
        }
    return;
}

void printvalue(values thisvalue) {
    switch (thisvalue) {
        case two:
            printf("2");    
            break;
        case three:
            printf("3");
            break;
        case four:
            printf("4");
            break;
        case five:
            printf("5");
            break;
        case six:
            printf("6");
            break;
        case seven:
            printf("7");
            break;
        case eight:
            printf("8");
            break;
        case nine:
            printf("9");
            break;
        case ten:
            printf("10");
            break;
        case jack:
            printf("J");
            break;
        case queen:
            printf("Q");
            break;
        case king:
            printf("K");
            break;
        case ace:
            printf("A");
            break;
    }

    return;
}


void printhand(cards *hand, int nr_of_cards) {
    int i;

    for (i=0; i<nr_of_cards; i++) {
        printsuit((hand[i]).suit);
        printvalue((hand[i]).value);
        printf(" ");
    }
    printf("\n");

    return;
}

最佳答案

您为 7 张卡分配了足够的空间:

(*playerhand) = (cards *) malloc(7*sizeof(cards));

然后将七张牌复制到*playerhand+5中:

memcpy(*playerhand+5,player,7*sizeof(cards));

所以你会遇到缓冲区溢出。 *playerhand+5 与 &(*playerhand)[5] 相同。因此，您要将 7 张卡复制到一个数组中的第五个位置，该数组的容量足以容纳 7 张卡。

这是使用魔数(Magic Number)的一个问题。如果你不给它起个名字，我就猜不出这个“5”是什么意思(也不知道为什么是 7 张牌)。

关于c - 使用 malloc 进行指针分配出错，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/8408218/

文章推荐： C & Libevent : add binary data to output buffer

文章推荐： c - opendir 函数损坏目录名称

文章推荐： javascript - lodash 链式方法中的当前链

c - 从 : malloc/free/malloc/free 的设计模式中获取与 malloc 相关的错误
我是 C 的新手，在 Linux 中使用带有开关 gcc -g -std=c89 -Wall ... 的 gcc4.4.6 进行编程，我在许多函数深处遇到了这个错误我的程序名为 compute: **
malloc - 使用jemalloc，malloc，calloc和其他常见替代方法的优缺点是什么？
今天阅读Rust subreddit时，我发现以下评论： jemalloc针对（多线程）速度而不是内存使用进行了优化经过更多研究后，我发现还有更多选择（例如calloc）。我想了解不同内存分配器的
malloc - 所以 malloc 不调用任何系统调用？
相关代码: write(-1, "test", sizeof("test")); void * p = malloc(1024); void * p2 = malloc(510); w
malloc - 如何对 malloc 实现进行基准测试？
我正在比较不同的 malloc 实现，我想比较它们的运行时间和内存使用情况。特别是，我对运行时和最大常驻内存感兴趣。重要的是最大常驻内存将是真实的(没有代码段等)。我不能使用像 valgrind
c - malloc(sizeof(int)) vs malloc(sizeof(int *)) vs (int *)malloc(sizeof(int))
我承认这三个都有不同的含义。但是，我不明白这些具体情况适用于哪些特定情况。任何人都可以分享每个例子吗？谢谢。 malloc(sizeof(int)) malloc(size
malloc - g_slice 真的比 malloc 快吗
GLib 文档推荐使用 GLib Slice Allocator 而不是 malloc: "For newly written code it is recommended to use the ne
c - Malloced char* 但崩溃并说它没有 malloc
我正在分配一个字符串 int main(){ int buf = 1024; char *input = malloc(sizeof(char*) * buf); //CODE
ios - malloc 诊断和 malloc 相关环境变量之间有什么关系？
Here有一个关于 malloc 包的环境变量列表: MallocStackLogging MallocStackLoggingNoCompact MallocPreScribble MallocSc
c - 分配了 malloc 返回值的指针是否也需要 malloc
总体问题:当您将通过malloc分配的返回值分配给一个指针时，您是否需要malloc该指针以及，还是您可以简单地声明并分配它？例如，假设我有一个函数 foo，它在执行过程中使用 malloc 创建了
c - 无法正确释放另一个 malloc 的 malloc
这个问题在这里已经有了答案: String assignment in C (4 个答案) 关闭 7 年前。这是有问题的片段。 int main() { char** RESERV = (
c - 多个小 malloc 与一个大 malloc
任务是将一个二进制文件解析到内存中。但是，我事先不知道需要分配的内存量。哪种方法更可取:在解析例程中进行多个小 malloc，或者首先遍历文件以确定所需的内存量，然后再次解析？感谢任何提示。最佳
c - malloc-free-malloc 和严格别名
我最近一直在尝试理解严格别名的一个特定方面，我认为我已经制作了尽可能最小的有趣代码。 (对我来说很有趣，就是这样!) 更新:根据到目前为止的答案，很明显我需要澄清这个问题。从某个角度来看，这里的第一个
c - 使 malloc 自动失败以测试 malloc 失败时的情况
我一直在为我创建的一个简单程序创建测试。我总是使用类似这样的方法检查使用 malloc 分配内存是否失败 int* ptr = malloc(sizeof(int) * x); if(!ptr){
ubuntu - 对齐的 malloc 和标准 malloc 之间的区别？
我是 malloc 和对齐 malloc 的新手。我知道如何使用它们。但是，我不知道在什么情况下我们应该使用对齐的 malloc 而不是标准的 malloc。你能给我解释一下吗？最佳答案 glibc
c - 使用 malloc 循环来保证 malloc 的结果不好吗？
这样分配内存是不好的做法吗？: FOO *foo; while (!(foo = malloc(sizeof(FOO)))) ; 最佳答案我不知道有什么不好的做法，但这种情况并不常见。 malloc
c - 使用 malloc 和不使用 malloc 创建结构体的区别
有人可以向我解释使用和不使用 malloc 创建结构之间的区别吗？什么时候应该使用 malloc，什么时候应该使用常规初始化？例如: struct person { char* name;
c - 在为该结构本身调用 malloc 之后，是否需要为该结构中的其他字段调用 malloc()？
假设我有一个类型 node_t typedef struct node{ char* value; struct node *next; }node_t; 当我想创建一个名为 n1 的
c - malloc 实际分配了多少内存，并且可以将变量存储在分配有 malloc 的数组中以节省内存？
关闭。这个问题需要更多 focused .它目前不接受答案。想改进这个问题？更新问题，使其仅关注一个问题 editing this post . 3年前关闭。 Improve this questi
c - 使用非 malloced 指针变量交换名称(malloced 名称)
我对指针感到困惑。这是交换两个名称的代码。请看代码。考虑输入:hellohai(对于 d)和 asd(对于 e)。我得到的输出:asd 1ellohai 1ellohai #include #incl
c - 添加到 malloc 数组的值将成为 malloc 数组中的所有值
我已经编写了这个函数(如下)。它应该逐行读取文件。编辑该行并将某些单词/字符放入各种功能中。然后将这些函数放入“entrant”结构的数组(malloc)中。问题是，当我退出循环并尝试打印数组时，放

行者123

个人简介

我是一名优秀的程序员,十分优秀！

作者热门文章

滴滴打车优惠券免费领取

全站热门文章

首页

博学

6Ren·AI

商城

c - 使用 malloc 进行指针分配出错