例外関数内で配列をreallocをしようとしている

Question

は、以下のstruct

struct HotelManagement 
{ 
    Hotel_t *hotel; 
    Customer_t *customers; 
    reservation_t *reservations; 
    int physicalSize; 
    int registerdSize; 

    int physicalSizeReserv; 
    int registerdSizeReserv; 

} typedef HotelManagement_t; 

そして次mainを付けながら：

printf("-----DETAILS FOR CUSTOMER-----\n"); 

printf("enter name "); 
gets(tempName);//"cleans" the buffer 
gets(tempName); 
customerName = strdup(tempName); 

printf("nenter credit card "); 
gets(tempCredit); 
creditCardNumber = strdup(tempCredit); 

printf("enter credit card expiration month(mm)\n"); 
printf("enter credit card expiration year(yyyy)\n"); 

scanf("%d",&month); 
scanf("%d",&year); 

addCustomer(customerName,creditCardNumber, month,year,&hm); 

とaddCustomerの次の実装：

void addCustomer(char *customerName, char *numberOfCreditCard,int month,int year,HotelManagement_t *hotelMang) 
{ 
    int *m = &month; 
    int *y = &year; 
    int i; 

    if (hotelMang->physicalSize == hotelMang->registerdSize) 
    { 
     hotelMang->customers = (Customer_t*)realloc(hotelMang->customers, hotelMang->registerdSize * sizeof(Customer_t)); 
    } 
    hotelMang->customers[hotelMang->registerdSize].id = hotelMang->registerdSize+1;//id starts with '1' 
    hotelMang->customers[hotelMang->registerdSize].cName = (char*)malloc(strlen(customerName)*sizeof(char));//initalize space for the Customers name 
    strcpy(hotelMang->customers[hotelMang->registerdSize].cName ,customerName); 

    if (checkValidCreditCard(&month,&year,numberOfCreditCard) == 1) 
    { 
     hotelMang->customers[hotelMang->registerdSize].credit = (char*)malloc(strlen(numberOfCreditCard)*sizeof(char));//initalize size 
     strcpy(hotelMang->customers[hotelMang->registerdSize].credit, numberOfCreditCard); 
     hotelMang->customers[hotelMang->registerdSize].month = month; 
     hotelMang->customers[hotelMang->registerdSize].year = year; 
    } 
    //increments 
    for (i = 0; i <= hotelMang->registerdSize; i++) 
    { 
     printOneCustomer(hotelMang->customers[i]); 
    } 
    hotelMang->registerdSize= hotelMang->registerdSize +1; 
} 

theresの配列を初期化する関数...

hotelManagement->customers = (Customer_t*)calloc(1, sizeof(Customer_t)); 
    hotelManagement->hotel->roomsMat = (Room_t**)calloc(floors, sizeof(Room_t*)); 
    hotelManagement->reservations = (reservation_t*)calloc(1, sizeof(reservation_t)); 

私は顧客にスペースを再割り当てしたいのですが、私は顧客に配列を追加していますが、例外がスローされます。その理由は何でしょうか？

Answer 1

あなたは非常に初めてaddCustomerを呼び出すと、hotelManagement->registerdSizeの値はゼロです。サイズゼロのreallocをと定義した場合、何が起きるかはと定義されています。 realloc関数が元のポインタを返すことがあります。それはかもしれないを返しますnullポインタを返します。それも無料メモリがあります。

あなたは Customer_t構造の「アレイ」を指すようにhotelMang->customersを初期化することを考えると、あなたはおそらく1にhotelManagement->registerdSizeを初期化する必要があります。再割り当てする場合は(hotelManagement->registerdSize + 1)を使用してください。

さらに、エラーをチェックすることはありません。callocとreallocの両方がnullポインタを返す可能性があることに注意してください。 reallocへの呼び出しでは、元のポインタを失うため、メモリリークが発生します。 reallocの結果を得るには、常に一時変数を使用してください。