25-04-2017, 08:10
Witam,
Sugestie:
- jeśli chcesz mieć MENU to w pętli loop() powinna być obsługa przycisku digitalRead() umożliwiająca wejście w to MENU. Dodatkowe dwa przyciski do zwiększania/zmniejszania temperatury, czyli wciskając dwa przyciski jednocześnie następuje wejście do menu i zmiana temperatury (o ile MENU ma jedynie realizować tę funkcję),
- w pętli loop() nie trzeba co iterację wpisywać na przekaźnik stanu. Daj dodatkową zmienną typu boolean, która przechowuje obecny stan przekaźnika. Poza tym dlaczego co iterację bezwarunkowo wpisujesz stan HIGH na przekaźnik?,
- co prawda sam odczyt temperatury z czujnika trochę trwa, co powoduje spowolnienie wykonania pętli loop(), ale z uwagi na dużą bezwładność temperaturową układu, można do wnętrza loop() wrzucić jakiegoś delay-a,
- skoro używasz wewnątrz loop() funkcji to dlaczego za pomocą funkcji nie wyświetlasz temp. kotła? Kod będzie czytelniejszy.
Pozdrawiam.
Sugestie:
- jeśli chcesz mieć MENU to w pętli loop() powinna być obsługa przycisku digitalRead() umożliwiająca wejście w to MENU. Dodatkowe dwa przyciski do zwiększania/zmniejszania temperatury, czyli wciskając dwa przyciski jednocześnie następuje wejście do menu i zmiana temperatury (o ile MENU ma jedynie realizować tę funkcję),
- w pętli loop() nie trzeba co iterację wpisywać na przekaźnik stanu. Daj dodatkową zmienną typu boolean, która przechowuje obecny stan przekaźnika. Poza tym dlaczego co iterację bezwarunkowo wpisujesz stan HIGH na przekaźnik?,
- co prawda sam odczyt temperatury z czujnika trochę trwa, co powoduje spowolnienie wykonania pętli loop(), ale z uwagi na dużą bezwładność temperaturową układu, można do wnętrza loop() wrzucić jakiegoś delay-a,
- skoro używasz wewnątrz loop() funkcji to dlaczego za pomocą funkcji nie wyświetlasz temp. kotła? Kod będzie czytelniejszy.
Pozdrawiam.