Unübersichtlich und „aufgeblähter" Code?

[Jörg Reinholz]

Moin!

jemand meinte heute zu mir ich würde unübersichtlichen und „aufgeblähter" Code schreiben, wollte mir aber nicht sagen warm und wie bzw. was ich ändern könnte.

Naja, vieles davon ist definitiv Geschmackssache, aber

1.) die Einrückungen sind schon mal ziemlich tief. Vier Leerzeichen genügen als Tab vollauf, vielen sogar nur zwei. Bedenke was passiert, wenn die Verschachtelungstiefe steigt. Die Zeilenlänge steigt unnötig an und damit der Stress dabei, zu prüfen ob man eigentlich überhaupt noch in der richtigen Zeile ließt, sozusagen im Quadrat der Zeilenlänge. Das ist sehr unschön.

2.) Etwas wie

if (flag) {
        if (foo=bar)
                        ...
                {
}}

ist auch nicht "sauber", denn das wäre eigentlich als

if ( flag && foo == bar )  {
    ...
}

zu notieren. Meinetwegen gerne auch:

if ( 
  flag 
  && foo == bar 
)  {
    ...
}

3.)

if (function(foo)) {

sieht als

if ( function(foo) ) {

besser aus.

4.) Deine höchst ungewöhnliche Einrückung des schließenden Brackets:

if () {
    ...
    }

verwirrt schlicht und einfach. Das gehört sich so:

if () {
   ...
}

Jörg Reinholz