C++ Singleton: `Undefined reference to` error

Question

Я пытаюсь реализовать шаблон проектирования singleton без выделения памяти. Я попытался найти решение, но похоже, что каждое решение предназначено для одного сингла, определенного с распределением памяти.

Я сделал конструктор приватным и единственный код, который я добавил в файл заголовка, чтобы сделать это синглтон шаблон дизайна был:

static ParametersServerPC& ParametersServerPC::GetInstance() { 

    static ParametersServerPC instance; 
    return instance; 

} 

Это производный класс от базового класса ParametersServerABS который имеет пустой конструктор определение. ParametersServerABS - абстрактный класс.

Когда я пытаюсь создать экземпляр ParametersServerPC класс в отдельном файле:

ParametersServerPC& paramServer = ParametersServerPC::GetInstance(); 

Я получаю эту ошибку:

undefined reference to `ParametersServerPC::GetInstance()' 

здесь являются .cpp и .hpp файлы:

parameters_server_abs. hpp:

#ifndef PARAMETERS_SERVER_ABS_HPP_ 
#define PARAMETERS_SERVER_ABS_HPP_ 

class ParametersServerABS { 

    public: 

     ParametersServerABS(); 

     ~ParametersServerABS(); 

     virtual bool Load() = 0; 

}; 

#endif 

parameters_server_abs.cpp:

#include "mid_level/parameters_server_abs.hpp" 

ParametersServerABS::ParametersServerABS() {} 

ParametersServerABS::~ParametersServerABS() {} 

parameters_server_pc.hpp:

#ifndef PARAMETERS_SERVER_PC_HPP_ 
#define PARAMETERS_SERVER_PC_HPP_ 

#include <string> 

#include "mid_level/parameters_server_abs.hpp" 

class ParametersServerPC: public ParametersServerABS { 

    public: 

     ~ParametersServerPC(); 

     static ParametersServerPC& GetInstance(); 

     virtual bool Load(); 

    private: 

     ParametersServerPC(std::string parameterFileName = "parameters.txt"); 

     std::string _parameterFileName; 

}; 

parameters_server_pc.cpp:

#include "mid_level/parameters_server_pc.hpp" 

ParametersServerPC::ParametersServerPC(std::string parameterFileName = "parameters.txt") : 
     _parameterFileName(parameterFileName) { 

} 

ParametersServerPC::~ParametersServerPC() { 

} 

static ParametersServerPC& ParametersServerPC::GetInstance() { 

    static ParametersServerPC instance; 
    return instance; 

} 

virtual bool ParametersServerPC::Load() { 

    return true; // TODO 

} 

my_test_file.cpp

#include "mid_level/parameters_server_pc.hpp" 

ParametersServerPC& paramServer = ParametersServerPC::GetInstance(); 

Answer 1

undefined reference to `ParametersServerPC::GetInstance()'

Это, кажется, ошибка компоновщика. Если вы можете опубликовать вывод консоли компиляции, мы можем сузить его дальше.

Тем временем вы можете проверить свою систему сборки и посмотреть, не опущены ли некоторые исходные файлы из компиляции.

На одноэлементном образце уже есть некоторые хорошие ответы. Подробнее по теме in a description of the pattern и in a general question about singletons.

Answer 2

Это приемлемый шаблон. Вот MVCE демонстрации осуществимости:

#include <iostream> 
#include <string> 

using namespace std; 

class A { 
public: 
    int ival; 
    string strval; 

    static A& getInstance(); 
private: 
    A(int ival, string strval): ival(ival), strval(strval) {} 
    A(A& src): ival(src.ival), strval(src.strval) {} 
    ~A() {}; 
}; 

A& A::getInstance() { 
    static A instance(1, "foo"); 

    return instance; 
} 



int main() { 
    A& a = A::getInstance(); 
    cout << a.ival << endl; 

    // A a1 = A::getInstance(); error 
    // A a2 = a; error 
    // A a3(2, "bar"); error 

    return 0; 
} 

Answer 3

Во-первых, отметьте ваш ~ParametersServerABS(); деструктор virtual, чтобы иметь возможность удалить объекты должным образом. Во-вторых, вам нужно удалить virtual и static ключевые слова из parameters_server_pc.cpp файл: они предназначены только для определений (для вашего файла заголовка).

Далее, сделать это правильно:

class ParametersServerPC { 
    // your code 

    private: 
     ParametersServerPC(std::string parameterFileName = "parameters.txt"); 
     ParametersServerPC(ParametersServerPC const&) = delete; 
     void operator=(ParametersServerPC const&) = delete; 
}; 

Singleton означает, что вы не можете получить копии объекта: необходимо запретить использование конструктора копирования и копирующий оператор присваивания. И в любом случае я думаю, что ваша проблема в static в вашем parameters_server_pc.cpp файла. Удалите его из части реализации (файл cpp), чтобы исправить проблему, но ОСТАВИТЕ ее в части определения (заголовочный файл).