ATNEL tech-forum

Re: Błędnie obliczana moc wyjściowa - problem z mnożeniem li

2021-01-17T13:58:40+01:00

anonimowy napisał(a):

https://atnel.pl/domyslna-promocja-do-typu-int.html

Bingo! Przejrzałem właśnie całego BB i nie znalazłem niczego na ten temat. Bardzo ci dziękuję.

Makowka napisał(a):

Policzyłem to korzystając z windowsowego kalkulatora i mi również wyszło 8 Dlaczego? Zobacz na linię 10. kodu twojej funkcji i zastanów się jak wykona się operacja dzielenia.

Wykonujesz rzutowanie na typ 16 bitowy. Wcześniej, wynik (jak mówiłeś) wynosi 0100 0010 0000 1000 0000.

Po wyrzuceniu 4 najbardziej znaczących bitów podziel 0b0010 0000 1000 0000 (0x2080) / 0b0011 1110 1000(0x3E8) i kalkulator pokaże 8 jak na zdjęciu poniżej.

Mam nadzieję, że się nie pomyliłem, ale proszę mnie poprawić jeżeli jednak to nie to

No właśnie według mojego rozumowania do zmiennej uint16_t powinien zostać zapisany wynik dzielenia zmiennej uint32_t przez 1000 a nie najpierw sama zmienna a później operacja dzielenia na niej. Z resztą w następnych wersjach funkcji wyeliminowałem ten problem jednak bez skutku.

Ostatecznie działająca funkcja wygląda tak:

[syntax=c]void disp_Pbatt (uint16_t ubatt, uint16_t ibatt, uint8_t posx, uint8_t posy){

//uint16_t a = (ibatt + 3) * 2;//przelicznik na mA (50 = 500mA)
//uint16_t b = ubatt * 2;//przelicznik na napięcie x100 (15V = 1500)

uint16_t a = ibatt;
uint16_t b = ubatt;

uint16_t p = ( ( (uint32_t)a * (uint32_t)b) ) / 1000;
lcd_locate(posy,posx);
lcd_int(p);
}[/syntax]

Problem leżał w tym że kompilator zapisywał wynik mnożenia a * b do jakiejśc zmiennej tymczasowej dwubajtowej pomimo iż wynik miał być zapisany do uint32_t. Wystarczyło potraktować zmienne a i b jako zmienne czterobajtowe i wynik jest już prawidłowy.

Makowka w tej funkcji też zapisałem na próbę wszystko w jednej linii i problem nie występuje

Miłego dnia wszystkim!

Statystyki: Napisane przez wolek14 — 17 sty 2021, o 13:58

Re: Błędnie obliczana moc wyjściowa - problem z mnożeniem li

2021-01-17T12:52:18+01:00

Statystyki: Napisane przez Makowka — 17 sty 2021, o 12:52

Re: Błędnie obliczana moc wyjściowa - problem z mnożeniem li

2021-01-17T12:47:14+01:00

https://atnel.pl/domyslna-promocja-do-typu-int.html

Statystyki: Napisane przez anonimowy — 17 sty 2021, o 12:47

Błędnie obliczana moc wyjściowa - problem z mnożeniem liczb

2021-01-17T11:22:34+01:00

Witajcie!

Napisałem sobie funkcję służącą do obliczania mocy wyjściowej pewnego urządzenia i wyświetlania go na LCD (HD44780 4 x 20):
[syntax=c]void disp_Pbatt (uint16_t ubatt, uint16_t ibatt, uint8_t posx, uint8_t posy){

//uint16_t a = (ibatt + 3) * 2;//przelicznik na mA (50 = 500mA)
//uint16_t b = ubatt * 2;//przelicznik na napięcie x100 (15V = 1500)

uint16_t a = ibatt;
uint16_t b = ubatt;

uint32_t p = (a * b);
uint16_t power = p/1000;
lcd_locate(posy,posx);
lcd_int(power);
}[/syntax]

Na potrzeby testu wywołuję go z funkcji głównej programu stałymi wartościami:
[syntax=c]disp_Pbatt(1680, 200, 10, 0);[/syntax]

Pomijam też przeliczanie surowej wartości z adc na fizyczne jednostki (są "zakomentowane").

Zgodnie z oczekiwaniami gdy wartości przekazywane do funkcji wynoszą odpowiednio 1680 i 200, wynikiem mnożenia jest liczba 336000. Ta powinna znajdować się w zmiennej p zapisana binarnie tak:

[syntax=c]0101 0010 0000 1000 0000[/syntax]

następnie w wyniku dzielenia do zmiennej power trafić powinna liczba 336 zapisana tak:

[syntax=c]0001 0101 0000[/syntax]

po czym zostać wyświetlona na lcd.

Niestety jak bym nie próbował - na wyświetlaczu mam cyfrę 8...

Spróbowałem wyświetlić zawartość zmiennej power bez dzielenia na 1000:

[syntax=c]void disp_Pbatt (uint16_t ubatt, uint16_t ibatt, uint8_t posx, uint8_t posy){

//uint16_t a = (ibatt + 3) * 2;//przelicznik na mA (50 = 500mA)
//uint16_t b = ubatt * 2;//przelicznik na napięcie x100 (15V = 1500)

uint16_t a = ibatt;
uint16_t b = ubatt;

uint32_t p = (a * b);
uint16_t power = p;
lcd_locate(posy,posx);
lcd_int(power);
}[/syntax]

i otrzymuję wynik 8320, binarnie to będzie:

[syntax=c]0010 0000 1000 0000[/syntax]

Okazuje się, że jest to wynik mnożenia (336000) z którego znikają cztery najstarsze bity. Wszystko się zgadza. Czemu natomiast dzielenie przez tysiąc wykonywane jest już na obciętej wartości? Tzn dzielone jest 8320 a nie 336000? Próbowałem też zrobić tak:

[syntax=c]void disp_Pbatt (uint16_t ubatt, uint16_t ibatt, uint8_t posx, uint8_t posy){

//uint16_t a = (ibatt + 3) * 2;//przelicznik na mA (50 = 500mA)
//uint16_t b = ubatt * 2;//przelicznik na napięcie x100 (15V = 1500)

uint16_t a = ibatt;
uint16_t b = ubatt;

uint32_t p = (a * b) / 1000;
uint16_t power = p;
lcd_locate(posy,posx);
lcd_int(power);
}[/syntax]

Niestety również wynikiem jest liczba 8.

Dajcie proszę jakąś wskazówkę - nie mam już za bardzo pomysłu. Myślałem że ostatni przykład załatwi sprawę i do zmiennej uint32_t zostanie zapisany wynik 336000 i podzielony przez 1000, jednak tam również ucięte są cztery najstarsze bity.

Mógłbym teoretycznie dzielić liczby przed mnożeniem ale:
- to jest ominięcie problemu a nie rozwiązanie + zerowa wartość edukacyjna
- nie uzyskam zakładanej dokładności

Procesor to Atmega8 taktowana fcpu = 16MHZ.

Proszę o wyrozumiałość - jestem amatorem hobbystą i dopiero się uczę

Łukasz

Statystyki: Napisane przez wolek14 — 17 sty 2021, o 11:22