Warnungen und Hints

[OlafSt]

Ich persönlich lasse keine von mir geschriebene Software aus dem Haus, das nicht 100% ohne Warnungen und Hints compiliert (Fremd-Libs mal ausgenommen, da wird noch massig geschlampt). Allerdings ist das "geh mal durch den Source und fixe das alles - es lohnt sich" auch nicht immer der Weisheit letzter Schluß.

Eines meiner übernommenen Alt-Projekte ist so ein Fall, da hagelt es Warnungen und Hints. Okay, keine 10k davon, aber 500-600 sind es schon. Ich habe in einer Unit angefangen, das zu beheben und dabei haarsträubendes zu Gesicht bekommen... Aber egal. Dummerweise funktionierte die Software in einzelnen Teilen bei einigen Kunden schlagartig nicht mehr :(

Hier wurden die klaren Unsauberkeiten übernommen, stillschweigend akzeptiert, was drumrum gebastelt und damit zu Features, die im Laufe der Jahre zu noch besseren Features erweitert wurden - ihr kennt das Prozedere.

Meine Aufräumarbeiten zerlegten damit einige "Features" und ich war nie glücklicher über ein funktionierendes SVN ;)

[Stevie]

Tja, nicht umsonst schreiben Leute Bücher darüber, wie man mit solchem Code umgeht ;)

[Ralle1]

Ich unterscheide zwischen Code, der einfach nur läuft und praktisch nie verändert wird und Code an dem regelmäßig gearbeitet wird. Ersteres soll keine Warnungen mehr bringen, weil die Zeit fehlt sie zu bearbeiten und weil "never touch the running....". An Code der ständig angefasst wird soll die Compilerdirektive beim nächsten Anfassen der Unit wieder raus genommen werden. Dann hat man eine überschaubare Menge an alten Warnungen, die man im Alltag mit ständig fehlender Zeit mal eben weg machen kann.

Weiter oben hieß es {$WARNINGS OFF} {$HINTS OFF} gilt immer für die ganze Unit. Kann man eine Warnungen für eine einzelne Methode oder Zeile unterdrücken? Es gibt ja immer wieder diese Stellen, wo man dann etwas macht weil es aus irgendeinem Grund so sein muss...

[Stevie]

Zitat:

Zitat von Ralle1 (Beitrag 1275201)

Weiter oben hieß es {$WARNINGS OFF} {$HINTS OFF} gilt immer für die ganze Unit. Kann man eine Warnungen für eine einzelne Methode oder Zeile unterdrücken? Es gibt ja immer wieder diese Stellen, wo man dann etwas macht weil es aus irgendeinem Grund so sein muss...

Klar, einfach wieder anschalten nach der Methode/Zeile :)

[Ralle1]

Ah das ist ja einfach :wink: :thumb:
Vielen Dank!!

[Dejan Vu]

Zitat:

Zitat von OlafSt (Beitrag 1275192)

Eines meiner übernommenen Alt-Projekte ist so ein Fall, da hagelt es Warnungen und Hints. Okay, keine 10k davon, aber 500-600 sind es schon. Ich habe in einer Unit angefangen, das zu beheben und dabei haarsträubendes zu Gesicht bekommen... Aber egal. Dummerweise funktionierte die Software in einzelnen Teilen bei einigen Kunden schlagartig nicht mehr :(

Hier wurden die klaren Unsauberkeiten übernommen, stillschweigend akzeptiert, was drumrum gebastelt und damit zu Features, die im Laufe der Jahre zu noch besseren Features erweitert wurden - ihr kennt das Prozedere.

Ein, zwei Beispiele / Anekdoten wären klasse.

Mir ist gerade sowas eingefallen:

Delphi-Quellcode:

Function RandomishFoolishness:Integer;
Var
  foo : Integer;

Begin
  Result := Foo;
End;

K.a. ob das auch ohne Foo funktioniert.

[Bernhard Geyer]

Zitat:

Zitat von OlafSt (Beitrag 1275192)

Eines meiner übernommenen Alt-Projekte ist so ein Fall, da hagelt es Warnungen und Hints. Okay, keine 10k davon, aber 500-600 sind es schon. Ich habe in einer Unit angefangen, das zu beheben und dabei haarsträubendes zu Gesicht bekommen... Aber egal. Dummerweise funktionierte die Software in einzelnen Teilen bei einigen Kunden schlagartig nicht mehr :(

Hier wurden die klaren Unsauberkeiten übernommen, stillschweigend akzeptiert, was drumrum gebastelt und damit zu Features, die im Laufe der Jahre zu noch besseren Features erweitert wurden - ihr kennt das Prozedere.

Dieser Fall i.m.E. ein 1a Fall von Lava Flow