Filestream Datei nicht gefunden bei FOpenWrite

[Zacherl]

Bei so tiefer Verschachtelung bietet sich übrigens wieder die Methode mit try ..finally Block an, über die wir schonmal in einem anderen Thread geredet hatten. Also Statt:

Delphi-Quellcode:

if (X) then
begin
  if (Y) then
  begin
    if (Z) then
    begin
      ..
      Free(Z);
    end;
    Free(Y);
  end;
  Free(X);
end;

einfach

Delphi-Quellcode:

try
  if (not X) then Exit;
  if (not Y) then Exit;
  if (not Z) then Exit;
  ..
finally
  if (X) then Free(X);
  if (Y) then Free(Y);
  if (Z) then Free(Z);
end;

Vorher natürlich Luckies Tipp befolgen und möglichst viel in logisch sinnvolle Funktionen/Methoden auslagern.

[EWeiss]

Zitat von Luckie:

Nur, wie sieht es mit der Wiederverwertbarkeit aus? Oder du baust die Oberfläche um? Dann musst du alles auseinander zupfeln und wieder neu zusammenflicken. Und wie gesagt, die Länge und die ein Dutzend lokalen Variablen sind das Schlimme. In Funktionen zerlegen (besser Klasse) und in eine separate Unit (Datei) packen.

Da ist nicht viel umzubauen.
Ist ja nur ein kleines Dialog Window.

gruss

[EWeiss]

Zitat von Zacherl:

Bei so tiefer Verschachtelung bietet sich übrigens wieder die Methode mit try ..finally Block an, über die wir schonmal in einem anderen Thread geredet hatten. Also Statt:

Delphi-Quellcode:

if (X) then
begin
  if (Y) then
  begin
    if (Z) then
    begin
      ..
      Free(Z);
    end;
    Free(Y);
  end;
  Free(X);
end;

einfach

Delphi-Quellcode:

try
  if (not X) then Exit;
  if (not Y) then Exit;
  if (not Z) then Exit;
  ..
finally
  if (X) then Free(X);
  if (Y) then Free(Y);
  if (Z) then Free(Z);
end;

Vorher natürlich Luckies Tipp befolgen und möglichst viel in logisch sinnvolle Funktionen/Methoden auslagern.

Auf jeden fall. Ich werde eure Ratschläge beherzigen.
Aber wie gesagt heute morgen war ich erst mal froh das alles funktioniert hat.

gruss

[Luckie]

Zitat von EWeiss:

Da ist nicht viel umzubauen.
Ist ja nur ein kleines Dialog Window.

Ich denke schon etwas weiter in die Zukunft. Vielleicht brauchst du den Code bei einem anderen Projekt oder willst ihn wem anders zur Verfügung stellen?

[EWeiss]

Zitat von Luckie:

Zitat von EWeiss:

Da ist nicht viel umzubauen.
Ist ja nur ein kleines Dialog Window.

Ich denke schon etwas weiter in die Zukunft. Vielleicht brauchst du den Code bei einem anderen Projekt oder willst ihn wem anders zur Verfügung stellen?

Ja logisch hier dafür mache ich doch das ganze.. Hobby

gruss

[Hobbycoder]

Meine Erfahrung mit Umfangreicheren Aufgaben, sind dass ich da den umgekehrten Weg gehe.
Früher habe ich auch erstmal alles der Nase nach runtergeschrieben, und danach in sinnvolle Funktionen und Klassen umgebaut.
Heute mache ich mir schon vorher Gedanken über den Ablauf und mache mir gleich die passenden Funktionen und lege mir vereinfachende Klassen an. Das hat sich bewährt, weil man sich einfach weniger verzettelt, und viele Probleme von Anfang an vermeidet.
Darüber hinaus schreibt man dann den Code oft nur einmal.