코드의 나쁜 맛: 제어 구조가 너무 깊게 박혀 있다

재구성 전 코드

    def newPrj(self,db,userId,prjName,password,privacy,intro='',status='active'):
        s = ''
        flag = False
        prjId=0
        
        try:
            t = db.transaction()
            if privacy == 'public' and len(password) < 1:
                return '{"result":"error","message":"           ，            "}'
            
            folderId,deep = tn.newFolder(db,prjName,userId,0,0)
            if folderId != 0:
                prjId = db.insert('Project',folderId=folderId,userId=userId,projectName=prjName,
                    password=password,privacy=privacy,intro=intro,status=status,
                    createTime=str(datetime.datetime.today()),revision=util.new_revision(db,userId))
                db.update("Category",where="catId=$folderId",projectId=prjId,deep=-1,vars=locals())
                
                import json
                from role import RoleMgr
                roleMgr = RoleMgr()
                s = roleMgr.newRole(db,"    ",prjId)
                dict = json.JSONDecoder().decode(s)
                if dict['result'] == 'ok':
                    roleId = int(dict['roleId'])
                    if roleMgr.bindPermission(db,roleId,folderId,read=1,write=0) == True:
                        flag = True
                        s = '{"result":"ok","prjId": "%d"}' %(int(prjId))
                    else:
                        s = '{"result":"error","message":"          "}'
                else:
                    s = '{"result":"error","message":"        "}'
            else:
                s = '{"result":"error","message":"       "}'
            
            if flag == True:
                t.commit()
            else:
                t.rollback()
                
            return s
        
        except:
            t.rollback()

재구성된 코드

def newPrj(db,userId,projectName,password,privacy,intro,status='active'):
    try:
        t = db.transaction()
        projectName = projectName.lstrip().rstrip()
        if not _checkName(projectName):
            t.rollback()
            return (False,"      ")
        folderId,deep = tn.newFolder(db,projectName,userId,0,0)
        if folderId == 0:
            t.rollback()
            return (False,"         ")
        projectId = _insert(db,projectName,folderId,userId,password,privacy,intro,status)
        if projectId == 0:
            t.rollback()
            return (False,"      ")
        if folder.setRootFolder(db,folderId,projectId) == False:
            t.rollback()
            return (False,"      ")
        if _preSetRole(db,userId,folderId,projectId) == False:
            t.rollback()
            return (False,"        ")
        if _preSetFolder(db,userId,folderId) == False:
            t.rollback()
            return (False,"        ")

        t.commit()
        return (True,projectId)
    except:
        t.rollback()
        raise