Die häufigsten Programmierirrtümer

[Helmi]

Zitat:

Zitat von Wolfgang Mix

@ Helmi :) Mit 60 ohne Brille ist auch schlecht

*auf die Bildschirmlupe verweis* :mrgreen:

Hier hätt ich noch einen:

Delphi-Quellcode:

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

[Wolfgang Mix]

Zitat:

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:

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

Delphi-Quellcode:

 xyz := (y = 2010);

[Reinhardtinho]

Zitat:

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.