Elektroda.pl
Elektroda.pl
X
Proszę, dodaj wyjątek www.elektroda.pl do Adblock.
Dzięki temu, że oglądasz reklamy, wspierasz portal i użytkowników.

[AVR][C] - Jak by ktos chciał - taka procedurka obsługi klawisza

06 Lis 2015 17:52 870 8
  • #1 06 Lis 2015 17:52
    2675900
    Użytkownik usunął konto  
  • #2 06 Lis 2015 20:21
    tadzik85
    Poziom 38  

    Trudniej się nie dało?

    Szczególnie z zastosowaniem tego goto??

    1
  • #3 06 Lis 2015 20:33
    2675900
    Użytkownik usunął konto  
  • #4 06 Lis 2015 20:34
    grko
    Poziom 33  

    Mam kilka uwag co do kodu:
    1. Tylko key_event powino miec atrybut volatile (poniewaz jest wspoldzielone pomiedzy kontekstem przerwania i programu)

    2. Dla pozostalych zmiennych powinienes zastosowac atrybut static aby nie zasmiecac przestrzeni symboli.

    3. Zmienne moglyby byc opakowane w strukture.

    4. Kod jest niekonsekwentnie sformatowany więc źle się go czyta (180 znakow w jednej linii to za duzo). Proponuje clang-format.

    5. Zamiast goto mogles uzyc po prostu return.

    0
  • #5 06 Lis 2015 20:34
    tadzik85
    Poziom 38  

    GrzegorzKostka napisał:
    ...

    5. Zamiast goto mogles uzyc po prostu return.


    wystarczyłby odwrotny warunek....

    0
  • #7 06 Lis 2015 21:05
    2675900
    Użytkownik usunął konto  
  • #8 06 Lis 2015 21:35
    tadzik85
    Poziom 38  

    GrzegorzKostka napisał:
    @tadzik85 Niby po co ? Za etykieta TIMER0_exit nic nie ma.


    Bo nie jeden styl programistyczny zakazuje takich form.

    0
  • #9 06 Lis 2015 21:37
    2675900
    Użytkownik usunął konto