PROGRAM NIE DAJE POPRAWNEGO WYNIKU

Question

Dane jest zadanie:
Program ma najpierw pobrać ciąg n liczb, w następnej linijce pobrać 2 liczby (u mnie są to: a i b).  W kolejnym kroku program ma wyliczyć średnią ciągu liczb rozpoczynając od a liczby od lewej strony, kończąc na liczbie b (również licząc od lewej strony).

O to napisany przeze mnie kod:

#include <iostream>
#include <cmath>
using namespace std;

int main()
{
 int ile, n, z, a, b, k, suma=0;
 double srednia;
 cin>>ile;
 int tab[ile];

 for (n=0; n<ile; n++)
 {
     cin>>tab[n];
 }
    cin>>a>>b;
    k=(b-a-1);
    z=(b-a+1);

 for (int g=(b-a-1); g<=b; g++)
 {
    suma+=tab[k];
    k++;
 }
    srednia=suma/z;
    cout<<floor(srednia);
}

I tu pojawia się pytanie:
Czy ktoś ma może pomysł dlaczego program nie działa i chciałby zasugerować jakieś jego rozwiązanie?

Answer 1

--a; --b;

for (int i = a; i < b; ++i)  {
    suma += tab[i];
 }

A tak nie wystarczy?

 

cin>>ile;
int tab[ile];

Takie cuda to nie w C++ (g++ dopuszcza taką konstrukcję, ale to błąd). Nie ma w zadaniu określonej maksymalnej wartości n? Bo jeśli jest, to zwykła tablica wystarczy.

 

Z drugiej strony według tego co napisałeś, to najpierw podana jest długość ciągu n, a później wartości a i b. A w kodzie po n czytasz ciąg liczb. Jeśli po n następują a i b, to tablica nie jest potrzebna.

srednia=suma/z;
cout<<floor(srednia);

Ten floor jest tutaj o tyle bez sensu, że operacja dzielenia jest całkowitoliczbowa (oba operandy są typu int), więc siła rzeczy jej wynik będzie liczba całkowitą i nie ma czego zaokrąglać.