ATNEL tech-forum

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T14:37:30+01:00

Za bardzo kombinujesz.
Nic się samo z siebie nie ustawia.
Po prostu jak masz = to na porcie ustawia się to co wymuszasz przez = .
Jak dałeś &= to najpierw sprawdzany jest poprzedni stan PORT'u , a dopiero później kasowane były te PINY którym odpowiadało 0 za &=
Jak miałeś wcześniej cały port wyzerowany to już nic Ci się ni zapalało.

Statystyki: Napisane przez auers — 12 kwi 2020, o 14:37

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T14:27:27+01:00

o kurka... działa bez &.... hmm... no jeszcze sporo nauki mam w tym temacie:) Czyli za każdym razem ten port się ustawia w stan wysoki przy zmianie cyfry tak? a ja samym = gaszę tylko te bity które potrzebuje tak?

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 14:27

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T14:16:25+01:00

Na początek spróbuj z samym =
Operator & stosujesz np jak chcesz wygasić konkretne bity nie zmieniając innych.
Jeżeli tego portu nie wykorzystujesz do innych celów, to Cię nie interesuje co na nim było.
Po prostu ustaw to co potrzebujesz.

A jak opanujesz podstawy to poczytaj o maskowaniu.
Wówczas & będzie już niezbędny.

Statystyki: Napisane przez auers — 12 kwi 2020, o 14:16

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T14:07:51+01:00

auers napisał(a):

Poczytaj o tablicach, bo na piechotę z samymi if-ami daleko nie zajedziesz.
Przemyśl jeszcze raz sterowanie portem, bo ta funkcja ZGAS_C jet niepotrzebna.
Powinieneś od razu ustawiać port tak jak Ci potrzebny.
Przemyśl jeszcze raz te definiecje:
#define CYFRA0 PORTB &= 0xC0 // Wlacz Cyfry PORT B0-B6
czy na pewno tam powinien być operator &?

Tak, tak na pewno będę czytać kiedy przyjdzie książka a co do operatora & to jaki powinienem zastosować? przecież na każdą cyfrę muszę ustawić stan niski na konkretnych bitach i wysoki na innych na jednym porcie tak jak w tym przykładzie cyfry 0 tzn
0xC0-----> 0b11000000, cały port B mam ustawiony na cyfry które ustawiam stanem niskim, a że tak to rozumie iż cały PORTB jest ustawiony na stan wysoki to tym operatorem & wstawiam w konkretne bity stan niski. Czy źle to rozkminiam?

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 14:07

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T13:47:53+01:00

Statystyki: Napisane przez auers — 12 kwi 2020, o 13:47

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T13:39:12+01:00

Dziękuję koledze za okazaną pomoc w moim problemie, teraz będę działał dalej próbując uruchamiać dalsze pozycje wyświetlacza

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 13:39

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T13:25:19+01:00

W deklaracji czyli tu:
void L_DISPLAY( int LICZNIK );
masz LICZNIK

W definicji czyli tam gdzie zaczyna się ciało funkcji:
void L_DISPLAY( int L_SCAN ) {
masz już L_SCAN

kompilator nie krzyczy bo dla niego najważniejszy jest typ, ale dobrze jest mieć porządek.

Natomiast już jak wywołujesz funkcję to możesz tam wstawiać zmienną o innej nazwie niż w samej deklaracji i to jest normalne.

Statystyki: Napisane przez auers — 12 kwi 2020, o 13:25

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T13:15:42+01:00

Teraz działa jak należy (chociaż musiałem dodać jeszcze funkcje ZGAS_C; w obsłudze klawiszów, bo zapomniałem o tym i wychodziła kaszanka). Dziękuję za pomoc i cenne wskazówki teraz to będę sobie analizował dogłębnie, by opanować te pisanie funkcji.
Jakby kolega jeszcze wyjaśnił mi o co chodzi w tym zdaniu: "
Tak dla porządku, w deklaracjach funkcji dobrze używać tych samych nazw zmiennych co w ich definicjach." Tzn gdzie?

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 13:15

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T12:51:55+01:00

Brakuje ci nawiasów }
- zamykającego funkcję main
- zamykającego funkcję L_DISPLAY

