
<html>

<body>

At 02:14 PM 10/24/2007, bob gailer wrote:<br>

<blockquote type=cite class=cite cite="">Dick Moores wrote:<br>

<blockquote type=cite class=cite cite="">Please give me constructive

criticism of

&lt;<a href="http://www.rcblue.com/Python/stopWatchForWeb14-a.py" eudora="autourl">

http://www.rcblue.com/Python/stopWatchForWeb14-a.py</a>&gt;<br>

&nbsp; </blockquote>1 - lapCounter is not used.</blockquote><br>

Deleted.<br><br>

<br>

<blockquote type=cite class=cite cite="">2 - simplify:<br><br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

if lapTimeFlag == False:<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;Total Time is&quot;, secsToHMS(totalTime)<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;StopWatch is continuing to time..&quot;<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

else:<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

currentLapTime = markTime - lapEnd<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;Current time of this lap, Lap %d, is %s&quot; % (lapNum,

secsToHMS(currentLapTime))<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;Total Time is&quot;, secsToHMS(totalTime)<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;StopWatch is continuing to time..&quot;<br><br>

&nbsp;&nbsp; new version:<br><br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

if lapTimeFlag:<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

currentLapTime = markTime - lapEnd<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;Current time of this lap, Lap %d, is %s&quot; % (lapNum,

secsToHMS(currentLapTime))<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;Total Time is&quot;, secsToHMS(totalTime)<br>

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;

print &quot;StopWatch is continuing to time..&quot;</blockquote><br>

Thanks. Missed that. <br><br>

I'm wondering about &quot;if lapTimeFlag:&quot;. Following &quot;The Zen

of Python&quot;, by Tim Peters (&quot;Explicit is better than

implicit&quot;), isn't &quot;if lapTimeFlag == True&quot;

preferable?<br><br>

See the new revision

&lt;<a href="http://www.rcblue.com/Python/stopWatchForWeb14-b.py" eudora="autourl">

http://www.rcblue.com/Python/stopWatchForWeb14-b.py</a>&gt;<br><br>

Dick<br>

</body>

</html>