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

清单 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;
 }
}
 

把我对 Stack 类作出的所有修改和修复综合起来，可以得到清单 28 中的代码。

清单 28. 一个可正常工作的栈
               
import java.util.ArrayList;

public class Stack<E> {

 private ArrayList<E> list;

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