MySQL C++ 连接器在 ResultSet->getString() 处使我的应用程序崩溃

Question

又是我在问菜鸟 C++ 问题

我很头疼，使该死的（对语言感到抱歉）MySQL C++ 连接器工作。我不知道它是不是写得不好还是什么，但就我的经验而言，我从来没有遇到过这么麻烦的事情。

无论如何，我让它连接并在失败的连接/查询上抛出异常，这对我来说是件大事 :U :P 。实际问题来自我获取查询结果。无论我做什么，我的应用程序总是崩溃：S

我使用了 32 位安装程序和来自 32 位 MySQL 服务器的 libmysql.dll/lib（因为我正在编译一个 32 位应用程序，我认为这是正确的做法）

这是一些代码，所以你可以想象我在说什么

数据库管理器

#ifndef DBMANAGER_H
#define DBMANAGER_H
#define CPPCONN_PUBLIC_FUNC
#define CPPCONN_LIB_BUILD True

#include <string>
#include "mysql_connection.h"
#include "mysql_driver.h"
#include <cppconn/driver.h>
#include <cppconn/exception.h>
#include <cppconn/resultset.h>
#include <cppconn/statement.h>
#include <cppconn/prepared_statement.h>

class DBManager
{
public:
    static DBManager* Instance();
    bool Query(const char* Query);
    void Connect(const char* DbHost, unsigned short DbPort, const char* DbUser, const char* DbPass, const char* DbName);
    bool ValidCredentials(const char* Username, const char* Password);
    void ManageException(sql::SQLException &e);

    ~DBManager();

protected:
    static DBManager* pInstance;

private:
    DBManager() {};
    DBManager(DBManager const&){};
    DBManager& operator=(DBManager const&){};

    sql::mysql::MySQL_Driver* driver;
    sql::Connection *Con;
    sql::PreparedStatement *pstmt;
    sql::ResultSet *res;
    sql::Statement *stmt;

    bool isConnected;
};

#endif

现在cpp文件DBManager.cpp

#include "DBManager.h"

DBManager* DBManager::pInstance = NULL;

DBManager* DBManager::Instance()
{
    if (!pInstance)
    {
        pInstance = new DBManager();
    }

    return pInstance;
}

bool DBManager::Query(const char* Query)
{
    return true;
}

DBManager::~DBManager()
{   
    delete Con;
    delete pstmt;
    delete res;
    delete stmt;
}

void DBManager::ManageException(sql::SQLException& e)
{
    if (e.getErrorCode() != 0) {
        std::cout << "# ERR: SQLException in " << __FILE__;
        std::cout << "(" << __FUNCTION__ << ") on line " << __LINE__ << std::endl;
        std::cout << "# ERR: " << e.what();
        std::cout << " (MySQL error code: " << e.getErrorCode();
        std::cout << ", SQLState: " << e.getSQLState() << " )" << std::endl;
    }
}

void DBManager::Connect(const char* DbHost, unsigned short DbPort, const char* DbUser, const char* DbPass, const char* DbName)
{
    try {
        driver = sql::mysql::get_mysql_driver_instance();
        std::string connDSN = "tcp://" + std::string(DbHost) + ":3306";

        Con = driver->connect(connDSN, sql::SQLString(DbUser), sql::SQLString(DbPass));
        Con->setSchema(sql::SQLString(DbName));
        isConnected = true;

        std::cout<<"Database connection successul."<<std::endl;

    } catch(sql::SQLException &e) {
        ManageException(e);
        isConnected = false;

        return;
    }
}

bool DBManager::ValidCredentials(const char* Username, const char* Password)
{
    bool cred = false;

    try {
        pstmt = Con->prepareStatement("SELECT * FROM account WHERE account_name=? LIMIT 1"); // Smart use of indexing
        pstmt->setString(1, Username);
        res = pstmt->executeQuery();

        while(res->next())
        {
            if (res->getString("password") == Password)
            {
                cred = true;
            }
        }
    }
    catch(sql::SQLException &e) {
        ManageException(e);
        return false;
    }

    return cred;
}

