Hi,<br><br>Sorry for again throwing in my two cents, although not contributing to the actual implementation.<br><br><div class="gmail_quote">2008/5/1 Matt Williams <<a href="mailto:matt@milliams.com">matt@milliams.com</a>>:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d">> - It's missing an "int numberOfScores() const" method. I don't really<br>
> love that name, but I haven't been able to come up with anything much<br>
> better. Maybe "scoreBoardLength"?<br>
<br>
</div>Yes, scoreBoardLength() sounds good. In that case I'll change<br>
setNumberOfScores() to setScoreBoardLength().</blockquote><div><br>I just want to remind you that there is a difference between the number of highscore entries shown in the highscore dialog, and the number of entries stored in the highscore list.<br>
<br>If these two numbers don't match, you may get in trouble:<br>1) display more entries than stored: the lower entries always remain empty<br>2) store more entries than displayed: the player who just won a new low highscore doesn't find himself in the list, and the dialog also does not show a box where he could enter his name<br>
<br>So, I think there is no sense in using a different number of entries for storing and displaying.<br>Furthermore, the number should be fix, i.e., set only once in the program, and never changed. Otherwise you loose highscore entries, or create empty ones.<br>
Therefore, the number of entries is predestined to be set in the constructor of the KScoreManager, isn't it? (Maybe use a default value of 10?)<br><br>Burkhard<br><br></div></div><br>