2

所以我正在检查 NULL 指针/空字符串，如果找到，则返回 0。但是，我似乎Segmentation Fault在其中一个编译器中遇到了错误。如果有人可以帮助我找出可能导致它的原因。显然，当empty strings和null pointers用作输入时，就会出现错误。但是，这个错误并没有出现在我正在使用的编译器中，而是出现在另一个编译器（我正在评分的编译器）中。

rpsls.c

#include <string.h>

int rpsls(const char *player1, const char *player2)
{

    if (*player1 == '\0' || *player2 == '\0' || player1 == NULL || player2 == NULL || strcmp(player1, player2) == 0)
        return 0;

    char *r = "rock";
    char *p = "paper";
    char *si = "scissors";
    char *l = "lizard";
    char *s = "Spock";



    if (!strcmp(player1, r) && !strcmp(player2, si))
        return 1;
    else if (!strcmp(player1, p) && !strcmp(player2, r))
        return 1;
    else if (!strcmp(player1, si) && !strcmp(player2, p))
        return 1;
    else if (!strcmp(player1, l) && !strcmp(player2, s))
        return 1;
    else if (!strcmp(player1, s) && !strcmp(player2, si))
        return 1;
    else if (!strcmp(player1, r) && !strcmp(player2, l))
        return 1;
    else if (!strcmp(player1, p) && !strcmp(player2, s))
        return 1;
    else if (!strcmp(player1, si) && !strcmp(player2, l))
        return 1;
    else if (!strcmp(player1, l) && !strcmp(player2, p))
        return 1;
    else if (!strcmp(player1, s) && !strcmp(player2, r))
        return 1;

    if (!strcmp(player2, r) && !strcmp(player1, si))
        return -1;
    else if (!strcmp(player2, p) && !strcmp(player1, r))
        return -1;
    else if (!strcmp(player2, si) && !strcmp(player1, p))
        return -1;
    else if (!strcmp(player2, l) && !strcmp(player1, s))
        return -1;
    else if (!strcmp(player2, s) && !strcmp(player1, si))
        return -1;
    else if (!strcmp(player2, r) && !strcmp(player1, l))
        return -1;
    else if (!strcmp(player2, p) && !strcmp(player1, s))
        return -1;
    else if (!strcmp(player2, si) && !strcmp(player1, l))
        return -1;
    else if (!strcmp(player2, l) && !strcmp(player1, p))
        return -1;
    else if (!strcmp(player2, s) && !strcmp(player1, r))
        return -1;

    return 0;

}

main.c

#include <stdio.h>

int rpsls(const char *player1, const char *player2);

int main (void)
{
  printf ("%d\n", rpsls("rock","paper"));
  printf ("%d\n", rpsls("rock","rock"));
  printf ("%d\n", rpsls("paper","rock"));
  printf ("%d\n", rpsls("lizard",(char*)0));
  printf ("%d\n", rpsls("",(char*)0));

  return 0;
}
4

2 回答 2

6

表达式使用逻辑 OR 运算符从左到右进行计算，因此在

if (*player1 == '\0' || *player2 == '\0' ||
    player1 == NULL || player2 == NULL ||
    strcmp(player1, player2) == 0)

NULL你在*player1 == '\0'测试之前取消引用player1 == NULL。

*player1==NULL相当于player1[0]==NULL。如果player1指向您未分配的内存，则尝试读取此内存的效果是不确定的。在这一点上，任何事情都可能发生。在您的本地测试中，您报告这似乎有效（我对此表示怀疑）；在你老师的测试中，你的程序没有读取地址0和段错误的权限。

您需要重新排序表达式

if (player1 == NULL || *player1 == '\0' ||
    player2 == NULL || *player2 == '\0' ||
    strcmp(player1, player2) == 0)
于 2013-11-15T08:16:55.290 回答
3 
if (*player1 == '\0' || *player2 == '\0' || player1 == NULL || player2 == NULL || strcmp(player1, player2) == 0)
        return 0;

您在检查 NULL 之前取消引用指针。有点违背了 NULL 检查的目的。

首先检查 NULL，然后取消引用。

于 2013-11-15T08:16:38.370 回答