int Test(){       BYTE *ba = (byte *)malloc(iLen);       if(...){
      ...
      free(ba);
      return -1;
   }       if(...){
      ...
      free(ba);
      return -2;
   }       if(...){
      ...
      free(ba);
      return -3;
   }       free(ba);
   return 0;
}

引言: 假設在一函式中宣告了一塊記憶體, 接著在運算過程中某些狀況就終止return, 則在我 return 前應free, 這樣不是我有幾個return 就要有幾個 free, 請問有沒好一點的寫法

 
int Test(){       BYTE *ba = (byte *)malloc(iLen);       if(...){
      ...
      free(ba);
      return -1;
   }       if(...){
      ...
      free(ba);
      return -2;
   }       if(...){
      ...
      free(ba);
      return -3;
   }       free(ba);
   return 0;
}

函式中的 if 有時有 8, 9個 要是用 if else 那就很多層... 想不到好一點的寫法 請提供意見囉 謝謝

uj 你好: 試試看這樣子寫，不妨參考看看 < class="code"> int Test() { int iR=0; BYTE *ba=NULL; ba=(byte *)malloc(iLen); if(...) { ... iR=-1; } if(...) { ... iR=-2; } if(...) { ... iR=-3; } if(ba!=NULL) free(ba); return iR; } -- Enjoy Researching & Developing --

int Test()
{
   int iR=0;
   BYTE *ba=NULL;
   ba=(byte *)malloc(iLen);
   if(...)
   {
      ...
      goto FinalFun;
      iR=-1;
   }       if(...)
   {
      ...
      goto FinalFun;
      iR=-2;
   }       if(...)
   {
      ...
      goto FinalFun;
      iR=-3;
   }    FinalFun:
   if(ba!=NULL) free(ba);
   return iR;
}     

引言: 有一陣子沒來了, 記的您是包子大大吧, 謝謝您的回覆 可是我的情況是依序而下 假如有其中有一個if成立 就是直接 return error 不要再繼續此 function 了 說起來自己覺得是用 if else 好像比較合邏輯 但有好幾個判斷式, 這樣用又很醜 直接 return 也不錯, 但卡在有 memory 要 free 一個 malloc 要在每個return 之前都要free 總覺得這樣寫...不太順 還有用 goto 的話, 好像還不錯 但記得...應避免使用 goto

int Test()
{
   int iR=0;
   BYTE *ba=NULL;
   ba=(byte *)malloc(iLen);
   if(...)
   {
      ...
      goto FinalFun;
      iR=-1;
   }       if(...)
   {
      ...
      goto FinalFun;
      iR=-2;
   }       if(...)
   {
      ...
      goto FinalFun;
      iR=-3;
   }    FinalFun:
   if(ba!=NULL) free(ba);
   return iR;
}     

請指教 謝謝

uj 你好:
(1)如果照你所說的，只要前面那個條件一成立，後面的條件就都忽略的話，那
   麼也可以改成這樣子寫的
int Test()
{
   int iR=0;
   BYTE *ba=NULL;
   ba=(byte *)malloc(iLen);
   if(...)
   {
      ...
      iR=-1;
   }
   else if(...)
   {
      ...
      iR=-2;
   }
   else if(...)
   {
      ...
      iR=-3;
   }
   if(ba!=NULL) free(ba);
   return iR;
}
(2)其實用 goto 也是可以的，小弟我也常常使用
(3)你會發現一件事情，從一開始到現在我們所討論主要的目的，就在於每一個
   function 到底應該有很多出口，還是只保持一個出口？這就見仁見智了，有
   的人說，很多個出口 (你第一次的寫法)，效率比較好；也有的人說，保持一
   個出口 (後來的寫法)，比較容易偵錯，不容易出錯。你認為呢？
(>    < src="http://delphi.ktop.com.tw/loadfile.php?TOPICID=10003806&CC=223734"> -- Enjoy Researching & Developing --
        

 
int Test(BYTE *ba){
   if(...){
      ...
      return -1;
   }
   if(...){
      ...
      return -2;
   }
   if(...){
      ...
      return -3;
   }
   return 0;
}    BYTE *ba = (byte *)malloc(iLen);
if (ba)
{
  Test(ba);
  free(ba);
}

template 
struct FreeArrayedRes {
    const _Tp *m_pArrayedRes;       FreeArrayedRes (_Tp * ARes)
      { m_pArrayedRes = ARes; }
   virtual ~FreeArrayedRes ()
      { delete[] m_pArrayedRes; }
};    void funcA()
{
 char *p = new [MAX_PATH];
 FreeArrayedRes auto_free(p);     if(...){
      ...
      // free(ba); 不需要了
      return -1;
   }     if(...){
      ...
      // free(ba);
      return -2;
   }
  ... 
}