• <ins id="pjuwb"></ins>
    <blockquote id="pjuwb"><pre id="pjuwb"></pre></blockquote>
    <noscript id="pjuwb"></noscript>
          <sup id="pjuwb"><pre id="pjuwb"></pre></sup>
            <dd id="pjuwb"></dd>
            <abbr id="pjuwb"></abbr>
            隨筆-90  評(píng)論-947  文章-0  trackbacks-0

            灰常感謝各位達(dá)人昨天的熱心回帖,讓我受益匪淺。我仰望夜空,群星點(diǎn)點(diǎn),就如各位的點(diǎn)睛之語(yǔ),在無(wú)盡的蒼穹閃耀。這讓我深深地意識(shí)到,在這里,不僅可以分享成果,也可以分享困惑、分享寂寞。(開(kāi)場(chǎng)白到此結(jié)束~)

            在平常的編程中,我發(fā)現(xiàn)很容易遇到這種結(jié)構(gòu):

            (1號(hào)方案)

            BOOL foo()
            {
               
            BOOL bRet = FALSE;

               
            HANDLE hProcess = OpenProcess(...);

               
            if (hProcess != NULL)
                {
                   
            HANDLE hToken = OpenProcessToken(hProcess, ...);

                   
            if (hToken != NULL)
                    {
                       
            // ...

                       
            if (LookupPrivilegeValue(...))
                        {
                           
            if (AdjustTokenPrivileges(hToken, ...))
                            {
                               
            bRet = TRUE;
                            }
                        }

                       
            CloseHandle(hToken);
                    }

                   
            CloseHandle(hProcess);
                }

               
            return bRet;
            }

            如上寫法,容易造成縮進(jìn)級(jí)別不斷增加。為了避免這種情況,可以改成:

            (2號(hào)方案)

            BOOL foo()
            {
               
            HANDLE hProcess = OpenProcess(...);

               
            if (hProcess == NULL)
                {
                   
            return FALSE;
                }

               
            HANDLE hToken = OpenProcessToken(hProcess, ...);

               
            if (hToken == NULL)
                {
                   
            CloseHandle(hProcess);

                   
            return FALSE;
                }

               
            // ...

               
            if (!LookupPrivilegeValue(...))
                {
                   
            CloseHandle(hToken);
                   
            CloseHandle(hProcess);

                   
            return FALSE;
                }

               
            if (!AdjustTokenPrivileges(hToken, ...))
                {
                   
            CloseHandle(hToken);
                   
            CloseHandle(hProcess);

                   
            return FALSE;
                }

               
            CloseHandle(hToken);
               
            CloseHandle(hProcess);

               
            return TRUE;
            }

            這樣,又引來(lái)了新的問(wèn)題,每次 return FALSE 時(shí)的清理任務(wù)比較麻煩,要是每步操作都引進(jìn)新的 HANDLE 的話,后續(xù)的清理工作就變得非常繁重。有人推薦do…while(0)的結(jié)構(gòu),有人推薦goto。這兩種形式分別是——

            do…while(0)

            (3號(hào)方案)

            BOOL foo()
            {
               
            HANDLE hProcess = OpenProcess(...);

               
            if (hProcess == NULL)
                {
                   
            return FALSE;
                }

               
            BOOL bRet = FALSE;

               
            do
               
            {
                   
            HANDLE hToken = OpenProcessToken(hProcess, ...);

                   
            if (hToken == NULL)
                    {
                       
            break;
                    }

                   
            // ...

                   
            BOOL bRetInner = FALSE;

                   
            do
                   
            {
                       
            if (!LookupPrivilegeValue(...))
                        {
                           
            break;
                        }

                       
            if (!AdjustTokenPrivileges(hToken, ...))
                        {
                           
            break;
                        }

                       
            bRetInner = TRUE;

                    }
            while (0);

                   
            CloseHandle(hToken);

                   
            if (!bRetInner)
                    {
                       
            break;
                    }

                   
            bRet = TRUE;

                }
            while (0);

               
            CloseHandle(hProcess);

               
            return bRet;
            }

            這種結(jié)構(gòu)可以避免每次 return FALSE 前的一堆清理工作,但缺點(diǎn)是,有幾個(gè)依賴性的 HANDLE,就要嵌套幾層的 do…while(0),有時(shí)候也會(huì)遇到需要三四層嵌套的情形。

            goto

            (4.1號(hào)方案)

            BOOL foo
            ()
            {
               
            BOOL bRet = FALSE;

               
            HANDLE hProcess = OpenProcess(...);

               
            if (hProcess == NULL)
                {
                   
            goto CLEAR;
                }

               
            HANDLE hToken = OpenProcessToken(hProcess, ...);

               
            if (hToken == NULL)
                {
                   
            goto CLEAR;
                }

               
            // ...

               
            if (!LookupPrivilegeValue(...))
                {
                   
            goto CLEAR;
                }

               
            if (!AdjustTokenPrivileges(hToken, ...))
                {
                   
            goto CLEAR;
                }

               
            bRet = TRUE;

            CLEAR:
               
            if (hToken != NULL)
                {
                   
            CloseHandle(hToken);
                }

               
            if (hProcess != NULL)
                {
                   
            CloseHandle(hProcess);
                }

               
            return bRet;
            }
            (4.2號(hào)方案)

            BOOL foo
            ()
            {
               
            BOOL bRet = FALSE;

               
            HANDLE hProcess = OpenProcess(...);

               
            if (hProcess == NULL)
                {
                   
            goto ERROR_LEVEL0;
                }

               
            HANDLE hToken = OpenProcessToken(hProcess, ...);

               
            if (hToken == NULL)
                {
                   
            goto ERROR_LEVEL1;
                }

               
            // ...

               
            if (!LookupPrivilegeValue(...))
                {
                   
            goto ERROR_LEVEL2;
                }

               
            if (!AdjustTokenPrivileges(hToken, ...))
                {
                   
            goto ERROR_LEVEL2;
                }

               
            bRet = TRUE;

            ERROR_LEVEL2:
               
            CloseHandle(hToken);
            ERROR_LEVEL1:
               
            CloseHandle(hProcess);
            ERROR_LEVEL0:
               
            return bRet;
            }

            (左邊和右邊哪種好一點(diǎn)。。。?)

            在這種情形下,goto 的方案似乎是完美的。但是 goto 如果遇到 C++,缺點(diǎn)體現(xiàn)出來(lái)了。下面這一段,現(xiàn)在是 do…while(0) 結(jié)構(gòu)(只有一層嵌套,這種結(jié)構(gòu)用在這里還算合理):

            BOOL foo()
            {
               
            HRESULT hr = CoInitializeEx(0, COINIT_MULTITHREADED);

               
            while (true)
                {
                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            hr = CoInitializeSecurity(NULL, -1, NULL, NULL, RPC_C_AUTHN_LEVEL_DEFAULT, RPC_C_IMP_LEVEL_IMPERSONATE, NULL, EOAC_NONE, NULL);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            CComPtr<IWbemLocator> pLoc = NULL;
                   
            hr = pLoc.CoCreateInstance(CLSID_WbemLocator, NULL, CLSCTX_INPROC_SERVER);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            CComPtr<IWbemServices> pSvc = NULL;
                   
            hr = pLoc->ConnectServer(_T("ROOT\\CIMV2"), NULL, NULL, NULL, 0, NULL, NULL, &pSvc);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            hr = CoSetProxyBlanket(pSvc, RPC_C_AUTHN_WINNT, RPC_C_AUTHZ_NONE, NULL, RPC_C_AUTHN_LEVEL_CALL, RPC_C_IMP_LEVEL_IMPERSONATE, NULL, EOAC_NONE);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            CComPtr<IEnumWbemClassObject> pEnum = NULL;
                   
            _bstr_t bstrLang = _T("WQL");
                   
            _bstr_t bstrSql = _T("SELECT * FROM __InstanceCreationEvent WITHIN 10")
                       
            _T("WHERE TargetInstance ISA 'Win32_LogonSession' AND (TargetInstance.LogonType = 2 OR TargetInstance.LogonType = 11)");
                   
            hr = pSvc->ExecNotificationQuery(bstrLang, bstrSql, WBEM_FLAG_FORWARD_ONLY | WBEM_FLAG_RETURN_IMMEDIATELY, NULL, &pEnum);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            ULONG uCount = 1;
                   
            CComPtr<IWbemClassObject> pNext = NULL;
                   
            hr = pEnum->Next(WBEM_INFINITE, uCount, &pNext, &uCount);

                   
            if (FAILED(hr))
                    {
                       
            break;
                    }

                   
            // ...

                   
            break;
                }

               
            CoUninitialize();

               
            return SUCCEEDED(hr);
            }

            如果改成 goto,則需要把所有需要對(duì)象的定義全放到最前面來(lái),不然 goto 會(huì)跳過(guò)他們的初始化,編譯不過(guò)。但是,所有對(duì)象都放到最前面定義,又違反了即用即聲明的規(guī)則,而且太多了也容易混淆。

            最后,問(wèn)題是,如果遇到 C++ 的、多層嵌套的,大家一般如何組織代碼呢?

            謝謝!

            posted on 2010-03-30 09:55 溪流 閱讀(2780) 評(píng)論(26)  編輯 收藏 引用 所屬分類: C++

            評(píng)論:
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)?[未登錄](méi) 2010-03-30 10:02 | v
            封裝,讓它自己析構(gòu)  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)?[未登錄](méi) 2010-03-30 10:18 | ~
            個(gè)人覺(jué)得第二種最明了.

            本來(lái)就不可避免的問(wèn)題,為什么要想法設(shè)法去避免呢?  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 10:38 | 陳昱(CY)
            這個(gè)沒(méi)辦法解開(kāi)的,邏輯復(fù)雜時(shí),不可能 既完全沒(méi)有重復(fù)代碼,又效率最高(思路最清晰)的

            覺(jué)得1、2種都可以  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)?[未登錄](méi) 2010-03-30 10:54 | Dancefire
            我的觀點(diǎn)是這樣:

            1、如果只有5層左右if嵌套,不是什么大問(wèn)題。可以用一個(gè)函數(shù)解決,看到不少代碼中都是存在一個(gè)函數(shù)中有多層if嵌套。

            2、如果嵌套太多,影響函數(shù)邏輯的清晰了。那么拆解函數(shù),將功能獨(dú)立的內(nèi)層嵌套,拆解成獨(dú)立的函數(shù)。保證每個(gè)函數(shù)中嵌套不要太多,而拆解出的函數(shù),功能又比較獨(dú)立。

            3、對(duì)于需要重復(fù)寫每次清理時(shí)繁瑣的步驟的問(wèn)題,如果確實(shí)重復(fù)、確實(shí)繁瑣,可寫成一個(gè)函數(shù),每次清理前調(diào)用一下這個(gè)函數(shù)即可。不用擔(dān)心調(diào)用成本,C++會(huì)自動(dòng)分析,編譯Release版本時(shí),很有可能會(huì)將其視為inline。

            4、假如Process和Token的每次創(chuàng)建過(guò)程都比較復(fù)雜,比如有多種情況要考慮,多種錯(cuò)誤要處理,出現(xiàn)問(wèn)題后要有多種錯(cuò)誤處理方式。那么建議將Process和/或Token進(jìn)行類封裝。這樣使用時(shí),只要Process p;即可,發(fā)現(xiàn)錯(cuò)誤直接return false; p會(huì)自動(dòng)析構(gòu),調(diào)用相應(yīng)的析構(gòu)函數(shù)來(lái)清理垃圾。在這種情況下,我與第一個(gè)回復(fù)的朋友觀點(diǎn)一致。不要擔(dān)心類的成本,只要沒(méi)有繼承、虛函數(shù)之類的東西,類的效率與調(diào)用函數(shù)無(wú)太大差別。

            5、使用異常。異常的設(shè)計(jì)初衷本就是為了解決這種現(xiàn)象的,避免錯(cuò)誤處理影響了函數(shù)邏輯。如果出了錯(cuò)就是異常處理、清理、返回,而不會(huì)出現(xiàn)改正錯(cuò)誤重新執(zhí)行代碼的情況,可以使用異常。這樣只要出錯(cuò)就拋出異常,不同的錯(cuò)誤,異常不同。函數(shù)內(nèi)解決異常,不讓異常出函數(shù)。這樣也可以將錯(cuò)誤處理集中起來(lái)。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 11:17 | 空明流轉(zhuǎn)
            一般很少出現(xiàn)這種情況。。。
            偶爾用while(0),如果是一些特殊的析構(gòu)語(yǔ)義,那就用Functor+一個(gè)Execute on Destruction。不過(guò)這個(gè)也有弊端,就是不能扔異常。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 12:45 | 鷹擊長(zhǎng)空
            @Dancefire
            同意大部分 但個(gè)人不喜歡使用異常   回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 13:00 | 溪流
            @v
            不會(huì)每個(gè)HANDLE都搞個(gè)class吧?這樣這個(gè)class連一個(gè)相對(duì)完整的邏輯都沒(méi)包含  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 13:00 | 溪流
            @~
            我想確定下是不是真的無(wú)法避免了。。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 13:01 | 溪流
            @陳昱(CY)
            效率不重要,代碼沒(méi)重復(fù)、思路清晰就可以了  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 13:02 | 溪流
            @Dancefire
            @鷹擊長(zhǎng)空

            我也不喜歡異常~  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 13:03 | 溪流
            @空明流轉(zhuǎn)
            很少嗎?像文中的這兩種場(chǎng)合如何解決呢?Windows API應(yīng)該經(jīng)常會(huì)有這種場(chǎng)合吧,除非你不做應(yīng)用層的開(kāi)發(fā)。。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 14:51 | yrj
            觀點(diǎn)與第一個(gè)回復(fù)的朋友一致

            "不會(huì)每個(gè)HANDLE都搞個(gè)class吧?這樣這個(gè)class連一個(gè)相對(duì)完整的邏輯都沒(méi)包含"

            Google Chromium 源碼里就有不少這樣的 class
            http://src.chromium.org/svn/trunk/src/base/scoped_handle_win.h  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 15:28 | 溪流
            @yrj
            謝謝,這個(gè)例子讓我耳目一新~  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 15:31 | 溪流
            忽然覺(jué)得有點(diǎn)想通了,其實(shí)就是需要一個(gè)“智能句柄”。。。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 15:46 | OwnWaterloo
            大哥, C++有析構(gòu)函數(shù), 根本不會(huì)寫出這種代碼。
            而且, 也不需要一個(gè)handle一個(gè)RAII類, 有范型的Loki::ScopeGuard, 有boost::scope_exit, 愛(ài)怎么玩就怎么玩。

            C語(yǔ)言, do while(0) 一層就夠了。
            注意利用不可能的handle值。
              回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 16:01 | 陳梓瀚(vczh)
            把你的HANDLE封裝在類里,用析構(gòu)函數(shù)去搞定  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 16:06 | 溪流
            @OwnWaterloo
            大哥哥,do...while(0)一層怎么搞呀?可否給個(gè)示范?就用3號(hào)方案的場(chǎng)景好了~~  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 16:44 | OwnWaterloo
            cppblog插入代碼很繁瑣……
            你看是這樣嗎?
            int f(void)
            {
                  HANDLE process 
            = 0;
                  HANDLE token   
            = 0;
                  
            int r = 0;

                  
            do {

                        
            if ( process = OpenProcess(), !process)
                              
            break;
                        
            if ( token = OpenProcessToken(), !token )
                              
            break;
                        
            if ( !LookupPrivilegeValue( ) )
                              
            break;
                        
            if ( !AdjustTokenPrivileges( ) )
                              
            break;

                        r 
            = 1;

                  }
             while (0)


                  
            if ( token != N(OpenProcessToken) ) CloseHandle(token);
                  
            if ( process!= N(OpenProcess) ) CloseHandle(process);
                  
            return r;
            }

              回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 17:44 | yrj
            純 C 特例

            如果調(diào)用的每個(gè)函數(shù)返回同一種錯(cuò)誤碼, 可用宏和goto (4.1號(hào)方案)

            #define check_error(func) if ((error = (func)) < 0) goto Error;

            int foo()
            {
            int error;

            check_error(func1());

            check_error(func2());

            check_error(func3());

            /* */

            Error:
            /* your cleanup code */
            return error;
            }  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 17:48 | megahertz
            封裝再封裝,每個(gè)函數(shù)只做一件事。對(duì)象管理資源分配和釋放  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 18:55 | 溪流
            看到這里,我覺(jué)得可以接受的做法就是給每種需要清理的對(duì)象封裝一個(gè)類了:
            ScopedHandle
            ScopedCoInitialization
            ScopedSysMemory
            ScopedPointer
            CComPtr
            。。。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 19:00 | OwnWaterloo
            @溪流
            不是給你說(shuō)了有范型的RAII(RRID)類嗎?
             
            int f()
            {
                  HANDLE process 
            = OpenProcess(  );
                  
            if (!process)
                        
            return 0;
                  LOKI_ON_BLOCK_EXIT(process, CloseHandle);

                  HANDLE token 
            = OpenProcessTokon(  );
                  
            if (!token)
                        
            return 0;
                  LOKI_ON_BLOCK_EXIT(token, CloseHandle);

                  
            if ( !LookupPrivilegeValue(  ) )
                        
            return 0;

                  
            if ( !AdjustTokenPrivileges(  ) )
                        
            return 0;

                  
            return 1;
            }

              回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 19:12 | 溪流
            @OwnWaterloo
            啊,,,這是哪個(gè)庫(kù)里的?  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 19:18 | 溪流
            @OwnWaterloo
            哦,,,又是boost。。。  回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-30 19:22 | OwnWaterloo
            @溪流
            是Loki啦:
            http://en.wikipedia.org/wiki/Loki_(C%2B%2B)
            http://loki-lib.sourceforge.net/

            配套的《modern c++ design》是目前看過(guò)的最高級(jí)的C++書(shū)……

            boost也有scope_exit,是重量級(jí)的。
            Loki的ScopeGuard依賴"現(xiàn)有的函數(shù)", 比如CloseHandle。
            注冊(cè)之后, 在block退出時(shí), 會(huì)調(diào)用這個(gè)函數(shù)。

            而boost scope_exit是"就地編寫退出代碼"(調(diào)用函數(shù)也包括在內(nèi))。
            當(dāng)退出時(shí), 會(huì)執(zhí)行這些代碼。
              回復(fù)  更多評(píng)論
              
            # re: 這種代碼結(jié)構(gòu)如何組織?goto or do&hellip;while(0)? 2010-03-31 00:18 | 溪流
            @OwnWaterloo
            我突然也有想寫一個(gè)的沖動(dòng)。。。  回復(fù)  更多評(píng)論
              
            久久综合狠狠综合久久97色| 久久精品国产精品亚洲毛片| 日韩欧美亚洲综合久久影院Ds| 久久亚洲精品无码观看不卡| 亚洲国产天堂久久综合| 精品国产乱码久久久久软件| 久久久久无码精品国产不卡| 女人香蕉久久**毛片精品| 久久综合给合综合久久| 久久午夜伦鲁片免费无码| 93精91精品国产综合久久香蕉| 香蕉aa三级久久毛片| 99久久99久久久精品齐齐| 国产精品综合久久第一页| 国产成人精品综合久久久| 伊人久久免费视频| 伊人久久大香线蕉亚洲五月天| 久久99精品国产99久久6男男| 亚洲а∨天堂久久精品| 久久精品国产久精国产思思| 日本精品一区二区久久久 | 奇米影视7777久久精品人人爽| 97久久久久人妻精品专区 | 久久亚洲AV无码精品色午夜麻豆| 国产精品视频久久久| 人妻无码久久精品| 亚洲午夜久久久精品影院 | 精品国产综合区久久久久久| 无码AV中文字幕久久专区| 久久人人爽人人爽AV片| 久久se精品一区二区| 久久水蜜桃亚洲av无码精品麻豆| 久久久网中文字幕| 国产精品激情综合久久| 国产精品九九九久久九九| 久久偷看各类wc女厕嘘嘘| 久久中文字幕精品| 欧美日韩成人精品久久久免费看| 色综合久久最新中文字幕| 99久久超碰中文字幕伊人| 久久精品国产亚洲AV高清热 |