本周Code Retreat中,其中一次是Silent Pair,就是在Pair过程中不能说话,由于两个人都是第一次Pair,所以也不会熟悉对方套路的问题,我觉得这样在Pair中需要注意几点,1.步子不能太大,太大了对方可能很难理解。 2.写测试用例或者生产代码目的性更强,能让对方理解起来更容易 3.命名需要更准确。
总体来讲我和Pair虽然全程没有讲话,但基本上还是做到这几点,看来程序员之间还是比较默契的_,言归正传,这里介绍下在Pair中学到一点小技巧-快速通过用例,考虑到需要突出重点,所以以阶段来介绍,就不stepBystep了
题目还是Game Of Life,
思路大概是这样的
1.以任意一个Cell中心,根据8个邻居状态,判断该Cell下一个状态:如果2个活着那么保持状态不变,3个邻居活者也为活,其他情况都是死。
2.边界判定,在4条边上的Cell,邻居就不是8个而是3或者5个,需要判定边界,但判断边界的代码太不优雅,所以在原有“棋盘”上包一圈死的Cell(全是0,计算时就不会影响结果),如图:
阶段1:
先来看一下test代码
function test0InitLife(){
//given
var grid = [[0,0,0],[0,0,0],[0,0,0]];
var expected = [[0,0,0],[0,0,0],[0,0,0]];
//when and then
expected(next(grid),expected)
}
function test1InitLife(){
//given
var grid = [[0,0,0],[0,1,0],[0,0,0]];
var expected = [[0,0,0],[0,0,0],[0,0,0]];
//when and then
expected(next(grid),expected)
}
此时的生产代码代码如下:
int[][] grid;
function next(grid){
this.grid = grid;
var newGrid = cloneGrid();
//这里也是一样,判定(1,1)即可满足上面2个测试用例
newGrid[1][1] = getLiveNeigboursCount(1,1,grid[1][1]);
return newGrid;
}
function getLiveNeigboursCount(int i,int j,int currentStatus){
var liveCount = grid[i-1][j] + grid[i-1][j] + grid[i-1][j+1] +grid[i][j-1]
+grid[i][j+1] +grid[i+1][j-1] +grid[i+1][j] +grid[i+][j+] ;
if( liveCount == 2 ){
return currentStatus;
}else if( liveCount == 2 ){
return currentStatus;
}
return 0;
}
function cloneGrid(){
return this.grid;
}
````
在阶段1中,为快速让测试用例通过,cloneGrid方法以及next的判定是特定的,而不是通用的。
###阶段2:
增加一个用例,并且使用轮询的方式判定每个Cell,此时代码肯定不能通过了,并且抛出数组越界异常,因为getLiveNeigboursCount方法中没有判定边界。
增加一个测试用例:
```
function test3InitLife(){
//given
var grid = [[0,0,0],[1,1,1],[0,0,0]];
var expected = [[0,0,0],[0,0,0],[0,0,0]];
//when and then
expected(next(grid),expected)
}
````
next方法轮询判定变成这样
```
function next(grid){
this.grid = grid;
var newGrid = cloneGrid();
for(var i=0;i<newGrid.length;i++){
for(var j=0;j<newGrid[0].length;i++){
newGrid[i][j] = getLiveNeigboursCount(i,j,grid[i][j]);
}
}
return newGrid;
}
```
###阶段3:
根据设计,我们需要在原有棋盘上包装一圈,代码变成这样(这一步稍微快了一些,毕竟之前已经有过4次Pair^_^):
```
var grid;
function next(grid){
this.grid = grid;
var warpperGrid = wrapperGrid();
var newGrid = cloneGrid();
for(var i=0;i<newGrid.length;i++){
for(var j=0;j<newGrid[0].length;i++){
newGrid[i][j] = getLiveNeigboursCount(i+1,j+1,grid[i][j], warpperGrid);
}
}
return newGrid;
}
function getLiveNeigboursCount(int i,int j,int currentStatus, warpperGrid){
var liveCount = wrapperGrid[i-1][j] + wrapperGrid[i-1][j] + wrapperGrid[i-1][j+1] + wrapperGrid[i][j-1] + wrapperGrid[i][j+1] + wrapperGrid[i+1][j-1] + wrapperGrid[i+1][j] + wrapperGrid[i+][j+] ;
if( liveCount == 2 ){
return currentStatus;
}else if( liveCount == 2 ){
return currentStatus;
}
return 0;
}
function cloneGrid(){
var newGrid = new Array();
for(var i=0;i<newGrid.length;i++){
for(var j=0;j<newGrid[0].length;i++){
newGrid[i][j] = this.grid[i][j];
}
}
}
function wapperGrid(){
//TODO
}
```
到这里,貌似其他步骤的代码都已经完成,但是wapperGrid看起来好像很简单,但很难下手,为了让测试快速通过,于是代码变成这样:
```
function wapperGrid(){
var wrapperGrid = new Array();
warpperGrid.push([0,0,0,0,0,0]);
warpperGrid.push([0].concat(this.grid[0]).concat([0]));
warpperGrid.push([[0].concat(this.grid[1]).concat([0]));
warpperGrid.push([[0].concat(this.grid[2]).concat([0]));
warpperGrid.push([0,0,0,0,0,0]);
return warpperGrid;
}
```
这样写好后,逻辑就已经很清楚了^_^
*******
结尾:
在TDD的方法中有一条就是"写刚好通过测试用例的实现代码",虽然本次TDD没有严格按照TDD方法做,但我觉得在阶段1中的next方法,以及阶段3中的wapperGrid方法,也是一种合理的方式可以运用。