重构实例1：抽取冗余代码 - 未通过测试而进行的第二次修正

昨天重构了OPEN-API模块，挺高兴，觉得自己做的不错，结果昨天把程序重构完，除了挑出来作为测试接口的代码其他代码都没有测试，今天上午要做其他的维护工作，觉得还是先跑一次测试再说，测试程序并不是特地为这个接口进行开发，当时为这个OPEN-API写了一个SDK，这个程序其实是为了SDK开发的测试程序，且写的并不完善，但即使是这样，测试程序的断言（assert）仍然报告执行出了问题，因此我又去排查程序，发现有几个接口，它的函数参数不只一个参数（self参数），而我的几类再调具体子类的时，就没有提供其他的参数，因此产生运行时错误，了解到这一点，就翻了手上的《python学习手册》花了一个小时解决问题

class OpenApiBase:
def __init__(self):
self.funPOST = self.POST
self.POST = self.post

#将post函数声明为支持可变参数的函数，这样它可以处理子类的POST函数有多个参数的情况
def post(self,*args):
try:
wi =  web.input()
token = wi.token
self.serverToken = getServerToken(db,token)

web.debug(str(self.serverToken))

if self.serverToken == False:
return '{"result":"error","message":"token is error"}'

if checkExpires(self.serverToken):
return '{"result":"error","message":"token is expires"}'

#执行每个子类具体的代码，注意：apply将可变参数送入funPOST
return apply(self.funPOST,args)
except:
if DEBUG:
raise
return '{"result":"error","message":""}'

class getFolders(OpenApiBase):
#该函数有两个参数，action是业务逻辑里必须的，因此父类的post函数调用时必须传递进来
def POST(self,action):
wi = web.input()

userId = self.serverToken.userId #子类拿到包含身份信息的变量，该变量是父类创建，通过属性拿到

if action == "person": #使用action
list = tnuser.User.getPersonFolders(db,userId)
return '{"result":"ok","message":"","data":%s}' %(json.dumps(list))

if action == "project": #使用action
projectId = util.unhash17(int(wi.id))
folderId = getProjectRootFolderId(db,projectId)
if folderId == -1:
return '{"result":"error","message":"","data":[]}'

list = folder.getSubFolders(db,folderId,userId)
for i in list:
i["projectId"] = util.hash17(i["projectId"])
return '{"result":"ok","message":"","data":%s}' %(json.dumps(list))

if action == "subfolder": #使用action
folderId = int(wi.id)
list = folder.getSubFolders(db,folderId,userId)
for i in list:
i["projectId"] = util.hash17(i["projectId"])
return '{"result":"ok","message":"","data":%s}' %(json.dumps(list))

return '{"result":"error","message":"","data":[]}'

最后，对测试程序做个说明 镇宅神兽 13:36:58：

我理解调试指的是单步啥的，但是测试是根据接口，预先定义好输入输出，什么样的输入会产生什么样的输出，一旦不符合规则，立即报警，且可以重复执行的脚本性程序