今天,同事的程序出了个怪问题,叫我帮他调。程序的问题是把JSP页面上输入的数据向数据库中插入的时候抛异常。通过控制台上的出错信息得知,是主键重复。这个简单,看了一下Bean中的主键生成的方法getMaxID没有问题,又看了一下业务方法和SQL语句,也没有问题。于是,将生成的ID打印在控制台上,观察发现,生成的ID是正确的,但有问题的是打印了两个ID。马上检查代码寻找是否有逻辑上的错误,无果。接下来用了最笨也是最实用的一招,根据程序流程一步步的打印输出,看看是在哪一步出错。运行程序后发现所有的信息都打印了两次,这种错误并不稀奇,奇怪的是每条打印信息都是连续打印两次后再打印下一条信息。如下:
比如程序中写了三条打印语句
......代码......
System.out.println("==========A");
......代码......
System.out.println("==========B");
......代码......
System.out.println("==========C");
在控制台上打印出的语句如下:
==========A
==========A
==========B
==========B
==========C
==========C
至此,程序为何抛异常已经很明白了。因为每段程序都执行两次,所以在插入数据时,将相同主键ID的值连续插入两次导致主键重复。错误的起点就是错误的根源。于是把打印语句从Bean中一直写到JSP页面。发现JSP页面上的打印语句也打印了两次。这下问题已经很明显了,重复提交。于是,在Form的里加了一句 onsubmit="return false" 重新运行,成功。查看Html代码发现提交按钮的类型是submit,并且在onclick中调用了javascript方法,错误的根源是javascripte方法中也提交了一次(window.form.submit());将按钮的类型改为button,问题解决。
解决这个问题用了大概半小时的时间!其实我一直就很提倡Martin Fowler在他的《Refactoring》中所说的“为每个公开的方法写测试用例,一有改动就进行测试,所有的Bug都尽现眼前”。的确如此。