今天的一个bug

python,webpy

有bug的API：删除文件夹

参数：文件夹ID

实现：判断文件夹ID是不是等于用户的默认文件夹ID，如果是返回false,如果不是，删除指定文件夹并返回true

def removeFolder(folderId,userId)
    defaultId = getDefault(userId)
    if folderId == defaultId:
        return (False,"默认文件夹不能删除")
    else:
        #删除文件夹
        return True

结果: 从webpy传来的文件夹ID是unicode类型，而从数据库里查出来的默认文件夹ID是long类型,两者比较永远是False。

两个教训：1，是忘记了从客户端来的整数其实是字符串 2，因为没有单元测试，而且因为客户端的原因，它自己在客户端就判断了catId是不是和defaultId相等，如果相等就不向服务器发起请求，导致服务端的代码走的总是else分支，因此这个问题就一直没有暴露！直到客户端改版，新的程序员设计新的客户端时，就没有做这个判断，导致程序不该删除默认文件夹的结果删除了，于是就悲剧了。。。