carson-day / decaf-chess-interface

Automatically exported from code.google.com/p/decaf-chess-interface
1 stars 0 forks source link

Two subsequent half-moves with the same notation displayed as a single half-move in the move list #49

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

1. Try to play the following game using decaf:
1. c4 f5 2. Nf3 Nf6 3. g3 g6 4. Bg2 Bg7 5. O-O O-O 6. d4 d6
7. Qb3 Kh8 8. Nc3 c6 9. a4 a5 10. d5 c5 11. Nb5 Nbd7 12. Ng5 Ne5
13. Rd1 h6 14. Ne6 Bxe6 15. dxe6 Rb8 16. Be3 b6 17. h3 g5
18. Qc2 Qc8 19. f4 Nc6 20. Qxf5 Nh5 21. Qg4 gxf4 22. gxf4 Nf6

2. Pay attention to what happens with the move list after 5.O-O O-O 6.d4
and after 21. ... gxf4 22. gxf4 Nf6

What is the expected output? What do you see instead?

I expect that the O-O and gxf4 moves should be shown for both players who
made them, but the second consecutive occurence of a move with the same
algebraic notation is not shown. Instead the next move of the other player
is displayed.

What version of the product are you using? On what operating system?
Decaf 1.1.1 on Mac OS X 10.5.7.

Please provide any additional information below.
This is repeatable, at least with the above move sequence.

Original issue reported on code.google.com by ltdun...@gmail.com on 24 Jul 2009 at 10:03

GoogleCodeExporter commented 8 years ago
I may be wrong but at the first glance it seems the problem may be caused by
ChessAreaControllerBase.java at lines 465-466, when the appendMove is not 
called when
lastMove.equals(move). If such a kind of condition is really necessary to avoid 
some
repeating, perhaps compare the previous and current Positions, instead of 
moves?...

Original comment by ltdun...@gmail.com on 24 Jul 2009 at 10:36

GoogleCodeExporter commented 8 years ago
ChessAreaControllerBase is a nightmare i hated it and rewrote all this in 
raptor.
Might want to give that a try. All of this is fixed in it.

Original comment by carsonday on 11 Nov 2009 at 8:55