小酌重構系列[22]——儘快返回

来源:http://www.cnblogs.com/keepfool/archive/2016/05/31/5544601.html
-Advertisement-
Play Games

閱讀文章時,如果某個段落已經傳達了關鍵信息,我們可能就不會逐字逐句地將文章讀完,因為我們已經知道了這篇文章的核心內容。 與此類似,如果方法中某些條件判斷可以得到結果,我們應該儘快返回該結果。 儘快返回可以帶來三個好處:1. 節省閱讀代碼的時間 2. 避免執行無效的邏輯 3. 增強代碼的可讀性 ...


概述

閱讀文章時,如果某個段落已經傳達了關鍵信息,我們可能就不會逐字逐句地將文章讀完,因為我們已經知道了這篇文章的核心內容。
與此類似,如果方法中某些條件判斷可以得到結果,我們應該儘快返回該結果。

儘快返回可以帶來三個好處

  • 節省閱讀代碼的時間——如果方法能夠儘快返回,後面的代碼邏輯可以不必閱讀。
    見下圖,如果①已經返回了,就不必閱讀②部分的代碼
  • 避免執行無效的邏輯——如果方法能夠儘快返回,後面的代碼邏輯就不會被執行。
    見下圖,如果①已經返回了,②部分的邏輯不會被執行
  • 增強代碼的可讀性

 

image

分解大括弧這篇文章中,我們已經用到了“儘快返回”這個重構策略。
在實現複雜的業務邏輯時,可能會形成多層的代碼嵌套,如果某些條件判斷可以得到結果,我們應該儘快返回這個結果,這也能夠減少嵌套層數,從而增強代碼的可讀性。

示例

下麵沿用了分解大括弧這篇文章中的實例,但我會從“儘快返回”的角度去講解這個示例。

重構前

Security類的HasAccess方法為了判斷用戶是否有許可權訪問,直到方法的最後一行才返回結果。
這個方法有4層嵌套,它的可讀性本身是較差的,這意味著我們需要理解所有的嵌套邏輯,才能知道返回結果。

隱藏代碼
public class Security
{
    public ISecurityChecker SecurityChecker { get; set; }

    public Security(ISecurityChecker securityChecker)
    {
        SecurityChecker = securityChecker;
    }

    public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions)
    {
        bool hasPermission = false;

        if (user != null)
        {
            if (permission != null)
            {
                if (exemptions.Count() == 0)
                {
                    if (SecurityChecker.CheckPermission(user, permission) || exemptions.Contains(permission))
                    {
                        hasPermission = true;
                    }
                }
            }
        }

        return hasPermission;
    } 
}

public interface ISecurityChecker
{
    bool CheckPermission(User user, Permission permission);
}

HasAccess方法的邏輯可以分解為3個步驟:

  1. 當user或permission為null時,返回false
  2. 當參數permission是參數exemptions的一個元素時,返回true
  3. 返回SecurityChecker.CheckPermission(user, permission)的調用結果

下圖標示了這3個步驟。

image

這3個步驟都能決定方法的返回結果,它們之間沒有相互依賴性。
我們可以將這3個步驟分離開來,然後指定這3個步驟的執行順序,並儘快返回結果。

重構後

重構後,HasAccess方法的邏輯一目瞭然,它的3步邏輯都用到了return,易於閱讀和理解。

隱藏代碼
public class Security
{
    public ISecurityChecker SecurityChecker { get; set; }

    public Security(ISecurityChecker securityChecker)
    {
        SecurityChecker = securityChecker;
    }

    public bool HasAccess(User user, Permission permission, IEnumerable<Permission> exemptions)
    {
        if (user == null || permission == null)
            return false;

        if (exemptions.Contains(permission))
        {
            return true;
        }

        return SecurityChecker.CheckPermission(user, permission);
    } 
}

public interface ISecurityChecker
{
    bool CheckPermission(User user, Permission permission);
}

小結

如果能夠儘快得到結果,何必等到最後一刻?

【關註】keepfool
您的分享是我們最大的動力!

