Gặp chút rắt rối với singleton pattern

[chanphat01001]

trong sách designpartten có viết bằng c#, bây giờ mình chuyển qua C++, nhưng có một số cái mình không biết mình chuyển đúng không(có một số cái trong C# hỗ trợ mà C++ không có) mà nó chạy không được.Bạn nào biết sửa dùm với

////					MAU SINGLETON PATTERN
#include<iostream.h>
class DatabaseInfo
{
private:
	char* ServerName;
	char* DatabaseName;
	char* UserName;
	char* Password;
protected:
	static DatabaseInfo *m_Instance=NULL;//nó báo lỗi chổ này
	DatabaseInfo()
	{
		ServerName=new char[20];
		ServerName="HuyPhat";
		DatabaseName=new char[20];
		DatabaseName="PhatHuy";
		UserName=new char[20];
		UserName="ChanPhat";
		Password=new char[20];
		Password="PhatChan";
	}
public:
	static DatabaseInfo* Singleton()
	{
		if(m_Instance==NULL)
		{
			m_Instance=new DatabaseInfo;
		}
		return m_Instance;
	}
};

void main()
{
	DatabaseInfo *DB;
	/*DB=new DatabaseInfo;*/
	DB=DatabaseInfo::Singleton();//object duoc tao ra mot lan duy nhat
             cout<<DatabaseInfo::Singleton().ServerName;
             cout<<DatabaseInfo::Singleton().DatabaseName;
             cout<<DatabaseInfo::Singleton().UserName;
             cout<<DatabaseInfo::Singleton().Password;

}

[nguyentuan2]

cậu này thích dùng con trỏ nhỉ
static DatabaseInfo *m_Instance=NULL;//nó báo lỗi chổ này
C++ không cho khởi tạo inline với property member như thế này

toàn bộ là con trỏ
char* ServerName;
char* DatabaseName;
char* UserName;
char* Password;

trong khi đó cấp phát fix size
ServerName=new char[20];
DatabaseName=new char[20];
UserName=new char[20];
Password=new char[20];


đã cấp phát rồi lại gán hằng chuỗi
ServerName="HuyPhat";
DatabaseName="PhatHuy";
UserName="ChanPhat";
Password="PhatChan";

viết class mà phương thức trả ra con trỏ
static DatabaseInfo* Singleton()

không thấy destructor

lúc nào cũng là contro
DatabaseInfo *DB;
/*DB=new DatabaseInfo;*/

Nói chung là cậu nên tránh những điều tui nói ở trên

[ilovecplusplus]

Bác chanphat khuân mấy cuốn C++ căn bản về ngâm cứu đã, làm gì mà vội thế. Em nghĩ chắc bác cũng mới học C++ thôi có phải không ? Thiên hạ cứ oang oang "đi tắt" với chả "đón đầu" nhưng mà thực ra không thể "biến không thành có, biến khó thành dễ" mà không cần đến những kiến thức căn bản.

VD: Trong cái source của bác có đoạn

private:
    char * ServerName;
    ....

Cái ServerName (DatabaseName...) ở trong private section thì làm sao mà lôi nó ra được để cout như ở trong hàm main được hở bác.

Ngoài ra, những gì bác nguyentuan đã nói đều đúng, em chỉ bổ sung thêm là nếu bác không implement một cách tường minh thì C++ sẽ tự add vào các đồng chí sau: default constructor (không có param), copy constructor, operator=, destructor. Bởi vậy, nếu chỉ có một đồng chí default constructor được khai báo tường minh (không thuộc public section), thì không đảm bảo rằng chỉ có duy nhất một singleton được tạo ra trong suốt chương trình. Tức là, client có thể dùng copy constructor để tạo ra 2nd Singleton --> Error (Đấy là chưa kể trong môi trường multithread, nhiều thread có thể cùng gọi đến cái method chịu trách nhiệm khởi tạo - ở đây là static DatabaseInfo * Singleton, vì vậy, nếu không đảm bảo được rằng chỉ có đúng một thread là gọi thành công (các thread khác bị block) thì cũng dẫn đến có nhiều hơn một instance của Singleton được tạo ra).

[ilovecplusplus]

Đây là cái source của bác mà em mới fix lại (cũng không tốt lắm, single thread only):

#include <cstdlib>
#include <string>
#include <iostream>

using std::string;

class DatabaseInfo {
public:
    static DatabaseInfo * getInstance () 
    {
        if (hInstance_ == NULL) {
            hInstance_ = createInstance ();
        }
        return hInstance_;
    }
    
    static void deleteInstance ()
    {    
        if (hInstance_) {
            hInstance_ -> ~DatabaseInfo ();
            delete hInstance_;
        }
        hInstance_ = NULL;
    }
    
private:
    DatabaseInfo ()
    {
        databaseName_ = "example.db";
        serverName_ = "localhost";
        userName_ = "ilcpp";
        password_ = "secret";
    }
    
    DatabaseInfo (const DatabaseInfo & self)
    {
    }
    
    ~DatabaseInfo ()
    {
        if (!databaseName_.empty ())
            databaseName_.erase ();
        
        if (!serverName_.empty ())
            serverName_.erase ();
            
        if (!userName_.empty ())
            userName_.erase ();
            
        if (!password_.empty ())
            password_.erase ();
    }
    
private:
    DatabaseInfo & operator= (const DatabaseInfo & self)
    {
        return *this;
    }
    
    static DatabaseInfo * createInstance ()
    {
        hInstance_ = new DatabaseInfo ();
    }
    
private:
    static DatabaseInfo * hInstance_;
    string databaseName_;
    string serverName_;
    string userName_;
    string password_;    
};

DatabaseInfo * DatabaseInfo::hInstance_ = NULL;

int main(int argc, char *argv[])
{
    DatabaseInfo * uniqueDI = DatabaseInfo::getInstance ();
    /* 
          ** Do something with di and
          ** cleaning up before exit
          */
    DatabaseInfo::deleteInstance ();
    return EXIT_SUCCESS;
}

P.S: Bác hãy tìm thư viện Loki để ngâm cứu thêm.
http://sourceforge.net/projects/loki-lib/

[chanphat01001]

Nhờ mấy bác mà em đã được mở rộng tầm mắt rồi!!!!!!!!!

[hoangtran]

Đây là cái source của bác mà em mới fix lại (cũng không tốt lắm, single thread only):

Ý của bạn về cái single thread là sao? Bạn có ý gì về sử dụng thread??