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

清单 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 所示，问题出现了。

清单 26. 没有可执行的内容
               
1) StackBehavior should return null on peek without push:
java.lang.ArrayIndexOutOfBoundsException: -1
 

修复这个缺陷的逻辑类似于 pop() 的逻辑，如清单 27 所示。

清单 27. 这个 peek() 需要做一些修复
               
public E peek() {
 if(this.list.size() > 0){
  return this.list.get(this.list.size()-1);
 }else{
  return null;
 }
}
 

文章来源于领测软件测试网 https://www.ltesting.net/