erget / kingsnake Goto Github PK

Needs temporal tracking

Pickling previous moves makes sequential rollback to the first move possible, because each previous move also knows what move came before that. But pickling is making performance problems after just a few moves because so much data is being stored.

Alternatives:

• Pickle and unpickle using custom pickling?
• Serialize with JSON with a custom deserializer?

No matter what I do, the rollback method should move to the chessboard, rather than staying with the player. If I'm saving all this data anyway I might as well keep it where it makes sense. Before, the player had been keeping everything until a move was finalized and rolling back manually, but since that's not happening any more I'm moving things.

Only applies to black pawn, white pawn transforms.

Easy solution: Let show() take an optional string argument that it prints before the prompt.

When king castles, rook is not moved. Also, the rule isn't implemented correctly: The king is not allowed to castle if the rook is threatened, although that should actually be legal.