Problem mit /park befehl

John_Vendetta · 7. Januar 2017

Hallo Community,

Ich habe ein Problem mit meiem /park Befehl.

Wenn ich alle Fahrzeuge die in meinen Besitz sind entferne und mir dann ein neues erstelle kann ich es parken, aber wenn ich mir noch eins erstelle sagt er mir

"You don't have the keys for this vehicle"

C

ocmd:park(playerid,params[])
{
	if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid,COLOR_COMMENT,"You're not in any vehicle!");
	for(new i=0;i<sizeof(vInfo);i++)
	{
	    if(vInfo[i][id_x] == 0)continue;
	    if(vInfo[i][besitzer] == sInfo[playerid][p_ID])
	    {
	        if(!IsPlayerInVehicle(playerid,vInfo[i][id_x]))return SendClientMessage(playerid,COLOR_HRED,"You don't have the keys for this vehicle"); // hier ist wahrscheinlich der Fehler
	        new query[200],Float:Pos[4];
	        GetVehiclePos(vInfo[i][id_x],Pos[0],Pos[1],Pos[2]);
			GetVehicleZAngle(vInfo[i][id_x],Pos[3]);
			format(query,sizeof(query),"UPDATE autos SET x='%f',y='%f',z='%f',r='%f' WHERE id=%i",Pos[0],Pos[1],Pos[2],Pos[3],vInfo[i][db_id]);
			mysql_function_query(dbhandle,query,false,"","");
			SendClientMessage(playerid,COLOR_TURKEY,"From now on you'll find your car here");
			return 1;
	    }
	}
	return 1;
}

Alles anzeigen

Mein Befehl zum erstellen eines Fahrzeugs

C

ocmd:v(playerid,params[])
{
	if(!isAdmin(playerid,4))return SendClientMessage(playerid,COLOR_HRED,"You are not authorized to use this command!");
	new cID,pID;
	new Float:x, Float:y, Float:z, Float:za;
	GetPlayerPos(pID,x,y,z);
	GetPlayerFacingAngle(pID,za);
	if(sscanf(params,"ui",pID,cID)) return SendClientMessage(playerid,COLOR_DGREY,"Use: /v [playerid] [vehicle id]");
	if(cID<400||cID>611)return SendClientMessage(playerid,COLOR_YELLOW,"Vehicle ID isn't existing");
	createPlayerCar(pID,cID,x,y,z,za);
	PutPlayerInVehicle(playerid,cID,0);
	SendClientMessage(playerid,COLOR_MESSAGE,"Player vehicle successfully created!");
	return 1;
}

Alles anzeigen

Hier erstellt er dann das Auto

C

createPlayerCar(playerid,modelid,Float:x,Float:y,Float:z,Float:r)
{
	for(new i=0; i<sizeof(vInfo);i++)
	{
	    if(vInfo[i][id_x]!=0)continue;
	    vInfo[i][besitzer]=sInfo[playerid][p_ID];
	    vInfo[i][c_x]=x;
	    vInfo[i][c_y]=y;
	    vInfo[i][c_z]=z;
	    vInfo[i][c_r]=r;
		vInfo[i][model]=modelid;
	    vInfo[i][id_x] = CreateVehicle(modelid,x,y,z,r,-1,-1,-1);
	    saveCarToDB(playerid,i);
	    return 1;
	}
	return 1;
}

Alles anzeigen

Ich hoffe ihr könnt mir helfen.

MfG

JayWayy · 7. Januar 2017

Code

new Float:X,Float:Y,Float:Z;
new Float:z_rot;
GetPlayerPos(playerid, X,Y,Z);

Zim30 · 7. Januar 2017

DU fragst ab ob 'i' dem Spieler gehört und wenn ja soll er überprüfen ob er auch in diesem Fahrzeug sitzt.
Wenn nicht, dann soll er die Nachricht ausgeben und die Funktion beenden.

Dadurch prüft er aber immer nur DAS ERSTE Fahrzeug bei dem der Besitzer mit der Spielerid überein stimmt.

Das verhinderst du indem du die Nachricht einfach nach der For Schleife ausgibst und hinter :" if(!IsPlayerInAnyVehicle(playerid))"

ein "continue;" statt "return;" einsetzt

Code

