C++ singleton, странная ошибка

Question

Я реализую потокобезопасный синглтон. Но этот аспект (singleton & thread-safe) не является частью моего вопроса.

Сравните два кода. Код 1:

#include <iostream> 
using namespace std; 
class DataLocation { 
private: 
    DataLocation(std::string) { 
    } 
public: 
    DataLocation& getInstance() { 
    std::string s = " "; 
    static DataLocation instance(s); 
    return instance; 
    } 
}; 
int main() { 
} 

и код 2:

#include <iostream> 
using namespace std; 
class DataLocation { 
private: 
    DataLocation() { 
    } 
public: 
    DataLocation& getInstance() { 
    static DataLocation instance();  
    return instance; 
    } 
}; 
int main() { 
} 

код 1 компилируется нормально. Код 2 дает следующее сообщение об ошибке:

15_singleton.cpp: In member function ‘DataLocation& DataLocation::getInstance()’: 
15_singleton.cpp:15:34: error: cannot declare static function inside another function 
    static DataLocation instance();  
           ^
15_singleton.cpp:16:12: error: invalid initialization of non-const reference of type ‘DataLocation&’ from an rvalue of type ‘DataLocation (*)()’ 
    return instance; 
      ^

С моей точки зрения единственное различие заключается в том, что частный конструктор имеет один, соответственно нулевые параметры.

Как я могу помочь компилятору понять, что я не определяю ничего нового, но я просто вызываю конструктор? Компилятор может понять это, когда есть один параметр.

Answer 1

Снимите кронштейны

static DataLocation instance; 

Чтобы создать экземпляр с помощью конструктора по умолчанию.

В качестве альтернативы используйте форматированную форму.

static DataLocation instance {}; 

Answer 2

Метод getInstance() должен быть объявлен как статический.

Answer 3

static DataLocation instance = DataLocation(); 

или

static DataLocation instance; 

И вы, вероятно, хотите объявить DataLocation& getInstance(); как метод static.

Answer 4

Вы используете скобки () при объявлении

static DataLocation instance(); 

которого намерения создать как объект. Но это просто объявление функции. И вы не можете создать функцию static внутри какой-либо другой функции не static. Это то, к чему относится ваша ошибка.