Программа с структурами

[Olya90]

Приветик всем!!
Вообщем у меня такая задачка:
С помощью структуры

struct pupil
{
char*name;
int age;
float rating
};

Нужно сделать чтобы пользователь с помощью меню мог вводить данные школьника а потом когда будет введено достаточно школьника, пользователь также с помощью меню мог найти школьников которым 13 лет и они имеют средний бал больше 4. Все ти дела нужно организовать функциями.
И еще вопрос правильно ли я сделала прогу на ввод школьников...

#include <cstdlib>
#include <iostream>

using namespace std;

struct pupil
{
char*name;
int age;
float rating;
};

void make_person()
{
	int Age; float rating_q; char Name[20];
	cout<<"Name?:";
	cin>>Name;
	cout<<"Age?:";
	cin>>Age;
	cout << "Rating?:";
	cin >>rating_q;
	pupil p;
	p.name=new char[strlen(Name)+1];
	strcpy(p.name,Name);
	p.rating=rating_q;
    p.age=Age;
}

int main(int argc, char *argv[])
{
int k;
do
{
cout<<"1. Form pupil\n";

cin>>k;
switch (k)
{
case 1: make_person(); break;
}
}
while (k!=0);


    system("PAUSE");
    return EXIT_SUCCESS;
}

[Sazary]

Школьника-то вы, вроде как, создаете. Только при выходе из функции он теряется.
Нужно либо делать массив школьников, либо список.
От того, как это будет выглядеть зависит и то, как будет выглядеть функция поиска.

[Сергей089]

А если сделать так?

В главной функции вводите кол-во школьников. Потом создаете динамический массив, состоящий из pupil. В вашей функции поменять тип с void на pupil и в конце добавить return p. И в цикле элементам массива присваивать функцию.

[Olya90]

Я сделала как вы написали

#include <cstdlib>
#include <iostream>

using namespace std;

struct pupil
{
char*name;
int age;
float rating;
};

pupil make_person()
{
	int Age; float rating_q; char Name[20];
	cout<<"Name?:";
	cin>>Name;
	cout<<"Age?:";
	cin>>Age;
	cout << "Rating?:";
	cin >>rating_q;
	pupil p;
	p.name=new char[strlen(Name)+1];
	strcpy(p.name,Name);
	p.rating=rating_q;
    p.age=Age;
return p;
}

int main(int argc, char *argv[])
{

int n;
cout << "Number of pupils:";
cin>> n;    
pupil mas=new pupil[n];

int k;
do
{
cout<<"1. Form pupil\n";

cin>>k;
switch (k)
{
case 1: make_person(); break;
}
}
while (k!=0);


    system("PAUSE");
    return EXIT_SUCCESS;
}

а как правильно тогда сделать чтобы в цикле элементам массива присваивать функцию??

[Sazary]

Ну вот, вроде:

#include <cstdlib>
#include <iostream>

using namespace std;

struct pupil
{
char* name;
int age;
float rating;
};

pupil make_person()
{
	int Age;
 float rating_q;
 char Name[20];
	cout<<"Name?:";
	cin>>Name;
	cout<<"Age?:";
	cin>>Age;
	cout << "Rating?:";
	cin >>rating_q;
	pupil p;
	p.name=new char[strlen(Name)+1];
	strcpy(p.name,Name);
	p.rating=rating_q;
 p.age=Age;
return p;
}

int main(int argc, char *argv[])
{

int n,i=0;
cout << "Number of pupils:";
cin>> n;
pupil *mas=new pupil[n];

int k;
do
{
 cout<<"1. Form pupil\n";

 cin>>k;
 switch (k)
  {
  case 1: mas[i++] = make_person(); break;
  }
 } while (k!=0);

system("PAUSE");
return EXIT_SUCCESS;
}

[Olya90]

Я вот немножко довела код до ума... Как думаете код нормальный ?? не сложный?

#include <cstdlib>
#include <iostream>

using namespace std;

struct pupil
{
char* name;
int age;
float rating;
};

pupil make_person()
{
	int Age;
 float rating_q;
 char Name[20];
	cout<<"Name?:";
	cin>>Name;
	cout<<"Age?:";
	cin>>Age;
	cout << "Rating?:";
	cin >>rating_q;
	pupil p;
	p.name=new char[strlen(Name)+1];
	strcpy(p.name,Name);
	p.rating=rating_q;
 p.age=Age;
return p;
}

void print_mas(int n, pupil *mas)
{
cout<<"Printig..."<<endl;
for(int count=1,i=0;i<n;i++,count++)
if(mas[i].age==13 && mas[i].rating>4)
cout<<"\nName:  "<<mas[i].name<<"\t"<<"Age:  "<<mas[i].age<<"\t"<<"Rating: "<<mas[i].rating<<endl;
}

int main(int argc, char *argv[])
{

int n,i=0;
cout << "Number of pupils:";
cin>> n;
pupil *mas=new pupil[n];
 for(int i=0;i<n;i++)
  mas[i] = make_person();

int k;
do
{
cout<<"1. Print Array\n";
cout<<"0. Exit\n";

cin>>k;
switch (k)
{
case 1: print_mas(n,mas); break;
}}
while (k!=0);

system("PAUSE");
return EXIT_SUCCESS;
}

[Sazary]

Как думаете код нормальный ?? не сложный?

В функции make_person не обязательно объявлять 3 дополнительные переменные. Можно сразу писать в поля структуры.

[Olya90]

А не могли бы показать...

[Sazary]

pupil make_person()
{
	pupil p;
	cout<<"Name?: ";
	p.name=new char[strlen(Name)+1];
	cin>>p.name;
	cout<<"Age?: ";
	cin>>p.age;
	cout << "Rating?: ";
	cin >>p.rating;
return p;
}

[Olya90]

ОК спасибо !!!