I przeanalizuj ten kod

_________________
Dragonus Cracovus: Biomagia

Funkcja działa ok.

1.Dodam, że stworzyłem jeszcze 4 funkcje i działają poprawnie. Ale w każdej z nich jak i w changeCounter pojawia się warning:

passing argument 1/2 of "changeCounter" makes pointer from integer without cast

2.Jeżeli mam ++i to licznik jest okej tzn. nie muszę na lcd wpisywać licznik-1, ale w ustawianiu minut potrzebne mi jest zero a tutaj licznik nalicza od jedynki itd.

3.lcd_locate(1,7). Czy w funkcji da się ustawić zmienną lcd_locate(1,x), żeby zmieniać położenie na LCD

1. Zapomniałeś gdzieś przy wywołaniu funkcji o &. Jak argumentem funkcji jest wskaźnik to wywołanie funkcji musi pobierać adres zmiennej, a nie wartość.
2. Możesz zwiększanie tej zmiennej przenieść, gdzie indziej. Przeanalizuj po kolei co się z nią dzieje.
3. Pewnie, że tak.

_________________
Dragonus Cracovus: Biomagia

Funkcja wygląda tak i działa:




lcd_locate() też śmiga

------------------------ [ Dodano po: 2 minutach ]

A co do warningu to wszędzie jest &. Wyskakuje mi takich błędów 62

Kompilator pokazuje miejsce, którego dotyczy ostrzeże. Możesz skopiować taką linię.

_________________
Dragonus Cracovus: Biomagia

passing argument 1/2 of "changeCounter" makes pointer from integer without cast

Taki błąd wyświetla się w każdym wywołaniu funkcji dla &time_prog oraz &time_plus_prog. 62 błędy dotyczą każdego odwołania do funkcji w których jest uint8_t* time_prog i uint8_t* time_plus_prog

Mam problem. Gdy ustawiam wybór poszczególnych przekaźników i wybiorę przekaźnik 1 oraz 2 to program załączy przekaźnik 1 potem 2, ale cała sekwencja się nie powtarza. Powtarza się jedynie gdy cztery przekaźniki są zaprogramowanie.

W tym przypadku licznik13, 16, 19, 22 jest równy 0:

Problem chyba polega na tym, że flagi na końcu przekaźnika 4 nie są ponownie ustawiane na 1. Jak to rozwiązać?

To może nie ustawiaj tych 4 flag w obsłudze przekaźnika 4 tylko zamiast zerowania ich na końcu np. flaga_p1_p1=0; //off ustaw na 1 np. flaga_p1_p1=1; każdą z osobna.

_________________
Dragonus Cracovus: Biomagia

Czy dobrze kombinuje z tą funkcją jeśli chodzi o pracę przekaźnika w zależności od ustawionej godziny włączenia i wyłączenia ( np 12:30:20 do 16:20:10). Jak wiadomo godziny, minuty i sekundy właczenie mogą być większe lub mniejsze
od wyłączenia

Krauser bardzo proszę o opinie na temat mojej funkcji

tak się nie robi .... bo jak byś miał jeszcze daty dodać to za chwilę musiałbyś przekazywać 1000 argumentów do funkcji a i tak już masz 500

zamień godziny i minuty na liczby

h_start=(hh*60)+mm;
h_stop=(hhs*60)+mms;

i potem porównój liczby

if( h_stop > h_start )...

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Pierwszy problem z kodem to zbytnie rozbudowanie, a drugi to taki, że w funkcji wielokrotne wykorzystujesz zmienne ze specyfikatorem volatile, a przypomnę, że w Twoim pierwszym poście masz:

W każdym obiegu tych porównań wystąpi wielokrotne niepotrzebne odczytywanie tych zmiennych z pamięci RAM zamiast z rejestrów. Ta pierwsza opcja wymaga większej ilości instrukcji i pociągnie większe zużycie pamięci programu. W takich przypadkach lepiej skopiować takie zmienne do innych i na nich wykonać te porównania lub nawet operacje i zapisać z powrotem do zmiennych z przydomkiem volatile.
Skorzystaj z powyższej rady lub nawet wykorzystaj czas uniksowy.
Volatile nie jest potrzebne do tych zmiennych, bo czas odczytujesz tylko w pętli głównej.

