ATNEL tech-forum

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-21T11:55:16+01:00

SylwekK napisał(a):

Zrób tryb debugowania na lcd. Wyślij tam wszystkie istotne zmienne na czas testów i obserwuj co się dzieje. Ja tak robię z każdym bardziej skomplikowanym programem, a i w prostszych też szybko można zlokalizować błąd - np. jeśli coś pisze pod tiny13 to najpierw pisze to na mega32 z lcd i jeśli algorytm działa dopiero robię konwersję na inny procek.

Sprawdziłem tym sposobem co się dzieje ze zmiennymi result, wilg i inne. Wartość wpisana z klawiatury jak i z czujnika wpisują się i pokazują poprawnie, więc nie wiem gdzie leży problem z odczytem wartości.

Może zapisuje zły warunek, który ma za zadanie działać według wskazań czujnika i zmiennej result.

Statystyki: Napisane przez KampoPampo — 21 lis 2018, o 11:55

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-15T15:29:52+01:00

Statystyki: Napisane przez SylwekK — 15 lis 2018, o 15:29

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-15T15:16:24+01:00

Próbowałem dodać jakieś usprawnienia, ale problem cały czas jest ten sam, gdyż po zatwierdzeniu jakiejkolwiek wartości z klawiatury (od 1 do 100) pompka włącza się i działa cały czas nie reagując na Zmiany wartości wilgotności. Chciałbym wiedzieć o co chodzi, ale nie wiem. Jest ktoś w stanie podrzucić jakąś wskazówkę, bo na razie nie daje rady tego rozwiązać. Bardzo dziękuję za wcześniejsze odpowiedzi.

Statystyki: Napisane przez KampoPampo — 15 lis 2018, o 15:16

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-12T10:35:15+01:00

Daro69 napisał(a):

witam,

KampoPampo napisał(a):

[syntax=c]void check_result(int result)
{
result = liczba; //instrukcja sprawdzająca wprowadzony wynik
}[/syntax]

czy nie chodzilo o
[syntax=c]void check_result(int liczba) // ? liczba wprowadzona z klawiatury. ?
{
result = liczba; // zapis liczby do zmiennej result ? liczba jest argumentem funkcji. a nie result. hmm.. ?

}[/syntax]
trochę miesza mi się.

Początkowo było tak i spełniało swoje działanie, ale dodałem zmienna liczba, bo myślałem, że to pomoże jakoś w odczycie wyniku z result.

[syntax=c]void check_result(int result)
{
//instrukcja sprawdzająca wprowadzony wynik
}[/syntax]

Za bardzo utrudniam. Proszę o wybaczenie

Statystyki: Napisane przez KampoPampo — 12 lis 2018, o 10:35

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T22:01:45+01:00

witam,

KampoPampo napisał(a):

[syntax=c]void check_result(int result)
{
result = liczba; //instrukcja sprawdzająca wprowadzony wynik
}[/syntax]

Statystyki: Napisane przez Daro69 — 11 lis 2018, o 22:01

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T21:58:55+01:00

Porównujesz tam widzę "wilg i liczba", a nie widzę aby result był przepisany do zmiennej liczba. Tak ma być?
Sorki, ale strasznie chaotycznie wydaje mi się napisany program i jego analiza jest po prostu koszmarna jak dla mnie.

Statystyki: Napisane przez SylwekK — 11 lis 2018, o 21:58

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T21:48:40+01:00

