Średnia arytmetyczna elementów z przedziału

Question

Cześć! Napotkałem na pewne trudności w zadaniu i kompletnie nie wiem, gdzie popełniłem błąd. Mam do napisania funkcję, która dla n-elementowej tablicy, liczy średnią arytmetyczną jej elementów z przedziału. Kombinowałem na wiele sposobów posiłkując się przykładami robionymi na zajęciach i doszedłem do tego punktu. Oto kod:

    function sredniaDane(tab, n) {

var i, suma;

suma = 0;

        for (i=0; i<n; ++i) suma += tab[i];

        return suma/n;

    }

    function sredniaPrzedz(tab, n, a, b) {

        var srw = new Array(n);

        var i, num, tab1;



        tab1 = Array(n);

        num = 0;

        for (i=0; i<n; ++i) srw[i] = sredniaDane(tab[i], n);

        for (i=0; i<n; ++i)

            if (tab[i]>a && tab[i]<b) tab1[num++] = tab[i];

        for (i=0; i<n; ++i) document.writeln("średnia ", " ", srw[i], "<BR>");

    }

Po wywołaniu: sredniaPrzedz(new Array(1, 2, 3, 4, 5, 6, 2), 7, 1, 6) otrzymuje następujący wynik:

 

Czy znajdzie się jakieś wprawne oko i dobra duszyczka, która wskaże błędy?  
Zaznaczam, że w temacie JS jestem raczkujący.

Answer 1

do sredniaDane w parametrze tab przekazujesz jedną liczbę a traktujesz przy sumowaniu jako tablicę (linia 7)

offtop: używając pętli używaj klamer, żeby to ładnie wyglądało {}, w js nie trzeba podawać wielkości Array tworząc go, możesz użyć kwadratowych nawiasów żeby zrobić pustą tablicę []

Comments

Wielkie dzięki, jeden błąd zniwelowany!  Natomiast w tym momencie strona zwraca następujące wartości:

 

Finalnie średnia z całego Array powinna zostać obliczana w jednym wierszu i tam zwracany całkowity wynik. Jak osiągnąć taki efekt końcowy? 

---

zapewne usunąłeś [i] z 7 linii (tam powinieneś zostawić iterowanie po tablicy), zamiast usunąć [i] z 25 gdzie podajesz tą te liczby do przemielenia,

srw[i] = sredniaDane(tab, n);

bo ta średnia powinna wynosić coś koło 3.2, a u Ciebie pokazuje tylko te cyfry które podałeś :P

 

---

Rzeczywiście, masz rację usunąłem tam, gdzie nie potrzeba i do tego za dużo  (błędy początkującego). Natomiast teraz owszem, średnia liczy się elegancko, ale po wpisaniu: sredniaPrzedz(new Array(1, 3, 5, 6, 7), 5, 1, 6) - ostatnie dwie liczby wyjściowo mają określać przedział liczonej średniej. A aktualnie mam dwa wiersze (sądzę, że są one z powodu tego, że 3 i 5 mieszczą się w przedziale) z tą samą średnią. 

---

mógłbyś napisać, jaki rezultat chciałbyś uzyskać? czy jeśli przedział to 1, 6 to nie chcesz brać pod uwagę tej liczby 7 z tablicy? jeśli w ten sposób to musisz zrobić filtrowanie

var wejscie = [1,3,5,6,7];

var a = 1; // minimalna liczba
var b = 6; // maksymalna liczba
var liczby = wejscie.filter(sprawdzCzyJestWPrzedziale); // [1,3,5,6]

function sprawdzCzyJestWPrzedziale(liczba) {
  return (liczba >= a && liczba <= b);
}

---

Tak, dokładnie tak jak mówisz. Tylko wtedy gdzie to umieścić i jak odpowiednio wywołać ?
Oto kod:

    function sredniaDane(tab, n) {
        var i, suma;
        suma = 0;
        for (i=0; i<n; ++i) suma += tab[i];

        return suma/n;
    }
    function sredniaPrzedz(tab, n, a, b) {
        var srw = new Array(n);
        var i, num, tab1;

        tab1 = Array(n);
        num = 0;
        for (i=0; i<n; ++i) srw[i] = sredniaDane(tab, n);
        for (i=0; i<n; ++i)
            if (tab[i]>a && tab[i]<b) tab1[num++] = tab[i];

        for (i=0; i<num; ++i) document.writeln("średnia ", " ", srw[i], "<BR>");
    }

kombinuje na różne sposoby, a i tak średnią oblicza mi ze wszystkich liczb w tablicy