_________________
Dragonus Cracovus: Biomagia

Rozumiem, ale gdy mamy przedział np. od 11:12:30 - 02:03:00 to h_start> h_stop.

Minuty stop są mniejsze niż minuty start tzn., że przedział minut powinien zawierać się w granicy:

minuty>=minuty_start || minuty<minuty_stop

bo gdy będzie:

minuty>=minuty_start && minuty<minuty_stop

to minuty muszą być większe od 12 i mniejsze od 3, a nie ma takiej liczby. Czy dobrze myślę? Czy już mi się coś poplątało?

Więc w jaki sposób to rozwiązać w kodzie?

Dzięki waszym wskazówkom powstała taka funkcja:
Jednak nie działa. Wydaje mi się, że to z powodu nie zmieniającego się czasu, co skutkuje nie załączeniem przekaźnika

&& ma wyższy priorytet niż ||
A ty masz coś takiego

Porównaj z tym co kolega GwynBleidD napisał.

_________________
Dragonus Cracovus: Biomagia

Nie podoba mi się jedna rzecz w moim kodzie, próbowałem to zmienić ale nie mam pomysłu. Chodzi o wyczyszczenie zapisanego programu. Po ustawieniu godzin włączania i wyłączania przekaźników i zaakceptowaniu program 1 działa. Po wyjściu escapem i ponownym wejcciu w program jest możliwość wyczyszczenia

Po ponownym wejściu w program 1 zaczynamy wtedy programowanie od nowa.

Czy macie jakiś inny pomysł jak to rozwiązać, ponieważ źle to działa.

Problem rozwiązałem

Tak śledzę czasem ten temat, szczegółów nie znam ale gdy widzę takie coś



to aż serce się kroi i woła o to aby w końcu nauczyć wykorzystywać struktury danych, i do funkcji przesłać jeden argument w postaci wskaźnika do struktury a nie TYLE argumentów, przecież takie podejście poza koszmarem kodu, poza kłopotem z czytaniem, poza łatwością o pomyłkę - zjada ci stos na potęgę i gdy tą drogą będziesz szedł to się potem będziesz dopiero dziwił że coś ci tam w zaskakujący sposób nie działa. Bo niestety uszkodzenia stosu dają najbardziej mylące objawy i najgorsze efekty...

zatem taki przykład - to "NAJLEPSZY" przykład jak to jest gdy ktoś nie chce się nauczyć struktur w C i sięgnąć po nie.

_________________
zapraszam na blog: http://www.mirekk36.blogspot.com (mój nick Skype: mirekk36 ) [ obejrzyj Kurs EAGLE ] [ mój kanał YT TV www.youtube.com/mirekk36 ]

Dodaj sobie funkcję static void ledOn(uint8_t led) i static void ledOff(uint8_t led)

To stwierdzenie jest zbyt ogólne. Jak chcesz zaoszczędzić na pamięci programu to mógłbyś to nieco uporządkować wprowadzając zamiast np. licznik5 tablicę licznik i do większości operacji użyć pętli for.

Do porównywania czasu łącznie datą lepiej przejść na czas unixowy, bo lata mogą być również przestępne. Poszukaj gotowej funkcji do konwersji w jedną i drugą stronę.
W tym zegarze jest również funkcja alarmu, można by ją wykorzystać.

_________________
Dragonus Cracovus: Biomagia

Chcę się nauczyć dlatego zadaję pytania i proszę o pomoc.Dopiero to mój pierwszy poważny projekt i stąd taki nie ład w pewnych miejscach. Ale dzięki mirekk36, że wskazałeś mi dobrą drogę. Rzeczywiście muszę w swoim projekcie użyć struktur danych i tablic w celu optymalizacji kodu. Kiedyś wykorzystywałem struktury, ale już co nie co zapomniałem.

Tylko nie wiem w jaki sposób ma wyglądać odwołanie do wskaźnika struktury w mojej funkcji. Proszę o pomoc. Muszę załapać i wtedy już będzie z górki.


