C動態結構數組，沒有錯誤，程序在運行時終止

Question

運行時，程序立即終止於我認為與內存分配相關的問題。 main函數只調用此函數，另一個函數刪除分配的內存

DrinkMachine *create(void){
    FILE *inFile;
    //Pointer to data structure
    DrinkMachine *drinkMachine;
    // Memory allocation
    drinkMachine = calloc(1, sizeof(DrinkMachine));
    if(drinkMachine == NULL)    // Check success
        return NULL;

    // Open the input file for reading
    inFile = fopen("drink_machine.txt" , "r");
    if(!inFile){
        puts("Error opening file");
        return NULL;
    }
    // Read drink count from file
    fscanf(inFile, "%d", &(drinkMachine->numItems));
    printf("DEBUG read file arrayLen: %d\n", drinkMachine->numItems);

    // Dynamically create array of drink item structures 

    drinkMachine->drinkItem = malloc(drinkMachine->numItems*sizeof(DrinkItem));
    if(drinkMachine->drinkItem == NULL){
        puts("ERROR: Failed to allocate memory");
        return NULL;
    }

    // Put information from file into drinkItem structs
    for(int i=0; i < drinkMachine->numItems; ++i){
        fscanf(inFile, "%s %lf %d", (drinkMachine->drinkItem[i].name), &(drinkMachine->drinkItem[i].price), &(drinkMachine->drinkItem[i].drinkLeft));
        printf("DEBUG drink %d is: %s   %lf   %d\n", i, (drinkMachine->drinkItem[i].name), (drinkMachine->drinkItem[i].price), (drinkMachine->drinkItem[i].drinkLeft));
    }

    // Close inFile
    fclose(inFile);

    // Force output to screen
    puts("DEBUG readFile Success!");
    fflush(stdout);

    return drinkMachine;
}

該程序遇到錯誤或無法正確分配內存，並在運行時成功輸出錯誤消息，直到我放入行：

drinkMachine->drinkItem = malloc(drinkMachine->numItems*sizeof(DrinkItem));

此時程序編譯時沒有警告或錯誤，但在沒有任何輸出的情況下立即終止。 如果它有幫助，這里是結構：

typedef struct _DrinkItem{
    int id;
    char *name;
    double price;
    int drinkLeft;
    int drinkSold;
} DrinkItem;

typedef struct _DrinkMachine{
    int version;
    int numItems;
    int  drinkLocation;
    DrinkItem *drinkItem;
} DrinkMachine;

Answer 1

您還必須為每個名稱分配存儲空間。 您正在將字符讀入第30行的未分配指針。您應該將名稱讀入臨時數組，獲取名稱的長度，分配（長度+ 1）字節的存儲空間以命名，並將數據掃描。

Answer 2

你沒有分配名稱空間，你沒有處理任何輸入錯誤，你使用int而不驗證它對於一個大小不是負的，你沒有使用stderr的錯誤，你使用了保留標識符等等。

這里有一個代碼的命題，我認為我會修復你所有的錯誤（沒有測試它，因為你沒有提供輸入和輸出的例子）：

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

typedef struct DrinkItem {
  int id;
  char *name;
  double price;
  int drinkLeft;
  int drinkSold;
} DrinkItem;

typedef struct DrinkItem {
  int version;
  size_t numItems;
  int drinkLocation;
  DrinkItem *drinkItems;
} DrinkMachine;

static void helper_free(DrinkItem *drinkItems, size_t n) {
  for (size_t i = 0; i < n; i++) {
    free(drinkItems[i].name);
  }
  free(drinkItems);
}

DrinkMachine *create(char const *path) {
  FILE *inFile = fopen(path, "r");
  if (!inFile) {
    fprintf(stderr, "Error opening file");
    return NULL;
  }

  size_t numItems;
  if (fscanf(inFile, "%zu", &numItems) != 1) {
    fprintf(stderr, "Error parsing\n");
    return NULL;
  }
#ifndef NDEBUG
  printf("DEBUG read file arrayLen: %zu\n", numItems);
#endif

  DrinkItem *drinkItems = malloc(numItems * sizeof *drinkItems);
  if (!drinkItems) {
    fprintf(stderr, "ERROR: Failed to allocate memory");
    return NULL;
  }
  for (size_t i = 0; i < numItems; ++i) {
    char *name = malloc(100);
    if (!name) {
      helper_free(drinkItems, i);
      fprintf(stderr, "ERROR: Failed to allocate memory");
      return NULL;
    }
    double price;
    int drinkLeft;
    if (fscanf(inFile, "%99s %lf %d", name, &price, &drinkLeft) != 3) {
      free(name);
      helper_free(drinkItems, i);
      fprintf(stderr, "Error parsing\n");
      return NULL;
    }
    drinkItems[i] =
        (DrinkItem){.name = name, .price = price, .drinkLeft = drinkLeft};

#ifndef NDEBUG
    printf("DEBUG drink %zu is: %s   %lf   %d\n", i, name, price, drinkLeft);
#endif
  }

  fclose(inFile);

  DrinkMachine *drinkMachine = malloc(sizeof *drinkMachine);
  if (!drinkMachine) {
    helper_free(drinkItems, numItems);
    fprintf(stderr, "ERROR: Failed to allocate memory");
    return NULL;
  }
  *drinkMachine =
      (DrinkMachine){.drinkItems = drinkItems, .numItems = numItems};

#ifndef NDEBUG
  puts("DEBUG readFile Success!\n");
#endif

  return drinkMachine;
}