Zmienia mi wartosc a nie powinno

Question

Czy wie ktos dlaczego zmienia mi wartosc 'tablica[4]' po policzeniu sredniej?

#include <iostream>

using namespace std;

int main()
{
    int tablica[4];
    int suma=0;
    float srednia;
    float szuk[4];

    for(int i=0;i<5;i++)
    {
        cin>>tablica[i];
        suma+=tablica[i];
    }
    cout<<endl<<tablica[4]<<endl;
    srednia=suma/5;
    cout<<endl<<tablica[4]<<endl;

    return 0;
}

Answer 1

Błąd jest w definicji tablicy - widzę, że chciałeś zdefiniować 5 elementów tablicy, lecz zamiast tego zdefiniowałeś 4 (przy definicji tablicy nie numeruje się od zera). Dlatego kiedy pętla dochodzi do 4 to wychodzi ona poza tablice.

Comments

Dzieki

Answer 2

Zmienia się bo zakres indeksów (zmienna i w for(...)) powinien obejmować 0 do 3 jeśli tablica ma 4 elementy. Stąd pętla powinna być zrealizowana tak:

for(int i = 0; i < 4 ;i++)

Inną sprawą jest zachowanie niezdefiniowane pobrania danych z indeksu 4. Jeszcze raz, przy deklaracji tablicy tablica[4] masz maksymalny indeks 3.

No i ostatni błąd to liczenie średniej. Nie: srednia = suma / 5; tylko: srednia = suma / 4; bo masz 4 dane (od indeksu 0 do 3 włącznie).

Comments

Dzieki :)

Answer 3

Tablica ma 4 elementy. Do każdego z nich odwołujesz się tak:

n-1
Gdzie n równa się "numer elementu". Gdy n = 4:
tablica[n-1] -> tablica[4-1] -> tablica[3]
lub
*(tablica+n-1) -> *(tablica+4-1) -> *(tablica+3)

Odwołując się do 4 elementu (czyli w tym wypadku 5, bo w informatyce liczymy od zera, na tym opiera się ten "wzór" powyżej, a wynika to bodajże z systemu binarnego) dostajesz się do śmieci, do pamięci, która nie należy do Ciebie.