基本上，它编译没有问题，连接没有问题，执行查询没有问题，但是第二次我尝试检索数据时，在文件“xutils.cpp”中抛出了一些断点异常。我真的不知道我做错了什么。我在编译调试时使用 DEBUG 库。嗯，libmysql.dll 应该被释放，因为我是从服务器包中提取它的，但我似乎没有找到它作为编译我自己的源。

我真的不知道为什么它会像那样崩溃和燃烧：/

PS：不要介意密码没有散列，这对我来说真的只是一个概念证明......首先让它工作，然后保护它：U

PS：我还在项目中编译并准备好了 Boost 库，如果这有帮助的话：U

编辑：主要功能

bool ServerRunning = true;
int main(int argc, char** argv)
{
    #ifdef _WIN32
        std::string title = TEXT("Window Title Change");
        SetConsoleTitle(title.c_str());
    #endif;

    std::cout<<"Loading Configuration File..."<<std::endl<<std::endl;

    std::string path = boost::filesystem::path(boost::filesystem::current_path()).string();
    path += "\\Config.ini";

    INIParser* Config = new INIParser(path.c_str()); //MinINI

    // Sockets data
    std::string listenIP = Config->GetString("Network", "ListenIP", "127.0.0.1");
    unsigned short listenPort = Config->GetInt("Network", "ListenPort", 5000);

    // Database data
    std::string dbHost = Config->GetString("Database", "Host", "localhost");
    std::string dbUser = Config->GetString("Database", "User", "root");
    std::string dbPass = Config->GetString("Database", "Password", "");
    std::string dbName = Config->GetString("Database", "Database", "authserv");
    unsigned short dbPort = Config->GetInt("Database", "Post", 1000);

    // General settings
    int sessionTimeout = Config->GetInt("Settings", "SessionTimeout", 10);
    int maxClients = Config->GetInt("Settings", "MaxClients", 10);
    int serverTimeout = Config->GetInt("Settings", "GameserverTimeout", 1);

     // Begin Initialization
     DBManager::Instance()->Connect(dbHost.c_str(), dbPort, dbUser.c_str(), dbPass.c_str(), dbName.c_str());
     bool loginSuccess = DBManager::Instance()->ValidCredentials("Username", "Password");

    char c;
    while (ServerRunning)
    {
        std::cin>>c;

        if (c == 'q')
        {
            ServerRunning = false;
        }
    }

    return 0;
}

Answer 1

假设该password字段varchar在数据库中定义，则不能用于getString()检索它。您必须改用blob函数getBlob().

这是while循环的样子：

while(res->next())
{
    std::istream * retrievedPassword_stream = res->getBlob("password");
    if (retrievedPassword_stream)
    {
        char pws[PASSWORD_LENGTH+1]; // PASSWORD_LENGTH defined elsewhere; or use other functions to retrieve it
        retrievedPassword_stream->getline(pws, PASSWORD_LENGTH);
        std::string retrievedPassword(pws); // also, should handle case where Password length > PASSWORD_LENGTH
        if (retrievedPassword == std::string(Password))
        {
            cred = true;
        }
    }
}

旁注：请注意，代码还有一些其他问题。

• 必须删除语句句柄，因此您应该delete pstmt;在ValidCredentials()函数中的适当位置（而不是在析构函数中）执行 a。（但是，为什么在这种情况下使用准备好的语句？最好在构造函数（或调用查询的函数之外的其他地方）初始化准备好的语句，以及在析构函数或其他地方删除，如果您确实使用了准备好的语句。但是，请注意，准备好的语句对于使用率非常高和 CPU 密集度很高的查询最有用，因此在这里使用它进行密码验证可能并不重要（您可以只执行常规查询，而不是准备好的语句）。
• 同样，ResultSet 需要delete res在 try 块的末尾被删除 ( )，而不是在析构函数中。
• 在使用pstmt, res, 或之前一定要检查NULL Con。
• stmt 似乎未使用，不应删除。