[Pyro_]

I'm new to SourceMod/SourcePawn, and especially using SQL with it.

I'm trying to make a plugin that uses SQL neatly (and the best way (if this isn't the best way; what is the best way?)).

PHP Code:

public Handle:GetConnection()
{
    new Handle:db = SQL_Connect(NAME_OF_DATABASE_CONFIG, true, errorString, sizeof(errorString));

    if(db == INVALID_HANDLE)
    {
        LogAction(-1, -1, "[TG Loyalty] Could not connect: %s", errorString);
    }
    return db;
} 


The above is a function that returns a database handle, which is used in a lot of functions.
Example:

PHP Code:

public bool:QuickQuery(Handle:db, String:query[])
{
    if(db == INVALID_HANDLE)
    {
        db = GetConnection();
    }
    //This isn't the same, the first one checks if there already is a db connection, while this one checks if the possible new one didn't work
    if(db == INVALID_HANDLE)
    {
        CloseHandle(db);
        return false;
    }
    if(!SQL_FastQuery(db, query))
    {
        SQL_GetError(db, errorString, sizeof(errorString));
        LogAction(-1, -1, "Failed to query (error: %s)", errorString);
        CloseHandle(db);
        return false;
    }
    else
    {
        CloseHandle(db);
        return true;
    }
} 


I'm sorry if I'm doing extremely noobish things.

Whenever I call QuickQuery, or a function that uses QuickQuery
e.g.

PHP Code:

if(!QuickQuery(db, queryString)) 


It gives me this error in the console: Native "SQL_FastQuery" reported: Invalid database Handle b(random numbers) (error: 3)

What am i doing wrong? And if possible, how can I do it better?

[Powerlord]

When you CloseHandle a variable, you should set it to INVALID_HANDLE on the next line.

[Pyro_]

Originally Posted by Powerlord When you CloseHandle a variable, you should set it to INVALID_HANDLE on the next line.

Thanks for the response! However it didn't work, the error hasn't changed

[Dr. Greg House]

Which game?
I didn't look closely enough but it could be an issue with hibernation.

[Bimbo1]

why don't you use a global db and do the connection once? also, printtoserver the errorstring of the connection to see easily what's going on.

[Pyro_]

Originally Posted by Bimbo1 why don't you use a global db and do the connection once? also, printtoserver the errorstring of the connection to see easily what's going on.

Thanks!