Eure besten Quellcode Kommentare...

[MaBuSE]

Zitat von Mavarik:

Unter dem Strich hast Du natürlich zu 100% Recht. Also gibt es "eigentlich" nix was man hierzu erwidern kann

Das lass ich jetzt mal so stehen ;)

Zitat von Mavarik:

Die eigentlich Frage lautet : Was ist Dir wichtiger...?
...
Es gibt ein Softwareupdate das im Footer die Seitenanzahl ausgeben kann. In diese Routine hat sich ein Fehler eingeschlichen.
Dadurch wird die Seitenzahl nicht eingedruckt. Der try except end Abfänger verhindert eine Fehlermeldung...

Mir wäre wichtiger, dass der Fehler schnellstmöglich gefunden und behoben wird, statt dass sich viele nicht gefundene Fehler anhäufen und nie von dem Entwickler behoben werden, da er nichts davon weiß.

Wenn Du nicht in der Lage bist Deine Software (wenigstens einigermaßen) zu testen, solltest Du von Deinen 5000 Kunden 25 fragen, ob sie an einen Beta Test teilnehmen möchten.
Im Beta Test kannst du schnell neue Versionen ausrollen und die Kunden sind oft dankbar, das sie schon die neuen Funktionen nutzen können.

Langfristig hast Du aber viel mehr zufriedene Kunden (, bei denen die Seitenzahlen richtig geduckt werden;)
Und diese Kunden haben dann auch zufriedene Kunden, da auf ihren Schreiben keine falschen Seitenzahlen gedruckt wurden. ;)


Verstehe mich nicht falsch. Es spricht nichts dagegen Fehler zu klassifizieren und "unwichtige" Fehler "nur" zu loggen und den Anwender nicht damit zu belästigen.
Aber durch das fehlende on E: Fehlerklasse oder das böse try ... except end; darauf zu verzichten nicht klassifizierte also unbekannte Fehler zu entdecken, finde ich fraglich.

Ich möchte beim Autofahren auch erfahren, wenn ein unbekanntes Problem bei meinen Bremsen auftritt.
Unwichtige "bekannte" also klassifizierte Fehler interessieren mich hingegen nicht.
Das erwarte ich von den Programmierern der Software in meinem Auto.

[Mavarik]

Zitat von MaBuSE:

Ich möchte beim Autofahren auch erfahren, wenn ein unbekanntes Problem bei meinen Bremsen auftritt.
Unwichtige "bekannte" also klassifizierte Fehler interessieren mich hingegen nicht.

Möchtest Du auch, dass durch einen Fehler im "Bremsen Sensor abfrage Programm" Dein Auto jedes mal stehen bleibt und der Motor aus geht?

Nee, Spaß beiseite - schon klar worum es Dir geht... (Ist auch der falsche Thread für diese Diskussion)

[MaBuSE]

Zitat von Mavarik:

Nee, Spaß beiseite - schon klar worum es Dir geht... (Ist auch der falsche Thread für diese Diskussion)

Jetzt kannich schreiben:
Unter dem Strich hast Du natürlich zu 100% Recht. Also gibt es "eigentlich" nix was man hierzu erwidern kann

[MaBuSE]

Zitat von Relicted:

Delphi-Quellcode:

  //...
  except
    // Fehler nur schlucken!
    // Anmerkung: JAAAAA SCHLUCK DU LUDER :-))
  end;

Das war übrigens aus dem 1. Beitrag in diesem Thema

[edit]
In 9 Tagen ist das 10 Jahre her !!!
Der 1. Beitrag in diesem Thema ist vom 20. Jul 2007
[/edit]

[Der schöne Günther]

Zitat von Mavarik:

Möchtest Du auch, dass durch einen Fehler im "Bremsen Sensor abfrage Programm" Dein Auto jedes mal stehen bleibt und der Motor aus geht?

Ja !?!

Delphi-Quellcode:

try
   if benutzerMöchteBremsen() then
      zieheBremsenAn();
except
   // Gibt schlimmeres
end;

[himitsu]

Delphi-Quellcode:

if BenutzerMöchteBremsen then
  ZieheBremsenAn;
// wenn's nicht klappt ... k.A. was passiert, aber der Benutzer bekommt wenigstens erstmal sofort mit, daß jetzt doch nicht gebremst wird und kann anders drauf reagieren (rausspringen)


try
  if BenutzerMöchteBremsen then
    ZieheBremsenAn;
except
  NotfallAnhalteProzedur;  // Anker abwerfen
end;

Das Microsoft-Auto der Zukunft: "Ein Unfall wurde erkannt, wollen sie, dass der Airbag jetzt auf geht?"

[Luckie]

Ihr werdet offtopic! Bitte diskutiert die Fehlerbehandlung mit try-ecept in einem eigenständigen Thread.

[Der schöne Günther]

Irgendwie liest sich

Delphi-Quellcode:

(...)
   else
      Result := Succ(self);

schon etwas komisch

[Glados]

Das ist zwar kein Kommentar, aber auch nicht schlecht:

ShowMessage('Ups! An error occured!' + sLineBreak + 'The developer didn''t remove some obsolete code.' + sLineBreak + 'Maybe you let him know about this mistake? =)');

[himitsu]

ShowMessage-Fehlerbehandlung?

raise EProgrammerNotFound.Create('Ups! An error occured!' + sLineBreak + 'The developer didn''t remove some obsolete code.' + sLineBreak + 'Maybe you let him know about this mistake? =)');
Wenn Delphi schon einen passenden Typ mitbringt, dann wäre es doch schön, wenn er öfters auch benutzt wird. (ich hab auch das Gefühl, dass Viele ihn nichtmal kennen)



Ich hatte auch mal 'nen Code in unserem Programm, der sah etwa so aus

Delphi-Quellcode:

else
  raise EProgrammerNotFound.Create('Das sollte eigentlich niemand sehen, ....');

Nja, es sah dann doch einer der Kunden und guckte nicht schlecht.
Bei einem späteren Umbau wurde was vergessen und es sprang dann doch da rein, nach nichtmal 3 Monaten.