追求代码质量: 亲身体验行为驱动开发

核心提示： 清单 14. 完成 push 方法publicvoidpush(Evalue){if(value==null){thrownewRuntimeException("Can'tpushnull");}else{this.list.add(value);}}但是，等一下

清单 14. 完成 push 方法
public　void　push(E　value)　{　
　if(value　==　null){　
　throw　new　RuntimeException("Can't　push　null");　
　}else{　
　this.list.add(value);　
　}　
}　

但是，等一下 — 当我重新运行该行为时，它仍然失败！

清单 15. JBehave 报告一个 null 值，而不是一个异常
1)　StackBehavior　should　pop　pushed　value:　
VerificationException:　Expected:　
same　instance　as　<test>　
but　got:　
null:　

至少清单 15 中的失败有别于清单 13 中的失败。在这种情况下，不是抛出一个异常，而是没有发现 "test" 值；实际弹出的是 null。仔细观察 清单 10 会发现：一开始我将 pop() 方法编写为当内部容器中有项目时，就返回 null。问题很容易修复。

清单 16. 是时候编写完这个 pop 方法了
public　E　pop()　{　
　if(this.list.size()　>　0){　
　return　this.list.remove(this.list.size());　
　}else{　
　throw　new　RuntimeException("nothing　to　pop");　
　}　
}　

但是，如果现在我重新运行该行为，我又收到一个新的错误。

清单 17. 另一个错误
1)　StackBehavior　should　pop　pushed　value:　
java.lang.IndexOutOfBoundsException:　Index:　1,　Size:　1　

仔细阅读清单 17 中的实现可以发现问题：在处理 ArrayList 时，我需要考虑 0。

清单 18. 通过考虑 0 修复问题
public　E　pop()　{　
　if(this.list.size()　>　0){　
　return　this.list.remove(this.list.size()-1);　
　}else{　
　throw　new　RuntimeException("Nothing　to　pop");　
　}　
}