ATNEL tech-forum

Re: problem z programem

2014-05-26T22:02:17+01:00

jakieś sugestie?

Statystyki: Napisane przez ifrook — 26 maja 2014, o 22:02

Re: problem z programem

2014-05-26T11:43:00+01:00

Próbowałem tego to nie działa teraz dioda zachowuje się tak że gdy D1 jest włączona moge właczyć D2 ale tylko na chwile bo ta zaraz gaśnie.

Założyłem sobie że ma to być tak że po wciśnięciu D1 nie mogę włączyć D2 i dodatkowo jesli włączyłem D1 podczas świecenia D2 to D2 gaśnie. Wydaje mi się że ta definicja musi być bardziej rozbudowana ale nie mam pomysłu jak;/

Statystyki: Napisane przez ifrook — 26 maja 2014, o 11:43

Re: problem z programem

2014-05-26T09:17:14+01:00

ifrook napisał(a):

A w którym miejscu napisałem, że trzeba coś tam sterować plusem? Kolega chyba nadal jest zmęczony i nieuważnie czyta.

A zamiast tego [syntax=c]#define LED_PIN2OFF !(1<Statystyki: Napisane przez jacekk232 — 26 maja 2014, o 09:17

Re: problem z programem

2014-05-26T00:17:41+01:00

wszystko działa, może wczoraj ze zmeczenia coś źle zmieniłem w kodzie nie wiem.
Napisałeś że na ATB diody zmieniane są stanem niskim przecież z kodu jaki wkleiłem a Ty poprawiłeś wynika że musi być sterowane plusem ?

Chciałbym dodać do kodu aby po zgaszeniu diody 1 automatycznie zgasła też dioda druga

wymyśliłem coś takiego ale wyskoczył mi żółty błąd

dodałem
include LED_PIN2OFF

[syntax=c]/*
* main.c
*
* Created on: 2010-03-27
* Autor: Mirosław Kardas
*/
#include
#include

// ************* definicje dla preprocesora
// dioda LED1
#define LED_PIN1 (1<#define LED_TOG1 PORTC ^= LED_PIN1 // makrodefinicja – zmiana stanu diody

// dioda led2

#define LED_PIN2 (1<#define LED_TOG2 PORTC ^= LED_PIN2
#define LED_PIN2OFF !(1<// klawisz K1
#define KEY_PIN1 (1<#define KEY_DOWN1 !(PINC & KEY_PIN1)

// klawisz K2

#define KEY_PIN2 (1<#define KEY_DOWN2 !(PINC & KEY_PIN2)

int klawisz_wcisniety1(void); // deklaracja funkcji
int klawisz_wcisniety2(void);

// *************************************************** pętla główna main()
int main(void)
{
// ****** inicjalizacja *********
DDRC |= LED_PIN1; // kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN1; // wyłączenie diody LED
DDRC &= ~KEY_PIN1; // kierunek pinu PC6 - wejściowy
PORTC |= KEY_PIN1; // podciągnięcie pinu do VCC
DDRC|= LED_PIN2;
PORTC |= LED_PIN2;
DDRC &= ~KEY_PIN2;
PORTC |= KEY_PIN2;

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety1() ) // jeśli klawisz wciśnięty
{
LED_TOG1; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}

if(PINC & LED_PIN1)
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}
if(PINC & LED_PIN1)
{
LED_PIN2OFF;
}

}

}
//******************************************************** koniec main()