Poza tym nie wiem czy flaga_p_p powinna być w strukturze. Jeżeli będę miał 10 programów i dla każdego 4 przekażniki do ustawienia ( np. flaga_p1_p1, flaga_p1_p2, flaga_p1_p3, flaga_p1_p4, flaga_p2_p1, flaga_p2_p2 .... itd)

A co wtedy z licznikami ustawiającymi czasy włączania i wyłączania przekaźników oraz daty

W twoim przypadku, gdzie masz wiele takich samych "obiektów" deklaracja struktury bez nazwy i jednoczesna definicja tego typu zmiennej jest bezużyteczna, bo nie możesz zdefiniować innych zmiennych. Takie coś odpada:

Do definiowania struktury najlepiej posługiwać się słowem typedef, a to dlatego, że później definiowanie zmiennych jest takie jak i tych podstawowych:
Bez typedef

a z użyciem typedef

Czyli powinno to być mniej więcej tak, ale zastanów się nad organizacją danych, co cechuje pojedynczy przekaźnik. Jakie dane mają być w programie. Rozpisz to sobie dokładnie na kartce, a będzie ci później łatwiej to przenieść do programu:


To zależy jak zamierzasz wykorzystać to sterowanie z datą. Na pewno do porównań będzie lepszy, ale jak chciałbyś coś sterować w cyklu dziennym, tygodniowym, miesięcznym to się nie nadaje.

_________________
Dragonus Cracovus: Biomagia

Krauser mając na myśli " wiele takich samych obiektów", chodzi Ci o to, że mam 4 przekaźniki? Mając strukturę:



Do każdego przekaźnika będę miał inne godziny start, godziny stop ... itd. (np. przekaznik1.h_start), ale po co? Nie mogę się odwołać do funkcji w której mam wszystko obliczone, a w wywołaniu funkcji odwołuję się do danego przekaźnika?

Może źle Cię zrozumiałem

Dokładnie chodziło mi o to, że tworząc strukturę bez nazwy więcej już nie możesz jej wykorzystać np. jako element kolejnej struktury, bo możesz zrobić też tak:


Albo ja Ciebie Myślałem, że chciałeś pobierać czasy włączenia i wyłączenia jako składową danego programu działania, a wtedy w każdym innym "programie sterującym" to by się różniło. Na tym etapie nie zamierzam już dyskutować o algorytmie działania. Dlatego jak masz jakiś problem to staraj się go dokładniej opisać.

_________________
Dragonus Cracovus: Biomagia

Zacznę może dokładnie od początku, wtedy będziesz mógł mi pomóc.
I teraz Krauser proszę powiedz mi w jaki sposób stworzyć do tego strukturę tak jak zalecił mirekk36, żeby nie przesadzić ze stosem, który może zjadać mi RAM i powodować dziwaczne błędy.

Struktura:

Funkcja

Wywołanie funkcji:

Mam jeszcze taką uwagę odnośnie nazywania zmiennych. Masz counter[24], a później opisujesz co znajduje się w kolejnych komórkach tej tablicy. Tablica jest dobra do wykorzystania w pętli for, ale wtedy nie masz intuicyjnych nazw przez co łatwo o pomyłkę. Można to połaczyć stosując unię. Np.

_________________
Dragonus Cracovus: Biomagia

Bardzo dziękuje za wytłumaczenie Krauser.Zaczynam już to rozumieć co mnie niezmiernie cieszy Jeszcze jedna rzecz mnie zastanawia.

To by było niepotrzebne zwiększanie rozmiaru programu. Jeśli dobrze myślę to zmienna device przekazuje numer przekaźnika i wtedy możesz zrobić tak:

_________________
Dragonus Cracovus: Biomagia

Wyskakują mi takie błędy:

../main.c: In function 'DeviceWorks':
../main.c:3990: error: 'program_wsk' undeclared (first use in this function)
../main.c:3990: error: (Each undeclared identifier is reported only once
../main.c:3990: error: for each function it appears in.)

To moja pomyłka. W końcu ta struktura nazywa PROGRAMY.
Zamiast:

ma być:

_________________
Dragonus Cracovus: Biomagia

Nadal pojawiają mi się 3 błędy: