Code Style Conventions

[juliusf]

Nehmen wir die gleichen wie bei GEP/GPP? Oder hast du mittlerweile schon wieder was cooleres entdeckt?

[Manuzor]

Im Grunde können wir die gleichen nehmen wie bei GEP/GPP, aber ich würde das trotzdem nochmal im Wiki festhalten.

[Manuzor]

Eine Sache hätte ich gerne anders gemacht. Ich hasse upper-case Würste. Wenn ich sowas sehe, wird mir schlecht: KRDOSOMETHINGSILLY. Die verbesserte variante, die von den meisten verwendet wird, ist zwar schon besser, aber trotzdem nicht schön: KR_DO_SOMETHING_SILLY.

Klar, alle Welt schreibt macros in UPPER_CASE aber ich finde das furchtbar zu lesen, ich bevorzuge lieber camel-case: KR_DoSomethingSilly.

Hier ist mein Argument gegen die Konvention aller Welt: Man prefixed seine Macros sowieso schon mit GEP_, EZ_, KR_, und das macht man im restlichen Code sonst auch nicht. Nach dem prefix ist einem also eigentlich schon klar, dass jetzt ein Macro benutzt wird.

Außerdem wird so ziemlich jeder Syntax Highlighter Macros in einer bestimmten Farbe darstellen, wenn er den richtigen Kontext bekommt (Visual Studio, Vim, sicher auch CLion).

Was sagst du dazu?

Man müsste sich halt noch festlegen, welche camelCase Variante wir dafür nehmen würden:

#define KR_DoSomethingSilly
#define KR_doSomethingSilly
#define KR_DoSomethingSilly_WithInts
#define KR_doSomethingSilly_withInts

[juliusf]

Die einzige Frage die ich mir stelle ist ob das nicht zu Verwirrungen führen kann wenn wir Makros von EZ benutzen.

Wenn wir ein

#define KR_deleteAndNull 

haben und ez ein

#define EZ_DELETE_AND_NULL

kommt man da nicht durcheinander?

Aber generell finde ich deinen Vorschlag schöner zu lesen. Ich bin auch der Ansicht dass der prefix ausreicht umd das Makro zu kennzeichnen.

[Manuzor]

Schon, andererseits sieht man so auch gleich, was von uns ist. Das ist ja aber nicht unbedingt wichtig :smiley: Vielleicht sollten wir uns tatsächlich einfach an die ez-Konventionen halten, was das angeht?

[Manuzor]

Wie siehts eigentlich aus mit bracing?

void something()
{
  printf("hello\n");
}

vs.

void something() {
  printf("hello\n");
}

Und was ist mit indentation-level? 4 spaces? ez benutzt 2 spaces, was alles ein bisschen kompakter macht. Wir könnten natürlich auch etwas exotischere 3 spaces benutzen :smiley: Aber tabs kommen mir nicht mehr ins Haus.

[juliusf]

void something() {
  printf("hello\n");
}

DU MONSTER! braces stehen untereinander. punkt. ;)

ich wünsche mir also das hier:

void something()
{
  printf("hello\n");
}

keine tabs. anzahl der whitespaces ist mir egal. zwei finde ich ein bisschen wenig, aber ist vermutlich angenehmer zu lesen auf kleinen screens. 4 ist auch okay

[juliusf]

see wiki page.