Tak dla porządku, w deklaracjach funkcji dobrze używać tych samych nazw zmiennych co w ich definicjach.

Statystyki: Napisane przez auers — 12 kwi 2020, o 12:51

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T12:44:55+01:00

dalej nic, nie wiem o co mu chodzi z tym nawiasem...

kod:
[syntax=c]#include
#include
#include

#define CYFRA0 PORTB &= 0xC0 // Wlacz Cyfry PORT B0-B6
#define CYFRA1 PORTB &= 0xF9
#define CYFRA2 PORTB &= 0xA4
#define CYFRA3 PORTB &= 0xB0
#define CYFRA4 PORTB &= 0x99
#define CYFRA5 PORTB &= 0x92
#define CYFRA6 PORTB &= 0x82
#define CYFRA7 PORTB &= 0xD8
#define CYFRA8 PORTB &= 0x80
#define CYFRA9 PORTB &= 0x90

#define POZYCJA1 PORTD &= ~0x01 // Wlacz Pozycja PORT D0-D3
#define POZYCJA2 PORTD &= ~0x02
#define POZYCJA3 PORTD &= ~0x04
#define POZYCJA4 PORTD &= ~0x08

#define GUZIK1 0x01 // GUZIK PORT C0-C1
#define GUZIK2 0x02

#define ZGAS_P PORTD |= 0X0F
#define ZGAS_C PORTB |= 0XFF

void INICJALIZACJA_PORTOW();
void ZGAS_CP();
void L_DISPLAY( int LICZNIK );

int LICZNIK = 0; // zmienna LICZNIK zakres od 0 do 255

