Разделить проект на несколько библиотек

@Pantheonptz · Регистрация: 05.05.2017

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

Есть проект эмуляция работы бд. Пытаюсь функции main.cpp вынести на menu.h и menu.cpp
В menu.h записываю все определения

C++

#ifndef MENU_H_INCLUDED
#define MENU_H_INCLUDED
#include <Windows.h>
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
typedef struct Elem Elem;
 
struct Elem{
    char* proiz;
    char* model;
    char* apert;
    int iso;
    char* razresh;
    char* format;
    struct Elem *next;
};
int CountElem = 0;
int log = 0;
Elem *head = NULL;
const int S_FIELD = 50;
 
void Click(int select);
void showFields(int select);
int getPosField();
void showMenu(int select);
void SetColor(int text, int background);
enum ConsoleColor{
    Black = 0, Blue = 1, Green = 2, Cyan = 3,
    Red = 4, Magenta = 5, Brown = 6, LightGray = 7,
    DarkGray = 8, LightBlue = 9, LightGreen = 10,
    LightCyan = 11, LightRed = 12, LightMagenta = 13,
    Yellow = 14, White = 15};
#endif // MENU_H_INCLUDED

Прописываю в файлах main.cpp и menu.cpp #include "menu.h" и получаю multiple defenition CountElem log head
Не понимаю куда тогда положить эти объявления?

ValeryS · 11.06.2018, 18:25

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

main.cpp

так Си или все таки С++

@Pantheonptz · 11.06.2018, 18:30 **[ТС]**

Си. cpp это описка.
main.c menu.h menu.c

ValeryS · 11.06.2018, 18:59

теперь смотри

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

C

1 2	int CountElem = 0; int log = 0;

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

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

multiple defenition

прочитай про ключевое слово extern

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

Си.

тогда структуру лучше объявить так

C

typedef struct _Elem{
    char* proiz;
    char* model;
    char* apert;
    int iso;
    char* razresh;
    char* format;
    _Elem *next;
}Elem;

@Pantheonptz · 11.06.2018, 19:15 **[ТС]**

в main.c

C

#include "menu.h"
 
int CountElem = 0;
int log = 0;
Elem *head = NULL;

Из menu.h объявления убрал
В menu.c записал объявления

C

1
2
3

extern int CountElem = 0;
extern int log = 0;
extern Elem *head = NULL;

Все равно пишет multiple defenition
причем ошибка показывает на строку кода в stdio.h в функции printf

ValeryS · 11.06.2018, 19:22

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

C

1
2
3

extern int CountElem = 0;
extern int log = 0;
extern Elem *head = NULL;

не надо инициализировать

C

1
2
3

extern int CountElem;
extern int log;
extern Elem *head;

Добавлено через 3 минуты
и поменяй имя переменной log ибо это одно из служебных слов (функция которая берет логарифм) возможны косяки
назови например MyLog

@Pantheonptz · 11.06.2018, 19:22 **[ТС]**

теперь выдает ошибку ld returned 1 exit status

ValeryS · 11.06.2018, 19:24

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

ld returned 1 exit status

приведи код который вызывает ошибку

@Pantheonptz · 11.06.2018, 19:29 **[ТС]**

ld возвратил статус 1. Честно говоря я не знаю что такое ld. В инете пишут может программа осталась работать и посмотреть в диспетчере или есть кириллица в пути к файлам. Но ни того не другого не присутствует. среда codeblocks.

@easybudda · 11.06.2018, 19:32

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

Честно говоря я не знаю что такое ld.

Линковщик. Если не отработал, скорее всего не нашёл что-то. Проверяйте имена и сигнатуры функций там, где они объявлены, описаны и вызываются.

ValeryS · 11.06.2018, 19:39

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

среда codeblocks.

ничем не могу помочь

никогда с ней не работал

@Pantheonptz · 11.06.2018, 19:40 **[ТС]**

Дак ведь я всего лишь откопировал все объявления в header и затем в menu.c . Вы имеете ввиду могут быть ошибки в названиях функций в хедере и в файле?

ValeryS · 11.06.2018, 19:45

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

Вы имеете ввиду могут быть ошибки в названиях функций

функция может быть объявлена но не реализована

@Pantheonptz · 11.06.2018, 19:51 **[ТС]**

Дело в том что я сначала реализовал все функции в main.c откомпилировал и проверил, а потом часть из них перенес в menu.h
menu.c

C

#include "menu.h"
 
extern int CountElem;
extern int log;
extern Elem *head;
 
