在 C 中遇到段错误答案

【问题标题】：Running into a segfault in C在 C 中遇到段错误
【发布时间】：2018-01-22 08:52:00
【问题描述】：

编辑：底部的工作代码

我对编程比较陌生，我试图理解为什么这不起作用。我决定通过一个函数而不是main() 来发牌，以使其尽可能模块化。下面是我的代码；我知道错误与指针有关，但我不明白我做错了什么。

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

int dealCards (char* cards[52][30], int* x, int* y, int* a, int* b)
{
    time_t t;
    srand((unsigned) time(&t));

    printf("\n\nNow let's give you two random cards!\n\n");

    *x = rand() % 53;
    if (x != 0) {
        *x = x - 1;
    }

    *y = rand() % 53;
    if (y == x) {
        while (y == x) {
            y = rand() % 53;
        }
    }
    if (y != 0) {
        y = y - 1;
    }

    int i;
    printf("Card 1 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*x][i]);
        }
    printf("\nCard 2 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*y][i]);
        }

    printf("\n%d\n", x);
    printf("%d", y);

    printf("\n\nNow let's give your opponent two random cards!\n\n");

    *a = rand() % 53;
    if (a != 0) {
        a = a - 1;
    }
    if ((a == x) || (a == y)) {
        while ((a == x) || (a == y)) {
            *a = rand() % 53;
        }
    }

    *b = rand() % 53;
    if (b == a) {
        while ((b == a) || (b == x) || (b == y)) {
                *b = rand() % 53;
        }
    }
    if (b != 0) {
        *b = b - 1;
    }

    printf("Card 1 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*a][i]);
        }
    printf("\nCard 2 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*b][i]);
        }

    printf("\n%d\n", a);
    printf("%d", b);
}

