중복 사용되는 코드 묶기

updateContent(user, content){

    if(user.login() == false)
        return
    if(user.getID() != content.getUserID())
        if(user.isAdmin() == false)
            return
            
     ...
     ...
}


deleteContent(user, content){

    if(user.login() == false)
        return
    if(user.getID() != content.getUserID())
        if(user.isAdmin() == false)
            return

     ...
     ...
}

<개선 전>

 

hasContentWriteAuthority(user, content){

    if(user.login() == false)
        return false
    if(user.getID() != content.getUserID())
        if(user.isAdmin() == false)
            return false
    
    return true
}


updateContent(user, content){

    if(hasContentWriteAuthority(user, content) == false)
        return
            
     ...
     ...
}


deleteContent(user, content){

    if(hasContentWriteAuthority(user, content) == false)
        return

     ...
     ...
}

<개선 후>

 

 

코드들이 여러 함수에서 중복해서 사용되고 있다면, 그 코드들을 하나의 함수로 묶어서 사용한다.