華爲軟件編程規範學習(六)--函數、過程

華爲軟件編程規範學習(六)--函數、過程

轉自:http://blog.csdn.net/ce123_zhouwei/article/details/8888977

6-1:對所調用函數的錯誤返回碼要仔細、全面地處理

6-2:明確函數功能,精確(而不是近似)地實現函數設計

6-3:編寫可重入函數時,應注意局部變量的使用(如編寫C/C++語言的可重入函數時,應使用auto即缺省態局部變量或寄存器變量)

說明:編寫C/C++語言的可重入函數時,不應使用static局部變量,否則必須經過特殊處理,才能使函數具有可重入性。

6-4:編寫可重入函數時,若使用全局變量,則應通過關中斷、信號量(即P、V操作)等手段對其加以保護

說明:若對所使用的全局變量不加以保護,則此函數就不具有可重入性,即當多個進程調用此函數時,很有可能使有關全局變量變爲不可知狀態。

示例:假設Exam是int型全局變量,函數Squre_Exam返回Exam平方值。那麼如下函數不具有可重入性。

[plain] view plain copy
  1. unsigned int example(int para )  
  2. {  
  3.    unsigned int temp;  
  4.    Exam = para; // (**)  
  5.    temp = Square_Exam( );  
  6.    return temp;  
  7. }  

此函數若被多個進程調用的話,其結果可能是未知的,因爲當(**)語句剛執行完後,另外一個使用本函數的進程可能正好被激活,那麼當新激活的進程執行到此函數時,將使Exam賦與另一個不同的para值,所以當控制重新回到“temp = Square_Exam()”後,計算出的temp很可能不是預想中的結果。此函數應如下改進。

[plain] view plain copy
  1. unsigned int example(int para )  
  2. {  
  3.    unsigned int temp;  
  4.     [申請信號量操作]       // 若申請不到“信號量”,說明另外的進程正處於  
  5.    Exam = para;           // 給Exam賦值並計算其平方過程中(即正在使用此  
  6.    temp = Square_Exam( );  // 信號),本進程必須等待其釋放信號後,纔可繼  
  7.     [釋放信號量操作]       // 續執行。若申請到信號,則可繼續執行,但其  
  8.                            // 它進程必須等待本進程釋放信號量後,才能再使  
  9.                            // 用本信號。  
  10.    return temp;  
  11. }  

6-5:在同一項目組應明確規定對接口函數參數的合法性檢查應由函數的調用者負責還是由接口函數本身負責,缺省是由函數調用者負責

說明:對於模塊間接口函數的參數的合法性檢查這一問題,往往有兩個極端現象,即:要麼是調用者和被調用者對參數均不作合法性檢查,結果就遺漏了合法性檢查這一必要的處理過程,造成問題隱患;要麼就是調用者和被調用者均對參數進行合法性檢查,這種情況雖不會造成問題,但產生了冗餘代碼,降低了效率。

其他

6-1:防止將函數的參數作爲工作變量

說明:將函數的參數作爲工作變量,有可能錯誤地改變參數內容,所以很危險。對必須改變的參數,最好先用局部變量代之,最後再將該局部變量的內容賦給該參數。

示例:下函數的實現不太好。

[plain] view plain copy
  1. void sum_data(unsigned int num, int *data, int *sum )  
  2. {  
  3.    unsigned int count;  
  4.    *sum = 0;  
  5.    for (count = 0; count < num; count++)  
  6.     {  
  7.        *sum  += data[count]; // sum成了工作變量,不太好。  
  8.     }  
  9. }  

若改爲如下,則更好些。

[plain] view plain copy
  1. void sum_data(unsigned int num, int *data, int *sum )  
  2. {  
  3.    unsigned int count ;  
  4.    int sum_temp;  
  5.    sum_temp = 0;  
  6.    for (count = 0; count < num; count ++)  
  7.     {  
  8.        sum_temp  += data[count];  
  9.     }  
  10.    *sum = sum_temp;  
  11. }  

6-2:函數的規模儘量限制在200行以內

說明:不包括註釋和空格行。

