ATNEL tech-forum

Re: Obroty silnika a Micro Switch

2013-12-12T21:15:40+01:00

Powiedziałbym, że nie tylko delay jest problemem.
W skrócie:
-zmienna key_lock - pozostałość po poprzednich wersjach kodu
-zmienna i - nie jest zerowana- wiadomo, że jak dojdzie do 255 to po kolejnej instrukcji zrobi się 0, ale chyba jakiś porządek musi być
-zamiast tego[syntax=c]else
{
if[/syntax] napisałbym else if'a- chociaż różnica żadna

Statystyki: Napisane przez Lukaszm — 12 gru 2013, o 21:15

Re: Obroty silnika a Micro Switch

2013-12-12T19:47:23+01:00

SunRiver napisał(a):

Noo Brawo ...
ale ... a zresztą ważne że działa

No Sun powiedz, że te delaye to masakra jest.
Oczywiście najważniejsze, że ruszyło i działa jednak trzeba przystąpić do poprawienia kodu
pod kątem eliminacji tych delay'ów

------------------------ [ Dodano po: 1 minucie ]

trochę się też Koledze formatowanie "rozjechało"

Statystyki: Napisane przez MirkoT — 12 gru 2013, o 19:47

Re: Obroty silnika a Micro Switch

2013-12-12T18:11:09+01:00

Noo Brawo ...
ale ... a zresztą ważne że działa

Statystyki: Napisane przez SunRiver — 12 gru 2013, o 18:11

Re: Obroty silnika a Micro Switch

2013-12-12T18:06:23+01:00

Witam,
udało mi się rozwiązać problem
[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<uint8_t key_lock;
uint8_t i=2;

int main (void){
DDRC |= (1<DDRC &= ~key_pin;
PORTC |= key_pin;
_delay_ms(10);

{
while(1)
{
if(i%2==0)
{
if( !key_lock && !(PINC & key_pin) ) {
DC_LEWO;
_delay_ms(5000);
DC_STOP;
i=i+1;
}
}
else
{
if( !key_lock && !(PINC & key_pin) ){
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
i=i+1;
}
}
}

}
}[/syntax]

Statystyki: Napisane przez Camillo123 — 12 gru 2013, o 18:06

Re: Obroty silnika a Micro Switch

2013-12-11T20:01:30+01:00

looknij na PW

Statystyki: Napisane przez SunRiver — 11 gru 2013, o 20:01

Re: Obroty silnika a Micro Switch

2013-12-11T19:32:19+01:00

Witam,
błąd był w obsłudze Micro Switcha. Kod zmodyfikowałem, więc już problem z przyciskiem nie powinno być, lecz teraz silnik kręci się cały czas
[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<

int main (void){
DDRC |= (1< PINC |= key_pin;
uint8_t i=2;
uint8_t key_lock;
_delay_ms(10);
while(1)
{
if( i % 2 == 0 )
{
if( !key_lock && !(PINC & key_pin) ) {
key_lock=1;
DC_LEWO;
_delay_ms(5000);
DC_STOP;
}
}
else
{
if( key_lock && (PINC & key_pin ) ) key_lock++;
{
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
}
}i = i + 1;
}

}[/syntax]

Statystyki: Napisane przez Camillo123 — 11 gru 2013, o 19:32

Re: Obroty silnika a Micro Switch

2013-12-10T19:45:05+01:00

Wątpię by błąd był w tych liniach... Sądzę, że to coś z "i"

Statystyki: Napisane przez Camillo123 — 10 gru 2013, o 19:45

Re: Obroty silnika a Micro Switch

2013-12-10T19:40:18+01:00

Stawiałbym na jakiś tyci błąd w liniach 7. - 9., jednak niestety będę mógł to później przeanalizować

Statystyki: Napisane przez Waszek — 10 gru 2013, o 19:40

Re: Obroty silnika a Micro Switch

2013-12-10T19:16:12+01:00

Po zmianie błąd powtarza się jeszcze częściej

Statystyki: Napisane przez Camillo123 — 10 gru 2013, o 19:16

Re: Obroty silnika a Micro Switch

2013-12-10T19:10:09+01:00

Spróbuj ustawić na początku i = 0

Statystyki: Napisane przez Waszek — 10 gru 2013, o 19:10

Re: Obroty silnika a Micro Switch

2013-12-10T19:05:39+01:00

Witam,
zmniejszyłem czas DC_STOP; do 50ms, Efekt jest taki: silnik kręci się raz w prawo, raz w lewo(prawidłowo), lecz raz na jakiś czas zdarza mu się dwa lub trzy razy obrócić w prawo lub lewo Wie ktoś jak temu zaradzić? Z góry dziękuje

Statystyki: Napisane przez Camillo123 — 10 gru 2013, o 19:05

Re: Obroty silnika a Micro Switch

2013-12-10T18:36:55+01:00

[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<#define key_down !(PINC & key_pin)

int main (void){
DDRC |= (1< DDRC &= ~key_pin;
PORTC |= key_pin;
uint8_t i=1;
while(1)
{
if( i % 2 == 0 )
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_LEWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
}
else
{
if (key_down)
{
_delay_ms(80);
if (key_down)
{
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}
}
}
i = i + 1;
}
}[/syntax]

Według mnie powinno hulać już, poprawiłem lekko kod, no bo zmienna zwiększa się za każdym razem nie zależnie od warunku
i % 2 == 0 - to sprawdza czy i jest parzyste
% to dzielenie modulo, czyli wynikiem jest reszta z dzielenia. Jeżeli liczba jest parzysta, to liczba z dzielenia przez dwa jest równa zero, w przeciwnym wypadku jest równa 1.
Jeżeli chodzi o stronę elektroniczną, pytaj starszych Kolegów
Zastanawiam się tylko nad tym, jak wyczerpie się zakres, który obsługuje tę zmienną. Co wtedy się stanie? Czy koło się zamknie i znowu zostanie liczona od 0 ta zmienna?

Statystyki: Napisane przez Waszek — 10 gru 2013, o 18:36

Re: Obroty silnika a Micro Switch

2013-12-10T16:57:24+01:00

Witam,
jeszcze raz poprawiłem kod. Czy teraz jest on poprawny? Z góry dziękuje.
[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<#define key_down !(PINC & key_pin)

int main (void){
DDRC |= (1<DDRC &= ~key_pin;
PORTC |= key_pin;
uint8_t i=1;
while(1)
{
if(i%1==0)
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_LEWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
i=i+1;
}
else
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
i=i+1;
}
}
}[/syntax]

Statystyki: Napisane przez Camillo123 — 10 gru 2013, o 16:57

Re: Obroty silnika a Micro Switch

2015-11-06T14:23:52+01:00

Statystyki: Napisane przez xor — 10 gru 2013, o 16:45

Re: Obroty silnika a Micro Switch

2013-12-10T14:35:29+01:00

Witam,
udało mi się napisać program bez błędów i ostrzeżeń
[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<#define key_down !(PINC & key_pin)

int main (void){
DDRC |= (1<DDRC &= ~key_pin;
PORTC |= key_pin;
uint8_t i=0;
while (i==i+1)
{

if (i%1==0)
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_LEWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
}
else
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
}
}
}[/syntax]
Dzięki wielkie za pomoc

Statystyki: Napisane przez Camillo123 — 10 gru 2013, o 14:35

Re: Obroty silnika a Micro Switch

2013-12-09T23:46:12+01:00

Kolego Camillo123
Na początku spróbuj sobie wyjaśnić jak to ma działać
Masz na początku licznik i = 0. Ważne słowa 'na początku' dalej w nieskończonej pętli while z każdym jej obrotem następuje inkrementacja - wartość licznika zwiększa się o 1 za każdym kółkiem za każdym obrotem pętli automatycznie jest sprawdzany warunek, czy wartość licznika jest nieparzysta ( i % 2 == 1 ). Jeżeli tak, to mają się wykonać określone instrukcje
Spróbuj przeanalizować swój program z moim tokkem myślenia i odpowiednio go naprawić
Lekka modyfikacja i będzie fruwało w tym miejscu
Pozdrawiam
Waszek

Statystyki: Napisane przez Waszek — 9 gru 2013, o 23:46

Re: Obroty silnika a Micro Switch

2013-12-09T22:34:47+01:00

no niestety totalnie źle - tzn kolega nie do końca rozumie jak stosuje się po prostu tak podstawowe słowa kluczowe języka C jak warunki IF

w skrócie

if( wyrażenie ) ....zrób_coś....

czyli jeśli wyrażenie jest prawdziwe to wykonaj to co jest dalej

przy czym wyrażenie logiczne może być proste lub złożone. Jeśli proste to np:

if( a == c ) ....

a jeśli złożone to stosujemy operatory logiczne np && lub ||

if( a==c && w==2 ) ....

czyli jeśli a jest równe c a także w jest równe 2 to spełnij warunek

tymczasem kolega powpisywał w nawiasach jakieś przedziwne rzeczy niestety no i stąd równie przedziwny komunikat

posiada może kolega Blueboka ?

http://atnel.pl/mikrokontrolery-avr-jezyk-c.html

jeśli nie to polecam od niego zacząć, myślę, że wiele rzeczy się w głowie wyjaśni - nie tylko warunki if

Statystyki: Napisane przez mirekk36 — 9 gru 2013, o 22:34

Re: Obroty silnika a Micro Switch

2013-12-09T21:55:58+01:00

Chodzi o to:
jeżeli liczba przyciśnięć Micro Switch'a jest nieparzysta to jest wykonywana ta instrukcja, z każdym kliknięciem "i" (liczba przyciśnięć) zwiększ się o jeden, a początkowo wynosi 0.
Bardzo prawdopodobne, że to źle zapisałem

Statystyki: Napisane przez Camillo123 — 9 gru 2013, o 21:55

Re: Obroty silnika a Micro Switch

2013-12-09T21:44:07+01:00

Camillo123 napisał(a):

if (i=0 (i%1==0) (i=i+1))

to spróbuj wytłumaczyć co ma niby robić ta linia ? może jak rozwiniesz to - to sam sobie nawet odpowiesz, a jak nie to oczywiście pomożemy

Statystyki: Napisane przez mirekk36 — 9 gru 2013, o 21:44

Obroty silnika a Micro Switch

2013-12-09T21:36:50+01:00

Witam,
napisałem program do starowania silnika tak, aby po 1. wciśnięciu przyciska obroty były lewe, a po 2. prawe i analogicznie później. Miałem kilka błędów, ale udało mi się z nimi uporać, ale z tym nie:

Compiling C: silnik.c
avr-gcc -c -mmcu=atmega16 -I. -gdwarf-2 -DF_CPU=1000000UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Wall -Wstrict-prototypes -Wa,-adhlns=./silnik.lst -std=gnu99 -MMD -MP -MF .dep/silnik.o.d silnik.c -o silnik.o
silnik.c: In function 'main':
silnik.c:21: error: called object '0' is not a function
make.exe: *** [silnik.o] Error 1

[syntax=c]#include
#include

#define WE_A PC7
#define WE_B PC6

#define DC_PRAWO PORTC &= ~(1<#define DC_LEWO PORTC |= (1<#define DC_STOP PORTC &= ~(1<#define key_pin (1<#define key_down !(PINC & key_pin)

int main (void){
DDRC |= (1<DDRC &= ~key_pin;
PORTC |= key_pin;
uint8_t i;
while(1)
{

if (i=0 (i%1==0) (i=i+1))
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_LEWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
}
else
{
if (key_down)
{_delay_ms(80);
if (key_down)
{
DC_PRAWO;
_delay_ms(5000);
DC_STOP;
_delay_ms(3000);
}

}
}
}
}[/syntax]
Z góry dziękuje

Statystyki: Napisane przez Camillo123 — 9 gru 2013, o 21:36