【问题标题】:Why does assigning my input to enum cause segmentation fault?为什么将我的输入分配给枚举会导致分段错误?
【发布时间】:2014-12-23 20:11:07
【问题描述】:

我有以下程序

#include <stdio.h>
#include "dbg.h"

#define MAX_DATA 100

typedef enum EyeColor{
    BLUE_EYES, GREEN_EYES, BROWN_EYES, BLACK_EYES, OTHER_EYES
} EyeColor;

const char *EYE_COLOR_NAMES[] = {
    "Blue", "Green", "Brown", "Black", "Other"
};

typedef struct Person {
    int age;
    char first_name[MAX_DATA];
    char last_name[MAX_DATA];
    EyeColor eyes;
    float income;
} Person;

int main(int argc, char *argv[]){
    Person you = {.age = 0};
    int i = 0;
    char *in = NULL;

    printf("What's your First Name? ");
    in = fgets(you.first_name, MAX_DATA-1, stdin);  //fgets is much better
    check(in != NULL, "Failed to read first name.");

    printf("What's your Last Name? ");
    in = fgets(you.last_name, MAX_DATA-1, stdin);
    check(in != NULL, "Failed to read last name.");

    printf("How old are you? ");
    int rc = fscanf(stdin, "%d", &you.age);
    check(rc > 0, "You have to enter a number.");

    printf("What color are your eyes:\n");
    for(i = 0; i <= OTHER_EYES; i++) {
        printf("%d) %s\n", i+1, EYE_COLOR_NAMES[i]);
    }
    printf("> ");

    int eyes = -1;
    rc = fscanf(stdin, "%d", &eyes);
    check(rc > 0, "You have to enter a number.");

    you.eyes = eyes - 1;
    //check(you.eyes <= OTHER_EYES && you.eyes >= 0, "Do it right, that's not an option.");     // if you dont check this, seg fault can happen because of ...

    printf("How much do you make an hour? ");
    rc = fscanf(stdin, "%f", &you.income);
    check(rc > 0, "Enter a floating point number.");

    printf("------ RESULTS ------\n ");

    printf("First Name: %s", you.first_name);
    printf("Last Name: %s", you.last_name);
    printf("Age: %d\n", you.age);
    printf("Eyes: %s\n", EYE_COLOR_NAMES[you.eyes]);
    printf("Income: %f\n", you.income);

    return 0;
error:
    return 1;
}

如果我跑了: $./ex24

我给出以下输入:

What's your First Name? a
What's your Last Name? b
How old are you? 1
What color are your eyes:
1) Blue
2) Green
3) Brown
4) Black
5) Other
> 1000
How much do you make an hour? 1
------ RESULTS ------
 First Name: a
Last Name: b
Age: 1
Segmentation fault (core dumped)

此段错误是如何发生的?那不是枚举在我的内存中分配了应该可以容纳1000的int类型吗?

编辑 1:将一些文本格式化为代码。 编辑 2:注释掉检查。

【问题讨论】:

  • 你的调试器说什么?
  • "正在分配我的内存中应该能够容纳 1000 的 int 类型?" – 噢,你对记忆的工作原理非常感到困惑。 (特别是,如果一个整数值不适合变量,那么高位就会被截断。)但我敢打赌,段错误不会仅仅因为分配给enum 类型的变量而引起。使用调试器!
  • 请发布check函数。
  • @iharob:我们不需要看到check 函数。他输入了1000 作为输入。他问为什么在删除对check 的调用时代码会出现段错误。原因很清楚。

标签: c memory segmentation-fault


【解决方案1】:

enum 能不能到 1000 无关紧要;您的数组没有 1000 个元素。
没有 1000th 种眼睛颜色可供打印。

这是你应该构建的测试用例:

#include <iostream>

const char* EYE_COLOR_NAMES[] = { "a", "b", "c", "d", "e" };
const int eyes = 1000;

int main()
{
    std::cout << EYE_COLOR_NAMES[eyes] << '\n';
}

我会说这显然行不通。

您已经在代码中对you.eyes 进行了范围检查,并且您自己发现只有在进行范围检查时才会发生分段错误。因此,只需将范围签入即可。

【讨论】:

  • 我没看到,它不可见。对不起。
  • 本来想回答一样的,突然看到chek,没看到评论。
  • @iharob:这个问题掩饰得很好。 OP不是很清楚,应该发布一个测试用例。
  • @LightnessRacesinOrbit,是的,你的评论是对的。我已经用支票注释掉了这条线。我会检查你的测试用例。
  • @dennycrane 该数组的有效索引是0, 1, 2, 3, 4。你期望EYE_COLOR_NAMES[1000] 究竟会做什么?
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 2021-07-18
  • 2020-04-15
  • 1970-01-01
  • 2015-08-06
  • 1970-01-01
  • 1970-01-01
  • 2020-10-28
相关资源
最近更新 更多