Die häufigsten Programmierirrtümer

[s.h.a.r.k]

@sx2008: Ist ein s := '' gleichzusetzen, wie s := nil? Wäre mir definitiv neu, wobei ich nicht weiß, wie das intern gehandelt wird.

Zitat von sx2008:

Delphi-Quellcode:

var
  s : string;
  i, anz_comment : integer;
begin
  // Sinn & Zweck: Zähle alle Zeilen, die mit * beginnen
  anz_comment : = 0;
  for i:=0 to Memo1.Lines.Count-1 do
  begin
    s := Memo1.Lines[i];

    // sollte die Zeile leer sein, also s ='' dann knallts hier richtig
    // es gibt eine Zugriffsverletzung, denn s zeigt auf nil
    // auch die Bereichsprüfung kann hier keinen Schutz bieten
    if s[1] = '*' then
      Inc(anz_comment);
  end;

Hier will ich noch anbringen, dass ich es schon mehrfach gesehen habe, dass bei Schleifen entweder bei 1 angefangen wird und der Zähler dann als Index verwendet wird, wobei dieser dann nicht so verwendet wird Zähler - 1. Beim letzten Element schlägt dies dann folglich fehl. Oder dass zwar bei 0 begonnen wird zu zählen, aber bis (in diesem Fall) Memo1.Lines.Count läuft und nicht Memo1.Lines.Count - 1.

[R2009]

Hi,

aus meinen C Erfahrungen:

if (a = 0)

ist nicht das gleiche wie

if (a == 0)

Das ist ein Fehler der im nachhinein kaum noch zu finden ist.
Compiler meckert nicht. Im ersten Fall wird kein Vergleich durchgeführt sondern a die Null zugewiesen.

Ist zwar kein Fehler aber was ich überhaupt nicht leiden kann ist die C Schreibweise in Pascal:

Delphi-Quellcode:

extrem unschön:
If blabla then begin
...
end
sehr viel übersichtlicher:
If blabla then 
   begin
     ...
   end

Was mich auch nervt ist mein eigener unkommentierter Kode,
den ich dann nach 4 Wochen selbst nicht mehr versteh!

Grüsse
Rainer

[alzaimar]

Zitat von R2009:

extrem unschön:
...
sehr viel übersichtlicher:

der 'extrem unschöne' Code wird von Programmierern, die viel Code lesen, bevorzugt. Das "Begin" auf einer separaten Zeile hat keine eigene Aussage: Das hier bedingter Code folgt, ist durch die Einrückung (horizontale Formatierung) schon hinreichend dokumentiert. Letztendlich ist es jedoch Geschmackssache und muss jeweils Teamweit festgelegt werden. Diskussionen > 5 min sind hier überflüssig. Wir haben das z.B. durch Abstimmung geregelt.

Zitat:

Was mich auch nervt ist mein eigener unkommentierter Kode, den ich dann nach 4 Wochen selbst nicht mehr versteh!

Dann schreib ihn so, das Du (man) ihn versteht. Das geht auch ohne Kommentare. Kommentare sind eh böse.

Was Du aber vielleicht eigentlich meinst, ist:
Ein großer Programmiererirrtum ist es, zu meinen, das Maß aller Dinge sei funktionierender Code.
Das Maß aller Dinge muss lesbarer Code sein (der kann im Nachhinein viel einfacher entwanzt werden).

Das schließt deine oben zitierte Meinung mit ein, hoffe ich.

[R2009]

Hi alzaimar,

Zitat:

der 'extrem unschöne' Code wird von Programmierern, die viel Code lesen, bevorzugt. Das "Begin" auf einer separaten Zeile hat keine eigene Aussage: Das hier bedingter Code folgt, ist durch die Einrückung (horizontale Formatierung) schon hinreichend dokumentiert. Letztendlich ist es jedoch Geschmackssache und muss jeweils Teamweit festgelegt werden. Diskussionen > 5 min sind hier überflüssig. Wir haben das z.B. durch Abstimmung geregelt.

Dies ist ganz einfach meine persönliche Meinung.
Ich will niemand davon überzeugen und ich will auch keine Diskussion darüber anfangen.

Zitat:

der 'extrem unschöne' Code wird von Programmierern, die viel Code lesen, bevorzugt.

das ist eine subjektive Aussage die für unser Umfeld hier nicht stimmt.

Grüsse
Rainer
Ps: konnte das einfach so nicht stehen lassen. Bitte keine Diskussion darüber anfangen!

[Luckie]

Zitat von alzaimar:

Zitat von R2009:

extrem unschön:
...
sehr viel übersichtlicher:

der 'extrem unschöne' Code wird von Programmierern, die viel Code lesen, bevorzugt. Das "Begin" auf einer separaten Zeile hat keine eigene Aussage: Das hier bedingter Code folgt, ist durch die Einrückung (horizontale Formatierung) schon hinreichend dokumentiert.

Was ich mir aber abgewöhnt habe ist folgendes:

Delphi-Quellcode:

if ... then
   ...;  // eine Zeile

stattdessen:

Delphi-Quellcode:

if ... then
begin
  ...; // eine Zeile
end;

Ich fasse auch Rümpfe in begin und end ein, auch wenn sie nur eine Zeile umfassen. Einfach aus dem Grund, weil der Rumpf eventuell später erweitert wird und man dann vergisst das begin und end zu ergänzen. Bei dem daraus resultierenden Fehlverhalten kann man sich manchmal dumm und dämlich suchen. So was vermeide ich somit von vorne herein.

