Eräässä sulautetussa järjestelmässäni olisi sellainen tilanne, että ohjelman alkaessa pitäisi tulostaa LCD-näytölle tiettyjä muuttujan arvoja, jotka saatiin edellisellä ajokerralla, virtakatkoksen jälkeen. Nämä muuttujan arvot saadaan taulukosta, joka sijaitsee mikro-ohjaimen sisäisessä EEPROM-muistissa. Yritän suorittaa tulostusta käyttäen for-silmukassa pyörivää switch-lausetta seuraavaan tapaan:
for(lcd_eeprom_alustus = 1; lcd_eeprom_alustus
Switch-lause
5
866
Vastaukset
- C-noviisi
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;).Olen edellisen C-noviisin kanssa molemmista tapauksista muuten eri mieltä. Noin yksinkertaiset switschit kannattaa laittaa yhdelle riville luettavuuden parantamiseksi, ja pidemmät muuttujannimet ovat usein parempia kuin lyhyemmät (kuvaavampia). Toki jos niitä joutuu paljon kirjoittelemaan, jne.
Okei, for-loopissa taulukkoa läpikäytäessä indeksille annetaan perinteisesti yksikirjaiminen muuttuja (alkaen i:stä i, j, k, ... Harvoin enempää tarvitseekaan).
Mitä itse ongelmaan tulee, jotenkin epäilyttää tuo unsigned charin vertaaminen inttiin, kai sen pitäisi mennä, mutta silti... Eikös jotenkin voisi merkata, että tuo case 0: tarkoittaa juurikini unsigned char 0:aa? En tiedä onko tällä väliä.
Joka tapauksessa hieman ihmetyttää tuo tarve tulostaa numero stringinä noin. Eikös olisi helpompi vain muuttaa se stringiksi ja tyrkätä menemään ilman mitään valintoja?
- iziot
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- :)
Tota switch/case käyttää kokonaislukua ei char:ia, sen takia menee mönkään.
unsigned char lcd_taulu[14];
^^^^^^^^^^^^^^^^^^^^^^^^^^^
char merkki-> numero = kokonaisluku
0 = 48
1 = 49
2 = 50
jne...
- cmies
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.
Ketjusta on poistettu 0 sääntöjenvastaista viestiä.
Luetuimmat keskustelut
En usko et meistä tulee jotain
Se ei kuitenkaan estä toivomasta et tulisi. Toivon et voitas suudella ja se sais asioita loksahtamaan paikoilleen. Jutel102640- 911976
Eini paljastaa nuorekkuutensa salaisuuden - Tämä nousee framille: "Se on pakko, että jaksaa!"
Discokuningatar Eini on täyttänyt upeat 64 vuotta. Lavoilla ja keikoilla nähdään entistä vapautuneempi artisti, joka ei391511- 2601398
Yli puolella maahanmuuttajalapsista ei ole tietoja ja taitoja, joilla selviää yhteiskunnassa
Miksi Suomeen otetaan väkeä jolla on älyvajetta? https://www.hs.fi/politiikka/art-2000010730220.html2731038- 67933
Olen J-mies
Jos kerrot sukunimeni alkukirjaimen, ja asuinpaikkakuntani. Lupaan ottaa yhteyttä sinuun.47891- 55864
Ei sitten, ei olla enää
Missään tekemisissä. Unohdetaan kaikki myös se että tunsimme. Tätä halusit tämän saat. J miehelle. Rakkaudella vaalea na77860- 44775