[Rozwiązano] [Bascom]Gotowy projekt sterownika pompy - analiza kodu

Bardzo proszę o analizę i ewentualne poprawienie kodu który pracuje aktualnie w "Sterowniku odwadniania piwnicy". Będę bardzo wdzięczny za pomoc.

Link do tematu: https://www.elektroda.pl/rtvforum/topic3606939.html
Link do postu z ostatnią wersją kodu: https://www.elektroda.pl/rtvforum/topic3606939-30.html#18157054

Będę bardzo wdzięczny za pomoc.

Witam.

Dlaczego nie dałeś pływaka ( na Hall ) tylko styki?
Dwuczujnikowy.

Pierwszy włącza pompę. Drugi wszczyna alarm że pompa nie daje rady lub uległa awarii.

A skoro działa to znaczy że algorytm jest w porządku.

Dobre rozwiązanie, ale ja użyłem sond konduktancyjnych które również działają dobrze. Logikę mogę zmienić i będzie tak samo działało tyle że bez histerezy. Tak czy owak chodzi o mój program. Działa ale chciał bym by fachowiec ocenił czy jest w porządku, by nie spotkały mnie jakieś niemiłe niespodzianki. Wydaje mi się że trochę w nim namieszałem, ale jak wspomniałem wcześniej dopiero uczę się programowania.

Coś tu chyba nie pasuje
W opisie projektu autor napisał : "Na wspólną sondę dodatnią (poziomu i suchobiegu) podawane jest napięcie raz na ok. 1s przez ok. 100ms, o prądzie rzędu kilku mA. "

...a weź podziel 8MHz przez preskaler 64 i interwał timera 500 (Load Timer 500) to wyjdzie Tobie 250Hz...
Zmienna typu Word sonda_Hz kręci się w kółko i przepełnia przy 65536 co przy 250Hz daje 4.369min ... Czas impulsu ustawiony na 400 daje 1.6sec ...
Może by to i działało tak jak zakładałeś, ale ... w procedurze obsługi przerwania jest błąd.
Z racji tego, że napięcie na sondy włączasz transoptorem i uruchamiasz diodę nadawczą transoptora stanem niskim na pinie procesora to chyba przy komentarzu "załącz sondę COM" powinno być zero a nie jeden.

... wtedy sondy byłyby włączane co 4.4min na czas 1.6sec a tak...
Sondy są wyłączane na okres 1.6s co 4.4min

[EDIT] Przepraszam. Nie zauważyłem tego "sonda_Hz=0"
Czyli chyba jest to tylko źle opisane bo wychodzi na to, że sonda jest wyłączona na 1.6sec a włączona na jeden tick timera czyli 1/250Hz=0.004s=4ms

Kolego "bart-projects" Czy mógł bym liczyć na Twoje wsparcie?
Obecnie mój program wygląda tak:



Mam wrażenie że namieszałem coś w logice, i boje się żeby któregoś pięknego dnia program się po prostu nie zwiesił. Poza tym program jest napisany trochę chaotycznie, zupełnie nie zgodnie ze sztuką. Zerknij proszę jak możesz. Chodzi o linijki od 276 do 356. Pogubiłem się w nim całkowicie.

Dlaczego nie numerujesz linii w kodzie?

Gdzieś kiedyś widziałem taką opcję a Bascom, ale jej nie szukałem i nie włączałem. Liczyłem na to że jeśli ktoś podejmie się analizy, również ma Bascom i po przyklejeniu kodu będzie miał już ponumerowane.

Temat zamykam, gdyż postanowiłem sam napisać program od nowa bardziej spójniej. Dziękuję.