如何使用指针使用 malloc 存储多个字符串

Question

我试图制作一个程序来动态分配内存以使用指针存储多个字符串，但它似乎无法正常工作。

#include <stdio.h>
#include<conio.h>

void main()
{
    FILE *fp;
    int num;

    printf("enter no of students");
    scanf("%d",&num);
    char *names = (char*) malloc(num * 100 * sizeof(char));

    printf("enter the names");
    fp = fopen("file.txt","w");

    for (int i = 0; i < num; i++)
    {
        fgets(*(names + i * 100), 100, stdin);
        fprintf(fp, "%s\n", (names + i * 100));
    }

    fclose(fp);

    fp = fopen("file.txt", "r");
    printf("names stored in *ptr");
}

Answer 1

你有几个问题。首先scanf 不能很好地与fgets 配合使用——它会在输入流中留下一个换行符，这意味着当你调用fgets 时它只会读取它。解决这个问题的最简单方法是在询问学生人数时使用fgets 读取整行，并使用sscanf 获取实际人数，如下所示：

char temp[100];
fgets(temp,100,stdin);
sscanf(temp,"%d",&num);

其次，您只创建了一个非常大的长字符串，而不是多个字符串。因此，要创建num 字符串，您首先要分配这样的字符串“数组”

names=malloc(sizeof(char *)*num);

然后使用循环为字符串分配空间

for(int i=0; i<num; i++)
{
     names[i]=malloc(100);
} 

您还会看到不需要从malloc 转换返回值，但您确实需要包含正确的文件，即“stdlib.h”。从技术上讲，您不需要sizeof(char)，因为它始终是 1。

你不检查来自fopen的返回值，所以你不处理它无法打开文件进行写入的情况。

不知道为什么你最后还要打开文件进行阅读，因为你没有对它做任何事情，它只会在程序结束时自动关闭。

而且您应该始终养成编写代码以释放分配的内存的习惯，即使您不需要这样做，因为代码就是这么简单。

从技术上讲，main 应该返回 int。

代码的最终版本可能如下所示：

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

int main()
{
    FILE *fp;
    int num;
    char temp[100];

    printf("enter no of students");
    fgets(temp,100,stdin);
    sscanf(temp,"%d",&num);
    char **names=malloc(num*sizeof(char*));


    printf("enter the names");
    fp=fopen("file.txt","w");
    if(fp==NULL)
    {
        printf("Failed to open\n");
        return 1;
    }
    for(int i=0;i<num;i++)
    {
        names[i]=malloc(100);
        fgets(names[i],100,stdin);
        fprintf(fp,"%s\n",names[i]);
    }
    fclose(fp);

    for(int i=0;i<num;i++)
    {
        free(names[i]);
    }
    free(names);

    printf("names stored in *ptr");
}

Answer 2

您需要有一个指向字符的指针表。

char **strtable = NULL
size_t nstrings = 0; 

int addstring(const char *str)
{
    int result = -1;
    char **tmp = realloc(strtable, (nstrings + 1)* sizeof(*strtable));

    if(tmp)
    {
        tmp[nstrings] = malloc(strlen(str) + 1);
        if(tmp[nstrings])
        {
            result = 0;
            strtable = tmp;
            strcpy(tmp[nstrings++], str);
        }
    }
    return result;
}

使用它，您可以添加输入（读取）的字符串并轻松迭代。

Answer 3

最简单的方法是将names 声明为“指向100 个字符的数组的指针”并仅使用一个malloc。之后，您可以使用names[i] 访问个人姓名。

像这样：

char (*names)[100] = malloc(num * sizeof *names);

printf("enter the names\n");
fp = fopen("file.txt","w");

for (int i = 0; i < num; i++)
{
    fgets(names[i], 100, stdin);
    fprintf(fp, "%s", names[i]);
}

fclose(fp);

一些额外的cmets：

1) sizeof(char) 始终为 1

2) 请注意scanf("%d",&num); 将在输入缓冲区中留下一个换行符。这会导致您的名字为空。您需要在扫描名称之前删除该换行符

3) 另请注意，fgets 将包含换行符（至少在输入少于 99 个字符时）。

4) 似乎您并不需要在将所有名称立即写入文件时将它们存储在数组中。所以你可以简单地这样做：

printf("enter the names\n");
fp = fopen("file.txt","w");

for (int i = 0; i < num; i++)
{
    char name[100];
    fgets(name, 100, stdin);
    fprintf(fp, "%s", name);
}

fclose(fp);

Answer 4

我同意上面的答案，但如果你想保持相同的结构，你可以去

    #include<stdio.h>
    #include<conio.h>
    void main()
    {
    FILE *fp;
    int num;

    printf("enter no of students ");
    scanf("%d",&num);
    char *names=malloc(num*100);


    printf("enter the names\n");
    fp=fopen("file.txt","w");
                fseek(stdin,0,SEEK_END);

    for(int i=0;i<num;i++)
    {

    fgets((names + i * 100), 100, stdin);
    fprintf(fp, "%s", (names + i * 100));
        fseek(stdin,0,SEEK_END);
    }
    fclose(fp);

    fp=fopen("file.txt","r");
    printf("names stored in *ptr");
    fclose(fp);
    }