Создание многофайлового проекта

@dimazavt · Регистрация: 11.09.2017

Здравствуйте, я написал программу которая работает. Мне сказали разбить программу на функции и распределить их по файлам. Я сделал это, но выдаёт всякую хрень. Можете объяснить мне как это сделать.

C++

#define _CRT_SECURE_NO_WARNINGS// хрень чтобы компилятор не ругался на strcpy
#include <iostream>
#include <cstring>
#include <conio.h>
#include <cstdlib>
#include <windows.h>
#define M 15
#define N 7
using namespace std;
 
struct AEROFLOT {
    char kuda[M];
    int num;
    char type[M];
};
 
int main() {
    //  static int N = 3;
    system("chcp 1251");
    system("cls");
    AEROFLOT fly[N];
    int i;
    int j = 0;
    bool flg;
    for (i = 0; i < N; i++) {
 
        cout << "\nВведите номер рейса: ";
 
        while (1) {
            flg = true;
            fflush(stdin);
            cin >> fly[i].num;
            
            for (j = 0; j < i; j++) {
                if (fly[i].num==fly[j].num) {
                    flg = false;
                    cout << endl
                        << "|!| В системе уже имеется запись о самолёте" << endl
                        << "|!| (" << fly[i].num << ")" << endl
                        << "|!| Скорректируйте данные" << endl
                        << endl << "Введите номер рейса: ";
                    break;
                }
            }
            if (flg) {
                break;
            }
        }
        cout << "\nВведите тип самолёта "; cin >> fly[i].type;
        cout << "\nВведите пункт назначения "; cin >> fly[i].kuda;
    }
    char cur[M];
    for (i = 0; i < N - 1; i++)
        for (int j = i + 1; j < N; j++)
            if (strcmp(fly[i].kuda, fly[j].kuda) > 0)
            {
                strcpy(cur, fly[i].kuda);
                strcpy(fly[i].kuda, fly[j].kuda);
                strcpy(fly[j].kuda, cur);
            }
    cout << "\n ________________________________________________________\n";
    for (i = 0; i < N; i++) {
        cout << fly[i].kuda << "    " << fly[i].num << "    " << fly[i].type << endl;
 
    }
    cout << "\n ________________________________________________________\n";
    char temp[M];
    int count;
    while (strcmp(temp, "exit")) {
        cout << "\nВведите желаемый тип самолёта: "; cin >> temp;
        count = 0;
        for (i = 0; i < N; i++) {
            if (strcmp(fly[i].type, temp) == 0) {
                cout << endl << fly[i].kuda << "    " << fly[i].num << endl;
                count++;
            }
        }
        if (count == 0) cout << endl << " Нет таких рейсов\n";
    }
    _getch();
}

это код который сделал я. Если надо, могу скинуть как я пытался распределить это чудо.

@Raali · 05.12.2019, 12:29

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

Я сделал это, но выдаёт всякую хрень.

какую?
почему никто не пишет ошибку никогда

@dimazavt · 05.12.2019, 12:36 **[ТС]**

Raali, вот набор ошибок

@Джон Кофи · 05.12.2019, 12:36

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

Я сделал это

в твоем примере всё свалено в одну кучу в main() , как именно ты Это сделал? код, пожалуйста.

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

всякую хрень

текст ошибок, пожалуйста.

@Raali · 05.12.2019, 12:44

вот теперь по одной даблклик на каждую и смотреть какая строчка и файл

@dimazavt · 05.12.2019, 12:44 **[ТС]**

Джон Кофи,
основной файл с main

C++

#define _CRT_SECURE_NO_WARNINGS// хрень чтобы компилятор не ругался на strcpy
#include <iostream>
#include <cstring>
#include <conio.h>
#include <cstdlib>
#include <windows.h>
#include "mysor.h"
#include "vvod.cpp"
#include "vihod.cpp"
//#define M 15
 
using namespace std;
 
 
 
 
int main() {
    //  static int N = 3;
    system("chcp 1251");
    system("cls");
    AEROFLOT fly[N];
 
    vvod();
    return 0;
    char temp[M];
    int count;
    while (strcmp(temp, "exit")) {
        cout << "\nВведите желаемый тип самолёта: "; cin >> temp;
        count = 0;
        for (i = 0; i < N; i++) {
            if (strcmp(fly[i].type, temp) == 0) {
                cout << endl << fly[i].kuda << "    " << fly[i].num << endl;
                count++;
            }   vihod();
        }
        if (count == 0) cout << endl << " Нет таких рейсов\n";    vihod();
    }
    
    _getch();
}

файл ввода, vvod.cpp

C++

