AMXX.pl: Support AMX Mod X i SourceMod

[Save] Blad w zapytaniu: Commands out of sync; you can't run this command now

Wie ktoś co może oznaczać ten błąd ?

Hm, generalnie nie można wysłać dwóch zapytań jednocześnie, to oznacza ten błąd.

Prawdopodobnie masz gdzieś SQL_PrepareQuery, które nie zostało wysłane przez SQL_ExecuteQuery (tak się domyślam, bazując na Twoim wczorajszym poście).

                new qCommand[36000], Handle:hQuery, iError, szError[512], Handle:hConnection = SQL_Connect(hTuple, iError, szError, charsmax(szError));
		if(!hConnection)
		{
			log_amx("[Save] Nie mozna polaczyc sie z baza danych.");
			log_amx("[Save] Blad: %s", szError);
			SQL_FreeHandle(hConnection);
			return;
		}
		
		//...
		hQuery = SQL_PrepareQuery(hConnection, qCommand);
		
		if(!SQL_Execute(hQuery))
		{
			SQL_QueryError(hQuery, szError, charsmax(szError));
			log_amx("[Save] Blad w zapytaniu: %s", szError);
			SQL_FreeHandle(hConnection);
			return;
		}
		
		//...
		hQuery = SQL_PrepareQuery(hConnection, qCommand);
		if(!SQL_Execute(hQuery))
		{
			SQL_QueryError(hQuery, szError, charsmax(szError));
			log_amx("[Save] Blad w zapytaniu: %s", szError);
			SQL_FreeHandle(hConnection);
			return;
		}
                //...
		hQuery = SQL_PrepareQuery(hConnection, qCommand);
		if(!SQL_Execute(hQuery))
		{
			SQL_QueryError(hQuery, szError, charsmax(szError));
			log_amx("[Save] Blad w zapytaniu: %s", szError);
			SQL_FreeHandle(hConnection);
			return;
		}
		

Kopiuj + wklej z usunięciem formatex. Kod zatrzymuje się na ostatnim tutaj warunku który jednocześnie wykonuje zapytanie sql (warunek wykonuje się i na końcu zwraca return). Sporo czasu spędziłem nad tym fragmentem kodu ale nic nie widzę. Wszędzie gdzie jest SQL_PrepareQuery jest również po nim SQL_Execute. Co prawda w warunku ale to raczej nie ma znaczenia. Również nigdzie nie zapomniałem o SQL_FreeHandle. Wydaje mi się że wszystko jest dobrze, no jednak trzecie zapytanie (i nie ostatnie) kończy się błędem.

To co zwraca SQL_PrepareQuery też trzeba zwalniać tak jak to napisane w dokumentacji:

/**
 * Prepares a query.
 * 
 * @note This does not actually do a query!
 * 
 * @param db            Connection handle, returned from SQL_Connect().
 * @param fmt           Query string. Can be formated with format specifiers.
 * @param ...           Additional format specifiers used to format the query.
 * 
 * @return              Returns an SQL query handle, which must always be freed.
 *                      Returns Empty_Handle on failure.
 */
native Handle:SQL_PrepareQuery(Handle:db, const fmt[], any:...);

W kodzie tego nie widać. Trzeba zwolnić to zarówno w bloku, który obsługuje błąd jak i poza nim.

Warto dodać, że zamiast robić tak długich bloków kodu, które robią praktycznie to samo, możesz użyć funkcji, której tylko podasz query. Jeśli się nie myle to w sqlx.inc jest coś takiego.

/**
 * Use this for executing a query where you don't care about the result.
 *
 * @param db            Connection handle returned from SQL_Connect().
 * @param query         The query string.
 * @param error         If an error occurs, it will be placed into this buffer.
 * @param maxlength     Maximum length of the error buffer.
 * @param rows          Optional. If put, retrieves the number of rows the query returned.
 * 
 * @return              1 on success, 0 on failure.
 */
