0

程序退出时没有调用我的析构函数。该对象是一个单身人士，也许我错过了什么？

这是头文件和cpp文件：

#ifndef MYSQLCONNECTOR_H
#define MYSQLCONNECTOR_H

/* Standard C++ headers */
#include <iostream>
#include <string>

/* MySQL Connector/C++ specific headers */
#include <driver.h>
#include <connection.h>
#include <statement.h>
#include <prepared_statement.h>
#include <resultset.h>
#include <metadata.h>
#include <resultset_metadata.h>
#include <exception.h>
#include <warning.h>

class MysqlConnector {
private:
    static bool instanceFlag;
    static MysqlConnector* mysqlConnector;

    MysqlConnector() {

    };
public:
    static sql::Driver *driver;
    static sql::Connection *conn;
    static MysqlConnector* getInstance();

    virtual ~MysqlConnector() {
        instanceFlag = false;
        conn->close();
        delete conn;
        std::cout << "called" << std::endl;
    };
private:

};

#endif  /* MYSQLCONNECTOR_H */

和cpp文件

#include "MysqlConnector.h"

using namespace std;
using namespace sql;

bool MysqlConnector::instanceFlag = false;
MysqlConnector* MysqlConnector::mysqlConnector = NULL;

MysqlConnector* MysqlConnector::getInstance() {
    if (!instanceFlag) {
        mysqlConnector = new MysqlConnector();
        instanceFlag = true;

        try {
            driver = get_driver_instance();

            /* create a database connection using the Driver */
            conn = driver->connect("tcp://127.0.0.1:3306", "root", "root");

            /* turn off the autocommit */
            conn -> setAutoCommit(0);

            /* select appropriate database schema */
            conn -> setSchema("exchange");

        } catch (SQLException &e) {
            cout << "ERROR: SQLException in " << __FILE__;
            cout << " (" << __func__ << ") on line " << __LINE__ << endl;
            cout << "ERROR: " << e.what();
            cout << " (MySQL error code: " << e.getErrorCode();
            cout << ", SQLState: " << e.getSQLState() << ")" << endl;

            if (e.getErrorCode() == 1047) {
                cout << "\nYour server does not seem to support Prepared Statements at all. ";
                cout << "Perhaps MYSQL < 4.1?" << endl;
            }
        } catch (std::runtime_error &e) {
            cout << "ERROR: runtime_error in " << __FILE__;
            cout << " (" << __func__ << ") on line " << __LINE__ << endl;
            cout << "ERROR: " << e.what() << endl;
        }

        return mysqlConnector;
    } else {
        return mysqlConnector;
    }
}
4

2 回答 2

7

您的析构函数没有被调用，因为没有人调用delete创建的对象new：

mysqlConnector = new MysqlConnector(); // Where's the corresponding call to delete?

您可以考虑改用智能指针（std::unique_ptr如果您负担得起 C++11，我会建议）。delete当智能指针本身被销毁时，这将自动调用封装的对象。

另一种可能性是根本不使用指针，并具有类型的静态数据成员MysqlConnector。getInstance()然后可以返回对该对象的引用（而不是指针）。

于 2013-05-23T15:23:19.967 回答
0 
static MysqlConnector* mysqlConnector;

是指向类的指针。当您获得实例时，它只返回此指针。您需要调用delete此静态指针才能调用析构函数。

如果您继续在此指针上调用 delete，请确保仅在程序终止时才这样做 - 因为下一次调用getInstance()将创建一个新对象。

编辑 - 刚刚发现 instanceFlag 所以它不会创建一个新实例但会返回一个 NULL 指针

这是有问题的，因为您可能希望 Singleton 是同一个对象，但是当然创建它的新实例将意味着您的数据将不会持久化（即使您正在访问单例）。

无论如何，操作系统都会回收内存，因此您最好公开连接关闭代码并自己调用它？

于 2013-05-23T15:25:25.087 回答