Правильно записать вложенные структуры

[alexboliam]

Здравствуйте, мое задание звучало так: создать массив структур. Каждая структура состоит из следующих элементов:факультет и студент. Для факультета задать его название, курс и массив групп. Для студента задать его фамилию и массив экз. оценок...
Там дальше идет о записи в файл, редактировании, сортировке и тд, но меня интересует сама конструкция этого массива. Сначала я сделал без массива курсов, к которым будут относиться группы и это вышло вот так:

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <conio.h>
#include <cstdio>
#include <conio.h>
#include <stdlib.h>
#include <fstream>
#include <cstring>
using namespace std;
//////////////////////////////////////////////////////////////////////////////////////////////   
//Структуры
struct student {
    char lastname[20];
    int marks[15];
};
struct group {
    char name[10];
    student students[30];
};
struct faculty {
    char name[30];
    int course;
    group groups[20];
};
//////////////////////////////////////////////////////////////////////////////////////////////   
//Прототипы
 
void enterdata(faculty**);
 
//////////////////////////////////////////////////////////////////////////////////////////////     
//Функции
 
int main() {
    faculty **Faculty = new faculty*[10];
    for (int i = 0; i < 10; i++) {
        Faculty[i] = new faculty;
    }
    enterdata(Faculty);
    system("pause");
    return 0;
}
void enterdata(faculty **Faculty) {
    int i = 0,j,k,m;
    char key, key1, key2;
    do
    {
        system("cls");
        cout << "Enter name of faculty: ";
        cin >> Faculty[i]->name;
        cout << "Enter the course: ";
        cin >> Faculty[i]->course;
        j = 0;
        do
        {
            system("cls");
            cout << "| Faculty: " << Faculty[i]->name << " | Course: " << Faculty[i]->course << " |\n";
            cout << "Enter name of group: ";
            cin >> Faculty[i]->groups[j].name;
            k = 0;
            do
            {
                system("cls");
                m = 0;
                cout << "| Faculty: " << Faculty[i]->name << " | Course: " << Faculty[i]->course << "| Group: " << Faculty[i]->groups[j].name << " |" << " No. " << k+1 << " |\n";
                cout << "Enter student's last name: ";
                cin >> Faculty[i]->groups[j].students[k].lastname;
                cout << "Enter student's exam marks (enter \"111\" to stop):\n ";
                int ch = 0;
                while (ch != 111 && m < 15) {
                    cin >> ch;
                    if (ch > -1 && ch < 101) {
                        Faculty[i]->groups[j].students[k].marks[m] = ch;
                        m++;
                    }
                }
                k++;
                cout << "\nDo you want to continue entering students?(y - yes, any other button to stop): ";
                key2 = _getch();
            } while (key2 == 'Y' || key2 == 'y' || key2 == 'н' || key2 == 'Н');
            j++;
            cout << "\nDo you want to continue entering groups?(y - yes, any other button to stop): ";
            key1 = _getch();
        } while (key1 == 'Y' || key1 == 'y' || key1 == 'н' || key1 == 'Н');
        i++;
        cout << "\nDo you want to continue entering faculties?(y - yes, any other button to stop): ";
        key = _getch();
    } while (key == 'Y' || key == 'y' || key == 'н' || key == 'Н');
    system("cls");
}

Но после этого я прочитал доп. задание (определить факультет и курс с макс. количеством отличников) и мне кажется что надо всё-таки сделать массив курсов, за которыми будут привязаны группы со студентами. Как это легче сделать? Я не уверен, что надо добавить еще одну вложенную структуру с курсами и в нее перенести структуру групп, ведь тогда придется делать тройной указатель на массив Faculty и все переделывать?

[min@y™]

Может, уже пора вытащить сиську изо рта и юзать ООП? Инкапсуляция, полиморфизм и всё такое?
Вместо структур - иерархия классов, с внутренними методами сохранения/загрузки, а?

А то я смотрю на твой исходник и мне курить хочется. Хотя бросил 4 года назад.

[alexboliam]

Сообщение от min@y™

Может, уже пора вытащить сиську изо рта и юзать ООП? Инкапсуляция, полиморфизм и всё такое?
Вместо структур - иерархия классов, с внутренними методами сохранения/загрузки, а?

А то я смотрю на твой исходник и мне курить хочется. Хотя бросил 4 года назад.

Сообщение от min@y™

Может, уже пора вытащить сиську изо рта и юзать ООП? Инкапсуляция, полиморфизм и всё такое?
Вместо структур - иерархия классов, с внутренними методами сохранения/загрузки, а?

А то я смотрю на твой исходник и мне курить хочется. Хотя бросил 4 года назад.

Была бы моя воля, то да. Но в универе решили что лучшей идеей будет изучать ООП в 3 семестре,а во втором структуры и работу с данными.

[min@y™]

Сообщение от alexboliam

Была бы моя воля, то да. Но в универе решили что лучшей идеей будет изучать ООП в 3 семестре,а во втором структуры и работу с данными.

[alexzk]

Сообщение от alexboliam

Была бы моя воля, то да. Но в универе решили что лучшей идеей будет изучать ООП в 3 семестре,а во втором структуры и работу с данными.

Ну оставьте им слово struct. В С++ там разница с class только в видимости по умолчанию. И сделайте, как правильно.

[alexboliam]

Сообщение от alexzk

Ну оставьте им слово struct. В С++ там разница с class только в видимости по умолчанию. И сделайте, как правильно.

В ООП я не сильно разбираюсь ещё, но вот за ночь сделал эту программу со структурами.

[alexzk]

Сообщение от alexboliam

В ООП я не сильно разбираюсь ещё, но вот за ночь сделал эту программу со структурами.

..ну там ..чепуха вобщем) с первых строк

faculty **Faculty = new faculty*[10];

Вы же понимаете, что это массив УКАЗАТЕЛЕЙ НА УКАЗАТЕЛИ? Там нет структур, их еще каждую нада снова сделать new в такой записи (которая тоже избыточна и нафиг нада, но раз начали...).

А где delete ? Причем опять же, сначала каждую из 10 delete лично, а потом весь блок в спец. записи:

delete [] Faculty; //так делается удаление массива, в нашем случае - массив указателей на указатели.


Попробуйте почитать std::array. Его специально внесли в стандарт, т.к. люди слишком часто чепуху пишут + к нему можно применить <algorithm> - все оттуда.