2

所以我已经做了几个小时我认为会快速简单的项目，但我无法让它工作！这让我很沮丧，哈哈，我必须接近，但也许我不是。

我将在我的代码中包含解释它应该做什么的注释。本质上它使用私有构造函数和析构函数。一个成员整数，然后是一个公共静态函数，用于返回对类中对象的引用 - 以及一个应该显示成员整数并递增它的公共函数。我不断收到范围和初始化错误。

以下是错误：

Singleton.h: In function ‘int main(int, char**)’:
Singleton.h:28:2: error: ‘Singleton::Singleton()’ is private
main.cpp:38:12: error: within this context
Singleton.h:29:2: error: ‘Singleton::~Singleton()’ is private
main.cpp:38:12: error: within this context

任何帮助或建议将不胜感激！

这是我的代码（Singleton.h、Singleton.cpp、main.cpp）：

单例.h：

#ifndef SINGLETON_H
#define SINGLETON_H
#include <iostream>
#include <string>
#include <fstream>
#include <cstdlib>

class Singleton {

public:
    static Singleton& instance(); //returns a reference to a Singleton obj
    void sendOutput(); //prints member variable and increments it
private:
    int myInt; //member integer-variable
    Singleton(); //constructor
    ~Singleton(); //destructor

};
#endif

单例.cpp： 

#include <string>
#include "Singleton.h"

using namespace std;

//Displays to console that the obj was created. Initializes the member variable
Singleton::Singleton(){
    myInt = 0; //member variable
    cout << "Singleton Object Created!" << endl;
}

//destructor - displays to console that the Singleton object was destructed
Singleton::~Singleton(){
   // delete myObj;
    cout << "Singleton Object Destructed!" << endl;
}

//display member variable value and increment
void Singleton::sendOutput(){
    cout << "Request to send data to output to console" << endl;
    cout << "Integer Value: " << myInt << endl;
    myInt++;
}

//REQUIRED: Static method with a reference to an object of this class return type
//create a static Singleton object and return it
Singleton& Singleton::instance(){
    static Singleton myObj;
    return myObj;
}

主.cpp：

#include "Singleton.h"
#include <string>
#include <iostream>
#include <cstdlib>

using namespace std;

//Another requirement - demo static variables
void static_test(){
    static int a = 1;
    cout << a << endl;
    a++;
}

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

    static_test();
    static_test();
    static_test();


//messed around with this for awhile - got it to work a few times
//messed it up a few more times O.o 

    Singleton mySingletonObj;
    Singleton& sRef = Singleton::instance();
    //sRef = Singleton::instance();
    sRef.sendOutput();

    return 0;
}

想法/建议/问题/疑虑？任何事情都将有助于减轻这给我带来的挫败感，哈哈。给我造成这样的问题似乎太简单了。谢谢！

4

3 回答 3

2

禁止在该范围内：

单例 mySingletonObj;

应该可以：

Singleton& sRef = Singleton::instance();

引用不可重新分配：

sRef = Singleton::instance();

应该可以：

sRef.sendOutput();

另外，删除这个——语言已经指定了对象的生命周期和存储，并负责销毁它：

删除我的对象；

您还应该delete能够复制类型：

Singleton(); //constructor
Singleton(const Singleton&) = delete; // deleted copy constructor
于 2013-02-22T23:43:31.013 回答
2

只需delete myObj;在此处删除：

Singleton::~Singleton(){
    delete myObj;
    cout << "Singleton Object Destructed!" << endl;
}

它应该可以工作。myObj 是静态分配的，因此将在进程退出时由运行时删除。

于 2013-02-22T23:38:32.037 回答
0

首先，在开始编写程序之前，您应该了解该语言的基础知识。从您的代码看起来您​​尝试删除 myObj，它是来自另一个方法的本地静态对象，在析构函数中不可见，更不用说您只能在指针上调用 delete，并且应该只删除由 operator new 初始化的指针。此外，您的单例应该具有私有或已删除（对于 C++11）复制构造函数和赋值运算符。如果你这样做：“sRef = Singleton::instance();” 不会编译（而且它不应该在逻辑上）。除此之外一切都很好。

于 2013-02-22T23:47:45.133 回答