Nie mogę znaleźć błędu w kodzie :(

Question

Witam, robiłem zadanie, w którym użytkownik ma podać ile ma znaków, po czym ma je podać. Program wtedy powinien obliczyć średnią z jego znaków i podać liczbę najbardziej zbliżoną do średniej. Nie rozumiem skąd wynika problem i proszę o pomoc. Może się ten kłopot wydawać bardzo trywialny, ale już trochę siedziałem nad tym zadaniem i nie mam pojęcia gdzie jest usterka.

#include <iostream>
#include <math.h>

using namespace std;
int n, naj, sum, sr;
int tab[100];
int main()
{
    cout << "How many markss do u have: " ;
    cin >> n ;

    for(int i=0; i<n; i++)
    {
        cout << "Enter number: " << i+1 << " ";
        cin >> tab[i];
        sum+=tab[i];
    }

    sr=sum/n;


    for(int i; i<n; i++)
    {
        naj=tab[0];
        if((abs(sr-tab[i]))<(abs(sr-naj)))
        naj=tab[i];
    }

    cout << sr <<endl;
    cout << naj ;

    return 0;
}

 

Comments

W drugiej pętli nie przypisałeś wartości iteratorowi

---

No i średnią dałeś jako int, jeśli chcesz liczbę przecinkową to powinno być float, a jak zależy Ci na int to nie jestem pewny, ale wynik może być np. 0 albo 1 i wtedy można usunąć najpierw resztę z dzielenia, którą można obliczyć modulo (%) i potem podzielić

Answer 1

for(int i; i<n; i++)
    {
        naj=tab[0];
        if((abs(sr-tab[i]))<(abs(sr-naj)))
        naj=tab[i];
    }

W każdym obiegu pętli na samym początku nadpisujesz zmienną naj pierwszym elementem tablicy. Poza tym zmienna i nie jest inicjowana na 0.

sr=sum/n;

To jest dzielenie całkowito-liczbowe. Ucinasz w ten sposób część ułamkową średniej.

Zmienne globalne i using namespace std to złe nawyki. Wszystkie zmienne w tym programie powinny być deklarowane w mainie.

Comments

Dziękuję bardzo za pomoc, Tylko nie rozumiem w jaki sposób miał bym wyznaczyć średnią.

---

Napotkałem się również na drugi problem jakim jest to że pokazuje mi funkcje abs, jako nie zdeklarowaną.

---

Skoro średnia może być liczbą ułamkową, to powinieneś ją zapisać do zmiennej, która przechowuje liczby zmiennoprzecinkowe, czyli float lub double. Poza tym jak zostawisz zmienne sum i n jako inty, to musisz też rzutować przynajmniej jedną z nich na float (lub double) przy dzieleniu.

Co do abs, to wklej treść błędu. Jak includujesz math.h, to nie powinno być problemu z deklaracją.

---

Średnią już dostosowałem w miedzy czasie do liczb zmiennoprzcinkowych (float), a w przypadku funkcji abs podmieniłem ją na fabs. Wszystko działa tak jak miało, dziekuję za chęci i czas.