#include <iostream>
#include <cstring>
#include <conio.h>
#include <cstdlib>
#include <windows.h>
#include "mysor.h"
using namespace std;
int vvod() {
    system("chcp 1251");
    system("cls");
    AEROFLOT fly[N];
 
    for (i = 0; i < N; i++) {
 
        cout << "\nВведите номер рейса: ";
 
        while (true) {
            flg = true;
            fflush(stdin);
            cin >> fly[i].num;
 
            for (j = 0; j < i; j++) {
                if (fly[i].num == fly[j].num) {
                    flg = false;
                    cout << endl
                        << "|!| В системе уже имеется запись о самолёте" << endl
                        << "|!| (" << fly[i].num << ")" << endl
                        << "|!| Скорректируйте данные" << endl
                        << endl << "Введите номер рейса: ";
                    break;
                }
            }
            if (flg) {
                break;
            }
        }
        cout << "\nВведите тип самолёта "; cin >> fly[i].type;
        cout << "\nВведите пункт назначения "; cin >> fly[i].kuda;
    }
    char cur[M];
    for (i = 0; i < N - 1; i++)
        for (int j = i + 1; j < N; j++)
            if (strcmp(fly[i].kuda, fly[j].kuda) > 0)
            {
                swap(fly[i], fly[j]);
            }
    cout << "\n ________________________________________________________\n";
    for (i = 0; i < N; i++) {
        cout << fly[i].kuda << "    " << fly[i].num << "    " << fly[i].type << endl;
 
    }
    cout << "\n ________________________________________________________\n";
}

Файл выхода vihod.cpp

C++

#include <iostream>
#include <cstring>
#include <conio.h>
#include <cstdlib>
#include <windows.h>
#include "mysor.h"
using namespace std;
int vihod() {
    char vi;
    cout << "Хотите выйти из программы? da/net" << endl;
    cout << "->"; cin >> vi;
    if (vi == 'da') { exit(0); }
}

как я понимаю ту сделано не правильно.
файл со структурой mysor.h

C++

#define M 15
#define N 1
int i;
int j = 0;
bool flg;
struct AEROFLOT {
    char kuda[M];
    int num;
    char type[M];
};

@Raali · 05.12.2019, 12:47

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

файл со структурой mysor.h

глобальная переменная int i; для какой цели она создана?
почему она должна быть видна в обоих CPP, если не должна то зачем ее создавать глобально

@dimazavt · 05.12.2019, 12:50 **[ТС]**

Raali, для циклов. Я чувствую что лучше её как-нибудь объявлять перед каждой функцией?

@Raali · 05.12.2019, 12:53

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

для циклов.

для циклов самое простое делать так

C++

1	for(int i=0; i<10; ++i)

во вторых ошибка в том что нельзя одно и то же имя использовать в 2 единицах трансляции
допустим ты включил хедер в оба cpp - main1 и main2

C++

1 2	//main1.cpp int i;

C++

1 2	//main2.cpp int i;

будет ошибка линковки

@Джон Кофи · 05.12.2019, 12:58

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

C++

1	int vihod()

вИход

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

C++

1	for (j = 0; j < i; j++)

объявляйте такие counter-ы при объявлении цикла , пожалуйста.

C++

1	for(int i(0); i < size; ++i) {}

@Raali 855 / 445 / 111 Регистрация: 06.07.2013 Сообщений: 1,483
	05.12.2019, 12:29	2
	Сообщение от dimazavt Я сделал это, но выдаёт всякую хрень. какую? почему никто не пишет ошибку никогда 0

@dimazavt 3 / 3 / 0 Регистрация: 11.09.2017 Сообщений: 73
	05.12.2019, 12:36 [ТС]	3
	Raali, вот набор ошибок 0 Миниатюры

@Джон Кофи 228 / 49 / 10 Регистрация: 05.04.2018 Сообщений: 858 Записей в блоге: 1
	05.12.2019, 12:36	4
	Сообщение от dimazavt Я сделал это в твоем примере всё свалено в одну кучу в main() , как именно ты Это сделал? код, пожалуйста. Сообщение от dimazavt всякую хрень текст ошибок, пожалуйста. 0

@Raali 855 / 445 / 111 Регистрация: 06.07.2013 Сообщений: 1,483
	05.12.2019, 12:44	5
	вот теперь по одной даблклик на каждую и смотреть какая строчка и файл 0

@Raali 855 / 445 / 111 Регистрация: 06.07.2013 Сообщений: 1,483
	05.12.2019, 12:47	7
	Сообщение от dimazavt файл со структурой mysor.h глобальная переменная int i; для какой цели она создана? почему она должна быть видна в обоих CPP, если не должна то зачем ее создавать глобально 0

@dimazavt 3 / 3 / 0 Регистрация: 11.09.2017 Сообщений: 73
	05.12.2019, 12:50 [ТС]	8
	Raali, для циклов. Я чувствую что лучше её как-нибудь объявлять перед каждой функцией? 0