ocmd:park(playerid,params[])
{
	if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid,COLOR_COMMENT,"You're not in any vehicle!");
	for(new i=0;i<sizeof(vInfo);i++)
	{
	    if(vInfo[i][id_x] == 0)continue;
	    if(vInfo[i][besitzer] == sInfo[playerid][p_ID])
	    {
	        if(!IsPlayerInVehicle(playerid,vInfo[i][id_x]))continue;
	        new query[200],Float:Pos[4];
	        GetVehiclePos(vInfo[i][id_x],Pos[0],Pos[1],Pos[2]);
			GetVehicleZAngle(vInfo[i][id_x],Pos[3]);
			format(query,sizeof(query),"UPDATE autos SET x='%f',y='%f',z='%f',r='%f' WHERE id=%i",Pos[0],Pos[1],Pos[2],Pos[3],vInfo[i][db_id]);
			mysql_function_query(dbhandle,query,false,"","");
			SendClientMessage(playerid,COLOR_TURKEY,"From now on you'll find your car here");
			return 1;
	    }
	}
	SendClientMessage(playerid,COLOR_HRED,"You don't have the keys for this vehicle");
	return 1;
}

Alles anzeigen

Johnny · 7. Januar 2017

Folgendes unter "/v" kann schonmal nicht funktionieren: PutPlayerInVehicle(playerid,cID,0);
Du versuchst den Spieler in ein Fahrzeug zu setzen, die das angegebene Modell als ID hat.

new cID, pID, vID;
[...]
vID = createPlayerCar(pID, cID, x, y, z, za);
PutPlayerInVehicle(playerid, vID, 0);

Wobei du deine erstellende Funktion auch ändern musst:

C

createPlayerCar(playerid,modelid,Float:x,Float:y,Float:z,Float:r)
{
	for(new i=0; i<sizeof(vInfo);i++)
	{
	    if(vInfo[i][id_x]!=0)continue;
	    vInfo[i][besitzer]=sInfo[playerid][p_ID];
	    vInfo[i][c_x]=x;
	    vInfo[i][c_y]=y;
	    vInfo[i][c_z]=z;
	    vInfo[i][c_r]=r;
		vInfo[i][model]=modelid;
	    vInfo[i][id_x] = CreateVehicle(modelid,x,y,z,r,-1,-1,-1);
	    saveCarToDB(playerid,i);
	    return vInfo[i][id_x];
	}
	return 0; //Hier Fehlermeldung -> alle Arrayblöcke in vInfo belegt
}

Alles anzeigen

Ansonsten sehe ich auf den ersten Blick nicht, wieso die Fehlermeldung auftaucht. //Edit: "return", wie es oben beschrieben ist ist falsch. Das hab ich übersehen.

John_Vendetta · 7. Januar 2017

Zitat von Zim30
DU fragst ab ob 'i' dem Spieler gehört und wenn ja soll er überprüfen ob er auch in diesem Fahrzeug sitzt.
Wenn nicht, dann soll er die Nachricht ausgeben und die Funktion beenden.

Dadurch prüft er aber immer nur DAS ERSTE Fahrzeug bei dem der Besitzer mit der Spielerid überein stimmt.

Das verhinderst du indem du die Nachricht einfach nach der For Schleife ausgibst und hinter :" if(!IsPlayerInAnyVehicle(playerid))"

ein "continue;" statt "return;" einsetzt
Code
ocmd:park(playerid,params[])
{
	if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid,COLOR_COMMENT,"You're not in any vehicle!");
	for(new i=0;i<sizeof(vInfo);i++)
	{
	    if(vInfo[i][id_x] == 0)continue;
	    if(vInfo[i][besitzer] == sInfo[playerid][p_ID])
	    {
	        if(!IsPlayerInVehicle(playerid,vInfo[i][id_x]))continue;
	        new query[200],Float:Pos[4];
	        GetVehiclePos(vInfo[i][id_x],Pos[0],Pos[1],Pos[2]);
			GetVehicleZAngle(vInfo[i][id_x],Pos[3]);
			format(query,sizeof(query),"UPDATE autos SET x='%f',y='%f',z='%f',r='%f' WHERE id=%i",Pos[0],Pos[1],Pos[2],Pos[3],vInfo[i][db_id]);
			mysql_function_query(dbhandle,query,false,"","");
			SendClientMessage(playerid,COLOR_TURKEY,"From now on you'll find your car here");
			return 1;
	    }
	}
	SendClientMessage(playerid,COLOR_HRED,"You don't have the keys for this vehicle");
	return 1;
}
Alles anzeigen
Alles anzeigen

Danke es funktioniert jetzt!

Gibt es hier irgendwo ein Tutorial wie man ein Fraktionsautohaus baut? Ich bin nicht so vertraut mit Filterscripts.

Habe schon ein Fraktions-System & Fraktions-Fahrzeug-System

Problem mit /park befehl

Ähnliche Themen

Contract funkt. nicht

/orten Befehl geht nicht.

Stats Problem

Textdraw anzeigen lassen

MSQL Speichert als "Null"