如何整理太多的if語句以提高可讀性

Question

我有一些我經常在網站上看到的代碼示例，我想改進它，並希望得到一些幫助。通常，我會在page_load方法中看到5-10個嵌套的if語句，旨在消除無效的用戶輸入，但這看起來很難看，很難讀取和維護。

您會如何建議清理以下代碼示例？我試圖消除的主要事情是嵌套的if語句。

string userid = Request.QueryString["userid"]; 

if (userid != ""){ 
    user = new user(userid); 

    if (user != null){ 
     if (user.hasAccess){ 
      //etc. 
     } 
     else{ 
      denyAccess(INVALID_ACCESS); 
     } 
    } 
    else{ 
     denyAccess(INVALID_USER); 
    } 
} 
else{ 
    denyAccess(INVALID_PARAMETER); 
} 

正如你所看到的，這很快就會變得非常混亂！在這種情況下，我應該遵循哪些模式或做法？

Answer 1

通過使用Guard Clauses先生

string userid = Reuest.QueryString["userid"]; 

if(userid==null) 
return denyAccess(INVALID_PARAMETER); 

user = new user(userid); 
if(user==null) 
return denyAccess(INVALID_USER); 

if (!user.hasAccess) 
return denyAccess(INVALID_ACCESS); 

//do stuff 

PS。用返還或通過否定的條件拋出一個錯誤

Answer 2

可以清理嵌套了一下，寫的if-else鏈：

string userid = Reuest.QueryString["userid"]; 

if (userid == "") { 
    denyAccess(INVALID_PARAMETER); 

} else if (null == (user = new user(userid))){ 
    denyAccess(INVALID_USER); 

} else if (!user.hasAccess){ 
    denyAccess(INVALID_ACCESS); 

} else { 
    //etc. 
} 

Answer 3

最好將其拆分成多個方法（函數）。它會很容易understand.If一些新的人讀他/她理解的邏輯的代碼只是讀取的方法名稱本身（注：方法的名稱應該表達它做什麼測試）。樣品代碼：

string userid = Request.QueryString["userid"]; 

if(isValidParameter(userId)){ 
    User user=new User(userId); 
    if(isValidUser(user)&&isUserHasAccess(user)){ 
     //Do whatever you want 
    } 
} 

private boolean isUserHasAccess(User user){ 
    if (user.hasAccess){ 
     return true; 
    }else{ 
     denyAccess(INVALID_ACCESS); 
     return false; 
    } 
} 

private boolean isValidUser(User user){ 
    if(user !=null){ 
     return true; 
    }else{ 
    denyAccess(INVALID_USER); 
    return false; 
    } 
} 


private boolean isValidParameter(String userId){ 
    if(userid !=""){ 
     return true; 
    }else{ 
denyAccess(INVALID_PARAMETER); 
    return false; 
    } 
}