Switch-lause

Kokeile switch-lauseen default-haaraa ja katso tuleeko suoritus aina sinne. Muuttujien nimet ovat
aika pitkiä, olisi ehkä helpompi seurata koodia jos käytät lyhempia nimiä.

default:
lcd_putsf("9");
break;

Niin ja huono tapa on myös sijoittaa useita komentoja samalle riville (nuo break;).

Näyttää epäilyttävästi switchin väärinkäytöltä, etenkin kun käytetään totaali paskaa CodeVision kääntäjää.

Parempi tapa:

for(lcd_eeprom_alustus = 1; lcd_eeprom_alustus

Pieni tyyliseikka, tekee tiukempaa koodia kun ei pistä funktiokutsua joka on sama joka caselle sinne switchin sisään:

const char* s;
switch (a)
{
case 0: s = "0"; break;
case 1: s = "1"; break;
//
}
blabla_puts(s);

Sitten, kandee huomata että toi switch on symmetrinen, ton saman asian voi tehdä jotenkin näin:

char s[2];
s[0] = '0' a;
s[1] = 0; // terminate string
bla_puts(s);

Noi on vaan ehdotuksia, muokkaa siten että napsahtaa paremmin siihen koodiis kiinni. Mutta tosiaan tollanen switch missä noi menee ihan lineaarisesti noi vaihtoehdot ei o välttämättä paras ratkaisu.

Ja tosiaan ne puts() kutsut siellä switchin sisällä generoi aika ison kasan kakkaa koodia sinne, mm. koodi joka pistää parametrit esimerkiksi pinoon tulee joka caselle erikseen. Voi olla että kääntäjäs on ajantasalla ja optimoi ton hyvin, mutta miks riskeerata? Teet jotain embedded hommia niin luulisi että muistia ja tehoa ei ole ihan tolkuttomasti. Tää ei ole edes premature optimization vaan yritän opastaa vaan oikeaan suuntaan koodin kirjottamisessa heti ekalla yrityksellä.

Toivottavasti en vaikuta ihen sekopäältä. Ciao.