ATNEL tech-forum

Re: Prośba o weryfikację kodu.

2021-01-26T19:58:22+01:00

antosV10 napisał(a):

Czy reszta z dzielenia danej liczby jest tożsama z liczbą, którą dostaję po przecinku?

Nie. Reszta z dzielenia liczb całkowitych (% - operacja modulo) w twoim przypadku daje w wyniku wartość z zakresu od 0 do SR-1.

antosV10 napisał(a):

Tutaj dodaję kod, który jeszcze zmodyfikowałem czy on będzie poprawnie realizował swoje zadanie?

Zgubiłeś coś za ifem, jak już to zamiast "sr/SR+1;" powinno być "sr1 = sr1/SR+1;". Wydaje mi się że te obliczenia realizują zadanie. Ale można to zrobić o wiele prościej:
[syntax=c]adc_value = (sr1 + (SR/2)) / SR;[/syntax]
Czyli do sumy 8 próbek dodajesz jeszcze połowę tego przez co będziesz dzielił.

Statystyki: Napisane przez wojtekr51 — 26 sty 2021, o 19:58

Re: Prośba o weryfikację kodu.

2021-01-26T09:16:36+01:00

Tutaj dodaję kod, który jeszcze zmodyfikowałem czy on będzie poprawnie realizował swoje zadanie?

[syntax=c]void get_average(void)
{
static uint16_t sr[SR];
static uint8_t idx;
uint32_t sr1 = 0;
uint8_t i;

sr[idx] = measurement(PA5);
idx++;
if(idx > 7)
{
idx=0;
for(i = 0; i
if(sr1%SR*10/SR >= 5) sr/SR+1;
else sr1 /= SR;

adc_value = sr1;
}

}[/syntax]

Statystyki: Napisane przez antosV10 — 26 sty 2021, o 09:16

Re: Prośba o weryfikację kodu.

2021-01-26T00:10:40+01:00

Witam, bardzo dziękuję za cenną uwagę wiedziałem, że mogę coś przegapić . A więc do mojej funkcji dodałem 2 linie kodu. Myślę, że mój zamysł na rozwiązanie tego problemu jest poprawny, ale nie mam pewności czy dobrze wykrywam jaka liczba występuje po przecinku. Czy reszta z dzielenia danej liczby jest tożsama z liczbą, którą dostaję po przecinku?
[syntax=c]void get_average(void)
{
static uint16_t sr[SR];
static uint8_t idx;
uint32_t sr1 = 0;
uint8_t i;

sr[idx] = measurement(PA5);
idx++;
if(idx > 7)
{
idx=0;
for(i = 0; i
if(sr1%SR >= 5) sr/SR+1;
else sr1 /= SR;

adc_value = sr1;
}

}[/syntax]

Statystyki: Napisane przez antosV10 — 26 sty 2021, o 00:10

Re: Prośba o weryfikację kodu.

2021-01-25T23:20:32+01:00

Jedną rzecz proponowałbym Ci jeszcze uwzględnić, żeby otrzymać bardziej poprawny wynik. Aktualnie twoja funkcja troszkę zaniża średnią, konkretnie o wartość 0,5. Dla przykładu co oczekiwałbyś od twojej funkcji jeżeli miałaby obliczyć średnią z następujących 8 próbek: {1, 1, 1, 1, 1, 1, 1, 0}? Twoja funkcja da wynik 0. A według mnie wypadałoby żeby uśredniła to jako 1. Jeżeli tak ma być to ok, a jeżeli nie to czytaj dalej.
Przy dzieleniu zmiennych całkowitych w języku C wynik zaokrąglany jest zawsze w dół, np. 7/8 otrzymasz w wyniku 0. Jak chcesz troszkę pogłówkować to zakończę już na tym moją podpowiedź.

Statystyki: Napisane przez wojtekr51 — 25 sty 2021, o 23:20

Re: Prośba o weryfikację kodu.

2021-01-25T21:07:27+01:00

mtbchn napisał(a):

Testowałeś?

Tak, po testach okazuje się, że wszystko działa poprawnie. Pytałem tylko, w razie czego bo mogło się zdarzyć, że coś przegapiłem i coś może nie działać. Bardzo dziękuję za pomoc.

Statystyki: Napisane przez antosV10 — 25 sty 2021, o 21:07

Re: Prośba o weryfikację kodu.

2021-01-25T19:12:49+01:00

Moim zdaniem kod jest poprawny i zgodny z oczekiwaniami. Testowałeś?

Statystyki: Napisane przez mtbchn — 25 sty 2021, o 19:12

Re: Prośba o weryfikację kodu.

2021-01-22T21:03:26+01:00

wonsz napisał(a):

linia 18, czym jest adc_value? bo z tego co widzę przypisujesz temu wartość średnią, ale mniejsza.

To zmienna globalna, której przypisuję wartość ADC, żeby móc używać jej potem w głównej pętli programu.

wonsz napisał(a):

Code:

sr[idx]=measurement(PA5);

Na pewno nie chciałeś wypełnić całej tablicy tylko pierwszy jej element? :^)

Ale nie rozumiem, przecie wypełniam całą tablicę, bo zmienna "idx" jest inkrementowana z każdym wywołaniem funkcji.

Statystyki: Napisane przez antosV10 — 22 sty 2021, o 21:03

Re: Prośba o weryfikację kodu.

2021-01-22T19:44:14+01:00

linia 18, czym jest adc_value? bo z tego co widzę przypisujesz temu wartość średnią, ale mniejsza.

Code:

sr[idx]=measurement(PA5);

Na pewno nie chciałeś wypełnić całej tablicy tylko pierwszy jej element? :^)

Statystyki: Napisane przez wonsz — 22 sty 2021, o 19:44

Prośba o weryfikację kodu.

2021-01-22T17:26:53+01:00

Witam, chciałem prosić aby ktoś kto zna się trochę na języku C zweryfikował czy przedstawiona poniżej funkcja poprawnie liczy średnią wartość ADC. Dziękuję za pomoc.

[syntax=c]#define SR 8
...
...
void get_average(void)
{
static uint16_t sr[SR];
static uint8_t idx;
uint32_t sr1 = 0;
uint8_t i;

sr[idx] = measurement(PA5);
idx++;
if(idx > 7)
{
idx=0;
for(i = 0; isr1 /= SR;
adc_value = sr1;
}

}[/syntax]

Statystyki: Napisane przez antosV10 — 22 sty 2021, o 17:26