int main( void ) {

INICJALIZACJA_PORTOW();
POZYCJA1;

while ( 1 ) {

if ( LICZNIK == 0 ) {
ZGAS_C;
CYFRA0;
}

if ( !( PINC &( GUZIK1 ) ) ) {
LICZNIK++;
_delay_ms( 200 );

L_DISPLAY( LICZNIK );

}
if ( !( PINC &( GUZIK2 ) ) ) {
LICZNIK--;
_delay_ms( 200 );

L_DISPLAY( LICZNIK );

}

}

void INICJALIZACJA_PORTOW( void ) {

DDRB |= 0xFF; // PORT B jako wyjscie
DDRD |= 0xFF; // PORT D jako wyjscie
DDRC &= ~0x03; // PORT C jako wejscie -BIT 0 i 1 ustawione PULLUP-
_delay_ms( 10 ); // zwloka na ustawienie PULLUP
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-

}

void ZGAS_CP( void ) {
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-
}
void L_DISPLAY( int L_SCAN ) {

if ( L_SCAN == 0 ) {
CYFRA0;
} else if ( L_SCAN == 1 ) {
CYFRA1;
} else if ( L_SCAN == 2 ) {
CYFRA2;
} else if ( L_SCAN == 3 ) {
CYFRA3;
} else if ( L_SCAN == 4 ) {
CYFRA4;
} else if ( L_SCAN == 5 ) {
CYFRA5;
} else if ( L_SCAN == 6 ) {
CYFRA6;
} else if ( L_SCAN == 7 ) {
CYFRA7;
} else if ( L_SCAN == 8 ) {
CYFRA8;
} else if ( L_SCAN == 9 ) {
CYFRA9;
}[/syntax]

LOG:

compiling process, please wait...
"C:\Users\dom\OneDrive\Dokumenty\MkClipse\AVR Toolchain\bin\avr-gcc" -Wall -Os -Wstrict-aliasing=1 -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega32 -DF_CPU=8000000UL -MMD -MP -MF"Release/main.d" -MT"Release/main.o" -c -o "Release/main.o" "main.c""
main.c: In function 'L_DISPLAY':
main.c:106:9: error: expected declaration or statement at end of input
}
^
main.c: In function 'main':
main.c:106:9: error: expected declaration or statement at end of input
"C:\Users\dom\OneDrive\Dokumenty\MkClipse\AVR Toolchain\bin\avr-gcc" -Wall -Os -Wstrict-aliasing=1 -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega32 -DF_CPU=8000000UL -MMD -MP -MF"Release/mk_build.d" -MT"Release/mk_build.o" -c -o "Release/mk_build.o" "mk_build.c""

----------------
ERROR --> main.c --> Line: 106

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 12:44

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T12:27:36+01:00

Wywołujesz funkcję przed jej deklaracją.
Kompilator nie wie co to jest L_DISPLAY,

Odkomentuj:
//void L_DISPLAY(LICZNIK);
i w nawiasie dopisz typ zmiennej LICZNIK;

Statystyki: Napisane przez auers — 12 kwi 2020, o 12:27

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T12:21:24+01:00

mniej więcej w taki sposób, ale mnie to pokonało

[syntax=c]#include
#include
#include

#define CYFRA0 PORTB &= 0xC0 // Wlacz Cyfry PORT B0-B6
#define CYFRA1 PORTB &= 0xF9
#define CYFRA2 PORTB &= 0xA4
#define CYFRA3 PORTB &= 0xB0
#define CYFRA4 PORTB &= 0x99
#define CYFRA5 PORTB &= 0x92
#define CYFRA6 PORTB &= 0x82
#define CYFRA7 PORTB &= 0xD8
#define CYFRA8 PORTB &= 0x80
#define CYFRA9 PORTB &= 0x90

#define POZYCJA1 PORTD &= ~0x01 // Wlacz Pozycja PORT D0-D3
#define POZYCJA2 PORTD &= ~0x02
#define POZYCJA3 PORTD &= ~0x04
#define POZYCJA4 PORTD &= ~0x08

#define GUZIK1 0x01 // GUZIK PORT C0-C1
#define GUZIK2 0x02

#define ZGAS_P PORTD |= 0X0F
#define ZGAS_C PORTB |= 0XFF

void INICJALIZACJA_PORTOW();
void ZGAS_CP();
//void L_DISPLAY(LICZNIK);

int LICZNIK = 0; // zmienna LICZNIK zakres od 0 do 255

int main( void ) {

INICJALIZACJA_PORTOW();
POZYCJA1;

while ( 1 ) {

if ( LICZNIK == 0 ) {
ZGAS_C;
CYFRA0;
}

if ( !( PINC &( GUZIK1 ) ) ) {
LICZNIK++;
_delay_ms( 200 );

L_DISPLAY( LICZNIK );

}
if ( !( PINC &( GUZIK2 ) ) ) {
LICZNIK--;
_delay_ms( 200 );

L_DISPLAY( LICZNIK );

}

}

void INICJALIZACJA_PORTOW( void ) {

DDRB |= 0xFF; // PORT B jako wyjscie
DDRD |= 0xFF; // PORT D jako wyjscie
DDRC &= ~0x03; // PORT C jako wejscie -BIT 0 i 1 ustawione PULLUP-
_delay_ms( 10 ); // zwloka na ustawienie PULLUP
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-

}

void ZGAS_CP( void ) {
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-
}
void L_DISPLAY( uint8_t L_SCAN ) {

if ( L_SCAN == 0 ) {
CYFRA0;
} else if ( L_SCAN == 1 ) {
CYFRA1;
} else if ( L_SCAN == 2 ) {
CYFRA2;
} else if ( L_SCAN == 3 ) {
CYFRA3;
} else if ( L_SCAN == 4 ) {
CYFRA4;
} else if ( L_SCAN == 5 ) {
CYFRA5;
} else if ( L_SCAN == 6 ) {
CYFRA6;
} else if ( L_SCAN == 7 ) {
CYFRA7;
} else if ( L_SCAN == 8 ) {
CYFRA8;
} else if ( L_SCAN == 9 ) {
CYFRA9;
}[/syntax]

A tutaj LOG:

compiling process, please wait...
"C:\Users\dom\OneDrive\Dokumenty\MkClipse\AVR Toolchain\bin\avr-gcc" -Wall -Os -Wstrict-aliasing=1 -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega32 -DF_CPU=8000000UL -MMD -MP -MF"Release/main.d" -MT"Release/main.o" -c -o "Release/main.o" "main.c""
main.c: In function 'main':
main.c:54:13: warning: implicit declaration of function 'L_DISPLAY' [-Wimplicit-function-declaration]
L_DISPLAY( LICZNIK );
^
main.c: In function 'L_DISPLAY':
main.c:106:9: error: expected declaration or statement at end of input
}
^
main.c: In function 'main':
main.c:106:9: error: expected declaration or statement at end of input
"C:\Users\dom\OneDrive\Dokumenty\MkClipse\AVR Toolchain\bin\avr-gcc" -Wall -Os -Wstrict-aliasing=1 -fpack-struct -fshort-enums -ffunction-sections -fdata-sections -std=gnu99 -funsigned-char -funsigned-bitfields -mmcu=atmega32 -DF_CPU=8000000UL -MMD -MP -MF"Release/mk_build.d" -MT"Release/mk_build.o" -c -o "Release/mk_build.o" "mk_build.c""

----------------
WARNING --> main.c --> Line: 54

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 12:21

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T11:03:24+01:00

Pokaż jak ten licznik próbowałeś wstawić do funkcji.

Statystyki: Napisane przez auers — 12 kwi 2020, o 11:03

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T10:51:23+01:00

Faktycznie zmiana na int zmiennej LICZNIK pomogła Dziękuje. Co do tablic to jeszcze nie za bardzo rozumie ich działanie, czekam na książkę w celu dalszej edukacji.

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 10:51

Re: Problem nowicjusza. Prosty licznik.

2020-04-12T10:43:13+01:00

Źle zadeklarowałeś typ zmiennej LICZNIK.
kompilator traktuje je jako zmienna bez znaku czyli masz tylko liczby dodatnie.
Nigdy nie będzie spełniony warunek LICZNIK <0.
Zamiast -1 będzie 255, czyli LICZNIK >9 stąd LICZNIK =9.

Co do optymalizacji zamiast 2x10 if'ów stwórz sobie tablicę.

------------------------ [ Dodano po: 5 minutach ]

Zamiast testować czy LICZNIK po odejmowaniu jest ujemny.
Odejmowanie zrób dopiero po sprawdzeniu czy LICZNIK > 0.

Statystyki: Napisane przez auers — 12 kwi 2020, o 10:43

Problem nowicjusza. Prosty licznik.

2020-04-12T10:29:10+01:00

Witam kolegów i koleżanki. Nie tak dawno temu postanowiłem zgłębić tajniki programowania w języku C. Napisałem mega prosty program który ma zliczać kliknięcia od 0 do 9 jednym przyciskiem( to działa ), a drugim przyciskiem w drugą stronę (nie działa).
Problem w tym, że kiedy przyciskiem- zejdę do 0 i naciskam dalej, wyświetlacz pokazuje 0 (i to jest dobre, rzekł Pan) ale kiedy wcisnę przycisk+ to nie wskakuje 1 jakby można było się spodziewać tylko wskakuje 9 ?!? Dlaczego ja się pytam? .
Pomijam fakt optymalizacji kodu ( 3.6kB ), ani dlaczego nie udało mi się wstawić LICZNIKA do funkcji( wywala błąd ).

Coś o mnie: Przesiadłem się z Arduino (jakąś tam wiedzę posiadam), posiadam zestaw uruchomieniowy firmy innej, czekam na zamówioną książkę BB , piszę w MKClipsie, wiedzę do tej pory czerpie z poradników internetowych( jak na razie poradniki pana Mirka są dla mnie najbardziej zrozumiałe w porównaniu do innych dlatego zamówiłem BB). Liczę na jasne wytłumaczenie mojego błędu. Dziękuje.

[syntax=c]#include
#include
#include

#define CYFRA0 PORTB &= 0xC0 // Wlacz Cyfry PORT B0-B6
#define CYFRA1 PORTB &= 0xF9
#define CYFRA2 PORTB &= 0xA4
#define CYFRA3 PORTB &= 0xB0
#define CYFRA4 PORTB &= 0x99
#define CYFRA5 PORTB &= 0x92
#define CYFRA6 PORTB &= 0x82
#define CYFRA7 PORTB &= 0xD8
#define CYFRA8 PORTB &= 0x80
#define CYFRA9 PORTB &= 0x90

#define POZYCJA1 PORTD &= ~0x01 // Wlacz Pozycja PORT D0-D3
#define POZYCJA2 PORTD &= ~0x02
#define POZYCJA3 PORTD &= ~0x04
#define POZYCJA4 PORTD &= ~0x08

#define GUZIK1 0x01 // GUZIK PORT C0-C1
#define GUZIK2 0x02

#define ZGAS_P PORTD |= 0X0F
#define ZGAS_C PORTB |= 0XFF

void INICJALIZACJA_PORTOW();
void ZGAS_CP();

unsigned char LICZNIK = 0; // zmienna LICZNIK zakres od 0 do 255

int main( void ) {

INICJALIZACJA_PORTOW();
POZYCJA1;

while ( 1 ) {

if ( LICZNIK == 0 ) {
ZGAS_C;
CYFRA0;
}

if ( !( PINC &( GUZIK1 ) ) ) {
LICZNIK++;
_delay_ms( 200 );

if ( LICZNIK > 9 ) {
LICZNIK = 9;
}

if ( LICZNIK == 0 ) {
ZGAS_C;
CYFRA0;
} else if ( LICZNIK == 1 ) {
ZGAS_C;
CYFRA1;
} else if ( LICZNIK == 2 ) {
ZGAS_C;
CYFRA2;
} else if ( LICZNIK == 3 ) {
ZGAS_C;
CYFRA3;
} else if ( LICZNIK == 4 ) {
ZGAS_C;
CYFRA4;
} else if ( LICZNIK == 5 ) {
ZGAS_C;
CYFRA5;
} else if ( LICZNIK == 6 ) {
ZGAS_C;
CYFRA6;
} else if ( LICZNIK == 7 ) {
ZGAS_C;
CYFRA7;
} else if ( LICZNIK == 8 ) {
ZGAS_C;
CYFRA8;
} else if ( LICZNIK == 9 ) {
ZGAS_C;
CYFRA9;
}

}
if ( !( PINC &( GUZIK2 ) ) ) {
LICZNIK--;
_delay_ms( 200 );

if ( LICZNIK <= 0 ) {
LICZNIK = 0;
}

if ( LICZNIK == 0 ) {
ZGAS_C;
CYFRA0;
} else if ( LICZNIK == 1 ) {
ZGAS_C;
CYFRA1;
} else if ( LICZNIK == 2 ) {
ZGAS_C;
CYFRA2;
} else if ( LICZNIK == 3 ) {
ZGAS_C;
CYFRA3;
} else if ( LICZNIK == 4 ) {
ZGAS_C;
CYFRA4;
} else if ( LICZNIK == 5 ) {
ZGAS_C;
CYFRA5;
} else if ( LICZNIK == 6 ) {
ZGAS_C;
CYFRA6;
} else if ( LICZNIK == 7 ) {
ZGAS_C;
CYFRA7;
} else if ( LICZNIK == 8 ) {
ZGAS_C;
CYFRA8;
} else if ( LICZNIK == 9 ) {
ZGAS_C;
CYFRA9;
}
}

}

}

void INICJALIZACJA_PORTOW( void ) {

DDRB |= 0xFF; // PORT B jako wyjscie
DDRD |= 0xFF; // PORT D jako wyjscie
DDRC &= ~0x03; // PORT C jako wejscie -BIT 0 i 1 ustawione PULLUP-
_delay_ms( 10 ); // zwloka na ustawienie PULLUP
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-

}

void ZGAS_CP( void ) {
ZGAS_C; // wstawienie HIGH na PORT B -zgaszenie CYFR-
ZGAS_P; // wstawienie HIGH na PORT D -zgaszenie POZYCJI, stan wysoki na 4 mlodsze bity-
}[/syntax]

Statystyki: Napisane przez Marcin vel WiciaTG — 12 kwi 2020, o 10:29