Wyjaśnicie mi co jest złego w tym kodzie? (SPOJ - Średnia Arytmetyczna)

Question

#include <iostream>
#include <cmath>

using namespace std;

int najblizsza;
float srednia;
int suma;
int ile_testow;

int main()
{
    cin >> ile_testow;

    for (int i = 1; i <= ile_testow; i++)
    {

    int ile_liczb;

    cin >> ile_liczb;

    int liczby [ile_liczb];
    int wyniki [ile_liczb];

    for (int i = 0; i <= ile_liczb; i++)
    {
        cin >> liczby [ile_liczb];
        suma += srednia;
    }

    srednia = suma / ile_liczb;

    for (int i = 0; i <= ile_liczb; i++)
    {
        wyniki [ile_liczb] = abs(liczby [ile_liczb] - srednia);
    }

    najblizsza = wyniki[0];

    for (int i = 1; i <= ile_liczb; i++)
    {
        if( wyniki [i] < najblizsza )
        {
            najblizsza = wyniki[i];
        }
    }

        cout << najblizsza;
    }

    return 0;
}

Jak w temacie. Powiecie mi czemu Sędzia nie akceptuje mojego kodu? Zrobiłem coś źle konceptualnie? Jak to ugryźć? Może jakieś mniejsze niedociągnięcie, co? Kod dołączony.

Comments

Załączaj zawsze link do zadania.

---

Rozmiar tablicy musi być statyczny.

---

@mokrowski, okej, od teraz będę tak robił.  

---

@Velta, dobrze, zastosuję się do tego. Dzięki.

Answer 1

Cześć, próbowałeś chociaż wprowadzić przykładowane dane ?

Comments

Zacznij od tego a następnie staraj się likwidować napotkane błędy. Na start mogę wskazać:

•  Zmienne globalne.
•  Brakuje inicjalizacji sumy.
•  W pierwszej pętli wykonujesz ( suma += srednia ) bez przypisania lub inicjalizacji wartości zmiennej "srednia".

---

Okej. W takim razie tak zrobię. Dziękuję.

Answer 2

• Linie 6-9:  Te definicje powinny być w funkcji main.
• Linie 22 i 23: to nie są poprawne definicje tablic w C++.
• nie zerujesz suma dla każdego testu.

 

Comments

Dobrze wiedzieć, dzięki.