Правильно ли построен класс

@maxim43k · Регистрация: 04.09.2011

Author24 — интернет-сервис помощи студентам

STRUCT.cpp

C++

#include "STRUCT.h"
 
 STRUCT::STRUCT()
{
 clrscr();
 printf(" Введите количество спортсменов: ");
 scanf("%d", &count); //Ввод количества участников
 min = init_athletes(all, count);
 printf("\n Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s\nВозраст:"  
 "%d\n\Bремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
 all[min].time.min, all[min].time.sec);
 getch();
}
 
 int STRUCT::init_athletes(athlete*& all, int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {printf("%d. ", i + 1); //Порядковый номер записи спортсмена
    scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
      &all[i].time.min, &all[i].time.sec);
    if (i == 0) min = i;
    else if (all[i].age < all[min].age) min = i;
   }
 return min;
}

STRUCT.h

C++

#include <stdio.h>
#include <iostream.h>
#include <conio.h>
 
class STRUCT
{typedef struct 
   {
    int min; //минуты
    int sec; //секунды
   }time;
 
 typedef struct
   {
    char name[20]; //имя
    char f_name[20]; //фамилия
    int age; //возраст
    time time; //результат
   }athlete;
   
 int count = 0;
 int min = 0;
 athlete* all = NULL;
 
 public:
    STRUCT();
    int init_athletes(athlete*& all, int count)
}

@maxim43k · 09.09.2011, 11:39 **[ТС]**

*.cpp

C++

