ATNEL tech-forum

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T16:58:03+01:00

na C był buzer, ale go zlikwidowałem, to jest kod gdzie ćwiczę język C, więc jest trochę bałaganu. Dzisiaj bawiłem się z wyświetlaczem LCD i prawie pół dnia mi to zajęło. muszę to dokładnie rozgryźć to strasznie wciąga tak, tak że prawie cała niedziela przeleciała hehe. teraz chcę sobie coś wyświetlać na lcd, ale muszę jeszcze o tym poczytać. Ten kod , który jest wyżej sam sobie pisałem, więc jest w miarę czytelny, najgorsze jest to,że strasznie się męczyłem z drugim klawiszem i szukałem dlaczego nie działa, kombinowałem z kodem,a okazało się ,że miałem źle zdefiniowany pin. i na tym człowiek najwięcej traci czasu. Ale są tego plusy, ponieważ po drodze można się nauczyć czegoś innego. I tak powoli do przodu

Statystyki: Napisane przez wojciechgorzolka — 26 mar 2017, o 16:58

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T13:23:54+01:00

witaj ponownie,

do klawiszy używasz preprocesora dyrektywa:)
[syntax=c]#define KLAWISZ (1<#define KLAWISZ_WCISNIETY !(PIND & KLAWISZ) //makro sprawdzające czy klawisz jest wciśnięty czyli stan NISKI logicznie 0[/syntax]
a diodami bawisz się w kodzie bezpośrednio --- ciężko się to czyta,
spróbuj użyć tej samej techniki programowania ( preprocesora )

[syntax=c]#define DIODA1_LED (1<#define ZMIANA_STANU_DIODY PORTB ^=DIODA1_LED //zmienia stan diody na przeciwny[/syntax]