Das mache ich übrigens in allen Programmiersprachen so, sei es in Delphi oder PHP. Ich habe sogar schon Code vom Kollegen dahingehend angepasst.

Was Kommentare angeht: Sie generell als böse zu bezeichnen ist auch nicht ganz richtig, wie eigentlich fast jede Pauschlisierung. Aber wenn man etwas kommentiert, dann höchstens das Warum. Das Wie sehe ich am Code. Denn jemand der den Code liest wird sich meist fragen: Warum macht der das? Wenn dann da steht "Workaround wegen Bug XYZ", dann weiß derjenige Bescheid und ändert die Stelle nicht, weil er denkt es wäre überflüssig oder so und baut somit wieder einen schon beseitigten Fehler ein.
Auch habe ich mir angewöhnt, wenn ich fremden Code ändere meine Änderungen zu markieren mit:

Delphi-Quellcode:

// [mp:start]
...;
...;
...;
// [mp:end]

(mp ist mein Kürzel in der Firma.) Dann sehe ich sofort welcher Code von mir ist und ich kann ihn im Fall der Fälle über die Such-Funktion auch schneller wiederfinden. Leider habe ich meinen Kollegen noch nicht dazu bringen können dies auch zu tun. Wenn ich jetzt einen Fehler beheben muss, weiß ich teilweise nicht, ob der Fehler vom Framework stammt oder in vom Kollegen nachträglich hinzugefügten Code. Ich habe schon Stunden mit der Fehlersuche verbracht, wo es dann nachher hieß: "Äh, Moment, der Code ist von mir. Ich weiß, was da nicht stimmt."

[Cylence]

hmm

ich glaub ich hab auch noch was, ein bekannter von mir macht ständig sachen

onFormActivate

besser wär aber onFormshow, weil activate wird aufgerufen wenn man mal auf den desktop klickt und dann wieder auf das fenster....hat schon einige probleme gegeben hier...


Gruß

tom

[Luckie]

Nun ja, das ist aber weniger ein Irrtum, als ein Fehler würde ich sagen.

[Hansa]

Ein häufiger Irrtum ist :
for i := 1 to StringList.Count ... Das ist eben das Versehen, zu vergessen, dass das bei 0 anfängt und dementsprechend bei count-1 Schluss ist. "Listenindex überschreitet das Maximum XY".

Aber was ist hieran auszusetzen, bzw. unschön, mal davon abgesehen, dass es sich keineswegs um einen Irrtum handelt :

Zitat von R2009:

Ist zwar kein Fehler aber was ich überhaupt nicht leiden kann ist die C Schreibweise in Pascal:

Delphi-Quellcode:

extrem unschön:
If blabla then begin
...
end

sehr viel übersichtlicher:
If blabla then 
   begin
     ...
   end

Ich bevorzuge die erste Variante. Bei mir einzige Abweichung vom ansonsten IMHO guten Styleguide. Auch die zweite entspricht übrigens nicht dem offiziellen Styleguide. Grund für 1. Variante : wenn sich was verirrt, dann ein END; Oder ein begin wird vergessen. Ist so etwas passiert, dann will ich auch direkt sehen, ob das zu if, while oder sonstwas gehört. Das vom Styleguide, also das BEGIN bündig unter das zugehörige IF etc. zu schreiben läuft zwar ungefähr auf dasselbe hinaus. Notfalls geht man vom END; aus einfach nach oben, hat das entsprechende IF und sieht, ob das BEGIN am Ende der Zeile steht. Oder eben eine Zeile darunter. Aaaaber : es dürfte keine Seltenheit sein, dass eine Prozedur mit 10 IF...BEGIN usw. sagen wir mal 70 Zeilen hat. Die geht dann auf eine DIN A 4 Seite. Werden 10 BEGIN in extra Zeilen geschrieben, dann sinds schon 80 und die Seite reicht knapp eben nicht. Verschachtelungstiefe ist dann auch nicht sofort erkennbar. Für Bildschirm gilt ähnliches. Es muss unnötigerweise mehr gescrollt werden etc. Bei neueren Delphi-Versionen ist dieses Problem aber auch durch Ein/Ausklappen etwas entschärft. Betonung liegt dabei auf "etwas".

Wer nur nach Zeilen bezahlt wird, der richte sich allerdings wirklich besser 100% nach dem Styleguide.

[joachimd]

Zitat von Hansa:

Ein häufiger Irrtum ist :
for i := 1 to StringList.Count ... Das ist eben das Versehen, zu vergessen, dass das bei 0 anfängt und dementsprechend bei count-1 Schluss ist. "Listenindex überschreitet das Maximum XY".

Dazu die Worte meines (alten) Informatik-Professors, der inzwischen schon sicherlich nicht mehr auf dieser Erde weilt:

Zitat:

Es ist immer eins mehr oder weniger als man denkt

[BUG]

Zitat von R2009:

aus meinen C Erfahrungen:

if (a = 0)

ist nicht das gleiche wie

if (a == 0)

Das ist mir auch mehrfach passiert, besonders wenn man von Pascal = als Vergleichsoperator gewöhnt ist

Zitat von joachimd:

Dazu die Worte meines (alten) Informatik-Professors, der inzwischen schon sicherlich nicht mehr auf dieser Erde weilt:

Zitat:

Es ist immer eins mehr oder weniger als man denkt

Der Fehlertyp hat sogar seinen eigenen Namen: Off-by-One-Error