Średnia ważona

Question

Witam!

Bazując na wskaźnikach chciałem stworzyć kod, który będzie wyliczał średnią, ważoną jednak kod nie działa. Sprawdzałem i kod niepoprawnie sumuje oceny i nie mam pojęcia czemu. Dodam, że dopiero zacząłem używać wskaźników, więc możliwe, że nie do końca rozumiem zasadę ich działania. 

#include <iostream>

using namespace std;

float srwaga(float *marks, int *wag, int ilosc )
{
    int sumawag = 0;
    float sumaocen = 0;
        for(int i=0; i<ilosc;i++)
        {
            sumaocen+=*marks**wag;
            sumawag+=*wag;
            marks++;
            wag++;



        }
        return sumaocen/sumawag;




}




int main()
{
 int ile;

  cout<<"podaj ile ocen: ";
 cin>>ile;


 float *oceny;
 oceny = new float[ile];
 int *wagi;
 wagi=new int[ile];



 for(int i=0;i<ile;i++)
 {

     cout<< "podaj ocene i wage "<<i+1<<" : ";
     cin>>*oceny>>*wagi;
     oceny++;
     wagi++;


 }

 cout<<srwaga(oceny,wagi,ile);


delete [] oceny;
delete [] wagi;



    return 0;
}

Dodatkowo, czy postawienie obok siebie dwóch gwiazdek tj. od wskaźnika oraz mnożenia nie koliduje ze sobą?
Pozdrawiam i z góry dziękuję za pomoc :)

Answer 1

Moja rada jest taka - jak dopiero zaczynasz ze wskaźnikami to nie porywaj się od razu na tablice dynamicznie alokowane, tylko może najpierw poużywaj samych wskaźników - myślą że jak chcesz coś ze wskaźnikami porobić to pobaw się listą (łańcuchem odsyłaczowym) stosem, kolejką, drzewem, itp bo te struktury opierają się tylko na wskaźnikach, jest mnóstwo operacji które można na nich wykonać i przy okazji jak te struktury ogarniesz to wskaźniki nie będą miały przed tobą tajemnic (takie jest moje zdanie - najpierw same wskaźniki a potem struktury, dynamiczna alokacja pamięci gdzieś dalej), ale co do programu to wystarczy zmienić:

 cin>>*oceny>>*wagi;
 oceny++;
 wagi++;

na:

cin>>oceny[i]>>wagi[i];

 

Comments

Dzięki, działa :) Oraz dziękuje za radę :)

---

@k222

Hej, wiem że to nie ja założyłem ten temat ale mam małe pytanie, czemu właściwie zapis:

cin>>*oceny>>*wagi;
oceny++;
wagi++;

powoduje problemy?...  też chciałem udzielić odpowiedzi, zmieniłem ten zapis na tablicowy i wszystko działało, wolałem jednak nie pomagać gdyby miał być zapytany czemu to trzeba było zmienić :/  Z góry dziękuje za wyjaśnienie i pozdrawiam

---

Hmm dobre pytanie, też chciałbym wiedzieć ;)

---

Wydaje mi się, że ma to związek z tym, że nazwa wskaźnika ma adres pierwszego elementu tablicy. I teraz jak używasz oceny++; to nazwa wskaźnika reprezentuje drugi element tablicy. Czyli jak dojdziesz do końca to nazwa wskaźnika pokazuje na adres komórki pamięci która nie została wcześniej zarezerwowana dlatego wychodzą niepoprawne dane.

---

Jeżeli chcesz żeby działało tak jak to zrobiłeś to dodaj coś takiego :

int main()
{
	int ile;

	cout << "podaj ile ocen: ";
	cin >> ile;


	float *oceny;
	oceny = new float[ile];
	int *wagi;
	wagi = new int[ile];

	//To dodajesz 
	float* a = oceny;
	int* b = wagi;

	for (int i = 0; i<ile; i++)
	{

		cout << "podaj ocene i wage " << i + 1 << " : ";
		cin >> *oceny >> *wagi;
		oceny++;
		wagi++;


	}
	//Tutaj wskaźnik pokazuje "od nowa" na pierwszy element
	oceny = a;
	wagi = b;
	
	cout << srwaga(oceny, wagi, ile);

	cin.ignore();
	delete[] oceny;
	delete[] wagi;



	return 0;
}

 

---

To ma sens :) nie pomyślałem o tym