kalkulator średniej nie działa

Question

Cześć, przed chwilą głowiłem się nad zrobieniem kalkulatora średniej i wszystko działa, dopóki nie spróbuje wpisać 13 liczb. Wtedy wynik wychodzi źle.

#include <iostream>
using namespace std;
int x;
int liczba;
int i;
float wynikdodawania;
float wynik;
int main (){
    cout << "z ilu ocen chcesz srednia: ";
    cin >> liczba;
int ocena[liczba];
while (i<=liczba-1){
    i++;
    cout << "wprowadz ocene ";
    cout << i <<endl;
    cin >> ocena[i-1];

}
while (i>=0){
    wynikdodawania=wynikdodawania+ocena[i-1];
    i--;
}
wynik=wynikdodawania/liczba;
cout <<wynik;
return 0;
}

 

Comments

Gdzie inicjalizujesz i?

---

@Dod00,

Spostrzeżenie nie związane z problemem, ale tablica powinna być  tworzona dynamicznie.

int* ocena = new int [ile]; 

[...]


delete [] ocena; 

 

 

 

---

@amtrax Niektóre kompilatory przepuszczą tak zadeklarowaną tablicę

---

Nie jest to zgodne z żadnym standardem c++.

---

Ale g++ to przepuszcza

---

@Michałełe, 

tak, większość przepuszcza, wystosowane jest tylko ostrzeżenie. Warto o tym jednak pamiętać, przy większych projektach może to powodować problemy. Np. SPOJ nie akceptuje tak zadeklarowanych tablic. 

---

@Michałełe, To, że przepuszcza nie jest równoważne z dopuszczaniem przez standard. Jest to archaizm z C o nazwie VLA.

Answer 1

Zmień warunek w tej drugiej pętli while:

	while (i > 0) {
		wynikdodawania = wynikdodawania + ocena[i - 1];
		--i;
	}

Bo w Twoim przypadku pętla zadziała również w przypadku gdy i == 0 - a przecież w środku pętli wyjmujesz z tablicy element i - 1, przez co gdy i jest równe zero dodajesz jakieś "śmieci"

Comments

#include <iostream>

int main ()
{
    std::cout << "z ilu ocen chcesz srednia: ";
        int numOfElements;
    std::cin >> numOfElements;
        int *marks = new int [numOfElements];
        int i = 0;
    while (i <= numOfElements - 1){
        std::cout << "wprowadz ocene ";
        std::cout << i << std::endl;
        std::cin >> marks[i];
        i++;
    }
    double sum = 0;
    while (i >= 0){
        sum += marks[i-1];
        i --;
    }
    double avarage = sum / numOfElements;
    std::cout << avarage;
    delete [] marks;
    return 0;
}

 

Btw poprawilem kod i uwagi do autora:

1. Jak robisz odstepy miedzy roznymi znakami, to rob je w calym kodzie, a nie czasami tak czasami nie

2. zmienne lokalne > zmienne globalne

3. ciekawy pomysl z uzyciem tylko jednej zmiennej iteracyjnej

4. staraj sie deklarowac zmienne w miejscu gdzie ich uzywasz

5. staraj sie pisac po angielsku

6. to co robisz, nie jest wspierane w standardzie, wiec staraj sie omijac takie praktyki, a najlepiej przeczytaj o std::vector, std::list, std::map etc

Answer 2

Używasz  zmiennej i  w pętli while.

A jaka jest jej wartość początkowa?

Skoro w pętli stosujesz i++;