int main()
{
    char deck[52][30] = { 
        {"_____\n|2♥ |\n|   |\n|_2♥|\n"},
        {"_____\n|3♥ |\n|   |\n|_3♥|\n"},
        {"_____\n|4♥ |\n|   |\n|_4♥|\n"},
        {"_____\n|5♥ |\n|   |\n|_5♥|\n"},
        {"_____\n|6♥ |\n|   |\n|_6♥|\n"},
        {"_____\n|7♥ |\n|   |\n|_7♥|\n"},
        {"_____\n|8♥ |\n|   |\n|_8♥|\n"},
        {"_____\n|9♥ |\n|   |\n|_9♥|\n"},
        {"_____\n|10♥|\n|   |\n|10♥|\n"},
        {"_____\n|J♥ |\n|   |\n|_J♥|\n"},
        {"_____\n|Q♥ |\n|   |\n|_Q♥|\n"},
        {"_____\n|K♥ |\n|   |\n|_K♥|\n"},
        {"_____\n|A♥ |\n|   |\n|_A♥|\n"},

        {"_____\n|2♦ |\n|   |\n|_2♦|\n"},
        {"_____\n|3♦ |\n|   |\n|_3♦|\n"},
        {"_____\n|4♦ |\n|   |\n|_4♦|\n"},
        {"_____\n|5♦ |\n|   |\n|_5♦|\n"},
        {"_____\n|6♦ |\n|   |\n|_6♦|\n"},
        {"_____\n|7♦ |\n|   |\n|_7♦|\n"},
        {"_____\n|8♦ |\n|   |\n|_8♦|\n"},
        {"_____\n|9♦ |\n|   |\n|_9♦|\n"},
        {"_____\n|10♦|\n|   |\n|10♦|\n"},
        {"_____\n|J♦ |\n|   |\n|_J♦|\n"},
        {"_____\n|Q♦ |\n|   |\n|_Q♦|\n"},
        {"_____\n|K♦ |\n|   |\n|_K♦|\n"},
        {"_____\n|A♦ |\n|   |\n|_A♦|\n"},

        {"_____\n|2♠ |\n|   |\n|_2♠|\n"},
        {"_____\n|3♠ |\n|   |\n|_3♠|\n"},
        {"_____\n|4♠ |\n|   |\n|_4♠|\n"},
        {"_____\n|5♠ |\n|   |\n|_5♠|\n"},
        {"_____\n|6♠ |\n|   |\n|_6♠|\n"},
        {"_____\n|7♠ |\n|   |\n|_7♠|\n"},
        {"_____\n|8♠ |\n|   |\n|_8♠|\n"},
        {"_____\n|9♠ |\n|   |\n|_9♠|\n"},
        {"_____\n|10♠|\n|   |\n|10♠|\n"},
        {"_____\n|J♠ |\n|   |\n|_J♠|\n"},
        {"_____\n|Q♠ |\n|   |\n|_Q♠|\n"},
        {"_____\n|K♠ |\n|   |\n|_K♠|\n"},
        {"_____\n|A♠ |\n|   |\n|_A♠|\n"},

        {"_____\n|2♣ |\n|   |\n|_2♣|\n"},
        {"_____\n|3♣ |\n|   |\n|_3♣|\n"},
        {"_____\n|4♣ |\n|   |\n|_4♣|\n"},
        {"_____\n|5♣ |\n|   |\n|_5♣|\n"},
        {"_____\n|6♣ |\n|   |\n|_6♣|\n"},
        {"_____\n|7♣ |\n|   |\n|_7♣|\n"},
        {"_____\n|8♣ |\n|   |\n|_8♣|\n"},
        {"_____\n|9♣ |\n|   |\n|_9♣|\n"},
        {"_____\n|10♣|\n|   |\n|10♣|\n"},
        {"_____\n|J♣ |\n|   |\n|_J♣|\n"},
        {"_____\n|Q♣ |\n|   |\n|_Q♣|\n"},
        {"_____\n|K♣ |\n|   |\n|_K♣|\n"},
        {"_____\n|A♣ |\n|   |\n|_A♣|\n"},
    };

    int i, j;
    for (i = 0; i < 52; i++) {
        for (j = 0; j < 30; j++) {
            printf("%c", deck[i][j]);
        }
    }

    //dealing
    int heroFirst, heroSecond, villainFirst, villainSecond;

    dealCards(deck, &heroFirst, &heroSecond, &villainFirst, &villainSecond);


    //bets
}

工作代码：

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

int dealCards (char cards[52][30], int *x, int *y, int *a, int *b)
{
    time_t t;
    srand((unsigned) time(&t));

    printf("\n\nYour cards:\n\n");

    *x = rand() % 53;
    if (*x != 0) {
        *x = *x - 1;
    }

    *y = rand() % 53;
    if (*y == *x) {
        while (*y == *x) {
            *y = rand() % 53;
        }
    }
    if (*y != 0) {
        *y = *y - 1;
    }

    int i;
    printf("Card 1 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*x][i]);
        }
    printf("\nCard 2 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*y][i]);
        }

    printf("\n\nVillain cards:\n\n");

    *a = rand() % 53;
    if (*a != 0) {
        *a = *a - 1;
    }
    if ((*a == *x) || (*a == *y)) {
        while ((*a == *x) || (*a == *y)) {
            *a = rand() % 53;
        }
    }

    *b = rand() % 53;
    if (*b == *a) {
        while ((*b == *a) || (*b == *x) || (*b == *y)) {
                *b = rand() % 53;
        }
    }
    if (*b != 0) {
        *b = *b - 1;
    }

    printf("Card 1 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*a][i]);
        }
    printf("\nCard 2 >\n");
    for (i = 0; i < 30; i++) {
            printf("%c", cards[*b][i]);
        }
}

