Schritt 1
den Code sauber einrücken. Ich empfehle 3 Blanks pro Einrückung.
Schritt 2
Code entflechten. Da passiert einfach zu viel auf einmal.
Man kann z.B. diesen ganzen Block in eine eigene Unterprozedur verschieben:
Delphi-Quellcode:
procedure TviewWizWerbemittelVersandDetails.CreateColumns;
var
lCol: Integer;
begin
CheckClose(qryWerbemittel);
CheckOpen(qryWerbemittel);
while not (qryWerbemittel.Eof) do
begin
//Spalte erzeugen
lCol := grdDetails.Cells.AddColumn;
//es gibt wohl keine Möglichkeit, Daten in einer Header-Zelle zu speichern, also
//speichern wird die WEM-IDs in einer Liste
FWEMList.Add(TFloatClass.Create(qryWerbemittelwem_id.AsFloat));
grdDetails.cells.Columns[lCol].Caption := qryWerbemittelwem_display_name.AsString;
grdDetails.cells.Columns[lCol].Tag := qryWerbemittelwem_verfuegbar.AsInteger;
qryWerbemittel.Next;
end;
CheckClose(qryWerbemittel);
end;
Du kannst ruhig mehrere Struktogramme zeichnen.
Wenn du nur diesen Code betrachtest, dann hast du am Anfang einen Block ("qryWerbemittel neu öffnen") gefolgt von einer
abweisenden (vorprüfende – kopfgesteuerte) Schleife.
In der Schleife befindet sich ein Block mit mehreren Anweisungen.
Nach der Schleife kommt noch ein Block ("qryWerbemittel schliesen").
Schritt 3
es ist empfehlenswert die Struktogramme zuerst mit Bleistift auf ein Papier zu bringen.
Damit bist du 10 Mal schneller als wie mit einem Tool.
Pro Prozedur ein Struktogramm auf einem DIN A4 Blatt!
Schritt 4
Wiederholungen durch Hilfsprozedur/funktion ersetzen:
Delphi-Quellcode:
procedure TviewWizWerbemittelVersandDetails.MakeCellEditable(col, row:Integer; EditFlag:Boolean);
begin
grdDetails.Cells[Col, Row].CanEdit := EditFlag;
grdDetails.Cells[Col, Row].CanEnter := EditFlag;
end;
Es gibt 4 Stellen, an denen du diese Prozedur aufrufen kannst.