今天,发现昨天下午新作的页面中查询数据时,条件里出现了很多个0,程序是asp的。就开始向把0给直接删除。打开页面一看,这些个0是从另外一个函数返回的,就没有贸然删除,怕出错。
到另外一个函数中,发现返回0并没有错误,0是函数的默认值,不能从参数中直接把”0“直接改成空字符。那就调用端开始处理。在调用端把用一个新的变量把函数值给取到,然后判断,是"0",就直接处理,不是"0",就按照以前的规则处理。修改完,上传刷新页面。发现 条件里的0 少了一部分,但是还是有。我就直接把条件又再次处理了一下,",0",这种值直接replace成空值。
发现值中没有0这种空值了。顺手把这个函数放到另外一个文件里。同时,增加文件引用。再次上传更新,刷新页面没有问题。一切都安好。
这个是很简单的重构过程。没有改变页面的功能,在修改过程中增加判断,使函数return值的处理更加合理。经过测试,一切ok.越是asp这种IDE,不发达的语言,重构越难做,同时,重构的好处越多。在做了上述的重构后,还加一个小函数,使得功能更加完善了。
重构执行始于手下。