// definicja funkcji
int klawisz_wcisniety1(void)
{
if( KEY_DOWN1 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN1 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

int klawisz_wcisniety2(void)
{
if( KEY_DOWN2 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN2 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}[/syntax]

Statystyki: Napisane przez ifrook — 26 maja 2014, o 00:17

Re: problem z programem

2014-05-25T09:17:37+01:00

Jeśli chcesz zmieniać stan drugiej diody tylko wtedy gdy pierwsza świeci to powinieneś napisać tak[syntax=c]if( !(PINC & LED_PIN1) )[/syntax]jeśli pierwszą diodę zapalasz stanem niskim, albo tak[syntax=c]if( (PINC & LED_PIN1) )[/syntax]jesli zapalasz ją stanem wysokim.
W pierwszym przypadku warunek jest spełniony jeśli na pinie LED1 jest stan niski, a w drugim - jeśli jest stan wysoki.

Na ATB diody włączane są stanem niskim i mi działa pierwszy przykład.

ifrook napisał(a):

Gdy ten sam program napisze dla diód i przycisków które znajdują się na rejestrach C to dzieją się cuda

A jakiego procka używasz? Może używasz takiego, który na porcie C ma JTAG. I dlatego część pinów może Ci nie działać.

Statystyki: Napisane przez jacekk232 — 25 maja 2014, o 09:17

Re: problem z programem

2014-05-24T21:15:24+01:00

z przyzwyczajenia wiem że prawidłowo jest uint8_t

Pomogłeś mi dzięki, ale uwaga
if( !(PINC & LED_PIN1) ) dioda 2 da się włączać i wyłączać gdy 1 jest w stanie Low
dlatego wpisałem if( (PINC & LED_PIN1) ) i teraz działa jak chciałem ale dlaczego?

Gdy ten sam program napisze dla diód i przycisków które znajdują się na rejestrach C to dzieją się cuda

czy to ma związek jakiś z odczytem PINC ?

Statystyki: Napisane przez ifrook — 24 maja 2014, o 21:15

Re: problem z programem

2014-05-24T19:54:34+01:00

W pętli while zamiast [syntax=c]if(LED_PIN1)
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}[/syntax]
wstaw [syntax=c]if( !(PINC & LED_PIN1) )
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}[/syntax]
W pierwszym przykładzie zamiast LED_PIN1 preprocesor wstawi 0b10000000. Czyli ten warunek zawsze będzie spełniony i nie będzie to miało żadnego związku z diodą LED1.

A tak poza tym to dlaczego wszędzie używasz typu int zamiast wygodnych typów, które są w gcc np. uint8_t?

Statystyki: Napisane przez jacekk232 — 24 maja 2014, o 19:54

Re: problem z programem

2014-05-24T19:45:25+01:00

[syntax=c]/*
* main.c
*
* Created on: 2010-03-27
* Autor: Mirosław Kardas
*/
#include
#include

// ************* definicje dla preprocesora
// dioda LED1
#define LED_PIN1 (1< #define LED_TOG1 PORTC ^= LED_PIN1 // makrodefinicja – zmiana stanu diody

// dioda led2

#define LED_PIN2 (1< #define LED_TOG2 PORTA ^= LED_PIN2

// klawisz K1
#define KEY_PIN1 (1< #define KEY_DOWN1 !(PINC & KEY_PIN1)

// klawisz K2

#define KEY_PIN2 (1< #define KEY_DOWN2 !(PINA & KEY_PIN2)

int klawisz_wcisniety1(void); // deklaracja funkcji
int klawisz_wcisniety2(void);

// *************************************************** pętla główna main()
int main(void)
{
// ****** inicjalizacja *********
DDRC |= LED_PIN1; // kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN1; // wyłączenie diody LED
DDRC &= ~KEY_PIN1; // kierunek pinu PC6 - wejściowy
PORTC |= KEY_PIN1; // podciągnięcie pinu do VCC
DDRA|= LED_PIN2;
PORTA |= LED_PIN2;
DDRA &= ~KEY_PIN2;
PORTA |= KEY_PIN2;

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety1() ) // jeśli klawisz wciśnięty
{
LED_TOG1; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}

if(PINC & LED_PIN1)
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}
}

}
//******************************************************** koniec main()

