Enhance test readability by renaming couple variables
[?]
Dec 28, 2020, 7:02 PM
ZAZCOSBXPGEJ3VAG57UR67XKY7QI4XMMHIIAQ4W2WLSWAOE4LZXACDependencies
- [2]
2DRAL6A2Refactoring evaluate_test - [3]
G5KXCVILtictactoe: initial import - [4]
HSJSKGY7Refactoring evaluate functions - [5]
TYELLGSARename checkDiag to checkDiagLR and add checkDiagRL. Add more tests
Change contents
- replacement in snippets/tictactoe/tictactoe.go at line 15
Empty rune = '_'Pl1 rune = 'x'Pl2 rune = 'o'Emp rune = '_'P_1 rune = 'x'P_2 rune = 'o' - replacement in snippets/tictactoe/evaluate_test.go at line 16
{"Nothing", tictactoe.Board{{tictactoe.Pl1, tictactoe.Pl1, tictactoe.Pl2}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Empty}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Empty}}, 0},{"tictactoe.Pl1 row", tictactoe.Board{{tictactoe.Pl1, tictactoe.Pl1, tictactoe.Pl1}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Empty}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Empty}}, 10},{"tictactoe.Pl2 col", tictactoe.Board{{tictactoe.Pl2, tictactoe.Empty, tictactoe.Empty}, {tictactoe.Pl2, tictactoe.Empty, tictactoe.Empty}, {tictactoe.Pl2, tictactoe.Empty, tictactoe.Empty}}, -10},{"tictactoe.Pl1 last col", tictactoe.Board{{tictactoe.Empty, tictactoe.Empty, tictactoe.Pl1}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Pl1}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Pl1}}, 10},{"tictactoe.Pl2 diag lr", tictactoe.Board{{tictactoe.Pl2, tictactoe.Empty, tictactoe.Empty}, {tictactoe.Empty, tictactoe.Pl2, tictactoe.Empty}, {tictactoe.Empty, tictactoe.Empty, tictactoe.Pl2}}, -10},{"tictactoe.Pl1 diag rl", tictactoe.Board{{tictactoe.Empty, tictactoe.Empty, tictactoe.Pl1}, {tictactoe.Empty, tictactoe.Pl1, tictactoe.Empty}, {tictactoe.Pl1, tictactoe.Empty, tictactoe.Empty}}, 10},{"Nothing", tictactoe.Board{{tictactoe.P_1, tictactoe.P_1, tictactoe.P_2},{tictactoe.Emp, tictactoe.Emp, tictactoe.Emp},{tictactoe.Emp, tictactoe.Emp, tictactoe.Emp}},0},{"tictactoe.P_1 row", tictactoe.Board{{tictactoe.P_1, tictactoe.P_1, tictactoe.P_1},{tictactoe.Emp, tictactoe.Emp, tictactoe.Emp},{tictactoe.Emp, tictactoe.Emp, tictactoe.Emp}},10},{"tictactoe.P_2 col", tictactoe.Board{{tictactoe.P_2, tictactoe.Emp, tictactoe.Emp},{tictactoe.P_2, tictactoe.Emp, tictactoe.Emp},{tictactoe.P_2, tictactoe.Emp, tictactoe.Emp}},-10},{"tictactoe.P_1 last col", tictactoe.Board{{tictactoe.Emp, tictactoe.Emp, tictactoe.P_1},{tictactoe.Emp, tictactoe.Emp, tictactoe.P_1},{tictactoe.Emp, tictactoe.Emp, tictactoe.P_1}},10},{"tictactoe.P_2 diag lr", tictactoe.Board{{tictactoe.P_2, tictactoe.Emp, tictactoe.Emp},{tictactoe.Emp, tictactoe.P_2, tictactoe.Emp},{tictactoe.Emp, tictactoe.Emp, tictactoe.P_2}},-10},{"tictactoe.P_1 diag rl", tictactoe.Board{{tictactoe.Emp, tictactoe.Emp, tictactoe.P_1},{tictactoe.Emp, tictactoe.P_1, tictactoe.Emp},{tictactoe.P_1, tictactoe.Emp, tictactoe.Emp}},10}, - replacement in snippets/tictactoe/evaluate.go at line 7
if b.checkRow(row, col, WinSize, Pl1) == WinSize {// Skip empty!// Skip when remaining length < winsize, so no way to winif b.checkRow(row, col, WinSize, P_1) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 11
} else if b.checkRow(row, col, WinSize, Pl2) == WinSize {} else if b.checkRow(row, col, WinSize, P_2) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 14
if b.checkCol(row, col, WinSize, Pl1) == WinSize {if b.checkCol(row, col, WinSize, P_1) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 16
} else if b.checkCol(row, col, WinSize, Pl2) == WinSize {} else if b.checkCol(row, col, WinSize, P_2) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 19
if b.checkDiagLR(row, col, WinSize, Pl1) == WinSize {if b.checkDiagLR(row, col, WinSize, P_1) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 21
} else if b.checkDiagLR(row, col, WinSize, Pl2) == WinSize {} else if b.checkDiagLR(row, col, WinSize, P_2) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 24
if b.checkDiagRL(row, col, WinSize, Pl1) == WinSize {if b.checkDiagRL(row, col, WinSize, P_1) == WinSize { - replacement in snippets/tictactoe/evaluate.go at line 26
} else if b.checkDiagRL(row, col, WinSize, Pl2) == WinSize {} else if b.checkDiagRL(row, col, WinSize, P_2) == WinSize {