Lock Problem

TobiSagu · 29. Dezember 2015

Hallo liebe User,

nachdem ich gestern von Goldkiller einen sehr hilfreichen Tipp bekommen habe, und dieser soweit bis auf eines auch Funktioniert hat wende ich mich erneut an euch.

Ich habe den Befehl erneut bearbeitet und nun kann ich das Fahrzeug von außen wieder nicht öffnen. Warum habe ich es geändert ? if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1; hat mir gefehlt da andernfalls ja jeder das Fahrzeug auf und zu schließen kann.

Wie mache ich am schlausten weiter ?

Code

ocmd:fulock(playerid,params[])
{
	#pragma unused params
	new vehicleid;
	if(!IsPlayerInAnyVehicle(playerid))
	if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1;
	{
		vehicleid=GetNearestVehicle(playerid,6,true);
	}
	else
	{
	    vehicleid=GetPlayerVehicleID(playerid);
	}
	for(new i=1; i<sizeof(FraktAuto); i++)
	{
	   if(!IsPlayerInFrakt(playerid,1)&&!IsPlayerInFrakt(playerid,16)&&!IsPlayerInFrakt(playerid,6)&&!IsPlayerInFrakt(playerid,13)&&!IsPlayerInFrakt(playerid,12))return 1;
	    {
	        if(!isVehicleLocked(vehicleid))
	        {
	            lockVehicle(vehicleid);
			    GameTextForPlayer(playerid,"~w~Fahrzeug ~r~abgeschlossen",1500,3);
			    return 1;
	        }
	        else
	        {
	           unlockVehicle(vehicleid);
			   GameTextForPlayer(playerid,"~w~Fahrzeug ~g~aufgeschlossen",1500,3);
			   return 1;
	        }
	    }
	}
	return 1;
}

Alles anzeigen

Goldkiller · 29. Dezember 2015

Zitat von TobiSagu

if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return 1;

Du musst diese Abfrage machen, wenn der Spieler IM Fahrzeug ist. Aktuell hast du es so, dass die Abfrage kommt wenn er nicht im Fahrzeug ist. Wenn er nicht im Fahrzeug ist, ist er auch niemals der Fahrer. Daher ist die Abfrage immer wahr und die Funktion wird beendet.
Außerdem hast du die Klammer und das return komisch gesetzt. Das ist zum lesen verwirrent und ich bin mir ziemlich sicher, du möchtest nicht das,was da aktuell passiert.

Kurz gesagt:
Die Abfrage muss über die Zeile 12 bzw vor vehicleid=GetPlayerVehicleID(playerid); im Block kommen.

TobiSagu · 3. Januar 2016

Von außen und innen klappt nun nun nur die Frage, wie Frage ich genau ab welches Auto die jeweilige Fraktion öffnen darf ?

Also ist Player in der Fratkion vom Fraktionsfahrzeug

Code

ocmd:fulock(playerid,params[])
{
	#pragma unused params
	new vehicleid;
	if(!IsPlayerInAnyVehicle(playerid))


	{
		vehicleid=GetNearestVehicle(playerid,6,true);
	}
	else
	{
	    vehicleid=GetPlayerVehicleID(playerid);
	}
	for(new i=1; i<sizeof(FraktAuto); i++)
	{
	   if(!IsPlayerInFrakt(playerid,1)&&!IsPlayerInFrakt(playerid,16)&&!IsPlayerInFrakt(playerid,6)&&!IsPlayerInFrakt(playerid,13)&&!IsPlayerInFrakt(playerid,12))return 1;
	    {
	        if(!isVehicleLocked(vehicleid))
	        {
	            lockVehicle(vehicleid);
			    GameTextForPlayer(playerid,"~w~Fahrzeug ~r~abgeschlossen",1500,3);
			    return 1;
	        }
	        else
	        {
	           unlockVehicle(vehicleid);
			   GameTextForPlayer(playerid,"~w~Fahrzeug ~g~aufgeschlossen",1500,3);
			   return 1;
	        }
	    }
	}
	return 1;
}

Alles anzeigen

Jeffry · 3. Januar 2016

Wie weist du dem Fahrzeug die Fraktion zu? Beim erstellen? Poste gegebenenfalls mal den Code, mit dem du das machst.

TobiSagu · 3. Januar 2016

Ich erstelle das Fahrzeug mit folgenden Befehl

Code