int main()
{
    char deck[52][30] = { 
        {"_____\n|2♥ |\n|   |\n|_2♥|\n"},
        {"_____\n|3♥ |\n|   |\n|_3♥|\n"},
        {"_____\n|4♥ |\n|   |\n|_4♥|\n"},
        {"_____\n|5♥ |\n|   |\n|_5♥|\n"},
        {"_____\n|6♥ |\n|   |\n|_6♥|\n"},
        {"_____\n|7♥ |\n|   |\n|_7♥|\n"},
        {"_____\n|8♥ |\n|   |\n|_8♥|\n"},
        {"_____\n|9♥ |\n|   |\n|_9♥|\n"},
        {"_____\n|10♥|\n|   |\n|10♥|\n"},
        {"_____\n|J♥ |\n|   |\n|_J♥|\n"},
        {"_____\n|Q♥ |\n|   |\n|_Q♥|\n"},
        {"_____\n|K♥ |\n|   |\n|_K♥|\n"},
        {"_____\n|A♥ |\n|   |\n|_A♥|\n"},

        {"_____\n|2♦ |\n|   |\n|_2♦|\n"},
        {"_____\n|3♦ |\n|   |\n|_3♦|\n"},
        {"_____\n|4♦ |\n|   |\n|_4♦|\n"},
        {"_____\n|5♦ |\n|   |\n|_5♦|\n"},
        {"_____\n|6♦ |\n|   |\n|_6♦|\n"},
        {"_____\n|7♦ |\n|   |\n|_7♦|\n"},
        {"_____\n|8♦ |\n|   |\n|_8♦|\n"},
        {"_____\n|9♦ |\n|   |\n|_9♦|\n"},
        {"_____\n|10♦|\n|   |\n|10♦|\n"},
        {"_____\n|J♦ |\n|   |\n|_J♦|\n"},
        {"_____\n|Q♦ |\n|   |\n|_Q♦|\n"},
        {"_____\n|K♦ |\n|   |\n|_K♦|\n"},
        {"_____\n|A♦ |\n|   |\n|_A♦|\n"},

        {"_____\n|2♠ |\n|   |\n|_2♠|\n"},
        {"_____\n|3♠ |\n|   |\n|_3♠|\n"},
        {"_____\n|4♠ |\n|   |\n|_4♠|\n"},
        {"_____\n|5♠ |\n|   |\n|_5♠|\n"},
        {"_____\n|6♠ |\n|   |\n|_6♠|\n"},
        {"_____\n|7♠ |\n|   |\n|_7♠|\n"},
        {"_____\n|8♠ |\n|   |\n|_8♠|\n"},
        {"_____\n|9♠ |\n|   |\n|_9♠|\n"},
        {"_____\n|10♠|\n|   |\n|10♠|\n"},
        {"_____\n|J♠ |\n|   |\n|_J♠|\n"},
        {"_____\n|Q♠ |\n|   |\n|_Q♠|\n"},
        {"_____\n|K♠ |\n|   |\n|_K♠|\n"},
        {"_____\n|A♠ |\n|   |\n|_A♠|\n"},

        {"_____\n|2♣ |\n|   |\n|_2♣|\n"},
        {"_____\n|3♣ |\n|   |\n|_3♣|\n"},
        {"_____\n|4♣ |\n|   |\n|_4♣|\n"},
        {"_____\n|5♣ |\n|   |\n|_5♣|\n"},
        {"_____\n|6♣ |\n|   |\n|_6♣|\n"},
        {"_____\n|7♣ |\n|   |\n|_7♣|\n"},
        {"_____\n|8♣ |\n|   |\n|_8♣|\n"},
        {"_____\n|9♣ |\n|   |\n|_9♣|\n"},
        {"_____\n|10♣|\n|   |\n|10♣|\n"},
        {"_____\n|J♣ |\n|   |\n|_J♣|\n"},
        {"_____\n|Q♣ |\n|   |\n|_Q♣|\n"},
        {"_____\n|K♣ |\n|   |\n|_K♣|\n"},
        {"_____\n|A♣ |\n|   |\n|_A♣|\n"},
    };

    int i, j;
    for (i = 0; i < 52; i++) {
        for (j = 0; j < 30; j++) {
            printf("%c", deck[i][j]);
        }
    }

    double heroStack, villainStack;
    printf("\n-----------------------------");
    printf("\nWelcome to Heads Up Hold 'Em!");
    printf("\n-----------------------------");

    printf("\nHow many chips are your going to start off with?\n> ");
    scanf("%lf", &heroStack);
    villainStack = heroStack;

    printf("\nYour chipstack: %.lf\n", heroStack);
    printf("Villain's chipstack: %.lf\n", villainStack);

    //dealing
    int heroFirst, heroSecond, villainFirst, villainSecond;

    dealCards(deck, &heroFirst, &heroSecond, &villainFirst, &villainSecond);
    printf("\n%d\n", heroFirst);
    printf("%d\n", heroSecond);
    printf("%d\n", villainFirst);
    printf("%d\n", villainSecond);

    //bets
    //double heroBet, villainBet;
    //printf("\nYour move");
}