// вывод меню
void showMenu(int select){
    char menu[][80] = {"******************************************************************************\n",
                       "*                                                                            *\n",
                       "*  Добавить новый фотоаппарат                                                *\n",
                       "*  Редактировать                                                             *\n",
                       "*  Удалить                                                                   *\n",
                       "*  Сортировка                                                                *\n",
                       "*  Вывод на экран                                                            *\n",
                       "*  Загрузка из текстового файла                                              *\n",
                       "*  Загрузка из зашифрованного файла                                          *\n",
                       "*  Сохранение в текстовый файл                                               *\n",
                       "*  Сохранение в зашифрованный файл                                           *\n",
                       "*  Поиск                                                                     *\n",
                       "*  Нажмите Esc - для выхода из программы                                     *\n",
                       "*                                                                            *\n",
                       "******************************************************************************"};
    system("CLS");
    for(int i = 0; i < 15; i++){
        SetColor(DarkGray, Black);
        if(i == (select + 1))
            SetColor(Red, Black);
        printf("%s", menu[i]);
    }
SetColor(White, Black);
}
 
void SetColor(int text, int background){
    HANDLE hStdOut = GetStdHandle(STD_OUTPUT_HANDLE);
    SetConsoleTextAttribute(hStdOut, (WORD)((background << 4)|text));
}
 
 
void showFields(int select){
    system("CLS");
    for(int i = 0; i < CountElem; i++){
     SetColor(White, Black);
     if(i == select)
            SetColor(Red, White);
     Elem* temp = getElem(i);
     printf("Номер %i - %s %s\n", i, temp->proiz, temp->model);
    }
    SetColor(White, Black);
}
 
int getPosField(){
    int menuItem = 0;
 
    showFields(menuItem);
        while(1){
        if(GetAsyncKeyState(VK_UP)){
            keybd_event(VK_UP, 0, KEYEVENTF_KEYUP, 0);
            if(menuItem - 1 < 0)
                menuItem = CountElem - 1;
            else
                menuItem--;
            showFields(menuItem);
        }
        if(GetAsyncKeyState(VK_DOWN)){
            keybd_event(VK_DOWN, 0, KEYEVENTF_KEYUP, 0);
            if(menuItem + 1 >= CountElem)
                menuItem = 0;
            else
                menuItem++;
            showFields(menuItem);
        }
        if(GetAsyncKeyState(VK_RETURN)){
            keybd_event(VK_RETURN, 0, KEYEVENTF_KEYUP, 0);
            return menuItem;
        }
        if(GetAsyncKeyState(VK_ESCAPE)){
            keybd_event(VK_ESCAPE, 0, KEYEVENTF_KEYUP, 0);
            return -1;
        }
    }
}
 
void Click(int select){
    char proiz[S_FIELD];
    char model[S_FIELD];
    char apert[S_FIELD];
    int iso;
    char razresh[S_FIELD];
    char format[S_FIELD];
    int pos;
    int posf;
    char filename[S_FIELD];
    char pass[64];
    char find_str[50];
 
    fflush(stdin);
    switch(select){
    case 1:
        if(elementInput(proiz, model, apert, &iso, razresh, format) == -1){
            printf("ISO неправильно!");
            system("pause");
            return;
        }
        elementAdd(proiz, model, apert, iso, razresh, format);
        if(log == 1)
            toLog("Ввод с консоли\n");
    break;
    case 2:
        pos = getPosField();
        if( pos == -1){
            printf("Возврат в главное меню");
            system("pause");
            return;
        }
        printf("Введите изменяемое поле\n 1 - производитель 2 - модель 3 - аппертура 4 - ISO 5 - разрешение 6 - формат");
        do{
            posf = getch();
        } while ( posf < 49 || posf > 54);
 
        elementChange(pos, posf - 48);
        printf("Данные изменены!\n");
        if(log == 1)
            toLog("Произошло изменения данных\n");
        system("pause");
    break;
    case 3:
        pos = getPosField();
        if( pos == -1){
            printf("Возврат в главное меню");
            system("pause");
            return;
        }
 
        elementDel(pos);
        printf("Запись удалена!\n");
        if(log == 1)
            toLog("Произошло удаление данных\n");
        system("pause");
    break;
    case 4:
        printf("Введите поле по которому сортировать\n 1 - производитель 2 - модель 3 - аппертура 4 - ISO 5 - разрешение 6 - формат");
        do{
            pos = getch();
        } while ( pos < 49 || pos > 54);
        sort(pos - 48);
        if(log == 1)
            toLog("Сортировка данных\n");
    break;
    case 5:
         outToConsole();
         if(log == 1)
            toLog("Вывод данных в консоль\n");
         system("pause");
    break;
    case 6:
        printf("Введите имя файла - ");
        fflush(stdin);
        clearbuff();
        scanf("%s", filename);
        fromFileTxt(filename);
        printf("Данные загружены!\n");
        if(log == 1)
            toLog("Загрузка данных из текстового файла\n");
        system("pause");
    break;
    case 7:
        fflush(stdin);
        clearbuff();
        printf("Введите имя файла - ");
        scanf("%s", filename);
        printf("\nВведите пароль - ");
        scanf("%s", pass);
        fromFileCrypt(pass, filename);
        printf("Данные загружены!\n");
        if(log == 1)
            toLog("Загрузка данных из зашифрованного файла\n");
        system("pause");
    break;
    case 8:
        printf("Введите имя файла - ");
        fflush(stdin);
        clearbuff();
        //fgets(filename, S_FIELD, stdin);
        scanf("%s", filename);
        toFileTxt(filename);
        printf("Данные загружены в %s\n", filename);
        if(log == 1)
            toLog("Сохранение данных в текстовый файл\n");
        system("pause");
    break;
    case 9:
        printf("Введите имя файла - ");
        fflush(stdin);
        clearbuff();
        scanf("%s", filename);
        printf("\nВведите пароль - ");
        scanf("%s", pass);
        toFileCrypt(pass, filename);
        printf("Данные загружены в %s\n", filename);
        if(log == 1)
            toLog("Сохранение данных в зашифрованный файл\n");
        system("pause");
    break;
    case 10:
        printf("Поисковая строка - \n");
        scanf("%s", find_str);
        poisk(find_str);
        system("pause");
        break;
    }
}