da to takie zalety w jednym miejscu definiujesz "hardware" w sposób bardzo przejrzysty (zwiększasz przenośność kodu na inne połączenia (poprawiasz tylko deklaracje #define )
po za tym łatwiej będzie ten kod analizować
później w kodzie używasz
[syntax=c]if (KLAWISZ_WCISNIETY) {
_delay_ms(80);
ZMIANA_STANU_DIODY ;
} // koniec ifa od klawisza[/syntax]

Proszę poczytaj w BB rozdział 4.4 strona 187 do 192.
tam masz wszystko jak na tacy podane ..... ( rozbuduj o inne LEDY do robala)

Między wierszami czytam że diody na PORTB masz zapięte tylko pytanie co jest na PORTC pinie PC0

Statystyki: Napisane przez brat_kamyka — 26 mar 2017, o 13:23

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T11:18:30+01:00

dziękuję za cenne rady

Statystyki: Napisane przez wojciechgorzolka — 26 mar 2017, o 11:18

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T10:35:36+01:00

Daro69 napisał(a):

@brat_kamyka
Normalnie wróżka jesteś.
(bez fotki i kompletnego kodu)-...
Jakim cudem domyśliłeś się że problemem jest definicja przycisku. ??

bo błędów logicznych w funkcjach nie znalazłem ...
to jak nie kod ???? to co ???? HARDWARE !!!

Teraz pytanie do autora wątku po co tworzy się funkcje???
Odp. aby poprawić czytelność kodu,
funkcja to cześć kodu realizująca konkretne zadanie ( np. lcd_init(); itd....),
ja przeważnie jak pisze to bardzo podobnie do Mirka,
wpierw wszystko {co dotyczy jednego problemu} w funkcji main testuję czy działa zgodnie z założeniami,
potem przenoszę kod do osobnej funkcji ale uwaga nazwa funkcji musi mi mówić co ona robi (np: send_spi(string mojstring);)
i wszystko jasne nie.
Innymi słowy :
Programowanie to jak budowanie z klocków LEGO wpierw małe dzieci uczą składać klocki DUPLO (każdy klocek to prosta funkcja na przykład: void zapal_diode1(void);)
potem przychodzi czas na zwykłe klocki LEGO (np viod send_spi(string mojstring); która używa klocka DUPLO viod sendchar_spi(char A);)
Następnie dostajemy zestaw Lego Technic (np funkcja która ma za zadanie wysłanie napisów do wyświtlacza graficznego podpiętego po SPI ....)
a LEGO mindstorm to już wszystko (np prosty system czasu pseudo rzeczywistego -- timery programowe Mirka czy inne sposoby uruchamiania klocków lego Technic).

[syntax=c]while(1)
{
if( KLAWISZ_WCISNIETY == 0) PORTB = 0; //to się wykona tylko wtedy gdy KLAWISZ_WCISNIETY przyjmuje wartość 0
else if( KLAWISZ_WCISNIETY ) _delay_ms(80); //to się wykona tylko wtedy gdy KLAWISZ_WCISNIETY przyjmuje wartość większą od 0
if( KLAWISZ_WCISNIETY ) { //!!!!nowe sprawdzenie warunku!!!! to się wykona tylko wtedy gdy KLAWISZ_WCISNIETY przyjmuje wartość większą od 0
migacz_foor();
migacz_foor();
migacz_foor();
}[/syntax]

Pozdrawiam brat_kamyka.

Statystyki: Napisane przez brat_kamyka — 26 mar 2017, o 10:35

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T07:35:14+01:00

oo cześć Darku. na timerki przyjdzie czas

Statystyki: Napisane przez wojciechgorzolka — 26 mar 2017, o 07:35

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-26T01:19:51+01:00

hejka,

brat_kamyka napisał(a):

OK ale podpiąć możesz do dowolnego portu ATMEGI ...
napisz proszę Ledy port B przycisk pin PC2 czy jak ?

Najlepiej daj cały kod ...

@brat_kamyka
Normalnie wróżka jesteś.
(bez fotki i kompletnego kodu)-...
Jakim cudem domyśliłeś się że problemem jest definicja przycisku. ??

@Wojtek,
Widzę że zaczynasz ogarniać pisanie własnych funkcji. - super.
Przy najbliższej okazji proponuję zainteresować się timerami , bo widzę że używasz jeszcze delaye.
Wbrew pozorom timery to nie straszydło - tylko tak wygląda na pierwszy rzut oka. a są bardzo przydatne.
Jak wejdą w krew,
to nawet mruganie jedną diodą chce się pisać z wykorzystaniem timera.

Pozdrawiam Darek.

Statystyki: Napisane przez Daro69 — 26 mar 2017, o 01:19

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-25T23:09:29+01:00

ok, już kopiuję cały kod, chociaż nie chętnie, ponieważ nie chciał bym, aby inni mnie oskarżyli o pójście na łatwiznę . Nie oczekuję konkretnego rozwiązania, ponieważ w taki sposób się nie nauczę pisać kodu. [syntax=c]#include
#include

#define KLAWISZ (1<#define KLAWISZ_WCISNIETY !(PIND & KLAWISZ) //makro sprzwdzające czy klawisz jest wciśnięty

#define KLAWISZ_2 (2<#define KLAWISZ_2_WCISNIETY !(PIND & KLAWISZ_2)

void robal(void);//pierwsza funkcia
void robal_2(void);//druga funkcja
void pulsar(void);// trzecia funkcja

void migacz_foor(void);//funkcja migania diodami - pętla foor

//void klawisz_1_robal(void);
//********************************************
int main (void)
{
DDRB = 0xFF;// wejscie-wyjście - ustaw jako wyjście
PORTB = 0xFF;//podanie stanu wysokiego na PORTB - wygaszenie wszystkich diod
DDRC |= (1<PORTC &= ~(1<PORTB |= (1<DDRD &= ~KLAWISZ;
PORTD |= KLAWISZ;
DDRD &= ~KLAWISZ_2;
PORTD |= KLAWISZ_2;

_delay_ms(500);
while(1)
{
if( KLAWISZ_WCISNIETY == 0)

PORTB = 0;

else
if( KLAWISZ_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}

/*if(( KLAWISZ_WCISNIETY==0)&&( KLAWISZ_2_WCISNIETY==0 ))

PORTB = 0;

else if(( KLAWISZ_WCISNIETY )&&( KLAWISZ_2_WCISNIETY ))
_delay_ms(80);
if(( KLAWISZ_WCISNIETY )&&( KLAWISZ_2_WCISNIETY ))
{
migacz_foor();
migacz_foor();
migacz_foor();
}
*/
}

}

/* if( KLAWISZ_WCISNIETY == 0)

PORTB = 0;

else
{
if( KLAWISZ_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}
}

}
}
else
{
if( KLAWISZ_2_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_2_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}

}

}}

*/

//****************************************************************
//definicje funkcji

void robal(void)//definicja pierszej funkcji
{
//PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<PORTC ^= 1<_delay_ms(30);
PORTC ^= 1<
}

void robal_2(void)//definicja drugiej funkcji
{
//PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<_delay_ms(30);
PORTB ^= 1<
}

void pulsar(void)//definicja trzeciej funkcji
{
PORTB ^=1<_delay_ms(500);
PORTB ^=1<_delay_ms(500);

}

void migacz_foor(void)//definicja funkcji z pętlą foor
{
uint8_t i, liczba=1;
for( i=0; i<8; i++)
{
PORTB = ~liczba;
liczba = liczba * 2;
_delay_ms(30);
PORTB = 0xFF;
_delay_ms(10);
}

void klawisz_1_robal(void)//definicja funkcji obsługi klawisza z efektem fobala
{
if( KLAWISZ_WCISNIETY == 0)

PORTB = 0;

else
if( KLAWISZ_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}
}

}[/syntax]

Pytałem o to , czy funkcje jedna w drugiej będą działać, ponieważ chciałem sobie uprościć to co jest w pętli głównej i zastąpić większość kodu nowymi dwiema funkcjami (od pierwszego klawisza i drugiego). Ogólnie rzecz biorąc, to chcę uzyskać efekt następujący:
Gdy wcisnę klawisz pierwszy, to ma mi się wykonać robal trzy razy, to samo na drugim klawiszu. Jednak nie jest to takie proste dla mnie. Gdy napiszę instrukcję dla jednego klawisza, to działa pięknie, problem mam też z tym, aby w drugim klawiszu to samo się wykonywało.
Podaję przykład jaka jest moja koncepcja na dwa klawisze, niestety po wgraniu kodu nie działa mi to poprawnie :
[syntax=c]if(( KLAWISZ_WCISNIETY==0)&&( KLAWISZ_2_WCISNIETY==0 ))

PORTB = 0;

else if(( KLAWISZ_WCISNIETY )&&( KLAWISZ_2_WCISNIETY ))
_delay_ms(80);
if(( KLAWISZ_WCISNIETY )&&( KLAWISZ_2_WCISNIETY ))
{
migacz_foor();
migacz_foor();
migacz_foor();
}[/syntax]

Czy mogę połączyć obsługę dwóch klawiszy operatorem iloczynu && ?

Gdy klawisze nie są wciśnięte, to PORTB ma stan 0,
gdy wcisnę klawisz i osczekam 80ms , sprawdzę stan klawisza, to robal wykona się trzy razy. i koniec:)

Dodam jeszcze,że myślałem o koncepcji drugiej :
if.........
else if..........
else if.........

------------------------ [ Dodano po: 14 minutach ]

taka koncepcja, ale w niej działa tylko pierwszy klawisz, drugi niestety nie uruchamia robala, tylko dlaczego?
[syntax=c]while(1)
{
if( KLAWISZ_WCISNIETY == 0)

PORTB = 0;

else
if( KLAWISZ_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}

else
if( KLAWISZ_2_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_2_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}[/syntax]

po kompilacji błędy n ie wyskoczyły. a mimo to nie działa.

------------------------ [ Dodano po: 30 minutach ]

:D EUREKA!!!!
znalazłem błąd [syntax=c]#define KLAWISZ_2 (2<
źle zdefiniowałem KLAWISZ_2. ufff:))
znów jest satysfakcja zwłaszcza,że jestem bardzo zielony z programowania

Bardzo Ci dziękuję za zainteresowanie moim pytaniem

Statystyki: Napisane przez wojciechgorzolka — 25 mar 2017, o 23:09

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-25T22:46:23+01:00

OK ale podpiąć możesz do dowolnego portu ATMEGI ...
napisz proszę Ledy port B przycisk pin PC2 czy jak ?

Najlepiej daj cały kod ...

funkcja void może być wywoływana w innej funkcji.
przecież funkcja main jest typu void (na AVR oczywiście).

Statystyki: Napisane przez brat_kamyka — 25 mar 2017, o 22:46

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-25T22:41:29+01:00

podłączenie diod jak w zestawie ATB czyli katodą do uC, załączają się stanem niskim. kod już poprawiłem.

Statystyki: Napisane przez wojciechgorzolka — 25 mar 2017, o 22:41

Re: funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-25T22:36:48+01:00

funkcja migacz_foor() zadziała gdy KLAWISZ_WCISNIETY da wartość prawda czyli >0, tylko pytanie kiedy się to wykona...
u ciebie jest tak :

[syntax=c]if( KLAWISZ_WCISNIETY == 0)

PORTB = 0; // nie wiem co tam zawiszone jest klawisz czy diody cz coś innego bo nic nie piszesz
//

else
if( KLAWISZ_WCISNIETY )_delay_ms(80); // koniec if( KLAWISZ_WCISNIETY == 0)

//dalej nowy if
if( KLAWISZ_WCISNIETY ) // zwraca TRUE tylko jak większe od 0 jak zero to FALSE
{
migacz_foor();
migacz_foor();
migacz_foor();

}[/syntax]

nic nie piszesz o podłaczeniu hardware przyciski diody czy co tam masz dalej pomagać bedzie trudno .....
ps do umieszczenia kodów używaj znaczników syntax=c /syntax

Statystyki: Napisane przez brat_kamyka — 25 mar 2017, o 22:36

funkcja typu void w funkcji typu void- czy to zadziała?

2017-03-25T22:39:23+01:00

Witam.
Piszę sobie mały programik do obsługi klawisza i ledów z zestawu ATB i mam problem , bo nie wiem czy ja coś źle robię, czy moja funkcja jest abstrakcją jak obrazy niektórych malarzy. Czy to w ogóle ma prawo działać? Przytoczę przykład -ciało funkcji dla lepszego zobrazowania sytuacji:
[syntax=c]void klawisz_1_robal(void)//definicja funkcji obsługi klawisza z efektem fobala
{
if( KLAWISZ_WCISNIETY == 0)

PORTB = 0;

else
if( KLAWISZ_WCISNIETY )
_delay_ms(80);
if( KLAWISZ_WCISNIETY )
{
migacz_foor();
migacz_foor();
migacz_foor();

}
}[/syntax]

wszystko jest pisane dosyć liniowo, ponieważ uczę się posługiwania funkcjami i if-ami, ale jeśli moja funkcja migacz_foor(); jest funkcją typu void i jest wpisana do funkcji, która także jest typu void, to czy to w ogóle będzie działaś? Funkcja migacz_foor() jest typowym robakiem, czyli przesuwa zapaloną diodę z lewa na prawo. w

Statystyki: Napisane przez wojciechgorzolka — 25 mar 2017, o 22:14