Code Smells

[s.h.a.r.k]

Zitat:

Zitat von EWeiss (Beitrag 1110452)

Außerdem habe ich die Erfahrung gemacht das sich die nachfolgenden Variablen schlecht Debugen lassen.
Da Sie wegen Optimierung unterdrückt werden.

PS:
Und Zeilen sparrt man sich dadurch auch nicht wirklich.

gruss

Aber Schreibarbeit ;)

[EWeiss]

Zitat:

Zitat von s.h.a.r.k (Beitrag 1110453)

Zitat:

Zitat von EWeiss (Beitrag 1110452)

Außerdem habe ich die Erfahrung gemacht das sich die nachfolgenden Variablen schlecht Debugen lassen.
Da Sie wegen Optimierung unterdrückt werden.

PS:
Und Zeilen sparrt man sich dadurch auch nicht wirklich.

gruss

Aber Schreibarbeit ;)

Ahh jo Copy und Paste ... hehehehe :)

gruss

[Meflin]

Zitat:

Zitat von EWeiss (Beitrag 1110454)

Ahh jo Copy und Paste...

... ist sehr oft ein Zeichen dafür, dass ein Smell vorliegt :P

[Stevie]

Die "Notwendigkeit" für ein with ist wahrscheinlich Feature Envy, Indecent Exposure und/oder Message Chains

[FredlFesl]

Zitat:

Zitat von s.h.a.r.k (Beitrag 1110440)

Was ist an with so böse?

Zitat:

Zitat von Luckie (Beitrag 1110446)

...Und mitunter kommt der Compiler schon mal durcheinander...

Ganz sicher nicht. Der Compiler ist deterministisch, da der die Grammatik zumindest hier formal korrekt interpretiert.

Zitat:

Zitat von Stevie (Beitrag 1110457)

Die "Notwendigkeit" für ein with ist wahrscheinlich Feature Envy, Indecent Exposure und/oder Message Chains

So hab ich das noch nicht gesehen. Aber stimmt: Verwender des 'WITH' müssen nur ihren Code aufräumen (zur Not mit Class Helper), dann wird das 'WITH' meist überflüssig.

Mein Code riecht immer dann gut, wenn ich keine Kommentare benötige, da der Code flüssig zu lesen ist (wie eine Beschreibung) und einfach zu verstehen ist.
Weiterhin spüre ich, das ich in die richtige Richtung gehe, wenn die Klassen mit Scheuklappen durch die Gegend rennen, d.h. keine Abhängigkeiten zu anderen Klassen existieren. Dann kann ich mit den Klassen vieles anstellen, ohne mir einen abzubrechen.

Eine schöne Beschreibung für eine gut implementierte Methode ist von Dr.Bob: "Wenn Du das, was die Methode macht, beschreibst, und ohne das Wörtchen 'und' auskommst, hast Du etwas richtig gemacht".

[Deep-Sea]

Uhu, bricht hier wieder mal ein With-Glaubenskrieg aus? *schon mal das Popcorn hol* :stupid:

[DeddyH]

Delphi-Quellcode:

with Popcorn do
  Mampf;

:mrgreen:

[Coffeecoder]

Hey,

Ich danke für eure Feedbacks. Es ist mir klar, dass es schon bisschen her ist.

Zitat:

Zitat von MrOuzo (Beitrag 1110266)

Hallo Coffeecoder,

gefällt mir ganz gut, fällst bei deinem Beispiel dann über die eigenen Regeln.

ich denke

Delphi-Quellcode:

procedure anpassenChart(ch : TChart);
Begin
end;

hier ist ch auch nicht gerade ein aussagekräftiger Parameter, könnte auch ein Char oder so sein :-D

Gruß
MrOuzo

Das stimmt da falle ich selbst über die "Anti-Code-Smells"-Regeln. Besser soll heissen, wie bereits erwähnt:

Delphi-Quellcode:

procedure anpassenChart(pchart : TChart);
Begin
    ....
end;

Weiter gehts :)

Zitat:

Zitat von Luckie (Beitrag 1110268)

Außerdem wird hier als Parameter eine Zeichenkette übergeben:

Delphi-Quellcode:

anpassenChart('Chart'+IntToStr(i));

laut Deklaration müsste es aber ein Objekt vom Typ TChart sein. Mir ist zwar klar, was du aussagen willst, aber trotzdem sollte es doch ein Beispiel sein, was zumindest compiliert.

Da habe ich wieder nicht aufgepasst :?

Also ich bedanke mich für eure Hinweise und Vorschläge, ich werde das Tutorial wohl einmal überarbeiten müssen und gebe es als PDF auch frei.

Bis dahin, habt bisschen Geduld :)

[DeddyH]

Zitat:

Zitat von Coffeecoder (Beitrag 1145315)

Delphi-Quellcode:

procedure anpassenChart(pchart : TChart);
Begin
    ....
end;

Der Präfix P erweckt aber eher den Eindruck, es handle sich um einen Zeiger auf ein TChart. Wieso nicht einfach Chart, AChart oder meinetwegen TheChart?

[Coffeecoder]

Zitat:

Zitat von DeddyH (Beitrag 1145316)

Zitat:

Zitat von Coffeecoder (Beitrag 1145315)

Delphi-Quellcode:

procedure anpassenChart(pchart : TChart);
Begin
    ....
end;

Der Präfix P erweckt aber eher den Eindruck, es handle sich um einen Zeiger auf ein TChart. Wieso nicht einfach Chart, AChart oder meinetwegen TheChart?

Hmm ok. Ich habe gerade diesen Beitrag hier in der DP gefunden :)