Warnungen und Hints

[Stevie]

Zitat von Ralle1:

An alle die vor dem Abschalten der Warnungen warnen: nur so kann ich wieder sinnvoll auf Warnungen eingehen. Wenn es unzählige sind, ist es utopisch zu glauben, ich könnte sie alle aus der Welt schaffen.

Ich kann dir aus eigener Erfahrung (Source mit ehemals tausendenden Warnings/Hints) sagen, dass es nicht so ist. Man muss sich nur mal dran setzen. Klar, wird man die nicht mal ebend auf null bringen aber ich versprech dir, du wirst den einen oder anderen potenziellen oder gar tatsächlichen Bug damit beheben. Source will gepflegt werden, und das gehört dazu (auch wenn man in ner heilen Welt lieber gleich Source produziert, der keine hat)

[Bernhard Geyer]

Zitat von Stevie:

Ich kann dir aus eigener Erfahrung (Source mit ehemals tausendenden Warnings/Hints) sagen, dass es nicht so ist. Man muss sich nur mal dran setzen. Klar, wird man die nicht mal ebend auf null bringen aber ich versprech dir, du wirst den einen oder anderen potenziellen oder gar tatsächlichen Bug damit beheben. Source will gepflegt werden, und das gehört dazu (auch wenn man in ner heilen Welt lieber gleich Source produziert, der keine hat)

Vor allem bei so vielen Einträgen gehen auch neu Produzierte Warnungen unter.
Denn ob es 100 oder 105 Meldungen sind interessiert dann auch keinen mehr.
Als Entziel sollte sein das auch Warnungen ein Buildprozess zum Abbruch bringen.

[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 von Ralle1:

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
Vielen Dank!!

[Dejan Vu]

Zitat von OlafSt:

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 von OlafSt:

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