mnożenie tablic

Question

cześć, mam pytanie. ćwicze sobie c++, chce zrobić program, który pobierze oceny, a potem policzy średnią. korzystam z tablic - robię to dobrze? oceny niby pobiera, a gdy chce policzyć średnią i pisze "tablica[1]/tablica[2].. itd" to crashuje mi program. jak to zrobić dobrze?
i podobno 3 opcja, którą tam dodałem możliwa nie jest, to prawda?

pozdrawiam




#include <iostream>
#include <cstdlib>
#include <windows.h>
#include <stdio.h>

using namespace std;

int oceny[5];
int wybor;
float srednia;
int main()
{
    for(;;)
{

    cout<<endl;
    cout<<"MENU GLOWNE"<<endl;
    cout<<"--------------"<<endl;
    cout<<"1. Wypisz oceny. "<<endl;
    cout<<"2. Policz srednia. "<<endl;
    cout<<"3. Wyczysc pamiec ocen. "<<endl;

    cin>>wybor;

    switch(wybor)
    {
    case 1:

        cout<<"Wypisz oceny: "<<endl;

        for (int i=1; i<=5; i++)
        {
            cout<<"Podaj 1 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 2 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 3 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 4 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 5 ocene: "<<endl;
            cin>>oceny[i];

            break;

            getchar();

            system("cls");
        }
    case 2:
        {
            srednia = oceny[1]; // tu miała być podzielona każda tablica przez siebie, ale crashuje gdy wybiore opcje 2
            cout<<srednia<<endl;
        }

    }





}



    return 0;
}

 

Answer 1

Tablice nie są dynamiczne lecz globalne, więc pamięci nie możesz zwolnić w trakcie działania programu.

Neprawidłowo wpisujesz wartości do tablicy. Pętla wykonuje się od 1 do 5 włącznie. A indeksy tablicy mają wartości kolejno: 0, 1, 2, 3, 4. Więc Ty będziesz próbował wpisać wartość do elementu o indeksie [5], którego nie ma!
Pętla powinna wyglądać tak:
for( int i = 0; i < 5; i++ )
a poza tym, źle korzystasz z tej pętli! (na dole umieszczę poprawiony kod, bo tłumaczyć tutaj co jest źle, to za dużo roboty :-P)

Średnią policzymy tak:
srednia = (float)(oceny[0]+oceny[1]+oceny[2]+oceny[3]+oceny[4])/5;

W razie kłopotów — pytaj! :-P

#include <iostream>
#include <cstdio>
#include <windows.h>

using namespace std;

int oceny[ 5 ];
int wybor;
float srednia;
int main()
{
	for( ;; )
	{

		cout << endl;
		cout << "MENU GLOWNE" << endl;
		cout << "--------------" << endl;
		cout << "1. Wypisz oceny. " << endl;
		cout << "2. Policz srednia. " << endl;
		cout << "3. Wyczysc pamiec ocen. " << endl;

		cin >> wybor;

		switch( wybor )
		{
		case 1:
			cout << "Wypisz oceny: " << endl;

			for( int i = 0; i < 5; i++ )
			{
				cout << "Podaj " << i + 1 << " ocene: " << endl;
				cin >> oceny[ i ];
			}
			getchar();
			system( "cls" );
			break;

		case 2:
			srednia = (float)(oceny[ 0 ] + oceny[ 1 ] + oceny[ 2 ] + oceny[ 3 ] + oceny[ 4 ]) / 5; 
			cout << srednia << endl;
		}
	}

	return 0;
}

Poprawek jest sporo:
- dołączyłeś niepotrzebne biblioteki
- źle napisałeś pętlę, która wykonywała się raz
- ciągle nadpisywałeś drugi element tablicy o indeksie 1
- break w złym miejscu
....

Comments

wielkie dzięki, działa! teraz mam pare pytań czy dobrze załapałem:

- dołączyłem niepotrzebne biblioteki - to źle? wiadomo, ze jak są niepotrzebne to nie mam ich dołączać, ale czy bardzo to przeszkadza?
- prawdopodobnie jeszcze dobrze nie załapałem pętli :/

- a gdzie to nadpisywałem? bo nie widzę tego :P

 

dziękuje jeszcze raz, nie liczyłem aż na taką pomoc :D

---

dołączanie bibliotek, z których nie korzystasz nie jest problemem. Chodzi głównie o to, że nie wiesz która biblioteka za co odpowiada. Na przykład: <iostream> dodaje "cout" "cin", <Windows.h> nie dodaje tutaj nic, <cstdio> dodaje getchar(), <cstdlib> dodaje system() i tak dalej... trzeba to mniej więcej kojarzyć.

Gdzie nadpisywałeś? Tutaj:

           cout<<"Podaj 1 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 2 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 3 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 4 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 5 ocene: "<<endl;
            cin>>oceny[i];

Pętla, w której to umieściłeś wykonywała się tylko raz. Zmienna "i" była równa 1. Zauważ, że za każdym razem wczytujesz wartość od użytkownika do tej samej komórki w tabeli: oceny[ i ], więc za każdym razem nadpisujesz tę wartość

Answer 2

for (int i=1; i<=5; i++)
        {
            cout<<"Podaj 1 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 2 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 3 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 4 ocene: "<<endl;
            cin>>oceny[i];
            cout<<"Podaj 5 ocene: "<<endl;
            cin>>oceny[i];
 
            break;
 
            getchar();
 
            system("cls");
        }

W powyższej pętli masz błąd. Zauważ, że ciągle zczytujesz od użytkownika wartość tablicy o wartości i.

Możesz więc albo usunąć pętlę, albo zrobić to jedną linijką w pętli :)

Pozdrawiam :)

A i PS. Zrobiłeś kardynalny błąd tym: <=            Pamiętaj, że tablice numerowane są od zera.

EDIT: Nie zauważyłem, że Sebastian odpisał. On to dobrze wytłumaczył :)

Comments

Zawsze warto podać własną propozycję rozwiązania problemu. Oceniana jest jakość a nie pierwszość :-)

"Zrobiłeś kardynalny błąd tym"
Kardynalny i do tego elementarny, podstawowy, który na szczęście z upływem czasu zanika. Po kilkukrotnym wyjściu poza zakres będzie się już pamiętać o indeksowaniu od zera :-P

(nie zmienia to faktu, że w programie kolegi pętla była tylko dla wyglądu XD)

---

również dziękuje za pomoc, sebastian dobrze to wytłumaczył :)

---

o co chodzi, że dla wyglądu? :c

i tym kardynalnym błędem jest to, że nie dałem i=0 w pętli, przez co w tablicy nie liczyło o 1 "szufladki" (jeszcze mi brak fachowego słownika :D ) tylko od 2?

---

Nie, nie. Napisałem "dla wyglądu" ponieważ deklaracja pętli:
for (int i=1; i<=5; i++)
wskazuje na to, że wykona się 5 razy.

Natomiast Ty na końcu pętli umieściłeś:   break;
Czyli w tym miejscu pętla się kończy. Oznacza to, że Twoja pętla wykona się raz i koniec. W takim razie mogłoby jej w ogóle nie być. Czyli jest tylko dla wyglądu :-)