Zitat von
Jens Schumann:
Der Code verstößt in allen Punkten gegen die Styleguides in Luckie's Artikel
CodeDesign
Die Styleguides habe ich noch nicht gelesen, werde ich nachholen.
Zitat von
Jens Schumann:
Globale Variablen sind extrem schlechter Stil und sind unbedingt zu vermeiden.
Was ist daran so schlecht? Oder steht das auch in den Styleguides?
Zitat von
Jens Schumann:
Berechnung ist mit der Benutzeroberfläche verbunden. Evt wäre eine function, die ein dem Ereignis entsprechendes Ergebnis zurück liefert besser.
Könntest du das vielleicht noch ein bisschen erläutern und sagen, was an meiner Variante nicht so gut ist?
Zitat von
Jens Schumann:
Delphi-Quellcode:
for i := 0
to breite
do
for j := 0
to hoehe
do
if map[i, j]>0
then
Dec(map[i, j]);
Wenn man lange genug spielt könnte map[i,j] evt. den gültigen Bereich eines Integers verlassen !!! Das kann zu schwer auffindbaren Fehlern führen.
Das verstehe ich nun nicht. Wann sollte map[i, j] den gültigen Integerbereich verlassen? Der Wert kann nicht kleiner null werden. Und in der höhe ist er auch begrenzt, da das Spielfeld nach einiger Zeit zu voll ist, um weitere Punkte zu bekommen, also die Länge zu erhöhen.
MfG Pr0g