Code Analyse von (semi) Profis

[kwhk]

False = 0
True = <> 0 (Alles was nicht NULL ist)
Wenn ich also prüfe if xx = True , dann kann xx alles <> 0 sein - wenn das passt OK, dann aber gleich auf if xx <> 0 testen.

[Luckie]

Eventuell für dich interessant: http://michael-puff.de/Programmierun...leanCode.shtml

[p80286]

Zitat von Furtbichler:

Es gilt aber auch: Ich muss mich der Situation anpassen: Entwickle ich in PHP und weiß, das alle Kollegen nur Notepad benutzen, sieht mein Code so aus, das man ihn auch ohne IDE schnell versteht.

So sollte es sein!!

Zitat von Furtbichler:

Hast Du ein paar Beispiele parat, wann beim Debuggen der Typ einer Variablen wichtig ist?

Hir war doch gestern noch ein Thread mit Dateigrößen? Da wäre es z.B. gut zu wissen ob signed oder unsigned. Und jetzt sag nicht Namen sind Schall und Rauch.

Zitat von Furtbichler:

Wie würde ein Kommentar bei dir aussehen, bei dem Du beschreibst, warum Du die Lösung so und nicht anders implementiert hast?

Kommt natürlich auf die Situation an

Delphi-Quellcode:

irgendwas:=irgendwas and $7FFF;  //15Bit wrap around

irgendwas:=irgendwas and $7FFF;  //set highest bit=0

Das hat mit "so und nicht anders" allerdings wenig zu tun, beantwortet aber die Frage nach dem Warum?

Gruß
K-H

[Furtbichler]

Zitat von p80286:

Hir war doch gestern noch ein Thread mit Dateigrößen? Da wäre es z.B. gut zu wissen ob signed oder unsigned. Und jetzt sag nicht Namen sind Schall und Rauch.

Eine Dateigröße ist doch eine Zahl. Wieso muss man dann wissen, ob sie signed oder unsigned ist. Außer, man programmiert in C.

Zitat:

Delphi-Quellcode:

irgendwas:=irgendwas and $7FFF;  //15Bit wrap around

irgendwas:=irgendwas and $7FFF;  //set highest bit=0

Ich mache das so:

Delphi-Quellcode:

irgendwas:= WrapAround15Bit(irgendwas); // <-- Wobei ich mich frage, was das bedeuten soll.
irgendwas:= ClearHighestBit(irgendwas);

Drei Fliegen mit einer Klappe. Kommentiert, refaktorisiert und wiederverwendbar. Mach das mal mit Kommentaren

[Elvis]

Zitat von Furtbichler:

Zitat von p80286:

Hir war doch gestern noch ein Thread mit Dateigrößen? Da wäre es z.B. gut zu wissen ob signed oder unsigned. Und jetzt sag nicht Namen sind Schall und Rauch.

Eine Dateigröße ist doch eine Zahl. Wieso muss man dann wissen, ob sie signed oder unsigned ist. Außer, man programmiert in C.

Weil du sonst Blödsinn kriegst, wenn die Datei größer als 2^31 ist.
Es sind genau diese total saudummen Fehler, die dazu führten, dass man den 3GB-switch in Windows 2003 Produktiv fast nie einsetzen konnte:
Bei mindestens einer, leider kritischen, Software/Treiber hat mindestens ein Held (aka Pi**birne) einen Pointer als int anstatt als uint benutzt. -> BSOD oder Crash
Ganz toll! Wer sowas macht frisst auch kleine Kinder...

Das heißt nicht, dass man jetzt die Ungarn rausholen soll!
Typenhinweise im Namen von Bezeichnern machen nur Sinn, wenn es absolut wichtig ist (siehe StatusByte).
Für Franzosen mag dieser ungarische Mist ja vllt. sogar lesbar sein (die schreiben eh alles verkehrtrum ), aber nicht für Nicht-Franzosen.

Mit Kommentaren sehe ich es ähnlich wie Furtbichler. Wenn ich viele Kommentare in Code sehe, werde ich meistens sehr misstrauisch.
Entweder weil der Code tatsächlich viele Fallen umschiffen muss (was Domain-abhängig ja durchaus unvermeidbar sein kann), oder weil der Author keine Verhältnismäßigkeit gelernt hat. (Es werden in Unis teilweise die allerschlimmsten Comment-Nazis herangezüchtet... )

[BUG]

Zitat von Elvis:

Es werden in Unis teilweise die allerschlimmsten Comment-Nazis herangezüchtet...

Wobei ich unverständlichen kommentierten Code bevorzugen würde, wenn die Alternative unverständlicher Code ohne Kommentare ist.
Gut zu kommentieren üben kann man nur, wenn man überhaupt kommentiert und sich überflüssige Kommentare abzugewöhnen kosten meiner Erfahrung nach weniger Überwindung, als überhaupt mit kommentieren anzufangen.

Zitat von Elvis:

siehe StatusByte

Allerdings könnte man solche Sachen auch gut wegkapseln (Records in Delphi, Klassen in C++). Dank inline hat man dabei auch keine Geschwindigkeitsverluste.

[Furtbichler]

Zitat von Elvis:

Weil du sonst Blödsinn kriegst, wenn die Datei größer als 2^31 ist.

Ach, stimmt ja. Ich programmiere nicht mehr mit Delphi.

Zitat von BUG:

Wobei ich unverständlichen kommentierten Code bevorzugen würde, wenn die Alternative unverständlicher Code ohne Kommentare ist.

Das man 99% Schrott dem 100%igen vorzieht, ist ja wohl logisch und keine Erwähnung wert.

Zitat:

Gut zu kommentieren üben kann man nur, wenn man überhaupt kommentiert und sich überflüssige Kommentare abzugewöhnen kosten meiner Erfahrung nach weniger Überwindung, als überhaupt mit kommentieren anzufangen.

Ich habe nie kommentiert, sondern immer eine korrekte Nomenklatur angestrebt. Nach Lektüre der Clean-Code Philosophien wurde mir dann klar, das ich wohl 20 Jahre in die richtige Richtung marschiert bin.