구조체와 메모리 할당 루프

Question

나는 강사가 (현재 설정으로) 3 개의 malloc'd 포인터를 만들었다 고 말했지만 결국 메모리를 free()하려고 시도했다.

참고 : 실제로 메모리에서 진행되고있는 malloc에 ​​관해 자세한 설명이 필요합니다.

내가 메모리 누출이 없는지 확인하기 위해 할 수있는 것에 대한 지침을 주시면 감사하겠습니다.

본인은 다음을 작성했습니다.

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

    #define LUNCH_ITEMS 5 
    #define REMAINING 2 
    #define CHAR_SIZE 256 

    int main(void) 
    { 
     struct Food 
     { 
      char *name; //name attribute of food 
      int weight, calories; 
     } lunch[LUNCH_ITEMS] = {{"apple", 4, 100}, {"salad", 2, 80},}; 
     int loopCount; 
     //INPUT 
     char *namePtr = NULL; 
     for (loopCount = REMAINING; loopCount < LUNCH_ITEMS; ++loopCount) 
     { 
      char tempBuffer[CHAR_SIZE]; 
      printf("Enter name of item,the weight of item, and the calories in that item: \n"); 
      // store name string in a tempBuffer. weight and calories directly into lunch structure array address 
      scanf("%255s %d %d", tempBuffer, &lunch[loopCount].weight, &lunch[loopCount].calories); 
      // get the exact size of (memory of) the input string including add one for null char 
      size_t exactMemory = strlen(tempBuffer) + 1; 
      //dynamically allocate the exact amount of memory determined in the previous step 
      namePtr = (char *)malloc(exactMemory * sizeof(char)); 
      // check if no memory is allocated for foodPtr 
    if (namePtr == NULL) 
    { 
     fprintf(stderr, "Not enough memory available***\n Terminating Program"); 
     exit(1); 
    } 
    //store the pointer to it in the name member of the structure in 
    //the current lunch array element. 
    (lunch + loopCount)->name = namePtr; 
    // Copy the food name (stored in tempbuffer) into the dynamically-allocated 
    // memory using the memcpy function 

      memcpy(namePtr, tempBuffer, exactMemory); 
    //(lunch + loopCount)->name = namePtr; 
     } 
    //DISPLAY 
     printf("Item      Weight  Cals\n---------------------------------------------\n"); 
     for (loopCount = 0; loopCount < LUNCH_ITEMS; loopCount++) 
     { 
      int weight = lunch[loopCount].weight; 
      int cals = lunch[loopCount].calories; 
      printf("%-12.20s%22d%11d\n", (lunch + loopCount)->name, weight, cals); 
      if (loopCount > REMAINING) 
      { 
       //(lunch+loopCount)->name = NULL; 
       namePtr = NULL; 
       free(namePtr); 
       //free((lunch + loopCount)->name); 
      } 
     } 
     //De-allocate all malloc'd memory 
     return EXIT_SUCCESS; 
    } 

내 출력 -

Item Weight Cals 
----------------- 
apple 4 100 
salad 2 80 
hello 22 33 
maybe 44 45 
right 100 200 

Answer 1

봐 : 즉

for (loopCount = REMAINING; loopCount < LUNCH_ITEMS; ++loopCount) 
{ 
    // The code inside the loop will be executed for 
    // loopCount being equal to 
    //  REMAINING 
    //  REMAINING + 1 
    //  .... 
    //  LUNCH_ITEMS - 1 
    // 
    // So in your case, you execute this code for 
    // loopCount equal to 2, 3 and 4 
} 

는 루프 내부의 코드는 3 배, I, E, 당신이 3 회의 malloc에게 전화를 실행한다.

마찬가지로 메모리를 해제 한 루프를 살펴보십시오.

for (loopCount = 0; loopCount < LUNCH_ITEMS; loopCount++) 
{ 
    // you execute this code for 
    // loopCount equal to 0, 1, 2, 3 and 4 

    // .... 

    if (loopCount > REMAINING) 
    { 
     // Since REMAINING is 2, you execute this code for 
     // loopCount equal 3 and 4 
    } 
} 

는 그래서 if의 몸에있는 코드는 2 번 실행됩니다. 번 (즉, loopCount가 2, 3 및 4 인 경우).

if (loopCount >= REMAINING) // Notice the = sign 
    { 
     // Since REMAINING is 2, you execute this code for 
     // loopCount equal 2, 3 and 4 
    } 

이제 malloc 및 free에 관한 : 그래서 당신은에 코드를 변경해야합니다. 당신이 free를 사용하여 즉 메모리를 해제 할 때, 정확히 전달해야 값이 같은 포인터를 저장 초기화에서의 malloc

귀하에게 반환

namePtr = (char *)malloc(exactMemory * sizeof(char)); 
// ... 
(lunch + loopCount)->name = namePtr; // Save pointer 

그래서한다 (lunch + loopCount)->name입니다 free에 사용됩니다. 마찬가지로 :

if (loopCount >= REMAINING) // Notice the = sign 
    { 
     free((lunch + loopCount)->name); 

     // Optional part but always after calling free 
     (lunch + loopCount)->name = NULL; 
    } 

은 BTW :

(lunch + loopCount)->name 

이

많은 사람들이 쉽게 읽을 고려

lunch[loopCount].name 

과 동일 함을 알 수 있습니다.

Answer 2

내가 강사의 코멘트에 대한 3 malloc 된 문자열을 생각은 꽤 강력한 단서입니다. 나는 당신이 2 항목 미리 채워진 5 품목의 배열을 가지고 있다는 것을 알아 차렸다. 5 - 2는 3입니다.

또한 C의 색인은 0부터 시작합니다. 배열을 사전 초기화하는 두 항목에는 색인 0과 색인 1이 있습니다. 색인 2는 처음 입력 한 데이터가됩니다. 해당 인덱스를> 2를 사용하여 비교하면 실제로 사용자가 제공 한 첫 번째 데이터의 인덱스를 건너 뛸 수 있습니다. 초기화 루프에서