6-3:一個函數僅完成一件功能

6-4:爲簡單功能編寫函數

說明:雖然爲僅用一兩行就可完成的功能去編函數好象沒有必要,但用函數可使功能明確化,增加程序可讀性,亦可方便維護、測試。

示例:如下語句的功能不很明顯。

[plain] view plain copy
  1. value = ( a > b )? a : b ;  

改爲如下就很清晰了。

[plain] view plain copy
  1. int max (int a, intb)  
  2. {  
  3.    return ((a > b) ? a : b);  
  4. }  
  5. value = max (a, b);  

或改爲如下。

[plain] view plain copy
  1. #define MAX (a, b)(((a) > (b)) ? (a) : (b))  
  2. value = MAX (a, b);  

6-5:不要設計多用途面面俱到的函數

說明:多功能集於一身的函數,很可能使函數的理解、測試、維護等變得困難。

6-6:函數的功能應該是可以預測的,也就是隻要輸入數據相同就應產生同樣的輸出

說明:帶有內部“存儲器”的函數的功能可能是不可預測的,因爲它的輸出可能取決於內部存儲器(如某標記)的狀態。這樣的函數既不易於理解又不利於測試和維護。在C/C++語言中,函數的static局部變量是函數的內部存儲器,有可能使函數的功能不可預測,然而,當某函數的返回值爲指針類型時,則必須是STATIC的局部變量的地址作爲返回值,若爲AUTO類,則返回爲錯針。

示例:如下函數,其返回值(即功能)是不可預測的。

[plain] view plain copy
  1. unsigned intinteger_sum( unsigned int base )  
  2. {  
  3.    unsigned int index;  
  4.    static unsigned int sum = 0; // 注意,是static類型的。  
  5.                                 // 若改爲auto類型,則函數即變爲可預測。  
  6.     for(index = 1; index <= base; index++)  
  7.     {  
  8.        sum += index;  
  9.     }  
  10.    return sum;  
  11. }  

6-7:儘量不要編寫依賴於其他函數內部實現的函數

說明:此條爲函數獨立性的基本要求。由於目前大部分高級語言都是結構化的,所以通過具體語言的語法要求與編譯器功能,基本就可以防止這種情況發生。但在彙編語言中,由於其靈活性,很可能使函數出現這種情況。

示例:如下是在DOS下TASM的彙編程序例子。過程Print_Msg的實現依賴於Input_Msg的具體實現,這種程序是非結構化的,難以維護、修改。

[plain] view plain copy
  1. ...  // 程序代碼  
  2. proc Print_Msg // 過程(函數)Print_Msg  
  3.    ...  // 程序代碼  
  4.    jmp  LABEL  
  5.    ...  // 程序代碼  
  6. endp  
  7. proc Input_Msg // 過程(函數)Input_Msg  
  8.    ...  // 程序代碼  
  9. LABEL:  
  10.    ...  // 程序代碼  
  11. endp  

6-8:避免設計多參數函數,不使用的參數從接口中去掉

說明:目的減少函數間接口的複雜度。

6-9:非調度函數應減少或防止控制參數,儘量只使用數據參數

說明:本建議目的是防止函數間的控制耦合。調度函數是指根據輸入的消息類型或控制命令,來啓動相應的功能實體(即函數或過程),而本身並不完成具體功能。控制參數是指改變函數功能行爲的參數,即函數要根據此參數來決定具體怎樣工作。非調度函數的控制參數增加了函數間的控制耦合,很可能使函數間的耦合度增大,並使函數的功能不唯一。

示例:如下函數構造不太合理。

[plain] view plain copy
  1. int add_sub( int a,int b, unsigned char add_sub_flg )  
  2. {  
  3.     if(add_sub_flg == INTEGER_ADD)  
  4.     {  
  5.        return (a + b);  
  6.     }  
  7.    else  
  8.     {  
  9.        return (a  b);  
  10.     }  
  11. }  

不如分爲如下兩個函數清晰。

