Quellcode Kommentieren

[Dejan Vu]

Zitat von p80286:

bei einem

Code:

if boolean(shl(shr(wert,1),1) then

wäre ich mir da nicht so sicher.

Refactoring:

Delphi-Quellcode:

function SecondBitIsSet(aValue : Integer) : boolean;
Begin
  result := boolean(shl(shr(aValue,1),1)
End;
...
// statt kryptisch
if boolean(shl(shr(wert,1),1) then

// nun verständlich
if SecondBitIsSet(Wert) then ...

Zum Changelog: Einfach pro Task einchecken. Es gehört Disziplin dazu, aber das lernt man. Spätestens wenn man schon wieder eine Teamrunde spendieren muss, weil man zu schlampig war (Ich weiß, wovon ich rede). Ich brauche kein Changelog pro Prozedur. Ich brauche ein Changelog für die Change Requests und Bugfixes. Das Diff und 'Blame' zeigt mir dann, wer was wann wo geändert hat. Braucht man eigentlich nur beim Review.