Średnia arytmetyczna przez wskaźniki

Question

#include <iostream>

using namespace std;

void setNumbers(int &x,float *tab,float *ind)
{
    cout <<"Podaj ilosc liczb: ";
    cin >> x;

    tab = new float[x];
    ind = tab;

    for(int i=0;i<x;i++)
    {
        cout<<"Podaj "<<i+1<<" liczbe: ";
        cin >> *ind;
        ind++;
    }

}

float getAverage(int x,float &y,float *tab,float *ind)
{
    ind=tab;

    for(int i=0;i<x;i++)
    {
        y+=*ind;
        ind++;
    }

    return y/=x;
}


int main()
{
    int quantity;
    float *w;
    float *numbers;
    float average=0;

    setNumbers(quantity,numbers,w);
    cout <<"Srednia liczb wynosi: "<<getAverage(quantity,average,numbers,w)<<endl;

    delete[]numbers;

    return 0;
}

Po skompliowaniu program zamiast poprawnej średniej wyrzuca śmieci.

Wiecie może gdzie jest błąd i dlaczego tak się dzieje ? Mam problem z przekazywaniem dynamicznych tablic i wskaznikow do funkcji i ich wykorzystywanie...

Answer 1

Ok,problem został rozwiązany .

Po prostu popełniłam błąd deklarując tablicę dynamiczną w procedurze.

Niech ktoś oceni obecny kod:

#include <iostream>

using namespace std;

void setNumbers(int x,float *ind)
{

    for(int i=0;i<x;i++)
    {
        cout<<"Podaj "<<i+1<<" liczbe: ";
        cin >> *ind;
        ind++;
    }

}

void setQuantity(int &x)
{
    cout <<"Podaj ilosc liczb: ";
    cin >> x;
}

float getAverage(int x,float &y,float *ind)
{

    for(int i=0;i<x;i++)
    {
        y+=*ind;
        ind++;
    }

    return y/=x;
}


int main()
{
    int quantity;
    float *w;
    float *numbers;
    float average=0;

    setQuantity(quantity);
    numbers = new float[quantity];
    w = numbers;

    setNumbers(quantity,w);
    cout <<"Srednia liczb wynosi: "<<getAverage(quantity,average,w)<<endl;

    delete[]numbers;

    return 0;
}

 

Comments

Znaczące nazwy, zmienna "x" jest kompletnie nijaka. "size", albo "capacity" jest zdecydowanie lepsza. 

 return y/=x;

Dlaczego "/="? Co zyskujesz przypisaniem?

Działanie na zmiennych referencyjnych jest mylące i narusza dobre praktyki. 

Nie deklaruj zmiennych sporo przed ich użyciem.

    int quantity;
    setQuantity(quantity);
    float *numbers; = new float[quantity];
    float *w = numbers;
    float average=0;
    setNumbers(quantity,w);
    cout <<"Srednia liczb wynosi: "<<getAverage(quantity,average,w)<<endl;

Na koniec, dlaczego działasz na wskaźnikach?

---

Na koniec, dlaczego działasz na wskaźnikach?

Wybrałam sobie taki przykład,aby po prostu nauczyć się wprawy w ich wykorzystywaniu :P

 Dlaczego "/="? Co zyskujesz przypisaniem?

Mój błąd.

 Działanie na zmiennych referencyjnych jest mylące i narusza dobre praktyki. 

Z ciekawości,dlaczego ? Chodzi tutaj o ryzyko przypadkowej ingerencji w oryginał ? 

---

Między innymi, ale zadaj sobie pytanie, czy potrzebujesz, by Twoja funkcja modyfikowała/zwracała więcej niż jedną rzecz? Jeżeli tak, prawdopodobnie robi za dużo. Skoro chcesz wysłać coś przez referencję, a tego nie modyfikujesz, pamiętaj o const, uchroni przed błędami.

Oczywiście są wyjątki, i odpowiedź "to zależy".