Funkcja check_result jest po to, aby wartość wpisana i zatwierdzona na klawiaturze, została zapisana do zmiennej. Przetestowałem tę opcje i działa dobrze.
read_key stworzyłem tak, żeby odczytywało wartość z klawiatury, może na switchach byłoby łatwiej ale jakoś się nad tym nie skupiałem, ponieważ nie sądziłem, że może wpływać na opisany przeze mnie problem.
Instrukcje goto wiem, że jest beznadziejnym rozwiązaniem, ale użyłem to po to, żeby poruszać się łatwo po menu (powrót do głównego menu oraz powrót do początku wyboru wilgotnosci.

Teraz o działaniu programu. Działanie jego ma być następujące. Z głównego menu wybieram opcje A - podlewanie zależne od wilgotnosci. Przechodzę do punktu menu WILGOTNOŚĆ i program prosi mnie o wpisanie wartości wilgotności (wartość zapisuje się do result, czujnik podłączony pod ADC). Zatwierdzam liczbe i program uruchamia pompkę jeśli wartość wilgotności na czujniku jest mniejsza od wpisanej. I tak cały czas do zatrzymania działania.

To działa jako wykasowanie danych.

[syntax=c]if(key==16)
{
goto skok1;
}[/syntax]

A to jest powrót do głównego menu

[syntax=c]if(key == 13)
{

_delay_ms(400);
goto skok;
}[/syntax]

Statystyki: Napisane przez KampoPampo — 11 lis 2018, o 21:48

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T20:58:11+01:00

Zerknąłem w kod...
Po co funkcja check_result?
Funkcję read_key można by zrobić na instrukcji switch, mam wrażenie, że byłoby łatwiej.
Tak, jak pisałem wcześniej, zrezygnowałbym z instrukcji goto, to zły nawyk.
Napisz, jak ma program działać, może napiszemy go w inny sposób?

Statystyki: Napisane przez Marhef — 11 lis 2018, o 20:58

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T16:07:37+01:00

I pamiętaj że jedno '=' oznacza przypisanie w języku C!

Statystyki: Napisane przez micky — 11 lis 2018, o 16:07

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T14:55:20+01:00

Zobacz jeszcze raz mój post. Tam nie ma porównania zmiennej samej ze sobą. Te operatory logiczne powodują, że porównania ze zmienną użytą w warunku za każdym razem jest osobnym działaniem. Nie wiem jak Ci to prościej wytłumaczyć.

Statystyki: Napisane przez SylwekK — 11 lis 2018, o 14:55

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T14:19:06+01:00

Dostało mi się ale to nic przynajmniej jakoś się ten post rozruszał

Ale i dobrze bo i dla mnie z tego jakaś nauka wypłynie

wytłumaczcie mi jak można porównać jedną i tą samą zmienną z nią samą, nawet gdyby zmieniała
się jej wartość?

dla jasności zmienna result = result wartość z lewej strony będzie się równała wartości z prawej str.
dla mnie to wygląda tak mam zmienną resuit o wartości 5 i teraz to porównaj - ale z czym z nią samą
jeśli nawet porównam z nią samą to zawsze będzie równa samej sobie - masło maślane

gdybym przyjął do porównania result porównaj z result_1 to są dwie różne zmienne które można porównać i to jest OK

Czy znowu się mylę?

Statystyki: Napisane przez miki53 — 11 lis 2018, o 14:19

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-11T12:16:37+01:00

Dałem ciała i dyskusja poszła w inną stronę, ale nie uważam, że podpowiedzi odnośnie ustawiania warunków są złe. Zacznę od zawarcia kilku istotnych funkcji, które mogą wyjaśnić wam przyczynę mojego problemu, bo tak jak było podane, nawet ze szklanej kuli się tego nie wyczyta.. Czy coś..

Funkcja check_result()
[syntax=c]void check_result(int result)
{
result = liczba; //instrukcja sprawdzająca wprowadzony wynik
}[/syntax]

Rozpisanie działania klawiszy ze zmienną a
[syntax=c]int read_key ()
{

int key = read_keypad();

if(key == 12)
{

return key;
}
if(key == 13)
{
return key;
}

if(key == 15)

{

return key;

}

if(key == 16)
{

return key;
}
if(key == 1)
{
a=1;
return key;
}
if(key == 2)
{
a=2;
return key;
}
if(key == 3)
{
a=3;
return key;
}
if(key == 5)
{
a=4;
return key;
}
if(key == 6)
{
a=5;
return key;
}
if(key == 7)
{
a=6;
return key;
}
if(key == 9)
{
a=7;
return key;
}
if(key == 10)
{
a=8;
return key;
}
if(key == 11)
{
a=9;
return key;
}
if(key == 14)
{
a=0;
return key;
}

return -1;
}[/syntax]

Funkcja read_keypad() sprawdza kolejno wszystkie przyciski klawiatury i zwraca numer pierwszego wciśniętego przycisku, albo zero, gdy żaden przycisk nie został wciśnięty. Jest to instrukcja działania klawiatury matrycowej.

Funkcji ADC nie dodaje bo to, sam odczyt z adc jest prawidłowy, jednakże może gdzieś jest problem z wartością wilg.
[syntax=c]adc_value = ADC_odczyt();// wpisanie wartości odczytanej z adc do zmiennej adc_value
wilg = 100-(adc_value *100.00)/1023.00; // przetwarzanie pomiaru na %, 1023-100%[/syntax]

Pętla główna
[syntax=c]int main(void)
{
/* Konfiguracja portów we/wy */
ADC_set();

klaw_init();

lcd_init();

czuj();
zbiornik();

lcd_cls();
read_keypad();
//zmienna wilgotnosc typu zmiennoprzecinkowego
//wywołanie funkcji ADC
char array[10];

sei(); //Włączenie przerwań

skok:

lcd_locate(0,0);
lcd_str("MENU ZWD: A-WILG");
lcd_locate(1,0);
lcd_str("B-CZAS C-KON.AW.");

//lcd_locate(0,0);
//lcd_str("****MENU ZWD****");

//for(i=1; i<=3; ++i)
//{
//
//if(i==1){
//lcd_locate(1,0);
//lcd_str("A-WILG B-CZAS...");
//_delay_ms(2000);
//}
//
//if(i==2){
//lcd_locate(1,0);
//lcd_str("C-KON.AW. D-USUN");
//_delay_ms(2000);
//}
//if(i==3){
//lcd_locate(1,0);
//lcd_str("*-MENU #-WPISZ..");
//_delay_ms(2000);
//i=0;
//}
//}

/* Nieskończona pętla */

// for(;;)

for(;;)
if(key = read_keypad())
{

if(key == 4)
{

//wilgotnosc

_delay_ms(400);
skok1:
lcd_cls();
lcd_locate(0,0);
// lcd_cls();
lcd_str("Wilgotnosc: ");

int result = 0;
while (true) {

adc_value = ADC_odczyt();// wpisanie wartości odczytanej z adc do zmiennej adc_value
wilg = 100-(adc_value *100.00)/1023.00; // przetwarzanie pomiaru na %, 1023-100%
dtostrf(wilg,2,0,array); //konwertowanie liczby na tekst
lcd_locate(1,0);
lcd_str(array);
_delay_us(500);//

// if(wilg > liczba)
// {
// //PORTC &= ~(1<//
// PORTA |= (1<// }
//
// if (wilg <= liczba)
// {
// //PORTC |= (1<// PORTA &= ~(1<// }

int key = read_key();
if (key != -1) {

if(key==16)
{
goto skok1;
}
if(key == 13)
{

_delay_ms(400);
goto skok;
}

if (key == 15)
{

// lcd_cls();
if(result>0 && result<=100)
{

check_result(result);
result = 0;

}

else {
lcd_locate(0,0);
lcd_str("******BLAD******");
lcd_locate(1,0);
lcd_str("SPROBOJ PONOWNIE");
_delay_ms(2000);
goto skok1;
}

lcd_cls();
lcd_locate(0,0);
// lcd_cls();
lcd_str(" START PODLEW. ");
lcd_locate(1,0);
lcd_str("################");

}

else
{
result = result * 10 + a; //key;
//result = liczba;
lcd_locate(0,12);
//lcd_int(result);
lcd_int(result);
lcd_locate(0,15);
lcd_str(" ");

}
if(wilg >= liczba)
{
PORTA |= (1< }

else
{
PORTA &= ~(1< }

}

}

lcd_cls();
lcd_locate(0,12);
lcd_int(result);//result * 10 +

}
_delay_us(100);
}
return 0;
}[/syntax]

Zmienne

[syntax=c]int i, a=0, czas, poziom = 0, adc_value, liczba;

double wilg;[/syntax]

Mam nadzieję, że nic nie pominąłem.

Statystyki: Napisane przez KampoPampo — 11 lis 2018, o 12:16

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T20:50:41+01:00

Albo jeszcze inaczej. Dwa warunki, a pomiędzy nimi "&&" można rozbić na sekwencję jak poniżej czyli warunek zagnieżdżony:

[syntax=c]if (result>0)
{
if(result <=100)
{
zrób coś tam; // bo zakres 1-100 spełniony
}
}[/syntax]
Natomiast w przypadku znaku "||" mamy coś takiego:

[syntax=c]if (result>0)
{
zrób coś tam; // bo zakres >0 spełniony;
}
if (result<=100)
{
zrób coś tam; // bo <=100 spełniony;
}[/syntax]

Widać subtelną różnicę?

W drugim przypadku każde badanie uruchamia wykonanie "czegoś tam", a w pierwszym tylko jeśli faktycznie jeśli będzie zakres spełniony.

Swoją drogą takie rozbijanie warunków "AND" w Bascomie na pojedyncze bardzo zmniejszał kod. Nie wiedzieć czemu normalne użycie AND w jednej linii koszmarnie wydłużał program.

Statystyki: Napisane przez SylwekK — 10 lis 2018, o 20:50

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T20:20:50+01:00

kolego miki53, tego, co napisałem wcześniej, jestem w 100% pewny. Zauważ, że sprawdzasz, czy zmienna result jest w przedziale od 0 do 100. Więc sprawdzasz oba warunki z operatorem &&.
Tak, żeby przybliżyć:
result > 0 && result <= 100:
Sprawdźmy dla kilku liczb:
5: 5 > 0 && 5 <= 100 -> wartość logiczna prawda (prawda and prawda = prawda)
99: 99 > 0 && 99 <= 100 -> wartość logiczna prawda (prawda and prawda = prawda)
101 : 101 > 0 && 101 <= 100 -> wartość logiczna fałsz (prawda and fałsz = fałsz)
A teraz result > 0 || result <= 100:
5: 5 > 0 || 5 <= 100 -> wartość logiczna prawda (prawda or prawda = prawda)
101: 101 > 0 || 101 <= 100 -> wartość logiczna nadal prawda (prawda or fałsz = prawda)
Też tego kiedyś tego nie rozumiałem. Ale tak to działa. I działa dobrze

Statystyki: Napisane przez Marhef — 10 lis 2018, o 20:20

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T15:40:46+01:00

No nie, co za bzdury... Jeśli wstawi tam "lub" to każda wartość łapie się w warunek.

Statystyki: Napisane przez SylwekK — 10 lis 2018, o 15:40

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T14:34:54+01:00

Kolego KampoPampo

na samym początku zmień warunek if (result>0 && result <=100) czyli if (result>0 i result <=100)
jak widzisz ta sama zmienna jest porównywana nie może przyjąć -- dwóch -- równych wartości w tym samym czasie
bo to jest jedna i ta sama zmienna a nie dwie różne zmienne które chcesz porównać
warunek zawsze będzie nie spełniony

zamień go na warunek if (result>0 || result <=100) czyli if (result>0 lub result <=100)
warunek będzie spełniony jeśli zmienna result będzie większa od 0 -- lub -- mniejsza lub równa 100

przyjrzyj się jeszcze warunkowi if ( wilg >= liczba ) jaka jest wartość zmiennej - wilg - i zmiennej - liczba -
dlaczego ten warunek Ci przepuszcza i włącza pompkę

Na razie tylko tyle mogę pomóc może gdybyś wstawił większy fragment kodu to mógłbym pomóc bardziej

Statystyki: Napisane przez miki53 — 10 lis 2018, o 14:34

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T21:21:40+01:00

Witam,
Podpowiem.
if( a > 0 && a <=100) // zamień operator logiczny && na wyraz 'jednocześnie, oraz, także..'. oba przyrównanie muszą być spełnione jednocześnie.
if( a > 0 || a < 100) // zamień operator na wyraz ' albo, lub, ewentualnie'. musi być spełnione co najmniej jedno przyrównanie, mogą oba.
Pozdrawiam
.........................................20:10.10/11/2018..................................
dopiszę jeszcze w tym poście bo szkoda miejsca.
w pierwszym if'ie, warunek jest spełniony dla 'a' większego od '0' ale nie większego od '100'. !!!
w drugim if'ie, warunek jest spełniony zawsze. (tak jak niżej napisał kolega "SylwekK" ). ponieważ każda wartość jest albo większa od 0, albo mniejsza równa 100. a ten if sprawdza czy a > 0(czy jest liczbą dodatnią). Jeżeli nawet nie jest, to jeszcze sprawdza bo być może jest mniejsze równe 100.
@miki53, napisz wartość która nie jest większa od 0 i jednocześnie nie jest mniejsza równa od 100.

Wysłane z mojego Lenovo K33a42 przy użyciu Tapatalka
.........................................21:17 10/11/2018...................................
żeby nie zaśmiecać wątku - konsekwentnie w tym poście dografuję, może pomoże. choć to i tak off top wątku.

Statystyki: Napisane przez Daro69 — 10 lis 2018, o 09:13

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T02:09:02+01:00

Witam kolegę Marhef

chciałbym się zapytać czy na pewno ten warunek który napisał
kolega KampoPampo w swoim kodzie jest poprawny ?
bo jeśli nie to nie przeczytałeś uważnie mojego postu

if (result>0 && result <=100)

Myślę że w ten sposób nie można porównywać tą samą zmienną
z różnymi wartościami ?

Ale to tylko moja opinia

Tak się pytam dla własnej nauki i wiedzy

Reszta kodu jest mało czytelna i fragmentaryczna trudna do przeanalizowania
jak wspomniał kolega Daro69

Statystyki: Napisane przez miki53 — 10 lis 2018, o 02:09

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-09T18:27:12+01:00

prawie 200 wyświetleń i żadnej konkretnej podpowiedzi.
Powodem jest mało czytelny kod.
nie widzę np. dokąd prowadzi nieszczęsne "goto skok;".
nie widać wartości liczba.
bardzo ciężko przeanalizować - nawet ze szklaną kulą.

Statystyki: Napisane przez Daro69 — 9 lis 2018, o 18:27

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-09T15:51:43+01:00

miki53 napisał(a):

ak myślisz nie powinno to wyglądać tak if (result>0 || result <=100)
co oznacza jeśli result jest większy od 0 lub result jest mniejszy lub równy 100

Ale ten warunek "łapie" również wartości mniejsze od zera (bo spełniają warunek result <= 100) i większe od 100 (no bo przecież 101 jest większe od 0).
Tak że ten warunek zapisany przez kolegę @KampoPampo jest dobrze zapisany.

Natomiast reszty kodu nie rozumiem... strasznie chaotycznie to napisane... brakuje funkcji check_result(result)... brakuje pętli głównej programu... co to za zmienna liczba? A a? dałoby się uniknąć instrukcji goto (swoją drogą baaardzo zły nawyk)... wielokrotnie zagnieżdżone warunki - i może gdzieś w tym zagnieżdżeniu jest problem, ale nie umiem go znaleźć

Statystyki: Napisane przez Marhef — 9 lis 2018, o 15:51

Re: Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-10T15:14:02+01:00

Witaj KampoPampo

Nie jestem mocny w te klocki ale postaram się Ci pomóc
tak przeglądałem twój kod i wpadł mi w oko pewien fragment twojego kodu

[syntax=c]if(result>0 && result<=100) // result to liczba ktora zapisuje sie pod wplywem wprowadzania liczby przez klawiature[/syntax]

nie uważasz że ten if jest dziwny bo oznacza - jeśli result jest większy od 0 i result jest mniejszy lub równy 100
dla mnie to niewiadoma tak jak myślę dla procka również bo jaka ta zmienna ma być albo większa od 0 albo mniejsza lub równa 100
i z czym porównać skoro to ta sama zmienna

jak myślisz nie powinno to wyglądać tak if (result>0 || result <=100)
co oznacza jeśli result jest większy od 0 lub result jest mniejszy lub równy 100

troszkę więcej w tym sensu jak myślisz?
Wybacz za wymądrzanie się początkującego

Statystyki: Napisane przez miki53 — 9 lis 2018, o 15:11

Sterowanie pompka, ADC, Klawiatura 4x4

2018-11-08T12:53:28+01:00

Witam.

Chce dodać opcje, która będzie uruchamiać / wyłączać pompke przy odpowiednich warunkach. Wpisuje liczbe i od razu włącza mi się pompka (nawet nie zatwierdzę i działa ciągle, nawet jak wracam do głównego menu), a moim celem jest żeby pompka działała zależnie od wilgotności. Kod do pomiaru ADC działa poprawnie, jednak tu mam kłopot.

[syntax=c]if(key = read_keypad()) // start klawiatury
{

if(key == 4)
{

//wilgotnosc

_delay_ms(400);
skok1:
lcd_cls();
lcd_locate(0,0);
// lcd_cls();
lcd_str("Wilgotnosc: ");

int result = 0;
while (true) {

adc_value = ADC_odczyt();// wpisanie wartości odczytanej z adc do zmiennej adc_value
wilg = 100-(adc_value *100.00)/1023.00; // przetwarzanie pomiaru na %, 1023-100%
dtostrf(wilg,2,0,array); //konwertowanie liczby na tekst
lcd_locate(1,0);
lcd_str(array);
_delay_us(500);//

int key = read_key(); // odczyt wcisnietego klawisza
if (key != -1) {

if(key==16)
{
goto skok1;
}
if(key == 13) // powrot do glownego menu
{

_delay_ms(400);
goto skok;
}

if (key == 15) // zatwierdzenie wpisanej liczby
{

if(result>0 && result<=100) // result to liczba ktora zapisuje sie pod wplywem wprowadzania liczby przez klawiature
{

check_result(result);
result = 0;

}

else {
lcd_locate(0,0);
lcd_str("******BLAD******");
lcd_locate(1,0);
lcd_str("SPROBOJ PONOWNIE");
_delay_ms(2000);
goto skok1;
}

lcd_cls();
lcd_locate(0,0);
// lcd_cls();
lcd_str(" START PODLEW. ");
lcd_locate(1,0);
lcd_str("################");

if(wilg >= liczba)
{
PORTA |= (1< }

else
{
PORTA &= ~(1< }

}

else
{
result = result * 10 + a; // zsumowanie cyfr jako dwucyfrowa liczba
lcd_locate(0,12);
lcd_int(result);
lcd_locate(0,15);
lcd_str(" ");

}

}

}
}[/syntax]

Potraficie mi coś doradzić, bo niestety męczę sie nad tym troche? Jak przechodzę do tej pozycji menu, wpisuje liczbe i zatwierdzam to pompka się włącza niezależnie od wskazania czujnika i działa non stop. Próbowałem to jakoś rozwiązać, ale bez skutku. Z góry dziękuję i pozdrawiam.

Statystyki: Napisane przez KampoPampo — 8 lis 2018, o 12:53