ocmd:createfcar(playerid,params[])
{
    if(!DarfPlayerCommand(playerid))return ErrorMsg(playerid,"Du darfst keine Befehle nutzen.");
	if(!IsPlayerAnAdmin(playerid,5))return 1;
	new fID,vID,vName[32],farbe1,farbe2;
	if(sscanf(params,"is",fID,vName))return ErrorMsg(playerid,"INFO: /createfcar [fID][Fahrzeugname]");
	if(!IsNumeric(vName))
	{
		vID = GetVehicleModelIDFromName(vName);
	}
	else
	{
		vID = strval(vName);
	}
	if(vID<400||vID>611)return 1;
	if(fID>=30){if(!IsPlayerAnAdmin(playerid,6))return ErrorMsg(playerid,"Nicht möglich.");}
	if(vID == 537 || vID == 538 || vID == 569 || vID == 570 || vID == 590) return ErrorMsg(playerid, "Dieses Fahrzeug ist nicht erstellbar.");
	new i=getFreeFraktAuto();
    switch(fID)
    {
        case 1:{farbe1=53;farbe2=1;}
	    case 2:{farbe1=ffcarcolor1;farbe2=ffcarcolor2;}
	    case 3:{farbe1=6;farbe2=6;}
	    case 4:{farbe1=0;farbe2=0;}
	    case 5:{farbe1=86;farbe2=86;}
	    case 6:{farbe1=77;farbe2=77;}
	    case 7:{farbe1=6;farbe2=6;}
	    case 8:{farbe1=bikercarcolor1;farbe2=bikercarcolor2;}
	    case 9:{farbe1=ballascarcolor1;farbe2=ballascarcolor2;}
	    case 10:{farbe1=3;farbe2=3;}
	    case 11:{farbe1=2;farbe2=2;}
	    case 12:{farbe1=1;farbe2=3;}
	    case 13:{farbe1=1;farbe2=1;}
	    case 14:{farbe1=6;farbe2=6;}
	    case 15:{farbe1=hitmancarcolor1;farbe2=hitmancarcolor2;}
	    case 16:{farbe1=fbicarcolor1;farbe2=fbicarcolor2;}
	    case 17:{farbe1=kirchecarcolor1;farbe2=kirchecarcolor2;}
	    case 18:{farbe1=cncol1;farbe2=cncol2;}
    }
	new Float:bx,Float:by,Float:bz,Float:ba;
	GetPlayerPos(playerid,bx,by,bz);
	GetPlayerFacingAngle(playerid,ba);
	FraktAuto[i][v_xc] = bx;
	FraktAuto[i][v_yc] = by;
	FraktAuto[i][v_zc] = bz;
	FraktAuto[i][vfarbe1] = farbe1;
	FraktAuto[i][vfarbe2] = farbe2;
	FraktAuto[i][v_ac] = ba;


	format(query_,sizeof(query_),"INSERT INTO samp_fraktautos (besitzer,model,x,y,z,a,farbe1,farbe2,preis) VALUES (%d, %d, %f, %f, %f, %f, %d, %d, %d)",
	fID,
	vID,
	bx,
	by,
	bz,
	ba,
	farbe1,
	farbe2,
	1);
	mysql_query(query_);
    FraktAuto[i][id_x] = CreateVehicleEx(vID,bx,by,bz,ba,farbe1,farbe2,0);
	resetKofferraum(FraktAuto[i][id_x]);
	FraktAuto[i][besitzer]=fID;
    FraktAuto[i][preisx] = 1;
    FraktAuto[i][vmodel]=vID;
	FraktAuto[i][benzin]=100;
	FraktAuto[i][paintjob]=255;
	tank[FraktAuto[i][id_x]]=100;
    FraktAuto[i][d_id] = FactionVehicles+1;


	SendClientMessage(playerid,COLOR_GREEN,"SERVER: Ein Fraktionsfahrzeug wurde erfolgreich erstellt.");
	lockVehicle(FraktAuto[i][id_x]);
	istkaputt[FraktAuto[i][id_x]]=0;
	//DestroyDynamic3DTextLabel(autotext[FraktAuto[i][id_x]]);
	new Float:xc,Float:yc,Float:zc;
	GetVehiclePos(FraktAuto[i][id_x],xc,yc,zc);
	if(FraktAuto[i][besitzer]==13)
	{
	    format(string,sizeof(string),"Präsi");
	}
	else
	{
		format(string,sizeof(string),"%s-%d",getFCarOwner(fID),FraktAuto[i][id_x]);
	}
	SetVehicleNumberPlateEx(FraktAuto[i][id_x],string);
	//autotext[FraktAuto[i][id_x]]=CreateDynamic3DTextLabel(string,COLOR_BRIGHTRED,xc,yc,zc,25,INVALID_PLAYER_ID,FraktAuto[i][id_x],0,-1,-1,-1,25.0);
	format(autostring[FraktAuto[i][id_x]],128,string);
	PutPlayerInVehicle(playerid,FraktAuto[i][id_x],0);
	FactionVehicles ++;
	return 1;
}

Alles anzeigen

Jeffry · 3. Januar 2016

Dann kannst du es so machen:
ocmd:fulock(playerid)
{
if(!IsPlayerInFrakt(playerid,1)&&!IsPlayerInFrakt(playerid,16)&&!IsPlayerInFrakt(playerid,6)&&!IsPlayerInFrakt(playerid,13)&&!IsPlayerInFrakt(playerid,12))return 1;
new vehicleid;
if(!IsPlayerInAnyVehicle(playerid))
{
vehicleid=GetNearestVehicle(playerid,6,true);
}
else
{
vehicleid=GetPlayerVehicleID(playerid);
}
for(new i=1; i<sizeof(FraktAuto); i++)
{
if(vehicleid == FraktAuto[i][id_x])
{
if(!IsPlayerInFrakt(playerid,FraktAuto[i][besitzer]))
{
SendClientMessage(playerid, 0xFF0000FF, "Error: Du bist nicht in der Fraktion dieses Fahrzeugs.");
return 1;
}
if(!isVehicleLocked(vehicleid))
{
lockVehicle(vehicleid);
GameTextForPlayer(playerid,"~w~Fahrzeug ~r~abgeschlossen",1500,3);
}
else
{
unlockVehicle(vehicleid);
GameTextForPlayer(playerid,"~w~Fahrzeug ~g~aufgeschlossen",1500,3);
}
break;
}
}
return 1;
}

TobiSagu · 3. Januar 2016

Erledigt. Danke Jeffry

Lock Problem

breadfish.de 12. März 2022

Ähnliche Themen

[SAMMELTHREAD] Kleine Scripting Fragen

AHK | Kleines Problem

Privatautohaus Speicherung

Erneut Problem mit /lock Befehl

[Problem] bei /lock

/lock und befehl erstellen