Ошибка: multiple definition of ...

[andrey_snegovik]

Добрый день! Не получается скомпилировать программу, хотя вроде все переменные и функции объявлены правильно. Подскажите пожалуйста!

Программа состоит из четырёх файлов. Всё основное выполнение происходит в main.cpp, остальное - модули.

primer.cpp

#include <iostream>
using namespace std;

void Move_Disks (int nn, char sourse, char dest, char tmp){
if (nn==1) {
    cout <<"Perestavit disk 1 so stolbika "<< sourse <<" na stolbik " << dest <<endl;
}
else {
    Move_Disks (nn-1, sourse, tmp, dest);
    cout <<"Perestavit disk "<< nn <<"so stolbika " <<sourse <<" na stolbik " <<dest<<endl;
    Move_Disks (nn-1, tmp, dest, sourse);
}
}

procedure.cpp

#include <iostream>
using namespace std;

#define N 3
#define M N

void input_array (int A[N][M]) {
for (int i = 0; i < N; i++)
for (int j = 0; j < M; j++) {
cout << "Enter [" << i + 1 << ";" << j + 1 << "]:" << endl;
cin >> A[i][j];
}
}

void print_array (int A[N][M]) {
for (int i = 0; i < N; i++) {
for (int j = 0; j < M; j++) {
cout << A[i][j] << " ";
}
cout << endl;
}
}

// Процедура, обрабатывающая одномерный массив
int proc (int T[]) {
int sum = 0;
for (int i = 0; i < N; i++) {
sum += T[i];
}
return sum;
}

int proc1 (int T1[]) {
int sum1 = 0;
for (int i = 0 ; i < N ; i++) {
sum1 += T1[i];
}
return sum1;}

main.cpp

#include <iostream>
#include "procedure.cpp"
#include "primer.cpp"

using namespace std;

int main () {
//Выполнение второй программы
cout <<endl;
cout << "*******************"<<endl;
cout <<  "Программа с процедурой" <<endl;
cout << "*******************"<<endl;

cout << "Fill array" << endl;
int B[N][M];
// Заполняем массив
input_array(B);
// Выводим массив на экран
print_array(B);

// Обрабатываем двумерный массив
// Смотрим сначала главную диагональ "вырезаем" ее из двумерного массива

int GDIAG[N] = {0};
for (int i = 0; i < N ; i++)
{
GDIAG[i] = B[i][i];
}
int gSum = proc(GDIAG); // Сумма элементов на главной диагонали
// Смотрим побочную диагональ
int PDIAG[N] = {0};
for (int i = M - 1 ; i >= 0 ; i--)
{
PDIAG[i] = B[i][i];
}
int pSum = proc1(PDIAG); // Сумма элементов на побочной диагонали

cout << "Glavnaya diagonal : " << gSum << " ; Pobochnaya diagonal: " << pSum << endl;

if (gSum > pSum)
{
cout << "Glavnaya diagonal bol'she! " << endl;

}
else
if (gSum == pSum)
{
cout << "Odinakovo!!" << endl;
}
else {
cout << "Pobochnaya bol'she!!" << endl;
}
    //Конец выполнения второй программы

    //Выполнение второй программы
cout << "*******************************"<<endl;
cout <<  "Программа из методички" <<endl;
cout << "*******************************"<<endl;
int P;
cout << "Vvedite chislo disks:";
cin>> P;
Move_Disks (P, 'A', 'C','B');
    //Конец выполнения второй программы

}

Выдаёт 4 ошибки:

In function `Z10Move_Disksiccc'
multiple definition of `Move_Disks(int, char, char, char)'
first defined here
collect2.exe:-1: ошибка: error: ld returned 1 exit status

Не понимаю, в чем проблема? Подскажите пожалуйста!

[casekey]

во-первых, раскидай декларацию функций в заголовочные файлы и подключай их, а не напрямую cpp - это неправильно.

Например:
Файл Primer.h

#pragma once
#include <iostream>
using namespace std;

void Move_Disks (int, char, char, char); // Здесь мы объявляем функцию Move_Disks, но не определяем ее

Файл primer.cpp:

#include "primer.h"

void Move_Disks (int nn, char sourse, char dest, char tmp){
  if (nn==1) {
    cout <<"Perestavit disk 1 so stolbika "<< sourse <<" na stolbik " << dest <<endl;
  }
  else {
    Move_Disks (nn-1, sourse, tmp, dest);
    cout <<"Perestavit disk "<< nn <<"so stolbika " <<sourse <<" na stolbik <<dest<<endl;
    Move_Disks (nn-1, tmp, dest, sourse);
  }
}

А в main.cpp подключаете primer.h и вызываете Move_Disks. Аналогично надо сделать с procedure.cpp, разделив его на файл-заголовка и файл-реализации

[andrey_snegovik]

