Program przestał działać

Question

Witam wszystkich,

Ślęczę już nad kodem jakiś czas i nie potrafię znaleźć wytłumaczenia, dlaczego program przestaje działać pod koniec kodu.

Mój kod:

#include <iostream>
#include <string>
using namespace std;


struct student
{
	string imie;
	int  ocena_pol;
	int  ocena_mat;
	int  ocena_inf;
};

void drukuj(student pupil[], int ilosc)
{
		string name;
		
		cout << endl << endl;
	
		cout << "Wybierz studenta wpisujac jego imie: ";
		cin >> name;
		
		cout << endl;
		
		for(int i=0; i<ilosc; i++)
		{
			if(name==pupil[i].imie)
			{
				cout << "Imie: " << pupil[i].imie << endl;
				cout << "J.pol: " << pupil[i].ocena_pol << endl;
				cout << "Mat: " << pupil[i].ocena_mat << endl;
				cout << "Inf: " << pupil[i].ocena_inf << endl;
			}
		}
}

student studenci (int tablica[], int ilosc)
{
	student pupil[ilosc];
	
	for(int i=0; i<ilosc; i++)
	{
		cout << "Podaj imie studenta: ";
		cin.ignore();
		getline(cin, pupil[i].imie);
		
		cout << "Podaj ocene z j. polskiego: ";
		cin >> pupil[i].ocena_pol;
		
		cout << "Podaj ocene z matematyki: ";
		cin >> pupil[i].ocena_mat;
		
		cout << "Podaj ocene z informatyki: ";
		cin >> pupil[i].ocena_inf;
		cout << endl;
	}
	
		drukuj(pupil, ilosc);
		
		return pupil[ilosc];
}



int main()
{
	int ilosc;
	cout << "Podaj ilosc studentow: ";
	cin >> ilosc;
	
	cout << endl;
	int tablica[ilosc];
	studenci(tablica, ilosc);

	return 0;
}

Z góry dziękuję za pomoc!

Answer 1

Witam,

na początku popraw kod. Wtedy łatwiej będzie analizować kod. Zauważ że w funkcji studenci nie używasz zmiennej tablica oraz nie ma potrzeby zwracania struktury

#include <iostream>
#include <string>
using namespace std;


struct student
{
    string imie;
    int  ocena_pol;
    int  ocena_mat;
    int  ocena_inf;
};

void drukuj(student pupil[], int ilosc)
{
        string name;

        cout << endl << endl;

        cout << "Wybierz studenta wpisujac jego imie: ";
        cin >> name;

        cout << endl;

        for(int i=0; i<ilosc; i++)
        {
            if(name==pupil[i].imie)
            {
                cout << "Imie: " << pupil[i].imie << endl;
                cout << "J.pol: " << pupil[i].ocena_pol << endl;
                cout << "Mat: " << pupil[i].ocena_mat << endl;
                cout << "Inf: " << pupil[i].ocena_inf << endl;
            }
        }
}

void studenci (int ilosc)
{
    student pupil[ilosc];

    for(int i=0; i<ilosc; i++)
    {
        cout << "Podaj imie studenta: ";
        cin.ignore();
        getline(cin, pupil[i].imie);

        cout << "Podaj ocene z j. polskiego: ";
        cin >> pupil[i].ocena_pol;

        cout << "Podaj ocene z matematyki: ";
        cin >> pupil[i].ocena_mat;

        cout << "Podaj ocene z informatyki: ";
        cin >> pupil[i].ocena_inf;
        cout << endl;
    }

        drukuj(pupil, ilosc);

}



int main()
{
    int ilosc;
    cout << "Podaj ilosc studentow: ";
    cin >> ilosc;

    cout << endl;
    studenci(ilosc);
    cout << "Koniec " << endl;
    return 0;
}

Po drugie źle definiujesz tablice. Nie możesz używać statycznych tablic z nieznanym rozmiarze

int ilosc;
cin >> ilosc;
int tablica[ilosc];

w takim przypadku musisz użyć tablic dynamicznych

int ilosc;
cin >> ilosc;
int *tablica = new int[ilosc];
*
*
*
delete [] tablica;

https://www.p-programowanie.pl/cpp/tablice-dynamiczne/

Comments

Wyjaśnienie czemu nie możesz statycznie: 

https://forum.pasja-informatyki.pl/384088/tablica-statyczna-a-dynamiczna?show=384088#q384088

A program przestawał działać prawdopodobnie przez wyjście poza zakres tablicy:

return pupil[ilosc];

---

A gdybyście tak przestali tablice automatyczne nazywać statycznymi?