Beiträge von Whice

Habs grade übersehen aber vor dem WHERE kommt kein Komma mehr

stock SaveUserStats(playerid)
{
	//Wenn der Spieler nicht eingeloggt ist, dann speichere seine Statistiken nicht
	if(!PlayerInfo[playerid][pLoggedIn]) return 1;
	//Ansonsten speichere sie
	new query[256];
	mysql_format(handle, query, sizeof(query), "UPDATE users SET level = '%d', money = '%d', kills = '%d', deaths = '%d', alevel = '%d'  WHERE id = '%d'",
	PlayerInfo[playerid][pLevel], PlayerInfo[playerid][pMoney], PlayerInfo[playerid][pKills], PlayerInfo[playerid][pDeaths], PlayerInfo[playerid][pAlevel], PlayerInfo[playerid][p_id]);
	mysql_pquery(handle, query);
	return 1;
}

Der Fehler liegt an deiner MySQL-Abfrage, die du zum Speichern nutzt. Die Variable pid muss an letzter Stelle der Parameter sein, da diese ja auch als letztes in dem MySQL-Befehl gebraucht wird.

stock SavePlayer(playerid)
{ 
	new query[256]; 
	mysql_format(db_handle, query, sizeof(query), "UPDATE accounts SET level = '%d', money = '%d', kills = '%d', deaths = '%d', admin = '%d', ip = '%d' WHERE id = '%d'", 
	PlayerInfo[playerid][plevel], PlayerInfo[playerid][pmoney], PlayerInfo[playerid][pkills], PlayerInfo[playerid][pdeaths], PlayerInfo[playerid][padmin], PlayerInfo[playerid][pip], PlayerInfo[playerid][pid]); 
	mysql_pquery(db_handle, query); 
	return 1;
}

Wenn du alleine auf dem Server bist, solltest du eigentlich ID 0 haben. Ansonsten zeig uns mal dein makeadmin-Command

// EDIT

Grad am Handy gewesen deswegen hab ich es voll übersehen, aber deine Parameter der SQL abfrage sind falsch gesetzt und irgendwie 2 zu viel (Franktion, Rank)

mysql_format(handle, query, sizeof(query), "UPDATE users SET level = '%d', money = '%d', kills = '%d', deaths = '%d', alevel = '%d',  WHERE id = '%d'",
	PlayerInfo[playerid][pLevel], PlayerInfo[playerid][pMoney], PlayerInfo[playerid][pKills], PlayerInfo[playerid][pDeaths], PlayerInfo[playerid][pAlevel], PlayerInfo[playerid][p_id]);

wichtig ist, dasss PlayerInfo[playerid][p_id] an letzter Stelle steht, da dies ja deine letzte Variable in der SQL-Abfrage ist. Wenn du also einen neuen Parameter hinzufügen möchtest, einfach zwischen pAlevel und p_id eintragen.

Zitat von Incepted

Funktioniert nun, allerdings entsteht doch ein Dauerspam - wie kann ich diesem entgehen?

Du könntest ganz einfach nen Sleep, 5000 oder so einbauen, also einfach unter das SendChat

#IfWinActive, GTA:SA:MP
#UseHook


#Include SAMP-API.ahk


SetTimer, AutoGate, 500


AutoGate:
IfWinNotActive, GTA:SA:MP
	return
if(IsPlayerDriver()) {
	if(IsPlayerInRangeOfPoint(1545.0774,-1627.2709,13.3828, 10)) {
		SendChat("/gate")
	}
}
return

So sollte es dann allerdings klappen

Heißt deine api denn wirklich "SAMP-API.ahk" und befindet sie sich im selben Ordner?

Hast du denn eine offizielle Version von GTA V und diese installiert?

Zitat von KinGRaPTiiC

Wenn er doch 2500$ abzieht ich dann Disconnecte wird es doch gespeichert aber ich verstehe auch nicht was du damit meinst...

Es steht doch in deinem Code, dass PlayerInfo[playerid][pMoney] gespeichert wird, somit musst du auch diese Variable setzten

Zitat von KinGRaPTiiC

Müsste ich das dann auch so beim Adminlevel machen und normales Lvl etc?

Du musst eigentlich immer die Variable setzten

Das ist auch alles nicht das Problem, wie gesagt du speicherst das nicht in der Variable, nur diese wird gespeichert.

ocmd:geld(playerid,params[])
{
	PlayerInfo[playerid][pMoney] = PlayerInfo[playerid][pMoney]+2500;
	GivePlayerMoney(playerid, -2500);
	return 1;
}

Zitat von KinGRaPTiiC

@Whice. Habe mir Ingame Adminrang 0 gegeben bin Disconnectet schaue in die DB und bin trotzdem Rang 3 und die 5000 sind auch nicht da gewesen

/edit Nun habe ich irgendwie die $5000 bekommen habe mir einen Befehl ins Script gemacht wo ich $2500 verliere bin nachdem ich den Befehl ausgeführt habe Disconnectet und bin wieder connectet und habe wieder $5000

Das ganze wirkt irgendwie so als ob du einen Befehl erstellt hast aber die Variablen nicht verwendet hast bzw. das gespeichert hast

Es könnte bei dir daran liegen, dass du es in der DB änderst, du danach den Server restartet (Er also automatisch deinen Account mit den Ingame Variablen speichert) und der Wert somit wieder der alte ist. Also entweder du disconnectest vorher vom Server oder erstellst einen Ingame Command dafür

Zitat von KinGRaPTiiC

Nur wieso habe ich die $5000 die ich in der Datenbank hinzugefügt habe

Verstehe irgendwie die Frage grade nicht. Wenn du 5000 in der DB hinzufügst dann hast du natürlich 5000 mehr

Wird das ganze denn auch ausgeführt? Gibt es überhaupt eine Verbindung zur Datenbank?

Ein paar Punkte darunter OnUserRegister bzw. wenn er schon registriert ist OnUserLogin, dort fügst du dann nur dein AdminLevel hinzu

Ok, wenn das ein Tutorial ist gehe das ganze doch nochmal durch (BSP: Hier)
Ich denke nicht, dass der Jeffry ein Tutorial macht ohne eine Funktion, die das ganze lädt.

Magst du uns mal zeigen wie du die Spieler lädst?

Meinste sowas?

ocmd:levelup(playerid, params[])
{
	new score, i;
	if(sscanf(params, "d", score))return SendClientMessage(playerid, -1, "Use: /levelup [Score]");
	for(i=0; i<MAX_PLAYERS; i++)
	{
	    SetPlayerScore(i, GetPlayerScore(i)+score);
	}
	return 1;
}

Zitat von SekLoso

Das letzte return 1 macht den Fehler

Ist nur ein Warning, da dieses return 1; nie erreicht werden kann.
Entweder es passiert der obere Teil, wenn er in der Nähe ist oder das else.

Somit kannst du das return 1; rausnehmen, du returnst ja in jedem Fall davor schon mal

Tipps: Scripting-Probleme richtig erklären

Du könntest zum Beispiel eine Variable in einer Datenbank oder in einer .ini Datei hochsetzten.
Dafür wäre es wichtig, dass du uns deine Methode nennst, die du nutzt um deine Accounts abzuspeichern und am besten die Version davon.