menu.h

C

#ifndef MENU_H_INCLUDED
#define MENU_H_INCLUDED
#include <Windows.h>
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
typedef struct Elem Elem;
 
struct Elem{
    char* proiz;
    char* model;
    char* apert;
    int iso;
    char* razresh;
    char* format;
    struct Elem *next;
};
 
const int S_FIELD = 50;
 
void Click(int select);
void showFields(int select);
int getPosField();
void showMenu(int select);
void SetColor(int text, int background);
enum ConsoleColor{
    Black = 0, Blue = 1, Green = 2, Cyan = 3,
    Red = 4, Magenta = 5, Brown = 6, LightGray = 7,
    DarkGray = 8, LightBlue = 9, LightGreen = 10,
    LightCyan = 11, LightRed = 12, LightMagenta = 13,
    Yellow = 14, White = 15};
#endif // MENU_H_INCLUDED

ValeryS · 11.06.2018, 20:08

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

const int S_FIELD = 50;

вот это тоже вынеси из заголовочного файла
а лучше напиши

C

1	#define S_FIELD 50

const в Си работает не совсем так как в плюсах

@Pantheonptz · 11.06.2018, 20:20 **[ТС]**

сделал как вы написали и все заработало. видимо на это и ругался линковщик.

@Pantheonptz 1 / 1 / 4 Регистрация: 05.05.2017 Сообщений: 521
	11.06.2018, 20:20 [ТС]	16
	сделал как вы написали и все заработало. видимо на это и ругался линковщик. 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,511
	11.06.2018, 18:25	2
	Сообщение от Pantheonptz main.cpp так Си или все таки С++ 0

@Pantheonptz 1 / 1 / 4 Регистрация: 05.05.2017 Сообщений: 521
	11.06.2018, 18:30 [ТС]	3
	Си. cpp это описка. main.c menu.h menu.c 0

@Pantheonptz 1 / 1 / 4 Регистрация: 05.05.2017 Сообщений: 521
	11.06.2018, 19:22 [ТС]	7
	теперь выдает ошибку ld returned 1 exit status 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,511
	11.06.2018, 19:24	8
	Сообщение от Pantheonptz ld returned 1 exit status приведи код который вызывает ошибку 0

@Pantheonptz 1 / 1 / 4 Регистрация: 05.05.2017 Сообщений: 521
	11.06.2018, 19:29 [ТС]	9
	ld возвратил статус 1. Честно говоря я не знаю что такое ld. В инете пишут может программа осталась работать и посмотреть в диспетчере или есть кириллица в пути к файлам. Но ни того не другого не присутствует. среда codeblocks. 0

@easybudda Модератор 12453 / 7478 / 1752 Регистрация: 25.07.2009 Сообщений: 13,748
	11.06.2018, 19:32	10
	Сообщение от Pantheonptz Честно говоря я не знаю что такое ld. Линковщик. Если не отработал, скорее всего не нашёл что-то. Проверяйте имена и сигнатуры функций там, где они объявлены, описаны и вызываются. 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,511
	11.06.2018, 19:39	11
	Сообщение от Pantheonptz среда codeblocks. ничем не могу помочь никогда с ней не работал 0

@Pantheonptz 1 / 1 / 4 Регистрация: 05.05.2017 Сообщений: 521
	11.06.2018, 19:40 [ТС]	12
	Дак ведь я всего лишь откопировал все объявления в header и затем в menu.c . Вы имеете ввиду могут быть ошибки в названиях функций в хедере и в файле? 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,511
	11.06.2018, 19:45	13
	Сообщение от Pantheonptz Вы имеете ввиду могут быть ошибки в названиях функций функция может быть объявлена но не реализована 0