Svenska ElektronikForumet

Nerre skrev:Samma sak t.ex. med casting, ska man skriva tydlig kod så ska all casting vara explicit.

Nerre skrev:Har jag 10 000 element och vill visa i koden att de är initierade till 0 så gör jag en loop som intierar dem. Även om loopen är helt onödig ur funktionssynpunkt, så är den nödvändig ur läsbarhetssynpunkt.

Samma sak t.ex. med casting, ska man skriva tydlig kod så ska all casting vara explicit.

arvidb skrev:Tvärt om, oftast (om man inte drar det till extremer som i "obfuscation"-tävlingar eller liknande).

Palle500 skrev:Håller med Nerre till 100%
…..
Kolla bara på all gratis kod för tex arduino ofta riktigt trevlig kod.

Mr Andersson skrev:Jag håller med om att ifall man måste skriva kommentarer som förklarar vad koden gör är den för komplicerat eller dåligt skriven. Däremot måste man nog kunna förvänta sig att den som ska läsa koden har grundläggande kunskaper i språket. (Om inte syftet med koden är just att lära ut språket, då kan man vara övertydlig)
Kommentarer ska istället förklara varför man gör saker på ett visst sätt.

init_adc(); // initialize adc
init_timer(); // initialize timer

Riktigt skitdålig kod.

const int WAIT_ONE_HOUR = 2*60*60; // 1 hour in seconds

Sämst kod

TomasL skrev:Det finns ju en anledning till att kommentarerna och dokumentationen inom jpl är typ 100 ggr större än koden i sig.
Vill man skriva så buggfri kod som möjligt skall varje kodrad förklaras oavsett hur trivial den till synes verkar vara.

int x = 3; // initialize x to 3
y = 7; // assign 7 to y
++z; // increase z by 1

for(int i=0; i<sizeof(tmrs)/sizeof(*tmrs); ++i)  {
  disable_int(tmrs[i]);
}

for(auto timer : running_timers) {
  timer->disable_interrupt();
}