[Tutor] Looking for suggestions for improving chessTimer.py code

Dick Moores rdm at rcblue.com
Mon Oct 29 23:14:28 CET 2007


I've developed chessTimer a bit further:
<http://www.rcblue.com/Python/chessTimerForWebV9.py>

I'm still hoping for some constructive criticism. But I also thought
I'd mention again the points I have doubts about:

1. Is wrapping the long lines where I have done so OK?
2. I've used 1 for White player, -1 for Black player, and (-1)*player
to alternate players. Is there a better way?
3. I've used a lot of variables. Too Many?
4. Should more of the code be in functions?
5. Is there a way to collapse lines 130-137?:
if player == 1: # player is White
                 whiteMoveCounter += 1
                 print "Black to make move %d" % (blackMoveCounter)
                 remainingWhiteTime -= timeUsedThisMove
elif player == -1: # player is Black
                 blackMoveCounter += 1
                 print "White to make move %d" % (whiteMoveCounter)
                 remainingBlackTime -= timeUsedThisMove

6. I thought I had a way to make this script useable on unix as well 
as Windows. Thus the section with the 3 classes. But it won't run on 
unix, because it was necessary to import msvcrt outside of the 
classes--it wouldn't compile otherwise, and because of the need for 
the line  'if msvcrt.kbhit():'  (line 116).  I hope I'm wrong about 
this, and someone can show me how to fix it so that the unix people 
can give me some criticism/advice as well.

Thanks,

Dick Moores



More information about the Tutor mailing list