2016-12-06 103 views
-4

最近,我一直在嘗試瞭解動態編程的基礎知識,現在我面臨着堆腐敗的一些問題。當我嘗試釋放內存時,我的代碼中斷了。 我在互聯網上做了一些研究,我發現我收到這條消息,因爲我在一些未分配的內存空間中寫入。但是我仍然不知道錯誤在哪裏,或者如何解決。正常塊(#164)後檢測到堆損壞

#include <stdio.h> 
#include <malloc.h> 
#include <stdlib.h> 
#include <fstream> 
std::ifstream f("file.in"); 
void read(int *a, int &n) 
{ 
    f >> n;///scanf("%d", &n); 
    int i; 
    for (i = 0; i < n; i++) 
     f >> a[i];/// scanf_s("%d", &a[i]); 
} 
int main() 
{ 
    int *a, *b, *c; 
    int na = 0, nb = 0; 
    int i = 0, j = 0, k = 0, kk = 1, p; 
    a = (int*)malloc(na*sizeof(int)); 
    b = (int*)malloc(nb*sizeof(int)); 
    c = (int*)malloc(kk * sizeof(int)); 
    read(a, na); 
    read(b, nb); 
    while (i < na && j < nb) 
    { 
     if (a[i] < b[j]) 
     { 
      kk++; 
      c = (int*)realloc(c, kk * sizeof(int)); 
      c[k] = a[i]; 
      k++; 
      i++; 
     } 
     else 
     { 
      kk++; 
      c = (int*)realloc(c, kk * sizeof(int)); 
      c[k] = b[j]; 
      k++; 
      j++; 
     } 
    } 
    if (i < na) 
     for (p = i; p < na; p++) 
     { 
      kk++; 
      c = (int*)realloc(c, kk*sizeof(int)); 
      c[k++] = a[p]; 
     } 
    if (j < nb) 
     for (p = j; p < nb; p++) 
     { 
      kk++; 
      c = (int*)realloc(c, kk *sizeof(int)); 
      c[k++] = b[p]; 
     } 

    for (i = 0; i < k; i++) 
     printf("%d ", c[i]); 
    free(a); 
    free(b); 
    free(c); 
} 
+0

請[編輯]你的問題提供了[MCVE。 –

+0

** na **和** nb **爲null(等於0),所以當您執行'malloc(na * sizeof(int))'時,您不會分配內存,對於** b **分配使用* *注**。 –

+0

我是一名學習C的學生,之前學習過一些C++,現在我在編碼時使用它們。對不起,如果它困擾你。 –

回答

1
int na = 0; 
int *a = (int*)malloc(na*sizeof(int)); 
read(a, na); 

這裏的問題。你創建一個大小爲0的緩衝區,然後你寫入它。只有在確定尺寸後才能創建緩衝區,這是在讀取尺寸之後。重新排列代碼,使這條線是有道理的:

int na = 0; 
int *a = read(na); 

或者更好的是:

std::vector<int> a = read();