Hallo,
dies soll eine Diskussion starten.
Ich habe den Quelltext einiger plugins angeschaut.
Dort findet man Zeilen wie diese:
Dies ist ganz schlechter Stil.
Jetzt regt euch bitte gleich wieder ab, ich weiss dass viele das Programmieren als Hobby betreiben. Oft ohne es richtig gelernt zu haben. Aber das kann man ja ändern .-)
Zahlen haben im Code nichts zu suchen!
Dafür gibts #define, const int, oder eine Variable.
Dabei ist es wichtig, vernünftige Namen zu verwenden (nicht A, B oder C).
Der Namen soll genau Aussagen um was es sich handelt.
Die 40 im obigen Beispiel ist die Höhe der Zeile im DVD-Menü. Also wäre ein passender Name RowHight. (Und bitte nur Englische Bezeichner! Wir sind schlieslich international) .
Quelltext wird nur einmal geschrieben, aber sehr oft gelesen. Also lohnt es sich, etwas mehr zu schreiben und KURZ zu kommentieren was in etwa eine Funktion macht (Eine Zeile reicht oft aus).
Beispiel:
const int RowHight=40;
const int ButtonPosX=93;
const int ButtonPosY=118;
DrawImage(mJob->GetSkin()->Button(), ButtonPosX, ButtonPosY + i * RowHight);
Dieser Code ist viel "verstehbarer" und änderungsfreundlicher/robuster. Wenn wir die Höhe einer Zeile programmatisch ändern wollen, müssen wir jetzt nur die Definition von RowHight ändern damit es eine Variable wird.
int RowHight=40;
const int ButtonPosX=93;
const int ButtonPosY=118;
.....
RowHight=CalculateRowHight();
.....
DrawImage(mJob->GetSkin()->Button(), ButtonPosX, ButtonPosY + i * RowHight);
Man sieht, es erleichtert die Arbeit wenn man es richtig macht.