Z racji że kolega nie może jeszcze wrzucać na forum nowych tematów zakładam za niego temat

Witam wszystkich serdecznie i dziękuję koledze rezasurmar za założenie tematu. Od niedawna zacząłem się uczyć programowania w C. Jako na dobry początek wymyśliłem sobie sterowniczek do centrali wentylacyjnej który miałby za zadanie odnaleźć najcieplejszy kanał powietrza i przełączyć się na niego (kominek, wymiennik gruntowy i czerpnia na ścianie budynku). W związku z tym, iż zdecydowanie za mało wiem postanowiłem, że posklejam kod z tego co znajdę i co już mam wraz z niebieską księgą. Aż doszedłem do momentu kiedy musiałem sam napisać kawałek kodu, no i tu pojawiły się schody. Na początku porównywałem ze sobą całkowite części temperatury i tu nie było problemu, ale chciałbym, żeby urządzenie działało jeszcze dokładniej więc dodałem do porównania części po przecinku, kilka warunków if pod sobą i już problem. Proszę o pomoc.
Dziękuję i pozdrawiam,

Jak na mój gust to przy tych wszystkich if'ach brakuje klamerek {} potem nagle jest średnik nie wiadomo po co a potem klamerka, która dla kompilatora jest już bardzo nie ważna.

Proponuję koledze aby przeczytał on pierwszą część BB i ogólnie "zaprzyjaźnił" się z samą składnią C.

_________________
50 znaków Atnela ....

Skorzystam z okazji i zapytam, czy w sytuacji gdy jest tyle warunków które muszą się spełnić, zapisanie tego w formie jaką prezentuję poniżej jest lepszym czy może gorszym rozwiązaniem od zapisania tego w sposób jaki podał kolega, oczywiście po poprawieniu błędów w składni.

_________________
http://www.marcingibas.pl

Marcin_G, w obu przypadkach kompilator u mnie zwrócił ten sam wynik, więc ze względów optymalizacyjnych różnicy brak.

Co innego wygląd (ja wolę formę podaną przez Ciebie, zwięzłą) i możliwości znalezienia jakiś błędów/debagowania (tu zdecydowanie łatwiej dopisać jakieś else w formie rozpisanej). Niemniej - kwestia gustu.

Witam

...to znaczy: nie działa, źle działa czy brzydko wygląda.
Sory, ale ( chyba ) nie piszesz.

Pozdr.

Ja bym się nie bawił z tym porównywaniem wartości zmiennych w if-ach przed przecinkiem, po przecinku.
Nie żeby się nie dało wykonać ale strasznie zagmatwane.

Ja to realizuje tak że jeżeli chcę mieć dokładność do jednego miejsca po przecinku to przemnażam wartości przez 10 i na nich wykonuje wszystkich operacji.
Tutaj masz gotową funkcje której ja używam.


w miejscu gdzie przypisujesz wartości temperatur do kolejnych temp... zastosuj tą funkcje i przypisz wynik do zmiennej int16_t.
I wtedy porównujesz zmienne np temperatura 10.5C to wartość 105, -5,3C to wartość -53 itp...



Tylko zadeklaruj wartość temp2 jako int16_t a wartość temp21 usuń i tak analogicznie dla wszystkich czujników.

------------------------ [ Dodano po: 9 minutach ]

A tak sprawdzasz swoje temperatury

_________________
https://www.instagram.com/myfirstquadcopter/

Zrobiłem tak jak podpowiedział kolega kurnachatka, lecz kompilator dalej krzyczy "suggest explicit braces to avoid ambiguous 'else'"
w miejscu



cały kod wygląda tak:

llemar, nie masz klamerek po if'ach i masz else po wykonaniu PORTC &=...... czyli else nie wiadomo czego się dotyczy.

Przeczytaj podstawy C z książki zwłaszcza rozdział o IF i jej składni. Bez takich podstaw ani rusz dalej.

_________________
50 znaków Atnela ....

