這是否使用C指針可以防止泄漏內存？

Question

char *a() { 
    char *t = malloc(8); 
    t[0] = 'a'; 
    t[1] = 'b'; 
    //... 
    t[7] = 'h'; 
    return t; 
} 

int main(void) { 
    char *x = a(); 
    //do something with x 
    //... 
    free(x); 
    return 0; 
} 

此代碼是否有任何潛在的問題，因爲內存分配在a()和已用內存main()？

Answer 1

不，您明確不會有任何問題，因爲您分配了a中的功能並在main中使用它。

Answer 2

沒有問題—其實，這是分配函數返回—但你可能要free()內存當你用它完成後，將仍然使用的內存的正確方法。八個字節不會成爲問題，但針對內存泄漏進行編碼是一個很好的習慣。

Answer 3

只要你記得釋放x指向的內存，那麼你將不會有任何問題。通過使用malloc（），您可以在堆回到調用函數時獨立分配內存。

Answer 4

首先，a（）被聲明爲返回void，但您試圖返回一個char *。更改簽名以返回char *。

其次，你的函數很好，但是你的示例代碼有內存泄漏，因爲你永遠不會釋放返回的指針指向的內存。

第三，正如gbrandt指出的那樣，在調用malloc之後，您沒有檢查成功。 malloc可能會失敗，並檢查它是否成爲一個好習慣。

---

另一種方式來實現，這將是一個指針傳遞給一個指針變成（）來代替，然後調用者有其傳遞到（）之前創建的指針本身，但無論哪種方式，你仍然需要釋放內存。說實話，在這種情況下，我會用你的方法來解決這個問題。沒有令人信服的理由這樣做，只是想我會提到它。

void a(char **p) 
{ 
    *p = malloc(8); 
    if (*p) 
    { 
     **p[0] = 'a'; 
     **p[1] = 'b'; 
     ... 
     **p[7] = 'h'; 
    } 
} 

int main(void) 
{ 
    char *x; 
    a(&x); 
    //do something with x 
    ..... 
    free(x); 
} 

如果這種替代方法混淆了你，請讓我知道，我會很樂意提供解釋（雖然，此刻，我要回去工作！）

Answer 5

除了有a()（應該是char *而不是void）的錯誤返回類型，代碼沒有任何問題。

只要確保將free()分配給內存即可。

Answer 6

以上所有的好建議。代碼的小問題，最大的問題是...

您不在malloc中檢查成功或函數調用。不要忘記錯誤處理。

Answer 7

這裏的大多數答案表明這不是問題，只記得稍後釋放（）它。這在技術上是正確的，但確實是一個問題。任何時候你在一個範圍內分配內存，並期望它在另一個範圍內釋放，你就會要求內存泄漏。人們不記得釋放記憶。還有一個問題，調用者必須知道你使用了malloc，而不是alloca（）或new（）或其他東西。如果他們調用除匹配的解除分配例程之外的其他任何內容，則結果是不確定的。

總而言之，分配內存並將其傳回給調用者通常是一個錯誤。

最好是期望內存由調用者分配，因爲如果他們分配內存，他們會記住釋放內存並正確執行。

Answer 8

謝謝。我剛編輯了代碼。我想這次應該沒有問題。順便說一下，在C中，當函數完成時，編譯器是否清除堆棧中的所有變量（包括指針012）？