// definicja funkcji
int klawisz_wcisniety1(void)
{
if( KEY_DOWN1 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN1 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

int klawisz_wcisniety2(void)
{
if( KEY_DOWN2 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN2 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}[/syntax]

Statystyki: Napisane przez gumeni — 24 maja 2014, o 19:45

Re: problem z programem

2014-05-24T17:49:57+01:00

chciałbym stan diody drugiej można było zmieniać tylko gdy dioda pierwsza świeci.

wydawało mi się że powinno to być coś takiego

[syntax=c]/*
* main.c
*
* Created on: 2010-03-27
* Autor: Mirosław Kardas
*/
#include
#include

// ************* definicje dla preprocesora
// dioda LED1
#define LED_PIN1 (1<#define LED_TOG1 PORTC ^= LED_PIN1 // makrodefinicja – zmiana stanu diody

// dioda led2

#define LED_PIN2 (1<#define LED_TOG2 PORTA ^= LED_PIN2

// klawisz K1
#define KEY_PIN1 (1<#define KEY_DOWN1 !(PINC & KEY_PIN1)

// klawisz K2

#define KEY_PIN2 (1<#define KEY_DOWN2 !(PINA & KEY_PIN2)

int klawisz_wcisniety1(void); // deklaracja funkcji
int klawisz_wcisniety2(void);

// *************************************************** pętla główna main()
int main(void)
{
// ****** inicjalizacja *********
DDRC |= LED_PIN1; // kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN1; // wyłączenie diody LED
DDRC &= ~KEY_PIN1; // kierunek pinu PC6 - wejściowy
PORTC |= KEY_PIN1; // podciągnięcie pinu do VCC
DDRA|= LED_PIN2;
PORTA |= LED_PIN2;
DDRA &= ~KEY_PIN2;
PORTA |= KEY_PIN2;

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety1() ) // jeśli klawisz wciśnięty
{
LED_TOG1; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}

if(LED_PIN1)
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}
}

}
//******************************************************** koniec main()

// definicja funkcji
int klawisz_wcisniety1(void)
{
if( KEY_DOWN1 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN1 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

int klawisz_wcisniety2(void)
{
if( KEY_DOWN2 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN2 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}[/syntax]

Statystyki: Napisane przez ifrook — 24 maja 2014, o 17:49

Re: problem z programem

2014-05-24T15:31:38+01:00

udalo mi sie

Statystyki: Napisane przez ifrook — 24 maja 2014, o 15:31

Re: problem z programem

2014-05-24T15:27:18+01:00

Za dużo nawiasów miałeś.

[syntax=c]/*
* main.c
*
* Created on: 2010-03-27
* Autor: Mirosław Kardas
*/
#include
#include

// ************* definicje dla preprocesora
// dioda LED1
#define LED_PIN1 (1<#define LED_TOG1 PORTC ^= LED_PIN1 // makrodefinicja – zmiana stanu diody

// dioda led2

#define LED_PIN2 (1<#define LED_TOG2 PORTA ^= LED_PIN2

// klawisz K1
#define KEY_PIN1 (1<#define KEY_DOWN1 !(PINC & KEY_PIN1)

// klawisz K2

#define KEY_PIN2 (1<#define KEY_DOWN2 !(PINA & KEY_PIN2)

int klawisz_wcisniety1(void); // deklaracja funkcji
int klawisz_wcisniety2(void);

// *************************************************** pętla główna main()
int main(void)
{
// ****** inicjalizacja *********
DDRC |= LED_PIN1; // kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN1; // wyłączenie diody LED
DDRC &= ~KEY_PIN1; // kierunek pinu PC6 - wejściowy
PORTC |= KEY_PIN1; // podciągnięcie pinu do VCC
DDRA|= LED_PIN2;
PORTA |= LED_PIN2;
DDRA &= ~KEY_PIN2;
PORTA |= KEY_PIN2;

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety1() ) // jeśli klawisz wciśnięty
{
LED_TOG1; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}

if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2; // zmień stan diody na przeciwny
_delay_ms(200); // pauza 200ms
}
}

}
//******************************************************** koniec main()