#include "struct.h"
#include <iostream.h>
#include <stdio.h>
#include <conio.h>
 
 STRUCT::STRUCT() 
{
  all = NULL;
  int count = 0;
  int min = 0;
  clrscr();
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  min = init_athletes(all, count);
  
  printf("\n Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s\nВозраст:"  
  "%d\n\Bремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
  
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                             &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return min;
}

*.h

C++

class STRUCT 
{
     struct time_ms 
    {
        int min; //минуты
        int sec; //секунды
    };
 
     struct athlete 
    {
        char name[20]; //имя
        char f_name[20]; //фамилия
        int age; //возраст
        STRUCT::time_ms time_ms; //результат
    };
    
    athlete *all;  
    
public:
    STRUCT();
    int init_athletes(int count);
};

@maxim43k · 09.09.2011, 20:57 **[ТС]**

Из-за чего ругается? Я английский не понимаю

Добавлено через 16 минут

C++

#include "struct.h"
#include <iostream.h>
#include <stdio.h>
#include <conio.h>
 
 STRUCT::STRUCT() 
{
  all = NULL;
  int count = 0;
  int min = 0;
  clrscr();
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  init_athletes(count);
  printf("\n Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s\nВозраст:"
  "%d\n\Bремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
 
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                 &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return 0;
}

--------------------------------------------------------------------------------

C++

class STRUCT 
{
     struct time_ms 
    {
        int min; //минуты
        int sec; //секунды
    };
 
     struct athlete 
    {
        char name[20]; //имя
        char f_name[20]; //фамилия
        int age; //возраст
        STRUCT::time_ms time_ms; //результат
    };
    
    athlete *all;  
    
public:
    STRUCT();
    int STRUCT::init_athletes(int count);
};

Добавлено через 8 часов 48 минут
Укажите на ошибки, пожалуйста

@kravam · 09.09.2011, 21:22

Одним файлом всё, ни на что не ругается

C++

#include <stdio.h>
#include <iostream>
#include <conio.h>
using namespace std;
 
class STRUCT 
{
     struct time_ms 
        {
        int min; //минуты
        int sec; //секунды
    };
 
     struct athlete 
        {
        char name[20]; //имя
        char f_name[20]; //фамилия
        int age; //возраст
        STRUCT::time_ms time_ms; //результат
    };
        
    athlete *all;  
        
public:
    STRUCT();
    int STRUCT::init_athletes(int count);
};
 
int main() {
 getchar();
 return 0;
}
 
 
 
 STRUCT::STRUCT() 
{
  all = NULL;
  int count = 0;
  int min = 0;
//  clrscr();
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  init_athletes(count);
//  printf("Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s\nВозраст: %d\n\Bремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
  printf("%d\nФамилия: %s\nИмя: %s\nВозраст: %d\nBремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
 
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                                 &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return 0;
}

@maxim43k · 09.09.2011, 21:32 **[ТС]**

В том то и проблема, что мне нужно всё сделать в двух файлах *.cpp и *.h Вот класс с двумя файлами у меня никак не хочет работать(((

@kravam · 09.09.2011, 21:42

..А вообще фигню ты делаешь. Я так понял, прога должна выводить данные самого юного участника? Значит, кропай некое подобие сортировки и сортируй участников по полю athlete.age, а то у тебя самым юным выводится первый введённый перец , а он может старпень какой-нибудь...

Добавлено через 5 минут

C++

#include "STRUCT.h"
int main() {
 SetConsoleCP(1251);
 SetConsoleOutputCP(1251);
 getchar();
 STRUCT str;
 return 0;
}

C++

#include <windows.h>
#include <stdio.h>
#include <iostream>
#include <conio.h>
using namespace std;
 
class STRUCT 
{
     struct time_ms 
        {
        int min; //минуты
        int sec; //секунды
    };
 
     struct athlete 
        {
        char name[20]; //имя
        char f_name[20]; //фамилия
        int age; //возраст
        STRUCT::time_ms time_ms; //результат
    };
        
    athlete *all;  
        
public:
    STRUCT();
    int STRUCT::init_athletes(int count);
};
 
 
 
 STRUCT::STRUCT() 
{
  all = NULL;
  int count = 0;
  int min = 0;
//  clrscr();
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  init_athletes(count);
  printf("Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s\nВозраст: %d\nBремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
 
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                                 &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return 0;
}

кидай файлы в одну директорию, озаглавливай второй STRUCT.h, первый как хочешь и компиль и всё будет круто

@maxim43k · 09.09.2011, 21:57 **[ТС]**

Фишка в том, что у меня ВС++ 3.1. Ничего опять не компилится, не работает, исправлял под себя, а всё равно не хочет запускаться

@kravam · 09.09.2011, 22:08

А ошибки какие?

@maxim43k · 09.09.2011, 22:19 **[ТС]**

Я пытаюсь скомпилировать такой код:

STRUCT.cpp

C++

#include <stdio.h>
#include <iostream.h>
#include <conio.h>
#include "STRUCT.h"
 
 STRUCT::STRUCT()
{
  all = NULL;
  int count = 0;
  int min = 0;
  clrscr();
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  init_athletes(count);
  printf("Самый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: %s" 
  "\nВозраст: %d\nBремя: %d min. %d sec.", min + 1, all[min].name, all[min].f_name, all[min].age,
 
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                 &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return 0;
}

STRUCT.h

C++

class STRUCT
{
 struct time_ms
   {
    int min; //минуты
    int sec; //секунды
   };
 
 struct athlete
   {
    char name[20]; //имя
    char f_name[20]; //фамилия
    int age; //возраст
    STRUCT::time_ms time_ms; //результат
   };
 
 athlete *all;
 
 public:
    STRUCT();
    int STRUCT::init_athletes(int count);
};

В итоге:

@kravam · 09.09.2011, 22:24

Слушай, насколько я знаю в каждой проге должна присутствовать главная функция main (или Winmain), короче, чё-то такое. Если есть возражения по этому пункту, давай их сюда.

Ты же безо всякого предупреждения режешь на корню мой код, убираешь оттуда main и говоришь, что оно не компилится. А оно вроде и не должно компилиться- main-то отсутствует!

Компиль МОЙ код и будем разбираться.

@maxim43k · 09.09.2011, 22:34 **[ТС]**

Твой код как есть

@kravam · 09.09.2011, 22:42

Так не пойдёт. Я не вижу текста ошибки ПОЛНОСТЬЮ. И никто не увидит. Перепиши на бумажку и дай сюда.

@maxim43k · 09.09.2011, 22:57 **[ТС]**

Вообщем, создал я файлы заново, в итоге 3 ошибки:

@maxim43k · 10.09.2011, 00:02 **[ТС]**

Что-то самого молодого спортсмена не находит, хотя без классов всё работало. Наверное, в адресах что-то.

Добавлено через 9 минут
Делаю так:

*.cpp

C++

#include "STRUCT.h"
 
int main()
{
 getchar();
 STRUCT str;
 return 0;
}

*.h

C++

#include <windows.h>
#include <stdio.h>
#include <iostream.h>
#include <conio.h>
 
class STRUCT 
{
 struct time_ms 
   {
    int min; //минуты
    int sec; //секунды
   };
 
 struct athlete 
   {
    char name[20]; //имя
    char f_name[20]; //фамилия
    int age; //возраст
    STRUCT::time_ms time_ms; //результат
   };
        
 athlete *all;  
        
 public:
      STRUCT();
      int STRUCT::init_athletes(int count);
};
 
  STRUCT::STRUCT() 
{
  all = NULL;
  int count = 0;
  int min = 0;
  printf(" Введите количество спортсменов: ");
  scanf("%d", &count); //Ввод количества участников
  init_athletes(count);
  printf("\nСамый юный участник:\nМесто в реестре: %d\nФамилия: %s\nИмя: "
  "%s\nВозраст: %d\nBремя: %d min. %d sec.", min, all[min].name, all[min].f_name, all[min].age,
  all[min].time_ms.min, all[min].time_ms.sec);
  getch();
}
 
int STRUCT::init_athletes(int count)
{int i, min = 0;
 all = new athlete[count]; //массив данных типа структура
 printf(" Введите данные для каждого спортсмена:\r\n");
 for(i = 0; i < count; ++i)
   {
     printf("%d. ", i + 1); //Порядковый номер записи спортсмена
     scanf("%s %s %d %d %d", &all[i].name, &all[i].f_name, &all[i].age,
                                 &all[i].time_ms.min, &all[i].time_ms.sec);
     if (i == 0) min = i;
     else if (all[i].age < all[min].age) min = i;
   }
 return 0;
}

Но тут что-то не то с передачей min из функции. Результат некорректный. В чём причина?

@kravam · 10.09.2011, 00:20

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

@maxim43k · 10.09.2011, 00:27 **[ТС]**

А если писать ещё с файлами... Вот код, как в данном случае всё это сделать через класс? Причём тут такая же загвоздка, результат неправильно отображает, всегда выводит 1 спортсмена в списке((((

C++

#include <iostream.h>
#include <fstream.h>
#include <string.h>
#include <stdlib.h>
#include <iomanip.h>
#include <conio.h>
 
struct time_s
{
   int imin;
   int sec;
};
 
 struct athlete
{
   char name [20];
   char f_name [20];
   int age;
   time_s time;
};
 
void process(athlete* &all, int count, char *f1_name, char *f2_name);
 
 void main()
{
   athlete *all = NULL;
   char f1_name[20], f2_name[20];
   int count;
   count = 0;
   clrscr();
   cout << "Введите имя файла F1: ";
   cin >> f1_name;
   cout << "Введите имя файла F2 (бинарный): ";
   cin >> f2_name;
   cout << "Введите количество спортсменов: ";
   cin >> count;
   process(all, count, f1_name, f2_name);
   getch();
}
 
void process(athlete* &all, int count, char *f1_name, char *f2_name)
{
   int i, imin = 0;
   all = new athlete[count];
   fstream f1(f1_name, ios::in | ios::out);
   if (f1.fail())
     {
       cout << "F1 Ошибка!" << f1.rdstate() << endl;
     }
   fstream f2(f2_name, ios::in | ios::binary | ios::out);
   if (f2.fail())
     {
       cout << "F2 Ошибка!" << f2.rdstate() << endl;
     }
   cout << "Введите данные о спортсменах\n"
         "(имя, фамилия, возраст, результат в мин. и сек. через пробел):\n";
   for (i = 0; i < count; ++i)
     {
        cout << i + 1 << ". ";
        cin >> all[i].name;
        cin >> all[i].f_name;
        cin >> all[i].age;
        cin >> all[i].time.imin;
        cin >> all[i].time.sec;
     }
  
   f1 << "\nПроверка считывания из F2: \n";
   f2.seekg(0,ios::beg);
   for(i=0; i<count; i++)
   f2.write((char *)&all[i], sizeof(athlete));
   f2.seekp(0, ios::beg);
   for(i=0;i<count;i++)
   {
       f2.read((char *)&all[i], sizeof(athlete));
       f1.write((char *)&all[i], sizeof(athlete));
   }
   for (int k = 0; k < count; k++)
   if (k == 0)  imin = k;
   else if (all[k].age < all[imin].age)
      imin = k;
   cout << "\nСамый юный участник:\n";
   cout << "Порядковый номер в реестре: " << imin 
        << "\nИмя: " << all[imin - 1].name 
        << "\nФамилия: " << all[imin - 1].f_name
        << "\nВозраст: " << all[imin - 1].age 
        << "\nРезультат: " << all[imin - 1].time.imin 
        << " мин. " << all[imin - 1].time.sec << " сек. ";
 
   ofstream file("F2.bin", ios::out | ios::binary);
   file.write((char *)&all[imin], sizeof(athlete)); 
   file.close();
   f2.close();
   f1.close();
}

@kravam · 10.09.2011, 00:54

Нужны данные, при которых выводится некорректный результат

@maxim43k · 10.09.2011, 00:57 **[ТС]**

Тут не то чтобы обрабатывалось неправильно. Здесь сравнения похоже не происходит, потому что всегда выводит первого спортсмена из списка, а не сравнивает по возрасту.

@kravam · 10.09.2011, 01:05

Я понял. Дай данные, при которых сие происходит. Понимаешь, я могу тестировать твою программу. Но! Но зачем мне терять время на поиск "плохих" данных, если ты их знаешь? И да, я не уверен, что твоя программа всегда работает плохо. Опыт, знаешь ли. Поэтому не буду терять время на поиск "плохих" данных, дай их сюда.

@maxim43k · 10.09.2011, 01:15 **[ТС]**

Должны были вывестись данные 3 спортсмена

А здесь второго спортсмена

@kravam · 10.09.2011, 01:41

А потому так происходит, что переменная min у тебя в каждой функции СВОЯ. То есть в функции
STRUCT::STRUCT() переменная min объявлена и инициализирована нулём, поэтому и будет результаты нулевого спортсмена выводить. И эта функция знать не знает, что где-то там существует переменная тоже с именем min, которая меняется. Поэтому min надо делать одну на обе функции, глобальную или член класса, раз с классами работаешь.

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	09.09.2011, 21:57 [ТС]	26
	Фишка в том, что у меня ВС++ 3.1. Ничего опять не компилится, не работает, исправлял под себя, а всё равно не хочет запускаться 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	09.09.2011, 22:24	29
	Слушай, насколько я знаю в каждой проге должна присутствовать главная функция main (или Winmain), короче, чё-то такое. Если есть возражения по этому пункту, давай их сюда. Ты же безо всякого предупреждения режешь на корню мой код, убираешь оттуда main и говоришь, что оно не компилится. А оно вроде и не должно компилиться- main-то отсутствует! Компиль МОЙ код и будем разбираться. 0

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	09.09.2011, 22:34 [ТС]	30
	Твой код как есть 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	09.09.2011, 22:42	31
	Так не пойдёт. Я не вижу текста ошибки ПОЛНОСТЬЮ. И никто не увидит. Перепиши на бумажку и дай сюда. 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	10.09.2011, 00:54	36
	Нужны данные, при которых выводится некорректный результат 0

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	09.09.2011, 21:32 [ТС]	24
	В том то и проблема, что мне нужно всё сделать в двух файлах .cpp и .h Вот класс с двумя файлами у меня никак не хочет работать((( 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	09.09.2011, 22:08	27
	А ошибки какие? 0

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	09.09.2011, 22:57 [ТС]	32
	Вообщем, создал я файлы заново, в итоге 3 ошибки: 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	10.09.2011, 00:20	34
	Потом гляну, а те функции- они для русского языка, можешь их убрать и делать русский язык по-своему. 0

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	10.09.2011, 00:57 [ТС]	37
	Тут не то чтобы обрабатывалось неправильно. Здесь сравнения похоже не происходит, потому что всегда выводит первого спортсмена из списка, а не сравнивает по возрасту. 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	10.09.2011, 01:05	38
	Я понял. Дай данные, при которых сие происходит. Понимаешь, я могу тестировать твою программу. Но! Но зачем мне терять время на поиск "плохих" данных, если ты их знаешь? И да, я не уверен, что твоя программа всегда работает плохо. Опыт, знаешь ли. Поэтому не буду терять время на поиск "плохих" данных, дай их сюда. 0

@maxim43k 0 / 0 / 0 Регистрация: 04.09.2011 Сообщений: 106
	10.09.2011, 01:15 [ТС]	39
	Должны были вывестись данные 3 спортсмена А здесь второго спортсмена 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	10.09.2011, 01:41	40
	А потому так происходит, что переменная min у тебя в каждой функции СВОЯ. То есть в функции STRUCT::STRUCT() переменная min объявлена и инициализирована нулём, поэтому и будет результаты нулевого спортсмена выводить. И эта функция знать не знает, что где-то там существует переменная тоже с именем min, которая меняется. Поэтому min надо делать одну на обе функции, глобальную или член класса, раз с классами работаешь. 0