1

Пишу программу подбора фильмов пользователю на основе ранее понравившихся ему фильмов. Привел вроде как в рабочее состояние парсер файла лайков, но вот не задача: выдает 6 ошибок LNK2019 на неразрешенный внешний символ(судя по LINK1120 там 3 таких места), вроде как что-то не то с inline getId() const.

Пример ошибок:

Ошибка LNK1120 неразрешенных внешних элементов: 3 FilmSelection

C:\Users\Andrey\Documents\GitHub\FilmSelection\FilmSelection\FilmSelection\De bug\FilmSelection.exe 1


Ошибка LNK2019 ссылка на неразрешенный внешний символ "public: class std::basic_string,class std::allocator > __thiscall Channel::getId(void)const " (?getId@Channel@@QBE?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@XZ) в функции "public: class std::shared_ptr __thiscall User::findChannel(class std::basic_string,class std::allocator >)" (?findChannel@User@@QAE?AV?$shared_ptr@VChannel@@@std@@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@3@@Z) FilmSelection C:\Users\Andrey\Documents\GitHub\FilmSelection\FilmSelection\FilmSelection\FilmSelection\User.obj 1

inline string Channel::getId() const; // public метод, который я вызываю через shared pointer на данный класс
{
   return id; //id является private string
}
//поиск фильма, у Film есть аналогичный метод получения getId(), films - это list<shared_ptr<Film>> films 
//increaseLikes() увеличивает количество лайков для канала и аналогично для фильма
shared_ptr<Film> Channel::findFilm(string id_film) 
{
    for (auto it = films.begin(); it != films.end(); it++)
        if (id_film == (*it)->getId()) {
            (*it)->increaseLikes();
            return (*it);
        }
    return nullptr;
}

Вот исходник GITHUB

13
  • 1
    Эх, зачем же вы папки Debug и Release да в git.. 26 июл 2016 в 8:40
  • 1
    Попробуйте убрать inline.
    – free_ze
    26 июл 2016 в 8:40
  • @free_ze помогло, спасибо! Но почему???? Так же вроде принято делать при получении данных
    – Xambey
    26 июл 2016 в 8:46
  • @free_ze ошибку выдавал потому что Inline вставляет по месту вызова код целиком, а return в if() не очень?
    – Xambey
    26 июл 2016 в 8:51
  • 1
    @Xambey Потому что инлайн-функции должны определяться вместе с типом, т.е. в хэдере, аналогично шаблонам. Я скопипащу свой коммент в ответ.
    – free_ze
    26 июл 2016 в 9:02

2 ответа 2

4

Наличие спецификатора inline при определении функции с внешней компоновкой (external linkage) в .cpp требует дублирования определения функции во всех единицах трансляции, где она используется:

An inline function or variable shall be defined in every translation unit in which it is odr-used and shall have exactly the same definition in every case.

Ваш случай может быть упрощен до следующего:

// a.h 
#pragma once
struct A {
    void f();
};

// a.cpp
#include "a.h"
inline void A::f() {}

// main.cpp
#include "a.h"
int main() {
   A a;
   a.f(); 
}

Модуль main.cpp видит объявление A::f (из a.h), но из-за наличия inline не видит определение A::f (т.е. в main.cpp оно отсутствует). Решить проблему можно:

  • либо убрав inline, и сделав реализацию доступной другим единицам трансляции;
  • либо поместив в main.cpp точно такое же определение функции, что и в a.cpp.
2

Попробуйте убрать inline.

Говорят, смысла нет вручную писать это ключевое слово, ибо современные компиляторы на него не реагируют и инлайнят по своему разумению.

2
  • ну Вы бы уж полностью копировали свои комментарии, а то в текущем виде это не тянет на ответ, а только на комментарий. Пожалуйста, добавьте деталей, почему inline нужно убрать в этом случае.
    – ixSci
    26 июл 2016 в 15:18
  • @ixSci При желании вы можете править чужие ответы или добавлять свои. У меня нет ответа почему, потому я и не это пишу это в ответ.
    – free_ze
    26 июл 2016 в 15:27

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.