// definicja funkcji
int klawisz_wcisniety1(void)
{
if( KEY_DOWN1 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN1 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

int klawisz_wcisniety2(void)
{
if( KEY_DOWN2 ) // klawisz wciśnięty ?
{
_delay_ms(80); // czas drgań styków
if( KEY_DOWN2 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0; // jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}[/syntax]

Statystyki: Napisane przez gumeni — 24 maja 2014, o 15:27

Re: problem z programem

2014-05-24T14:46:13+01:00

Panie kolego to nie jest twój pierwszy post na forum, już wielokrotnie moderatorzy i ja także przypominaliśmy tobie i pokazywaliśmy jak się umieszcza kod źródłowy na forum w znacznikach [ syntax = c ] ....

jeśli więc nie zaczniesz tego stosować i nie poprawisz tego co wyżej to przykro mi ale będę usuwał takie posty

Statystyki: Napisane przez mirekk36 — 24 maja 2014, o 14:46

problem z programem

2014-05-24T15:31:05+01:00

W ramach ćwiczeń chciałem sobie dołożyć drugą diodę i przycisk do programu Mirka ale koniec końców ta właśnie dioda cały czas jest zasilona ;/ gdzie tkwi błąd?

Faktycznie poprawiam się już ;p

[syntax=c]
/*
* main.c
*
* Created on: 2010-03-27
* Autor: Mirosław Kardas
*/
#include
#include

// ************* definicje dla preprocesora
// dioda LED1
#define LED_PIN1 (1<#define LED_TOG1 PORTC ^= LED_PIN1// makrodefinicja – zmiana stanu diody

// dioda led2

#define LED_PIN2 (1<#define LED_TOG2 PORTA ^= LED_PIN2

// klawisz K1
#define KEY_PIN1 (1<#define KEY_DOWN1 !(PINC & KEY_PIN1)

// klawisz K2

#define KEY_PIN2 (1<#define KEY_DOWN2 !(PINA & KEY_PIN2)

int klawisz_wcisniety1(void);// deklaracja funkcji
int klawisz_wcisniety2(void);

// *************************************************** pętla główna main()
int main(void)
{
// ****** inicjalizacja *********
DDRC |= LED_PIN1;// kierunek pinu PC7 – wyjściowy
PORTC |= LED_PIN1;// wyłączenie diody LED
DDRC &= ~KEY_PIN1;// kierunek pinu PC6 - wejściowy
PORTC |= KEY_PIN1;// podciągnięcie pinu do VCC
DDRA|= LED_PIN2;
PORTA |= LED_PIN2;
DDRA &= ~KEY_PIN2;
PORTA |= KEY_PIN2;

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety1() ) // jeśli klawisz wciśnięty
{
LED_TOG1;// zmień stan diody na przeciwny
_delay_ms(200);// pauza 200ms
}
}
{
if( klawisz_wcisniety2() ) // jeśli klawisz wciśnięty
{
LED_TOG2;// zmień stan diody na przeciwny
_delay_ms(200);// pauza 200ms
}
}

}
//******************************************************** koniec main()

// definicja funkcji
int klawisz_wcisniety1(void)
{
if( KEY_DOWN1 )// klawisz wciśnięty ?
{
_delay_ms(80);// czas drgań styków
if( KEY_DOWN1 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0;// jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}

int klawisz_wcisniety2(void)
{
if( KEY_DOWN2 )// klawisz wciśnięty ?
{
_delay_ms(80);// czas drgań styków
if( KEY_DOWN2 ) return 1; // jeśli wciśnięty? zakończ funkcję - rezultat = 1
}

return 0;// jeśli nie wciśnięty klawisz, zakończ funkcję, rezultat = 0
}[/syntax]

Statystyki: Napisane przez ifrook — 24 maja 2014, o 14:36