Die häufigsten Programmierirrtümer

[Helmi]

Zitat von Wolfgang Mix:

@ Helmi Mit 60 ohne Brille ist auch schlecht

*auf die Bildschirmlupe verweis*

Hier hätt ich noch einen:

Delphi-Quellcode:

for i := 1 to StringList.Count do
  S := StringList.Strings[i];

[Wolfgang Mix]

Zitat von alzaimar:

Schlecht gewählte und nichtssagende Variablenbezeichner.
Schlecht lesbarer Code.

Der größte Programmiererirrtum: Zu meinen,von anderen keine Ratschläge annehmen zu müssen.

Das versuche ich hier gerade, bist du nachtragend?

[Noedel]

naja, kein Programmierfehler, eher ein Fehler im Umgang mit dem BB-Code:

"hab schon das alles geschafft:

[ganz viel Quelltext, nicht in Delphi-Tags eingefasst, nicht eingerückt]

Wer kann mir helfen?"

----------

Auch kein Programmierfehler, aber ein Mangel an Kenntnis im Umgang mit dem Debugger:

"[ganz viel Quelltext, vielleicht in Delphi-Tags eingefasst, eventuell eingerückt]

Da kommt dann ein Fehler. Wieso?"

[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) {
    // ...
}

[Reinhardtinho]

Etwas, was ich öfters sehe:


Fehler werden nicht gefunden, da Einrückung nicht korrekt ist.
- begin oder end zuviel/zuwenig
- if-Anweisung ohne begin/end und man wundert sich, dass zuviel ausgeführt wird
- ...


Abbruchbedingungen, die niemals zutreffen können.


GOTO (siehe xkcd).


Was ich persönlich einfach nicht leiden kann (ist ansich ja nicht falsch, aber ich finde es einfach unschön):

Delphi-Quellcode:

if cancelled then
  exit
else
begin
  ...
end;
//Ende der Methode

anstatt

Delphi-Quellcode:

if not cancelled then
begin
  ...
end;
//Ende der Methode

[Wolfgang Mix]

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) {
    // ...
}

Meintest du?
if (a == b) {

Darauf bin ich auch schon öfter 'reingefallen

[Luckie]

Delphi-Quellcode:

var
  sl: TStringList;
begin
  sl.Create;
  sl....;

[scrat1979]

Hätte auch noch was:

Delphi-Quellcode:

if y = 2010 Then
 xyz := true
else
 xyz := false;

statt

 xyz := (y = 2010);

[Reinhardtinho]

Zitat von Matze:

Code:

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

Diese Schreibweise der Klammernpaare (gilt auch für begin/end) - also nicht untereinander - halte ich auch für eine Quelle für Programmierirrtümer, gerade für Anfänger.

[Luckie]

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