[plain] view plain copy
  1. int add( int a, int b)  
  2. {  
  3.    return (a + b);  
  4. }  
  5. int sub( int a, int b)  
  6. {  
  7.    return (a  b);  
  8. }  

6-10:檢查函數所有參數輸入的有效性

6-11:檢查函數所有非參數輸入的有效性,如數據文件、公共變量等

說明:函數的輸入主要有兩種:一種是參數輸入;另一種是全局變量、數據文件的輸入,即非參數輸入。函數在使用輸入之前,應進行必要的檢查。

6-12:函數名應準確描述函數的功能

6-13:使用動賓詞組爲執行某操作的函數命名。如果是OOP方法,可以只有動詞(名詞是對象本身)

示例:參照如下方式命名函數。

[plain] view plain copy
  1. void print_record(unsigned int rec_ind ) ;  
  2. int input_record( void ) ;  
  3. unsigned charget_current_color( void ) ;  

建議6-14:避免使用無意義或含義不清的動詞爲函數命名

說明:避免用含義不清的動詞如process、handle等爲函數命名,因爲這些動詞並沒有說明要具體做什麼。

建議6-15:函數的返回值要清楚、明瞭,讓使用者不容易忽視錯誤情況

說明:函數的每種出錯返回值的意義要清晰、明瞭、準確,防止使用者誤用、理解錯誤或忽視錯誤返回碼。

6-16:除非必要,最好不要把與函數返回值類型不同的變量,以編譯系統默認的轉換方式或強制的轉換方式作爲返回值返回

6-17:讓函數在調用點顯得易懂、容易理解

6-18:在調用函數填寫參數時,應儘量減少沒有必要的默認數據類型轉換或強制數據類型轉換

說明:因爲數據類型轉換或多或少存在危險。

6-19:避免函數中不必要語句,防止程序中的垃圾代碼

說明:程序中的垃圾代碼不僅佔用額外的空間,而且還常常影響程序的功能與性能,很可能給程序的測試、維護等造成不必要的麻煩。

6-20:防止把沒有關聯的語句放到一個函數中

說明:防止函數或過程內出現隨機內聚。隨機內聚是指將沒有關聯或關聯很弱的語句放到同一個函數或過程中。隨機內聚給函數或過程的維護、測試及以後的升級等造成了不便,同時也使函數或過程的功能不明確。使用隨機內聚函數,常常容易出現在一種應用場合需要改進此函數,而另一種應用場合又不允許這種改進,從而陷入困境。

在編程時,經常遇到在不同函數中使用相同的代碼,許多開發人員都願把這些代碼提出來,並構成一個新函數。若這些代碼關聯較大並且是完成一個功能的,那麼這種構造是合理的,否則這種構造將產生隨機內聚的函數。

示例:如下函數就是一種隨機內聚。

[plain] view plain copy
  1. void Init_Var( void )  
  2. {  
  3.    Rect.length = 0;  
  4.    Rect.width = 0; /* 初始化矩形的長與寬 */  
  5.    Point.x = 10;  
  6.    Point.y = 10;   /* 初始化“點”的座標 */  
  7. }  

矩形的長、寬與點的座標基本沒有任何關係,故以上函數是隨機內聚。

應如下分爲兩個函數:

[plain] view plain copy
  1. void Init_Rect( void)  
  2. {  
  3.    Rect.length = 0;  
  4.    Rect.width = 0; /* 初始化矩形的長與寬 */  
  5. }  
  6. void Init_Point( void)  
  7. {  
  8.     Point.x= 10;  
  9.    Point.y = 10;   /* 初始化“點”的座標 */  
  10. }  

6-21:如果多段代碼重複做同一件事情,那麼在函數的劃分上可能存在問題

說明:若此段代碼各語句之間有實質性關聯並且是完成同一件功能的,那麼可考慮把此段代碼構造成一個新的函數。

6-22:功能不明確較小的函數,特別是僅有一個上級函數調用它時,應考慮把它合併到上級函數中,而不必單獨存在

說明:模塊中函數劃分的過多,一般會使函數間的接口變得複雜。所以過小的函數,特別是扇入很低的或功能不明確的函數,不值得單獨存在。

