Die häufigsten Programmierirrtümer

[scrat1979]

... und noch was

Delphi-Quellcode:

 sl : TStringList;
 [...]
begin

 sl := TStringList.Create(nil);
 [...]

 sl.Free; // wird oft vergessen :)
end;

P.S.: Wie zum Teufe rücke ich denn hier in den Tags ein?!? Im Antwort-Editor ist es eingerückt

[Wolfgang Mix]

@ Michael:
Sehe ich auch so

[Reinhardtinho]

Zitat von Luckie:

Das ist weder ein Irrtum noch ein Fehler. Der Programmierer fand eventuell seine Schreibweise lesbarer.

Ich habe weder das eine noch das andere behauptet.
Ich möchte nur darauf hinweisen, das Anfänger in solch formatiertem Code Fehler schlechter finden. Dabei handelt es sich allerdings nicht um eine pauschale Aussage, sondern ein Hinweis rein aus Erfahrung mit Anfängern.

[Helmi]

was auch ein Irrtum sein kann, wäre das hinzufügen eines Eintrages in eine StringList z. B. mittels "Insert" statt "Add"

[BUG]

Zitat von Reinhardtinho:

GOTO (siehe xkcd).

Ist aber eher kein typischer Anfängerfehler, da man dafür ja die passende Syntax (Label vor dem begin deklarieren) kennen muss.
GOTO habe ich in "Anfängercodes" hier in der DP noch nie gesehen.

Wenn man nicht ständig davor warnen würde, würden es die meisten (Anfänger) nicht mal kennen
(Mein erstes GOTO hab ich in einer Warnung gesehen.)

[Die Muhkuh]

Ich hätte da zu bieten:

Delphi-Quellcode:

// Damit es auf jeden Fall ausgeführt wird!!!
if 1 = 1 then
begin
  // Code
end;

Auch das Gegenteil hab ich schon oft genug gesehen:

Delphi-Quellcode:

// Darf nicht mehr ausgeführt werden!!!
if 1 = 2 then
begin
  // Code
end;

Toll fand ich auch:

Delphi-Quellcode:

if (Bedingung) then
begin
  Var1 := 'bla';
  Var2 := 'bla';
end
else
begin
  Var1 := 'blubb';
  Var2 := 'blubb';
end;

Var1 := 'bla';
Var2 := 'bla';

[Mithrandir]

Zitat von Matze:

Das sind keine "Programmierirrtümer", sondern Programmierfehler!
Da es in "Programmieren allgemein" steht, hier noch ein Fehler, der einen richtig frustrieren kann, da man ihn teils nur schwer findet:

Code:

// Der Code ist für C, C++, C#, ... (nicht für Delphi)
if (a = b) {
    // ...
}

Mit PHP geht das auch seeeeehr gut....

[s.h.a.r.k]

Zitat von Daniel G:

Mit PHP geht das auch seeeeehr gut....

OHHHHH ja... Und da soll man das erst mal finden...

Zitat von Helmi:

was auch ein Irrtum sein kann, wäre das hinzufügen eines Eintrages in eine StringList z. B. mittels "Insert" statt "Add"

Warum? Man kann doch via Insert einen Index angeben. Oder meinst du, dass jeder das Insert für das Add hält?

Zitat von scrat1979:

... und noch was

Delphi-Quellcode:

 sl : TStringList;
 [...]
begin

 sl := TStringList.Create(nil);
 [...]

 sl.Free; // wird oft vergessen :)
end;

P.S.: Wie zum Teufe rücke ich denn hier in den Tags ein?!? Im Antwort-Editor ist es eingerückt

Das hier mag ich noch dazu berichtigen, dass man da doch gleich das try-finally hinzufügt.

Delphi-Quellcode:

var
  sl : TStringList;
begin
  sl := TStringList.Create(nil);
  try

  finally
    sl.Free; // oder auch FreeAndNil(sl); welches ich all zu gerne nutze
  end;
end;

[sx2008]

Ein Klassiker: Zugriff auf leeres, dynamisches Array (auch Strings sind dynamische Arrays):

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;

[Delphi-Laie]

Oder:

if a=1 or 2 (anstatt: if (a=1) or (a=2)),

also so vereinfacht bzw. verkürzt, wie man in unserer natürlichen Sprache abgekürzt, aber dennoch verständlich sich auszudrücken pflegt.