Как разнести по файлам функцию с++

Question

Столкнулся с проблемой, как правильно разнести объявление и реализацию фунции между между заголовочным файлом .h и .cpp. Сейчас есть такая структура программы.

.h файл

namespace problem {
  class_1 {}
  class_2 {}
  ...
  static std::vector<int> coord;
static void solve(double&x, double&y) {
    //transform coord, use class_1, class_2;
  }
}

в .сpр вынесена реализация всех классов. Когда выношу реализацию функции solve в .cpp, то получаю ошибку при вызове функции problem::solve(x, y) (вызов происходит в main.cpp):

undefined reference to `problem::solve(double&, double&)'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

Не могу разобраться в чём проблема.

Возможный дубликат вопроса: Ссылка на неразрешенный внешний символ (возможные причины) — Mikhajlo, Jun 04 '22 at 06:11

score 0 · Accepted Answer · answered May 31 '22 at 16:53

0

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

При этом когда вы убираете реализацию в .h, при включении этого заголовочного файла в main.cpp, реализация этой функции фактически оказывается в файле main.cpp, то есть в том же файле, что и ее вызов, и ее статичности не мешает вызову.

Чтобы функцию можно было вызвать из main, уберите static

answered May 31 '22 at 16:53

Bearded Beaver

3,291

благодарю, с этим разобрался. Буду признателен, если поможете разобраться теперь с другим вопросом. static std::vector<int> coord; в этот вектор добавляю элементы так же в main.cpp. Дальнейшая обработка вектора идет в problem::solve(double&x, double&y). Как лучше организовать его передачу в функцию? Если убираю static в main.cpp вознимают ошибки. multiple definition of problem::solve. Заранее прошу прощения за такие ворпосы, только начинаю осваивать с++ – Loki May 31 '22 at 17:15
1

Я не уверен, что правильно понял, в чем именно проблема. Но если у вас coord это данные, необходимые для расчета в функции solve, я бы с точки зрения дизайна предложил передавать их в функцию solve как параметр, а заполнять видимо в main. А вообще лучше задайте отдельный вопрос, там опишите, какая структура у вас получается сейчас, посмотрим – Bearded Beaver May 31 '22 at 17:24

Как разнести по файлам функцию с++

1 Answers1