从提供地址而不是值的文件中读取

时间:2016-03-21 14:11:53

标签: c file-io

所以我正在从逗号分隔的文件中读取数据。由于某种原因,代码不会遍历整个文件,而只打印出一个特定的componentType。

同样对于numOfItems和price,我只是打印出地址而不是值!

这是我的代码:

Private Sub Worksheet_Change(ByVal Target As Range)
    If Target.Cells.Count = "" Then Exit Sub
    If Not Application.Intersect(Range("c5"), Target) Is Nothing Then
        If IsNumeric(Target.Value) And Target.Value = 1 Then
            Call Email
        End If
    End If
End Sub 

CSV示例

typedef struct inventory {
    char *componentType;
    char *stockCode;
    int numOfItems;
    int price;
} inventory;


int main(int argc, char** argv)
{

    char str[255];
    FILE *invf= fopen("inventory.txt", "r");
    // creating an array of structs
    struct inventory inv[384];
    // counter for array position
    int counter = 0;
    while (fgets(str, 256, invf) != NULL){
       char *componentType = strtok(str, " ,");
       // the NULL means it will pick up where it left off
       char *stockCode = strtok(NULL, " ,");
       int *numOfItems = strtok(NULL, " ,");
       int *price = strtok(NULL, " ,");

       // adding to the struct
       inv[counter].componentType = componentType;
       inv[counter].stockCode = stockCode;
       inv[counter].numOfItems = &numOfItems;
       inv[counter].price = &price;
       counter++;
    }
    int i = 0;
    for(i =0; i <300; i++){
        printf("%s %s %d %d \n", inv[i].componentType, inv[i].stockCode, inv[i].numOfItems, inv[i].price);
    }

    return EXIT_SUCCESS;
}

1 个答案:

答案 0 :(得分:0)

你的componenType是指向角色的指针。每次读取一行时,都将指针指向componentType的行。最后,所有库存项目都指向同一位置,这是最后读取的值。您必须使用malloc为其分配存储空间,然后复制该值。

以同样的方式将指针的整数元素设置为(字符串)变量。但是你必须得到。使用atoi()

进一步注意char str[255];可以包含255个字符,但是要求fgets最多读取256个字符。你忘了分配终止空字符。

不要忘记检查counter < 384

<小时/> 编辑(补充指导)

您正在构建包含库存物品的数组。您当前可以读取不超过384项,即数组的大小。如果项目数可以是“任意数量”,那么您需要一个更动态的数据结构。现在数组就足够了,但是在读取库存文件时必须检查是否超出了界限。

每次阅读lne时,都会将该行从左到右分成信息项。对于string类型的项目,您必须分配内存来保存字符串数据:

   inv[counter].componentType = malloc(strlen(componentType)+1);
   strcpy(inv[counter].componentType, componentType);

在读取数字信息时,它仍然是一个字符串,您必须将其转换为整数:

   int numOfItems = atoi(strtok(NULL, " ,"));

或:

  inv[counter].numOfItems = atoi(strtok(NULL, " ,"));

以这种方式构建阵列后,您可以在程序中使用它。要更改组件类型,例如:

   free(inv[counter].componentType );                               // release old memory
   inv[counter].componentType = malloc(strlen(newComponentType)+1); // get new
   strcpy(inv[counter].componentType, newComponentType);

完成后,您应该通过为每个项目调用malloc来释放分配了free的所有内存。