代码优化问题

写了一个逻辑上的代码，总觉得有问题，运行的也不对，现贴出来拜托大神们优化一下：

def handle_doc(self):
        '''
        初始化待处理文件列表页面，获得第一个文件的id号，如果有，则进行文件处理，如果没有则无待处理文件：
        一、跳转到该文件页面；
        二、获取该文件领导批示内容
        三、处理该文件
        1.如果批示内容为空：则录入execl,下载附件文件，提交部门领导审核
        2.如果批示内容不为空：说明已处理，则将批示内容更新至execl,根据批示内容进行下步处理        
        '''
        docs = DocsPage(self.driver)     #实例化待处理文件列表页面
        unid = docs.get_first_doc_unid()#获得第一个需要处理的文件的unid
        while unid:                        #如果存在待处理文件
            doc = DocPage(self.driver)    #实例化文件页面
            doc.open(url_parse.get_doc_url(unid))#打开该文件页面
            title = doc.get_doc_title()    #获得文件标题
            logger.info('正处理《%s》文件.' % title)
            if doc.get_approval():        #如果已批示，取得该文件领导批示
                logger.info('《%s》文件已被部门领导批示.' % title)
                data_lines.update_execl(unid, doc.get_approval()) #将领导批示更新到execl中
                #do some thing
                doc.finish_doc()        #结束处理
                break                    #break？
            else:                        #未批示文件
                logger.info('《%s》已接收，现进行写入数据库、下载文件附件、提交部门领导批示操作.' % title)
                data_lines.insert_execl(doc.get_doc_info())#写入execl
                data_lines.download_file('url', 'file_path', 'file_name')#下载文件附件
                doc.to_leader()            #提交领导批示
            doc.open(url_parse.get_default_url())    #处理完一个文件，跳转至待处理文件列表页面
            doc.refresh()
            self.handle_doc()                        #重新执行该函数
        else:
            logger.info('已处理完所有文件')

希望也能给出一些建议