6-23:設計高扇入、合理扇出(小於7)的函數

說明:扇出是指一個函數直接調用(控制)其它函數的數目,而扇入是指有多少上級函數調用它。

扇出過大,表明函數過分複雜,需要控制和協調過多的下級函數;而扇出過小,如總是1,表明函數的調用層次可能過多,這樣不利程序閱讀和函數結構的分析,並且程序運行時會對系統資源如堆棧空間等造成壓力。函數較合理的扇出(調度函數除外)通常是3-5。扇出太大,一般是由於缺乏中間層次,可適當增加中間層次的函數。扇出太小,可把下級函數進一步分解多個函數,或合併到上級函數中。當然分解或合併函數時,不能改變要實現的功能,也不能違背函數間的獨立性。

扇入越大,表明使用此函數的上級函數越多,這樣的函數使用效率高,但不能違背函數間的獨立性而單純地追求高扇入。公共模塊中的函數及底層函數應該有較高的扇入。

較良好的軟件結構通常是頂層函數的扇出較高,中層函數的扇出較少,而底層函數則扇入到公共模塊中。

6-24:減少函數本身或函數間的遞歸調用

說明:遞歸調用特別是函數間的遞歸調用(如A->B->C->A),影響程序的可理解性;遞歸調用一般都佔用較多的系統資源(如棧空間);遞歸調用對程序的測試有一定影響。故除非爲某些算法或功能的實現方便,應減少沒必要的遞歸調用。

6-25:仔細分析模塊的功能及性能需求,並進一步細分,同時若有必要畫出有關數據流圖,據此來進行模塊的函數劃分與組織

說明:函數的劃分與組織是模塊的實現過程中很關鍵的步驟,如何劃分出合理的函數結構,關係到模塊的最終效率和可維護性、可測性等。根據模塊的功能圖或/及數據流圖映射出函數結構是常用方法之一。

6-26:改進模塊中函數的結構,降低函數間的耦合度,並提高函數的獨立性以及代碼可讀性、效率和可維護性

優化函數結構時,要遵守以下原則:

(1)不能影響模塊功能的實現。

(2)仔細考查模塊或函數出錯處理及模塊的性能要求並進行完善。

(3)通過分解或合併函數來改進軟件結構。

(4)考查函數的規模,過大的要進行分解。

(5)降低函數間接口的複雜度。

(6)不同層次的函數調用要有較合理的扇入、扇出。

(7)函數功能應可預測。

(8)提高函數內聚。(單一功能的函數內聚最高)

說明:對初步劃分後的函數結構應進行改進、優化,使之更爲合理。

6-27:在多任務操作系統的環境下編程,要注意函數可重入性的構造

說明:可重入性是指函數可以被多個任務進程調用。在多任務操作系統中,函數是否具有可重入性是非常重要的,因爲這是多個進程可以共用此函數的必要條件。另外,編譯器是否提供可重入函數庫,與它所服務的操作系統有關,只有操作系統是多任務時,編譯器纔有可能提供可重入函數庫。如DOS下BC和MSC等就不具備可重入函數庫,因爲DOS是單用戶單任務操作系統。

6-28:避免使用BOOL參數

說明:原因有二,其一是BOOL參數值無意義,TURE/FALSE的含義是非常模糊的,在調用時很難知道該參數到底傳達的是什麼意思;其二是BOOL參數值不利於擴充。還有NULL也是一個無意義的單詞。

6-29:對於提供了返回值的函數,在引用時最好使用其返回值

6-30:當一個過程(函數)中對較長變量(一般是結構的成員)有較多引用時,可以用一個意義相當的宏代替

說明:這樣可以增加編程效率和程序的可讀性。

示例:在某過程中較多引用TheReceiveBuffer[FirstSocket].byDataPtr,

則可以通過以下宏定義來代替:

[plain] view plain copy
  1. # define pSOCKDATAT TheReceiveBuffer[FirstScoket].byDataPtr  
發佈了0 篇原創文章 · 獲贊 0 · 訪問量 1萬+
發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章