array[1..10] auf Wert prüfen?

[Delphi.Narium]

Wenn mir der Compiler sagt, dass eine Variabel, ein Rückgabewert ... eventuell nicht initialisiert sein könnte, dass irgendwas undefiniert sein könnte ..., dann wird das behoben.

Ohne Wenn und Aber.

Zitat von Sherlock:

Ein Entwickler, der einen Fall ignoriert, dessen Auftreten explizit erwähnt wird, sollte den Beruf wechseln.

Dem ist nichts hinzuzufügen.

Und wenn ein (semiprofessioneller) Hoppyprogrammierer ein Problem mit seinem Quelltext hat und hat nicht alle diese Warnungen (durch entsprechende Anpassung seines Quelltextes) behoben, sollte nicht wegen undefinierter Zustände, Zugriffverletzungen ... in irgendeinem Forum nachfragen, wo denn die Fehlerursache liegen könnte.

[hoika]

Hallo,
also ich habe Warnungen drin und das Programm stürzt nicht ab.

Also ist doch alles gut?

PS:

Zitat:

Habe schon unzählbare Warnungen dieser Art korrigiert und jedes einzelne Mal hatte der Compiler Recht und die Variable WAR nicht initialisiert.

Ich habe hier eine Warnung, da vertut sich der Compiler definitiv.
Ich kann das aber hier leider nicht posten.
Das ist aber die absolute Ausnahme.

[freimatz]

Zitat von hoika:

Also ist doch alles gut?

Nein. (Und nein, kein )

Und obwohl mir das Clean-Code-Pprinzip DRY sehr wichtig ist:

Zitat von Delphi.Narium:

Wenn mir der Compiler sagt, dass eine Variabel, ein Rückgabewert ... eventuell nicht initialisiert sein könnte, dass irgendwas undefiniert sein könnte ..., dann wird das behoben.

Ohne Wenn und Aber.

Zitat von Sherlock:

Ein Entwickler, der einen Fall ignoriert, dessen Auftreten explizit erwähnt wird, sollte den Beruf wechseln.

Dem ist nichts hinzuzufügen.

Und wenn ein (semiprofessioneller) Hoppyprogrammierer ein Problem mit seinem Quelltext hat und hat nicht alle diese Warnungen (durch entsprechende Anpassung seines Quelltextes) behoben, sollte nicht wegen undefinierter Zustände, Zugriffverletzungen ... in irgendeinem Forum nachfragen, wo denn die Fehlerursache liegen könnte.

Ja, es gibt Fälle wo der Compiler nicht wissen kann, dass die variable initialisert ist. Das beduet aber dann nur, dass der Code nicht sauber ist. Im Notfall initialisiert man halt einmal zuviel.
Denn: jede Warnung die man ignorieren kann führt dazu, dass man eher eine Warnung übersieht, die wichtig ist.

[Neutral General]

Zitat von hoika:

Hallo,
also ich habe Warnungen drin und das Programm stürzt nicht ab.

Also ist doch alles gut?

Du hast/hattest Glück.
Je nachdem was du machst oder falls du vllt den Code veränderst und sich dadurch zur Laufzeit der Inhalt des Stacks leicht verändert, dann hast du ggf. den Salat.
Deswegen gibt es keinen Grund solche Warnungen nicht ernst zu nehmen und die 10 Sekunden aufzubringen das Problem zu beheben.

Zitat von hoika:

Ich habe hier eine Warnung, da vertut sich der Compiler definitiv.
Ich kann das aber hier leider nicht posten.
Das ist aber die absolute Ausnahme.

Poste doch ein analoges Beispiel.
Bin mir fast sicher, dass der Compiler Recht hat

[himitsu]

Ich hatte gestern bei uns eine Funktion gefunden, da stand ganz am Ende etwas wie das

Delphi-Quellcode:

if xxx > 0 then
  zzz.FieldByName('fff').AsInsteger := xxx;

Früher war mal oben dem xxx was zugewiesen, aber das wurde gelöscht.

Dieses lokale xxx ist nicht initialisiert und kann potentiell jeden beliebigen Wert enthalten.
Zufällig war es zum glück lange Zeit 0 und es passierte nichts.

Tja, aber nun knallt es immernoch nicht, selbst wenn es plötzlich nicht mehr 0 ist ... im DataSet wird da "nur" das Feld mit "irgendeinem" beliebigen Wert überschieben.

[freimatz]

So aus dem Gedächtis war das so etwas in der Art:

Delphi-Quellcode:

var bla: Integer;  f:Boolean;
begin
  f := TrittDerFallXEin;
  if f then begin
     i := HoleDenIWert();
  end;
  MacheWasAnders();
  if f then begin
     MachWas(i);
  end;
end;

Aber wie gesagt:

Zitat von freimatz:

Ja, es gibt Fälle wo der Compiler nicht wissen kann, dass die variable initialisert ist. Das beduet aber dann nur, dass der Code nicht sauber ist. Im Notfall initialisiert man halt einmal zuviel.
Denn: jede Warnung die man ignorieren kann führt dazu, dass man eher eine Warnung übersieht, die wichtig ist.

[himitsu]

OK, hier stimmt es zufällig, aber irdenwann könnte mal wer auf die Idee kommen und eine der IF-Bedingungen ändern.

Und was spricht nun dagegen sich die 2 Sekunden zu nehmen und ein else i := 0; oder gantz oben einfach ein i := 0 einzufügen?

[Rollo62]

Die zwei Sekunden gehen wohl fürs Posten in DP drauf

WARNINGS: IMMER wie ERROR behandeln
HINTS : IMMER wie ERROR behandeln

Ein Release sollte man immer mit 0 Hints, Warnings, Errors kompilieren können.
So wird das bei mir gnerell als Vorgabe definiert.

Wenn ich Warnings/Hints akzeptiere, dann gibt es dazu immer eine spezielle "Ausnahmegenehigung".

Rollo

[himitsu]

Zitat von Rollo62:

Wenn ich Warnings/Hints akzeptiere, dann gibt es dazu immer eine spezielle "Ausnahmegenehigung".

Wobei man diese Hints/Warnings "nur" an der entsprechenden Stelle dann auch deaktivieren kann.

[DeddyH]

Diese ganzen Kylix-Altlasten "Unit Dingenskirchen ist plattformspezifisch" schalte ich auch immer aus, ansonsten wird jeder Warnung und jedem Hinweis nachgegangen und die Ursache abgestellt.