stock SQL_SimpleQuery(Handle:db, const query[], error[]="", maxlength=0, &rows=0)
{
	new Handle:hQuery = SQL_PrepareQuery(db, "%s", query);
	
	if (!SQL_Execute(hQuery))
	{
		SQL_QueryError(hQuery, error, maxlength);
		SQL_FreeHandle(hQuery);
		return 0;
	}
	
	rows = SQL_NumResults(hQuery);
	
	SQL_FreeHandle(hQuery);
	
	return 1;
}


/**
 * Use this for executing a query where you don't care about the result.
 * 
 * @note Differs from SQL_SimpleQuery() because the query can be formated.
 *
 * @param db            Connection handle returned from SQL_Connect().
 * @param error         If an error occurs, it will be placed into this buffer.
 * @param maxlength     The maximum length of the error buffer.
 * @param rows          Optional. If put, retrieves the number of rows the query returned.
 * @param fmt           The query string that can be formated with format specifiers.
 * @param ...           Additional arguments for formating the query.
 * 
 * @return              1 on success, 0 on failure.
 */
stock SQL_SimpleQueryFmt(Handle:db, error[]="", maxlength=0, &rows=0, const fmt[], any:...)
{
	static query_buf[2048];
	vformat(query_buf, 2047, fmt, 6);
	
	new Handle:hQuery = SQL_PrepareQuery(db, "%s", query_buf);
	
	if (!SQL_Execute(hQuery))
	{
		SQL_QueryError(hQuery, error, maxlength);
		SQL_FreeHandle(hQuery);
		return 0;
	}
	
	rows = SQL_NumResults(hQuery);
	
	SQL_FreeHandle(hQuery);
	
	return 1;
}

/**
 * Use this for executing a query and not caring about the error.
 *
 * @param db            A connection handle returned from SQL_Connect().
 * @param queryfmt      The query string that can be formated with format specifiers.
 * @pram ...            Additional arguments for formating the query.
 * 
 * @return              1 on error.
 *                      >= 0 on success (with the number of affected rows).
 */
stock SQL_QueryAndIgnore(Handle:db, const queryfmt[], any:...)
{
	static query[4096];
	new Handle:hQuery;
	new ret;
	
	vformat(query, sizeof(query)-1, queryfmt, 3);
	
	hQuery = SQL_PrepareQuery(db, "%s", query);
	
	if (SQL_Execute(hQuery))
	{
		ret = SQL_AffectedRows(hQuery);
	} else {
		ret = -1;
	}
	
	SQL_FreeHandle(hQuery);
	
	return ret;
} 

Also, jestem ciekaw jak długie zapytania masz, że aż taki bufor potrzebujesz

new qCommand[36000]

 

To co zwraca SQL_PrepareQuery też trzeba zwalniać tak jak to napisane w dokumentacji:

/**
 * Prepares a query.
 * 
 * @note This does not actually do a query!
 * 
 * @param db            Connection handle, returned from SQL_Connect().
 * @param fmt           Query string. Can be formated with format specifiers.
 * @param ...           Additional format specifiers used to format the query.
 * 
 * @return              Returns an SQL query handle, which must always be freed.
 *                      Returns Empty_Handle on failure.
 */
native Handle:SQL_PrepareQuery(Handle:db, const fmt[], any:...);

W kodzie tego nie widać. Trzeba zwolnić to zarówno w bloku, który obsługuje błąd jak i poza nim.

Już rozwiązałem ten problem, zmieniłem zapytanie na którym zatrzymywał się plugin, teoretycznie było to kilka zapytań w jednym SQL_PrepareQuery. Nie wiedziałem że to również powoduje taki błąd. Tak szczerze mówiąc bez żadnego SQL_FreeHandle wszystko działa, ale oczywiście zwolnię tym wszystkie uchwyty.

Also, jestem ciekaw jak długie zapytania masz, że aż taki bufor potrzebujesz

new qCommand[36000]

Tak długi potrzebny mi był przy odczytywaniu danych z bazy, chociaż chyba rzeczywiście trochę przesadziłem. Przy zapisie czego tyczył się ten temat najdłuższe zapytanie miało jakieś 3300 znaków, ale widocznie odruchowo napisałem 36000. I chyba powinienem to zmniejszyć bo to raczej nie wpływa dobrze na wydajność jeśli zmienna jest 10 razy większa niż powinna.