为结构数组元素（字符串）分配内存

Question

我正在尝试从键盘读取并将信息存储在我的结构书中。要求用户输入数组大小，然后我将大小动态分配给我的 book 数组。但是当 n > 1 时，运行时错误是 exc_bad_access。我不知道为什么它在 n = 1 时起作用，而在 n > 1 时不起作用。

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

struct book
{
    char *title;
    char *author;
    double price;
};

int main()
{
    int n, i = 0;
    printf("please enter the value for n\n");
    scanf("%d", &n);

    struct book *stack = malloc(sizeof(struct book) * n);
    stack->author = malloc(sizeof(char) * 100);
    stack->title = malloc(sizeof(char) * 100);
    //allocate memory for book and struct members
    while (i < n )
    {
        printf("Please enter the title\n");
        scanf("%s", stack[i].title);
        printf("Please enter the author\n");
        scanf("%s", stack[i].author);
        printf("Please enter the price\n");
        scanf("%lf", &stack[i].price);
        i ++;
    }
    free(stack);
    return 0;
}

Answer 1

您为输入的任何大小的元素分配了足够的结构，但没有为缓冲区分配空间：

struct book *stack = malloc(sizeof(struct book) * n);
stack->author = malloc(sizeof(char) * 100);
stack->title = malloc(sizeof(char) * 100);
// what about stack[1].author, stack[2].author and the rest??

这将起作用：

struct book *stack = malloc(sizeof(struct book) * n);
for (int i = 0; i < n; ++i) {
    stack[i].author = malloc(sizeof(char) * 100);
    stack[i].title = malloc(sizeof(char) * 100);
}

记住也要释放你的记忆。

Answer 2

对于堆栈中的每一本书，您必须为作者和标题分配内存。

for(int i = 0; i < n; i++)
{
    stack[i].author = malloc(sizeof(char) * 100);
    stack[i].title = malloc(sizeof(char) * 100);
}

您只是为第一个元素分配内存。

Answer 3

你需要为

分配内存

char *title;
char *author;  

当您执行struct book *stack = malloc(sizeof(struct book) * n); 时，内存分配给char* 指针（4 或8 个字节，具体取决于您正在构建的平台）。

所以你需要为内部值分配一些内存

while (i < n )
{
    char *title = malloc(255 * sizeof(char));
    printf("Please enter the title\n");
    scanf("%s", title);
    stack[i].title = title;
    .....
    i ++;
}