WEB开发网
开发学院软件开发Java 追求代码质量: 亲身体验行为驱动开发 阅读

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

 2009-11-19 00:00:00 来源:WEB开发网   
核心提示: 由于 peek() 还没有定义,因此清单 21 还不能编译,追求代码质量: 亲身体验行为驱动开发(10),在清单 22 中,我定义了 peek() 的一个最简单的实现,不会感到意外,如清单 26 所示,清单 22. 当前,peek 是必需的publicEpeek(){returnnull;}现在

由于 peek() 还没有定义,因此清单 21 还不能编译。在清单 22 中,我定义了 peek() 的一个最简单的实现。


清单 22. 当前,peek 是必需的
public E peek() { 
 return null; 
} 

现在 StackBehavior 类可以编译,但是它仍然不能运行。


清单 23. 返回 null 并不奇怪,对吗?
1) StackBehavior should leave value on stack after peep: 
VerificationException: Expected: 
same instance as <test 2> 
but got: 
null: 

在逻辑上,peek() 不会从内部集合中移除 项目,它只是传递指向那个项目的指针。因此,我将对 ArrayList 使用 get() 方法,而不是 remove() 方法,如清单 24 所示。


清单 24. 不要移除它
public E peek() { 
 return this.list.get(this.list.size()-1); 
} 

栈为空的情况

现在重新运行 清单 21 中的行为,结果顺利通过。但是,在这样做的过程中发现一个问题:如果栈为空,则 peek() 有怎样的行为?如果说栈为空时调用 pop() 会抛出一个异常,那么 peek() 是否也应该如此?

Linda 对此没有进行解释,所以,显然我需要自己添加新的行为。在清单 25 中,我为 “当之前没有调用 push() 时调用 peek() 会怎样” 这个场景编写了代码。


清单 25. 如果没有调用 push 就调用 peek,会怎样?
public void shouldReturnNullOnPeekWithoutPush() throws Exception{ 
 Ensure.that(stStack.peek(), m.is(null)); 
} 

同样,不会感到意外。如清单 26 所示,问题出现了。

上一页  5 6 7 8 9 10 

Tags:追求 代码 质量

编辑录入:爽爽 [复制链接] [打 印]
赞助商链接