为什么在实现单例类时会出现“未定义的引用错误”？ [复制]

Question

我正在尝试实现一个返回类实例的方法，但是当它第一次尝试创建实例时它崩溃了。我不知道如何在 C++/QT 中实现单例

主要

#include <QCoreApplication>
#include "carro.h"

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);

    Carta* nueva;
    nueva->getInstance();
    nueva->setColor("rojo");
    cout << nueva->getColor() << endl;

    return a.exec();
}

卡罗.h

#ifndef CARRO_H
#define CARRO_H

#include <string>
#include <iostream>

using namespace std;

class Carta{

private:
    string cara; //valor
    string palo; //simbolo
    string color;
    string direccion;

    static Carta* m_instance;

public:
    //constructor
    Carta(){
    }

   static Carta* getInstance(){
       if(!m_instance){
           m_instance = new Carta;
       }
       return m_instance;
    }

    string getDireccion(){
        return direccion;
    }

    void setColor(string pcolor){
        color = pcolor;
    }

    string getColor(){
        return this->color;
    }

    string getPalo(){
        return this->palo;
    }

    string getCara(){
        return this->cara;
    }

    //print
    string print(){
        return (cara + " de " + palo + " Color: "+color);
    }    
};

#endif // CARRO_H

Answer 1

您缺少定义static Carta* m_instance;，因此出现链接器错误：

    Carta* Carta::m_instance = nullptr;

---

不过，如果你确定需要单例，我更愿意推荐 this pattern：

static Carta* getInstance() {
    static Carta m_instance;
    return &m_instance;
}

或

static Carta& getInstance() {
    static Carta m_instance;
    return m_instance;
}

---

同样为了访问单例，你应该有一些类似的代码

Carta* nueva = Carta::getInstance();
nueva->setColor("rojo");
cout << nueva->getColor() << endl;

或参考版本

Carta& nueva = Carta::getInstance();
nueva.setColor("rojo");
cout << nueva.getColor() << endl;

Answer 2

这两行不正确：

Carta* nueva;
nueva->getInstance();

nueva 未初始化为任何内容，因此在下一行中将其推迟会导致未定义的行为。

看起来您打算执行以下操作：

Carta* nueva = Carta::getInstance();

Answer 3

Carta* nueva;
nueva->getInstance();
nueva->setColor("rojo");

这是不正确的。这几行代码对计算机准确地说：

1.Carta* nueva 创建一个指向 Carta 变量名 nueva 的指针。不要忘记此时，指针为 NULL。

2.nueva->getInstance() 这意味着访问 nueva 指针（仍然为空）并获取方法 getInstance()。由于 getInstance 是一个静态函数，它应该以这种方式使用 Carta::getInstance(); 您必须使用类而不是变量和运算符 :: 来访问静态成员。将其放入变量 nueva 后，它将给出： nueva = Carta::getInstance();

所以正确的写法是：

Carta* nueva;
nueva = Carta::getInstance();
nueva->setColor("rojo");