【问题标题】:C - Loading data from a file in an arrayC - 从数组中的文件加载数据
【发布时间】:2014-04-15 21:56:45
【问题描述】:

好的,所以我在 C 中工作,到目前为止,stackoverflow 一直在帮助我很多,但是我仍然卡住了!我不知道为什么这行不通!我正在尝试将标记化文件存储在一个数组中,但它似乎在这里不起作用是我正在尝试的:

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


int main(int argc, const char * argv[])
{
    int lines=0, i=0, a=0;
    char strings[50], *err=" ";

    FILE *file;
char *array[100][4];
    file=fopen("inventory.csv", "a+");
    if(file==NULL){
        return 1;}
    err=malloc(100);

        while(a<100 && fgets(strings, sizeof strings, file)!=NULL){



        int i;
        char *p;

        i = 0;
        p = strtok (strings,",");
        while (i<4 && p != NULL)
        {
            array[a][i++] = strdup(p);
            p = strtok (NULL, ",");
        }
            a++;
        }
   lines=a; 

    if(argv[1]!=NULL){
        if(strcmp(argv[1], "list")==0){
            printf("\nName, Quantity, Reorder limit, Cost\n\n");
            for(a=0;a<lines;a++){
                for(i=0;i<4;i++){
                    printf("%s ", array[a][i]);
                }
                printf("\n");
            }
            printf("\n");
        }

    else if(strcmp(argv[1], "reorder")==0){
        int i=0;
        printf("We need to purchase more of the following items: \n");
        for(i=0;i<lines;i++){
            if(strtod(array[i][1], &err)<=strtod(array[i][2],&err)){

                printf("%s\n", array[i][0]);


            }
        }
    }
    else if(strcmp(argv[1], "deduct")==0){
        if(argv[2]){
            int i=0;
            char str[100];

            for(i=0;i<lines;i++){
                if(strcmp(argv[2], array[i][0])==0){
                    if(strtod(array[i][1],&err)-1>0){
                        sprintf(str,"%f",strtod(array[i][1],&err)-1);
                        array[i][1]=str;
                        printf("Success\n");
                        fseek(file, 0, 1);
                        sprintf(str,"%s,%s,%s,%s", array[i][0], array[i][1], array[i][2], array[i][3]);
                        fwrite(str,1,sizeof(str+1),file);
                    }
                    else{
                        printf("There are none left!\n");
                    }
                }
            }
        }
        else{
            printf("Enter a food item");
        }
    }
    else if(strcmp(argv[1], "add")==0){
        fseek(file, 0, 2);
        char str[100];
        sprintf(str,"%s,%s,%s,%s\n",argv[2],argv[3],argv[4],argv[5]);
        fwrite(str,1,sizeof(str),file);

    }
    else{
        printf("Please enter an argument: list, reorder, deduct _name_, add _name_ _qty_ _reorderlimit_ _price_\n");
    }
}
int j;

for(i=0;i<lines;i++){
    for(j=0; j<3; j++){
        free(array[i][j]);
}
}

fclose(file);
return 0;
}

我已经修复了列表并添加了函数,但是,每次运行后我仍然收到核心转储,并且 deduct 不起作用,(说成功,核心转储,但不更新文件)并且 individulaAverage 仍然没有' t 找到输入(名称)为 == 到实验(i);

这是 csv 文件:

hotdog,10,2,1.5
bun,10,2,0.5
burger,100,10,2.0
cheese,100,10,2.25
pickle,10,2,1.0
ham,2,2,1.0
salmon,5,1,5.0
steak,5,1,7.0
pork,10,2,6.0
fries,1000,100,3.0

【问题讨论】:

  • 次要使用点:“存储”通常意味着将数据从内存移动到文件。我们大多数人会说您想将文件中的数据加载到数组中。
  • @keshlam 编辑标题!谢谢!
  • 您不断重复使用strings 中的空间来存储不同的数据(每行依次),但您将指向strings 中不同位置的指针保存在array 中。这会导致您看到的覆盖。在阅读时为每个字符串分配存储空间 (strdup()?)。然后记得释放分配的空间。
  • 您能解释一下 strdup() 的作用以及在这种情况下如何使用它吗? @JonathanLeffler
  • 实现:char *strdup(const char *str) { size_t len = strlen(str) + 1; char *dup = malloc(len); if (dup != 0) memmove(dup, str, len); return dup; } 函数strdup()通过为字符串的副本分配足够的内存来复制字符串(记住字符串末尾有一个零字节)并复制原始字符串如果分配成功,则为新字符串。无论如何,它都会返回重复的指针(null 或有效值)。您可以将 array[a][i++] = p; 替换为 array[a][i++] = strdup(p);

标签: c arrays list file csv


【解决方案1】:

您不断重复使用strings 中的空间来存储不同的数据(每行依次),但您将指向strings 中不同位置的指针保存在array 中。这会导致您看到的覆盖。分配存储空间 (strdup() ? — POSIX 的一部分,但不是标准 C) 中的每个字符串,当您阅读它时。然后记得释放分配的空间。

您能否解释一下 strdup() 的作用以及在这种情况下如何使用它?

实施

char *strdup(const char *str)
{
    size_t len = strlen(str) + 1;
    char *dup = malloc(len);
    if (dup != 0)
        memmove(dup, str, len);
    return dup;
}

评论

函数strdup() 通过为字符串的副本分配足够的内存来复制字符串(记住字符串末尾有一个零字节),如果分配成功,则将原始字符串复制到新字符串。无论如何,它都会返回重复的指针(null 或有效值)。

您可以将 array[a][i++] = p; 替换为 array[a][i++] = strdup(p);

您还需要一个类似于以下内容的循环:

for (int i = 0; i < lines; i++)
{
    for (int j = 0; j < 3; j++)
        free(a[i][j]);
}

(当然,如果你立即退出,循环就变成了可选的,但是确保你知道你执行的每个显式或隐式malloc()在何时何地都有一个free()是一个很好的主意。 )

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2019-12-10
    • 1970-01-01
    • 2014-09-27
    • 1970-01-01
    • 2017-10-30
    相关资源
    最近更新 更多