casekey, спасибо Вам добрый человек большое!) Ваш способ помог!
А если не секрет, в чём разница, как подключать, через .cpp или через .h? И почему через .cpp не работает?
Только сейчас начинаем c++ изучать в универе всех тонкостей не знаю...

Кстати, если отдельно запускать программу с функцией Move_Disks, то работает без определения. Вот код:

#include <iostream>
using namespace std;

void Move_Disks (int n, char sourse, char dest, char tmp){
if (n==1)
{cout <<"Perestavit disk 1 so stolbika"<< sourse <<"na stolbik" << dest <<endl;}
else {Move_Disks (n-1, sourse, tmp, dest);
cout <<"Perestavit disk"<< n <<"so stolbika" <<sourse <<"na stolbik" <<dest<<endl;
Move_Disks (n-1, tmp, dest, sourse);}}

int main ()
{int N;
cout << "Vvedite chislo disks:";
cin>> N;
Move_Disks (N, 'A', 'C','B');

return 0;
}

И ещё один момент. Файл-заголовок - это primer.h? А файл-реализации - это где у меня программа выполняется основная, так?)

[casekey]

А если не секрет, в чём разница, как подключать, через .cpp или через .h? И почему через .cpp не работает?

И ещё один момент. Файл-заголовок - это primer.h? А файл-реализации - это где у меня программа выполняется основная, так?)

В общем случае .h - это файл, где мы определяем имена, ф-ии, классы, но не указываем как они реализованы, их реализацию осуществляется в файле .cpp

К примеру, можете проверить такой код:

int a();
int a();
int a();

т.к эти ф-ии не имеют тела (т.е это только их определение, их может быть сколь угодно) - это компилируется
другое дело, если бы было так:

int a(){}
int a(){}

Вот тут уже будет ошибка.

Еще прочитайте про директиву include и что конкретно она делает с файлом при компиляции. В вашем примере функция Move_disks попала и в файл main.cpp и в файл primer.cpp. Линкер и сообщил что при линковке нашел две функции с определением.

Прочитайте так же про эту директиву

#pragma once
...

хотя лучше было бы написать так:

#ifndef PRIMER_H
#define PRIMER_H
...
#endif

[andrey_snegovik]

В принципе всё понятно, кроме одного момента.

Еще прочитайте про директиву include и что конкретно она делает с файлом при компиляции. В вашем примере функция Move_disks попала и в файл main.cpp и в файл primer.cpp. Линкер и сообщил что при линковке нашел две функции с определением.

Ведь в main.cpp я только вызываю функцию уже со значениями

Move_Disks (P, 'A', 'C','B');

Почему тогда получается две функции с определением? Если я вроде как определяю её только в primer.cpp?

[casekey]

Я же говорю посмотри как работает include. Include подставил содержимое файла primer.cpp в мейн перед компиляцией после чего скомпилировал обьектный файл скажем main.o. аналогично скомпилировался primer.cop в primer.o. После чего линкер пытаеться слинковать эти два файла и видит что в обоих есть функции с одинаковыми именами и обе имеют тело

[andrey_snegovik]

casekey, ещё раз к Вам обращусь. Проблема в том, что мне в основную программу (main.cpp) нужно включать модули через заголовочные файлы.
Пример:

#include <iostream>
#include "recursion.h"
#include "massiv.h"
int main () {
}

Но дело в том, что у меня в модуле massiv.cpp имеются глобальные переменные:

const int L=7, M=7;
int p, k, o;

И собственно при компиляции выдаёт ошибку. Вот скриншот: prntscr.com/38osxd

Как теперь эти переменные перенести? На всякий случай прикладываю текущую структуру программы.
lab_8.zip

Заранее спасибо, очень выручаете!

[ROD]

Во-первых перенести глобальные переменные в .h файл.
Во-вторых не использовать глобальные переменные (если без них можно обойтись).
В-третьих глобальные константы можно задефайнить или заменить на перечисления. (в .h файле, причем если они реально глобальные то я бы их вынес вообще в отдельный .h файл).

К стати, у меня компилятор ни на что не ругается. (Компилятор mingw32, IDE QtCreator 3.0.82)

upd.

И самое главное забыл - защитить .h файлы от повторного включения (include guard), как уже писалось выше.
(и, по возможности, не используйте #pragma once - не все компиляторы это понимают, да и не обязаны они эту директиву понимать)

[Пепел Феникса]

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

Во-первых перенести глобальные переменные в .h файл.

ток не забыть extern сделать, а в cpp оставить.

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

(и, по возможности, не используйте #pragma once - не все компиляторы это понимают, да и не обязаны они эту директиву понимать)

я обычно юзаю и то и то, и пока не видел компиля чтоб не понимал.

[ROD]

Сообщение от Пепел Феникса

я обычно юзаю и то и то, и пока не видел компиля чтоб не понимал.

keil uVision не понимает (правда, кажется, он вообще include guard как-то не очень понимает, по крайней мере то, что работает в gcc, в кейле не компилируется)