【问题讨论】：

char* cards[52][30] --> char cards[52][30]
*x = x - 1; - 看起来不可疑吗？ a = a - 1; - 而a 是一个指针？还有很多很多类似的问题。
始终认真对待编译器的警告。
您应该收到该代码的编译器警告。打开/打开编译器警告并修复它们！（对于gcc，将-Wall 添加到编译行。）
使用card = rand() % 52 并担心代码工作时RNG 分发的质量。但对于休闲游戏练习来说，这并不重要。

标签： c pointers

【解决方案1】：

问题是您在获取价值和计算价值之间混淆了这个if（和类似的），你知道你在做什么吗？

*x = rand() % 53; if (x != 0) { *x = x - 1; }

好吧，你的函数中的x是一个指针，这意味着，如果你想获取或赋值，你必须使用运算符*，如*x = rand() % 53 但是当你比较x != 0时，编译器会认为你在检查x是否为NULL（默认情况下，NULL是0）。所以，不是上面语句的值，会导致逻辑错误。

如果您仔细查看您的main，您应该会看到您传递的x、y 和类似的参数永远不会为NULL，因为它们是在main 函数中本地创建的，并且在您离开之前仍然存在它。因此，您的语句*x = x - 1 将意外工作，这可能会导致您的段错误

~~我认为您正在使用 deal 函数来获取这些变量的随机值~~

int heroFirst, heroSecond, villainFirst, villainSecond;

因此，您可以通过如下方式声明函数来通过引用传递这些值： int dealCards (char cards[52][30], int& x, int& y, int& a, int& b) 然后像这样调用你的函数 dealCards (deck, heroFirst, heroSecond, villainFirst, villainSecond)

在你的 dealCards 定义中，只需删除 * 之类的 *x 或类似的东西，你的功能就会正常工作（如果你知道你要做什么，那些 x y）此外，您可以安全地打印整个字符串，而无需像这样使用 for 循环，看起来很糟糕，不是吗？

for (i = 0; i < 30; i++) { printf("%c", cards[*y][i]); }

只需正常打印 printf("%s", cards[*y]);

编辑：

语法
您应该像上面的 cmets 一样重新声明您的 cardDecks 以使其工作

【讨论】：

这是一个 C 问题 - 如您所描述的那样通过引用传递不存在。也许您正在考虑 C++？
谢谢，但我最初选择使用诸如*x 之类的指针的原因是因为我不想从dealCards 函数返回任何内容，因为我不知道该怎么做。每当我尝试返回值时，我都会在main 中调用函数，然后通过printf("%d", heroFirst) 进行测试，它会抛出一个不在0 - 52 范围内的随机数
@ChrisTurner，哎呀，抱歉，谢谢你通知我（y）@MichaelKhoussid 因为这个和类似的行，它是随机的：*x = x - 1;你应该改为*x = *x - 1
@LoiLy 您应该编辑您的答案并删除那些破坏其他有用答案的误导性内容