某系統在某特殊情況下， 會出現bug, 經我非常保守地估計， 這個bug的定位修改費用至少3000元， 這還不包括其他的費用。 脫離具體場景， 我來抽象出一個簡單的模型， 示例程式碼如下：

#include <iostream>
using namespace std;

class A
{
private:
	static A *m_p;

public:
	static A *getSingleTon()
	{
		if(NULL == m_p)
		{
			m_p = new A();
		}

		return m_p;
	}


	~A()
	{
		if(NULL != m_p)
		{
			delete m_p;
			m_p = NULL;
		}
	}
};

A* A::m_p = NULL;


int main()
{
	return 0;
}
 

       如果沒有看出問題， 那你再執行一下如下程式：

#include <iostream>
using namespace std;

class A
{
private:
	static A *m_p;

public:
	static A *getSingleTon()
	{
		if(NULL == m_p)
		{
			m_p = new A();
		}

		return m_p;
	}


	~A()
	{
		if(NULL != m_p)
		{
			cout << "xxx" << endl;

			delete m_p; // 遞迴呼叫析構
			m_p = NULL;

			cout << "yyy" << endl; // 永遠也不會執行
		}
	}
};

A* A::m_p = NULL;


int main()
{
	A *p = A::getSingleTon();
	delete p;

	return 0;
}
 

        我們來反思一下， 為什麼會出上述問題呢？肯定是寫SingleTon的人牢牢記住了： 要在解構函式中釋放資源。  但是， 他不明白， 單例應該由類的使用者來釋放， 而不是類的提供者。 不要把角色搞錯了。 

        千萬不要說， 為什麼出這麼低階的問題！ 其實， 這個問題不低階， 是個比較隱蔽的錯誤。 而且， 當代碼多了（比如100w行）， 離職的人多了， 經幾次交接後， 那程式碼就可想而知了

         下面， 我們繼續看看：

#include <iostream>
using namespace std;

class A
{
private:
	static A *m_p;
	
	int x;

	A()
	{
		x = 1;
	}

public:
	static A *getSingleTon()
	{
		if(NULL == m_p)
		{
			m_p = new A();
		}

		return m_p;
	}


	~A()
	{
		if(NULL != m_p)
		{
			cout << "xxx" << x << endl; // 永遠是xxx1

			delete m_p; // 遞迴呼叫析構
			m_p = NULL;

			cout << "yyy" << x << endl; // 永遠也不會執行, 單例也不會被釋放
		}
	}
};

A* A::m_p = NULL;


int main()
{
	A *p = A::getSingleTon();
	delete p;

	return 0;
}

       實際上， 要快速定位到解構函式的問題， 還是很不容易的， 那麼多程式碼， 程序死掉， 咋快速定位？尤其是， 如果解構函式中沒有日誌列印， 根本就很難知道解構函式被多次執行了。 所以， 關於日誌， 我強烈建議：

       1. 所有的建構函式和解構函式都必須有日誌列印。

       2. 不被頻繁呼叫的函式中， 必須有日誌（很多人只喜歡在某些異常分支打日誌， 甚至連異常分支都不列印日誌， 確實太流氓了）。

       好吧， 一個小小的bug確實讓人蛋疼兩三天。 這個程式碼是誰寫的啊是不是應該在明天端午節請我吃一頓大餐呢我再次大聲疾呼， 軟體質量不是一句廢話。 

       要多反思， 多總結， 總會慢慢進步。本文先到此為止。