-Advertisement-
Play Games
更多相關文章
  • 目前比較流行的驗證做法:前端jquery-form-validate + 後端hibernate-validate 在pom中添加相關jar: spring mvc中配置: 配置錯誤信息:(當然也可以直接寫在javabean中) entity(setter getter 略): controller ...
  • 迴圈、控制語句 for: 迴圈、控制語句 while: ...
  • 一位數組: #include <stdio.h> #include<string.h> #define N 5 void luru(float s[],int n); void shuchu(float s[],int n); void chaxun(float s[],int n); void p ...
  • ...
  • 概述 當條件判斷語句較為複雜時(有多個不同的檢查項),就像下麵這幅圖所表示的,會使得代碼的可讀性會大打折扣,也難以清晰地傳達判斷意圖。 再者,當判斷邏輯變更時,我們不得不去修改if語句裡面的判斷代碼。如果判斷寫得有問題,則會影響方法的正確性,也會給該方法的單元測試帶來一些障礙。 我們可以根據檢查項是... ...
  • 新寵兒 DotNetCore是.net5.0版本,之所以不叫.net5.0為的就是不讓我們把它與前面的.net混為一淡,它將是真正意義的跨平臺開發語言,在網上也有相關介紹,中國的一些大牛也發了相關文章,像張善友大牛也發了幾個文章,來介紹.NetCore,這段時間,大叔將會一步一步說說這個跨平臺的新寵 ...
  • DAO層,Service層,Controller層、View層 http://hovertree.com/hvtart/bjae/sko15s3g.htm 推薦:http://www.cnblogs.com/roucheng/p/chengxuyuan.html ...
  • 原則,故名思議則是本質的意思。所謂擒賊先擒王,研究設計模式自然要先瞭解設計原則,所有的模式都是在這些原則的基礎之上發展起來的,有的是側重一個,有的是多個都有所涉及。看完設計模式之後,我感覺到每個模式都有這些原則的影子,還滲透著面向對象的三大屬性,也覺得這些原則也都有相通之處,,正是有了他們才使我們由 ...
一周排行
    -Advertisement-
    Play Games
  • 前言 本文介紹一款使用 C# 與 WPF 開發的音頻播放器,其界面簡潔大方,操作體驗流暢。該播放器支持多種音頻格式(如 MP4、WMA、OGG、FLAC 等),並具備標記、實時歌詞顯示等功能。 另外,還支持換膚及多語言(中英文)切換。核心音頻處理採用 FFmpeg 組件,獲得了廣泛認可,目前 Git ...
  • OAuth2.0授權驗證-gitee授權碼模式 本文主要介紹如何筆者自己是如何使用gitee提供的OAuth2.0協議完成授權驗證並登錄到自己的系統,完整模式如圖 1、創建應用 打開gitee個人中心->第三方應用->創建應用 創建應用後在我的應用界面,查看已創建應用的Client ID和Clien ...
  • 解決了這個問題:《winForm下,fastReport.net 從.net framework 升級到.net5遇到的錯誤“Operation is not supported on this platform.”》 本文內容轉載自:https://www.fcnsoft.com/Home/Sho ...
  • 國內文章 WPF 從裸 Win 32 的 WM_Pointer 消息獲取觸摸點繪製筆跡 https://www.cnblogs.com/lindexi/p/18390983 本文將告訴大家如何在 WPF 裡面,接收裸 Win 32 的 WM_Pointer 消息,從消息裡面獲取觸摸點信息,使用觸摸點 ...
  • 前言 給大家推薦一個專為新零售快消行業打造了一套高效的進銷存管理系統。 系統不僅具備強大的庫存管理功能,還集成了高性能的輕量級 POS 解決方案,確保頁面載入速度極快,提供良好的用戶體驗。 項目介紹 Dorisoy.POS 是一款基於 .NET 7 和 Angular 4 開發的新零售快消進銷存管理 ...
  • ABP CLI常用的代碼分享 一、確保環境配置正確 安裝.NET CLI: ABP CLI是基於.NET Core或.NET 5/6/7等更高版本構建的,因此首先需要在你的開發環境中安裝.NET CLI。這可以通過訪問Microsoft官網下載並安裝相應版本的.NET SDK來實現。 安裝ABP ...
  • 問題 問題是這樣的:第三方的webapi,需要先調用登陸介面獲取Cookie,訪問其它介面時攜帶Cookie信息。 但使用HttpClient類調用登陸介面,返回的Headers中沒有找到Cookie信息。 分析 首先,使用Postman測試該登陸介面,正常返回Cookie信息,說明是HttpCli ...
  • 國內文章 關於.NET在中國為什麼工資低的分析 https://www.cnblogs.com/thinkingmore/p/18406244 .NET在中國開發者的薪資偏低,主要因市場需求、技術棧選擇和企業文化等因素所致。歷史上,.NET曾因微軟的閉源策略發展受限,儘管後來推出了跨平臺的.NET ...
  • 在WPF開發應用中,動畫不僅可以引起用戶的註意與興趣,而且還使軟體更加便於使用。前面幾篇文章講解了畫筆(Brush),形狀(Shape),幾何圖形(Geometry),變換(Transform)等相關內容,今天繼續講解動畫相關內容和知識點,僅供學習分享使用,如有不足之處,還請指正。 ...
  • 什麼是委托? 委托可以說是把一個方法代入另一個方法執行,相當於指向函數的指針;事件就相當於保存委托的數組; 1.實例化委托的方式: 方式1:通過new創建實例: public delegate void ShowDelegate(); 或者 public delegate string ShowDe ...