Sichtbarkeit Prozduren in Prozeduren

[Bjoerk]

Zitat von Zacherl:

Ist in diesem Kontext aber nicht sehr sinnvoll. Allerhöchstens als private dann, aber lokal und ggfls. mit forward ist dann eventuell doch noch schöner.

Kann sein. Bei Kapselung macht das im Allgemeinen aber so. Man hat ja auch nicht z.B. die strings einer inifile als extra variable und die inifile nur lokal?

[Zacherl]

Zitat von Bjoerk:

Kann sein. Bei Kapselung macht das im Allgemeinen aber so. Man hat ja auch nicht z.B. die strings einer inifile als extra variable und die inifile nur lokal?

Der Vergleich hinkt aber ein wenig, oder? Aber hast natürlich Recht, es kommt immer auf den Kontext an. Hier werden die 2 Funktionen ja nur lokal (innerhalb von "IchBinGanzOben") aufgerufen, deshalb kann man auf jeden Fall schonmal private sagen. Jetzt kommt aber hinzu, dass die Funktionen nur an einer einzigen Stelle (aus "IchBinGanzOben" heraus) aufgerufen werden, weshalb ich hier zu lokalen Deklaration neigen würde. Ist prinzipiell aber sicher auch einfach Geschmackssache.

[nakaramia]

Vielen Dank.

[Furtbichler]

Das riecht eher nach einer eigenen Klasse, als nach Prozeduren, in Prozeduren, die zudem noch so komplex scheinen, das sie sich gegenseitig aufrufen. Muss nicht sein, wäre aber in jeder anderen Sprache ein Fall für eine neue Klasse.

Ich persönlich finde die kleinen Lokalenprozedurhelferlein sehr praktisch, weil das das 'R' in RAD betont. Es ist eben immer noch schneller, eine komplexe Methode in ein paar lokalen Prozeduren zu kapseln und auch einigermaßen lesbar zu machen (wenn man denn das erste Begin der Prozedur überhaupt findet).

Aber 'schneller' ist nicht gleich 'besser'.

Zitat von nakaramia:

...in dem in einer Prozedur ein Dutzend anderer enthalten sind...

Das riecht aber nicht nur nach einer neuen Klasse, es schreit förmlich danach. Die Lesbarkeit wird ja durch die 12 lokalen Prozeduren auch nicht gerade erhöht. Und in der separaten Klasse hast Du auch nicht die Sichtbarkeitsprobleme, weil sich eh jeder kennt.

[Perlsau]

Zitat von Furtbichler:

Es ist eben immer noch schneller, eine komplexe Methode in ein paar lokalen Prozeduren zu kapseln und auch einigermaßen lesbar zu machen (wenn man denn das erste Begin der Prozedur überhaupt findet).

Das wäre wohl das kleinste Problem: einfach mit entsprechendem Kommentar versehen und du findest deine Hauptprocedure sofort:

Delphi-Quellcode:

Procedure Komplex;
// Sub 1 DoSomeThing & DoItAgain
Procedure SubProcedure1;
Begin
  DoSomeThing;
  DoItAgain;
End;

// ********** Hauptprocedure *********************************
Begin
End;

[TiGü]

Zitat von Perlsau:

Zitat von Furtbichler:

Es ist eben immer noch schneller, eine komplexe Methode in ein paar lokalen Prozeduren zu kapseln und auch einigermaßen lesbar zu machen (wenn man denn das erste Begin der Prozedur überhaupt findet).

Das wäre wohl das kleinste Problem: einfach mit entsprechendem Kommentar versehen und du findest deine Hauptprocedure sofort:

Rücke doch einfach vernünftig ein?!

Delphi-Quellcode:

procedure Komplex;

  procedure SubProcedure1;
  begin
    DoSomeThing;
    DoItAgain;
  end;

begin
end;

[Perlsau]

Zitat von TiGü:

Zitat von Perlsau:

Zitat von Furtbichler:

Es ist eben immer noch schneller, eine komplexe Methode in ein paar lokalen Prozeduren zu kapseln und auch einigermaßen lesbar zu machen (wenn man denn das erste Begin der Prozedur überhaupt findet).

Das wäre wohl das kleinste Problem: einfach mit entsprechendem Kommentar versehen und du findest deine Hauptprocedure sofort:

Rücke doch einfach vernünftig ein?!

Sag das doch dem Furtbichler – er (und nicht ich) hatte angemerkt, daß man u.U. das Begin der Hauptprocedure nicht finden könne. Ich selbst hab damit keine Probleme, weshalb es auch vollkommen unnötig ist, mir hier subtil Unvernünftigkeit zu unterstellen. In diesem speziellen Fall ist es völlig wurscht: beides ist vernünftig, da es die Lesbarkeit fördert.