@Melany

Подскажите пожалуйста, в чем ошибка в данном коде:
1>Source.obj : error LNK2005: "bool (__cdecl** mark)(double,double)" (?mark@@3PAP6A_NNN@ZA) already defined in funcs.obj
Student.exe : fatal error LNK1169: one or more multiply defined symbols found

когда код запускается без массива указателей на функции все работает, но меня интересует использование именно данного массива, подскажите пожалуйста что тут не верно? тут выложена только та часть кода, которая затрагивает данный массив,
Заранее спасибо

в header:

int CntWithEqualMarks(const Group& gr, int * &StudList, int g, int wayofsearching);
void CaseSearchingbymark(const Group& gr, int * &StudList, int wayofsearching);

double GetMark(Student& a, int k);
int Search(const Group& gr, int * &StudList, int k, int wayofsearching);
bool MarkUp(double s1, double s2);
bool MarkDown(double s1, double s2);
typedef bool (*CmpMarks)(double , double);
CmpMarks mark[2]={MarkUp,MarkDown};

реализация функций:

void CaseSearchingbymark(const Group& gr, int * &StudList, int wayofsearching)
{
system("cls");
cout<<"Please select the parameter:\n";
cout<<"Average mark - 1\n";
cout<<"Programming mark - 2\n";
cout<<"Administ mark - 3\n";
cout<<"Design mark - 4\n";
enum Action { Average = 1, Progr = 2, Administ=3, Design=4};
int k(0), n(0);
cin >> k;
system("cls");
switch (k)
{
case Average:
n=Search(gr, StudList, k, wayofsearching);
break;
case Progr:
n=Search(gr, StudList, k, wayofsearching);
break;
case Administ:
n=Search(gr, StudList, k, wayofsearching);
break;
case Design:
n=Search(gr, StudList, k, wayofsearching);
break;
}

for(int i(0); i<n; i++)
Print(gr.stud[StudList[i]].name);
}
int Search(const Group& gr, int * &StudList, int k, int wayofsearching)
{
int qty=CntWithEqualMarks(gr,StudList, k, wayofsearching);
return qty;
}
int CntWithEqualMarks(const Group& gr, int * &StudList, int g, int wayofsearching)
{
double tmp=GetMark(gr.stud[0], g);
int n(0), k(0);
for(auto i(1); i<gr.cnt; i++)
{
if(mark[wayofsearching](tmp, GetMark(gr.stud[i],g)))
tmp=GetMark(gr.stud[i],g);
}
//StudList[gr.cnt]=NULL;
for(auto i(0); i<gr.cnt; i++)
{
if(tmp==GetMark(gr.stud[i],g))
{
StudList[k]=i;
k++;
}
}
return k;
}
double GetMark(Student& a, int k)
{
if(k==1)
return a.avgMark;
if(k==2)
return a.cslist.cs[0].mark;
if(k==3)
return a.cslist.cs[1].mark;
if(k==4)
return a.cslist.cs[2].mark;
}
bool MarkUp(double s1, double s2)
{
if (s1 < s2)
return true;
else
return false;
}
bool MarkDown(double s1, double s2)
{
if (s1 > s2)
return true;
else
return false;
}

0

@Melg

Вот это вот:
Является определением массива на функции размера 2. Если это определение Вы включаете в заголовочный файл - то оно попадает во все единицы трансляции(.cpp/.cc файлы) в которых есть директива препроцессора #include <имя данного заголовочного файла>.
Это приводит к тому, что в каждой из единиц трансляции есть своё определение массива mark. Притом, символы данного определения (mark) - видны вне единиц трансляции и экспортируются. Отсюда получается, что когда вы включили этот заголовочный файл в 2 .cpp файла - вы получили 2 разных определения одной и той же сущности mark. Далее на этапе линковки - это увидел линковщик о чём и рапортует.
Советы:
1) Используйте таги С++ (2 строчка, 1 в списке - у меня по крайней мере) над кодом, иначе Ваш код читать невозможно.
2) Подумайте над тем, для чего Вам нужен данный массив в 2-ух различных .cpp файлах и опишите тут суть его использования
3) Избавьтесь от ошибки - либо перенеся определение в .cpp файл и всё, либо оставив в заголовочном файле объявление, либо переделав получение данного массива в принципе (можно организовать получение как возвращаемое значение функции например)

1

@Melany

Melg, спасибо, у меня в данном проекте только 2 срр файла и 1 heаder, как видно, массив указателей был определен и создан в heаder,
по Вашему третьему совету, по переносу данного массива в срр, где этот массив используется, все заработало, но в принципе правильнее было бы определить эти все дела в header.h. можете поподробнее объяснить алгоритм, при котором данный массив можно оставить в header.h?
Спасибо еще раз огромное!!!

0

@Melg

Начнем с общих представлений:
Во-первых, глобальные переменные - это плохо. Они не дают мыслить о коде локально, прослеживать цепочки передачи и изменения данных. В Вашем случае - переменная mark будет носить характер глобальной, если оставить всё как есть и только избавиться от ошибки двойного определения.
Если решать проблему в лоб, тогда в .h у вас должно было бы остаться только объявление переменной:
А в одном .cpp файле должно было содержаться определение:
В общем случае Вам следует определять где конкретно и зачем используется данный массив функций. Если он не изменяется, а вызовы всегда идут по индексам - подумать о преобразовании массива в структуру. Если вы собираетесь его изменять - возможно следует инкапсулировать его в объекте класса и установить правила доступа к элементам массива на изменение. Наконец не стоит забывать о том что с++ является объектно-ориентированным языком. В стандартной библиотеке есть абстракция функции, std::function - которая может содержать любой функтор с правильной сигнатурой, кроме того, в стандартной библиотеке есть std::vector - который является абстракцией динамического массива с возможностью изменения размеров.
Подробнее читайте: http://www.cplusplus.com/refer... /function/
http://www.cplusplus.com/refer... or/vector/

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

1

@Melany

Melg, Спасибо, за разъяснения, очень полезно, так как класса я еще близко не знакома, то как раз будет повод почитать поподробнее.

0