acid3- zrobiłem tak jak jest na stronie 62 w niebieskiej książce. Czytam ją na bieżąco (nie zawsze ze zrozumieniem ale staram się to pojąć) Sugerujesz, żeby warunki były zagnieżdżone ?
Jeszcze jedno pytanko - jak zwrócić wynik funkcji do zmiennych (temp1, temp2, temp3)??

a czytałeś już o funkcjach w Bluebooku ? czy jeszcze nie ? ... bo jeśli nie - to warto

_________________
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 ]

Warto przeczytać i wiedzieć wszystko Być może za wcześnie zabieram się za ćwiczenia...

llemar, jeśli mnie wzrok nie zawodzi to teraz warunki się wykonują po kolei bez względu na wynik i na końcu dostajesz warninga za else które tam "nie pasuje".

Najpierw teoretyczna część ksiązki, potem miganie, a potem LCD z multitouchem

_________________
50 znaków Atnela ....

Tzn ja powtarzam zawsze jak mantrę ....

Bluebooka trzeba przeczytać RAZ przynajmniej JEDEN RAZ tak w całości od deski do deski a dopiero potem można czytać na wyrywki

dlaczego ?

ano dlatego że wtedy zanim sam sobie zadasz pytanie jak przekazać do zmiennych wartości z funkcji to w głowie odezwie ci się mała kotwiczka, którą zapuściłem wcześniej gdy czytałeś RAZ całość i będziesz wiedział że trzeba zajrzeć gdzieś tam do środka bo ... ooo tym właśnie było tam napisane

tak działa przekaz podprogowy w Bluebooku

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



Tzn tak jak pisałem wyżej - najpierw RAZ cała książka - bo w Bluebooku teoria miesza się w przykładach w drugiej jej części warsztatowej .... to nie jest typowo napisana książka że najpierw jakaś teoria a potem wybrane ćwiczenia ... to wszystko jest zmiksowane w odpowiedni sposób

_________________
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 ]

A niecierpliwość jest ogromna i chciałoby się wszystko robić od razu
No nic, trzeba wziąć jeszcze na wstrzymanie i czytać

Masz tu przerobiony swój kod znajdź różnice.


Jak go wkleisz powinno działać, munie kompiluje się bez błędów.
Podpowiem tylko że w kilku miejscach miałeś za if-ami niepotrzebne średniki i jeszcze tam kilka przeróbek.

_________________
https://www.instagram.com/myfirstquadcopter/

Dziękuję Ci kurnachatka program działa (gdzie się klika pomógł?)
Jednakże przed kolejnymi pytaniami poczytam jeszcze BlueBooka
Dzięki wam zrobiłem kroczek do przodu
Dziękuję.

Mam tak samo jak ty że tak przytoczę słowa piosenki.

Ogólnie wiedza moja na temat programowania w C jest mizerna, ale staram się zawsze uruchamiać po kolei poszczególne "klocki" programu
Dlatego idealnie jest podzielić sobie to na funkcje. Nauczyć się przekazywać parametry przez funkcje i przyjmować parametry do funkcji. Wtedy troszkę łatwiej zapanować nad kodem. Nawet jak ma być jeden if w funkcji to na początku dobrze to rozjaśnia kod.

Po za tym jak już koledzy wspominali, przyjmij jeden "model" czy "styl" klamrowania, tabulatorów. Masz kreatory od tego w eclipsie. Np. po enterze automatycznie wrzuca ci tabulacje. Sam staram się stosować styl Mirkowy, bo łatwiej wtedy o pomoc, a przejrzystość kodu na tym bardzo mocno zyskuje.

Np. robisz sobie szablon z ifów, elsów i od razu walisz za nimi klamry, nawet jak będzie tam jeden rozkaz na początku. Dzięki temu nie zgubisz żadnej klamerki po drodze.

Prawdopodobnie nie masz takiej możliwości bo temat założył kolega rezasurmar i tylko on może kliknąć pomógł.

_________________
https://www.instagram.com/myfirstquadcopter/

kliknąłem

rezasurmar a sobie też możesz kliknąć ? DZIĘKUJĘ

No sam sobie nie mogę