Wie breit sollte Quellcode sein?

volkard

borg schrieb:

was spricht dagegen auch während einer if-anweisung zeilenumbrüche zu machen?

nix.

if(field[x+drc[i][0]*distance][y+drc[i][1]*distance] == BISHOP ||
   field[x+drc[i][0]*distance][y+drc[i][1]*distance] == QUEEN)
  return true;

da seh ich keine möglichkeit es zu verkürzen.

siehe Optimizer

Piece const& piece=field[x+drc[i][0]*distance][y+drc[i][1]*distance];
if(piece == BISHOP || piece== QUEEN)
  return true;

aber sind BISHOP und QUEEN enums?

PAWN=1
ROOK=2
HORSE=4
BISHOP=8
QUEEN=ROOK+BISHOP
KING=16
if(piece & BISHOP)
return true;

//und dann
if(checkCastling(whiteMove))
aber ist das denn sinnvoll? ich brauchs nur einmal.

ist sinnvoll. dann kannste den haupt-code besser lesen.

<edit=kingruedi>Formatierungsfehler gefixt</edit>

Ponto

Es ist schön und gut wenn man einen breiten Monitor hat, der viele Zeichen lesbar darstellen kann, aber oft will man nicht nur eine Datei offen haben. Ebenso ist es beim Vergleichen zweier Dateien via kompare, besser, wenn die beiden Zeilen ganz sichtbar sind.

rüdiger

Ich mach immer 80 Zeichen. Das macht den Code idr. leserlicher und außerdem ist mein Editor auf 80 Zeichen eingestellt. Dh. wenn der Code länger wird, wird die Darstellung umgebrochen.

Walli

Ich breche nach Gefühl um und auch nur wenn ich eine sinnvolle Umbruchstelle ausmachen kann.

DEvent

mehr als 80 zeichen brauch ich nur sehr selten. aber nie >90 oder so. benutzt zZ eclipse, links ist package explorer, debug, rechts ist variables, expressions und outline ( als karteikarten ). dazwischen ist quellcode. bei 1280x1024.

Blue-Tiger

volkard schrieb:

//und dann
if(checkCastling(whiteMove))
aber ist das denn sinnvoll? ich brauchs nur einmal.
ist sinnvoll. dann kannste den haupt-code besser lesen.

wohin steckt ihr eigentlich solche kleinen Hilfsfunktionen? Als private in die aktuelle Klasse, in einen anonymen Namespace in der aktuellen Datei oder sonstwo?

DrGreenthumb

in die klasse nur, wenn auf private member zugegriffen werden muss. ansonsten in den anonymen namespace.