Question

愚蠢的小C程序，但对于我的生活，我无法弄清楚为什么我不断得到分段错误。我相信我正在为malloc分配足够的内存。任何帮助将非常感激。代码应该读取每月的一天和用户的“提醒”字符串，然后将这些读取到数组提醒指向的数据和malloc分配的内存空间。

/* Prints a one-month reminder list (dynamic string version) */

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

#define MAX_REMIND 50           /* maximum number of reminders */
#define MSG_LEN 60              /* maximum length of reminder message */

int read_line(char str[], int n);

int main(void)
{
    char *reminders[MAX_REMIND];
    char day_str[3], msg_str[MSG_LEN+1];
    int day, i, j, num_remind = 0;

    for (;;) {
        if (num_remind == MAX_REMIND) {
            printf("-- no space left --\n");
            break;
        }

        printf("Enter day and reminder: ");
        scanf("%2d", &day);
        if (day == 0) {
            break;
        } 
        sprintf(day_str, "%2d", day);
        read_line(msg_str, MSG_LEN);

        for (i = 0; i < num_remind; i++) {
            if (strcmp(day_str, reminders[i]) < 0) {
                break;
            }
        }
        for (j = num_remind; j > i; j--) {
            reminders[j] = reminders[j-1];
        }

        reminders[i] = malloc(2 + strlen(msg_str) + 10);
        if (reminders[i] = NULL) {
            printf("-- No Space Left --\n");
            break;
        }

        strcpy(reminders[i], day_str);
        strcat(reminders[i], msg_str);

        num_remind++;
    }

    printf("\nDay Reminder\n");
    for (i = 0; i < num_remind; i++) {
        printf(" %s\n", reminders[i]);
    }

    return 0;
}
int read_line(char str[], int n) {
    int ch, i = 0;

    while ((ch = getchar()) != '\n') {
        if (i < n) {
            str[i++] = ch;
        }
    }
    str[i] = '\0';
    return i;
}

Answer 1

    if (reminders[i] = NULL) {

在取消引用之前，您将reminders[i]设置为NULL。

Answer 2

可怕的作业/比较错字！这是一个提示，您的编译器警告级别设置得不够高，或者您忽略了警告。

// Should be '==' not '='
if (reminders[i] = NULL) {
    printf("-- No Space Left --\n");
    break;
}

此外，您的read_line功能类似于fgets，这可能会简化操作。

最后，始终验证您的用户输入。确保scanf返回您要求的项目数（对于简单的单项输入，通常为1）。否则，您可能冒险进入未定义的行为。

Answer 3

你在if子句中做作业。它应该是这样的：

if (reminders[i] == NULL) {
    printf("-- No Space Left --\n");
    break;
}

Answer 4

错误：

if (reminders[i] = NULL) {

右：

if (reminders[i] == NULL) {

另外，你正在联合day_str＆amp; msg_str，所以最好分配：

reminders[i] = malloc(2 + strlen(msg_str) + strlen(day_str));

使用char *和malloc数组的C程序中的分段错误

4 个答案: