來分析一個C語言代碼結(jié)構(gòu)的設(shè)計問題。

這段代碼，使用了兩次malloc，分別給 p1 和 p2 申請了內(nèi)存。用完后，內(nèi)存釋放，防止內(nèi)存泄漏。

大家覺得，這樣的代碼設(shè)計有沒有問題。

代碼是某位學(xué)員在項目中遇到的，我拿過來給簡化了一下。

其實這樣的代碼可能會導(dǎo)致內(nèi)存泄漏，如果第一個 malloc 申請成功，第二個 malloc 申請失敗，直接 return -1 退出函數(shù)，就會導(dǎo)致前面申請的內(nèi)存沒有釋放，長期運行，極有可能導(dǎo)致安全隱患。

對于這種連續(xù)初始化的代碼，一旦某個初始化失敗，應(yīng)該要考慮到如何釋放前面已經(jīng)申請的資源。

最簡單的方法，就是在 return 退出之前，把前面的資源清理掉。

不過這種方法，越往下初始化越多，退出的時候需要釋放的也就越多，重復(fù)的代碼也會很多。

或者使用 do while 語句，把代碼放在循環(huán)里面，初始化失敗后，直接跳出循環(huán)，不需要再進行下面的初始化，然后統(tǒng)一在某個地方釋放資源。

類似的還有 goto 語句，也是把釋放資源統(tǒng)一處理。

大家還有沒有其他更好的辦法，如果你也遇到過，歡迎在評論區(qū)交流。