Auto Rausschmiss problem. [WICHTIG]

HunterBoY998 · 21. Januar 2013

Hallo,
wenn man kein SAPD Beamter ist kann man trz. mit dem Auto fahren.

Code:

public OnPlayerStateChange(playerid, newstate, oldstate)
{ new newcar = GetPlayerVehicleID(playerid);
if(IsAMuelCar(newcar))
{
if(GetPVarInt(playerid,"Job") == 0)
{
SendClientMessage(playerid,rot,"Du bist kein Müllfahrer!");
RemovePlayerFromVehicle(playerid);
return 1;
}
}
if(IsAPoliceCar(newcar))
{
if(GetPVarInt(playerid,"Fraktion") == 0)
{
SendClientMessage(playerid,rot,"Du bist kein Mitglied des SAPD!");
RemovePlayerFromVehicle(playerid);
return 1;
}
}
return 1;
}

Nochmal zum vergleich habe ich das von dem Müllabfuhrwagen auch reingemacht weil da klappt es.

MFG

Kaliber · 21. Januar 2013

Schreibe das mal so:

public OnPlayerStateChange(playerid, newstate, oldstate)
{
new newcar = GetPlayerVehicleID(playerid);
if(newstate == PLAYER_STATE_DRIVER)
{
if(IsAMuelCar(newcar))
{
if(GetPVarInt(playerid,"Job") == 0)
{
SendClientMessage(playerid,rot,"Du bist kein Müllfahrer!");
RemovePlayerFromVehicle(playerid);
}
}
if(IsAPoliceCar(newcar))
{
if(GetPVarInt(playerid,"Fraktion") != sapdid) //sapid durch die Fraktionsid vom sapd ersetzen
{
SendClientMessage(playerid,rot,"Du bist kein Mitglied des SAPD!");
RemovePlayerFromVehicle(playerid);
}
}
}
return 1;
}

mfg.

HunterBoY998 · 21. Januar 2013

Geht immer noch nicht

Kaliber · 21. Januar 2013

Ja...:

if(GetPVarInt(playerid,"Fraktion") != sapdid) //sapid durch die Fraktionsid vom sapd ersetzen

mfg.

HunterBoY998 · 21. Januar 2013

Geht nicht

Kaliber · 21. Januar 2013

Zitat von HunterBoY998

Geht nicht

Muss gehen

Was hast du denn hingeschrieben

HunterBoY998 · 21. Januar 2013

if(GetPVarInt(playerid,"Fraktion") != 0)
{
SendClientMessage(playerid,rot,"Du bist kein Mitglied des SAPD!");
RemovePlayerFromVehicle(playerid);
}

Kaliber · 21. Januar 2013

Zitat von HunterBoY998

if(GetPVarInt(playerid,"Fraktion") != 0)

Ja, 0 ist doch nicht die Fraktionsid vom Sapd

HunterBoY998 · 21. Januar 2013

Aber das ist unlogisch bei Müllabfuhr gehts ja auch weil wenn man Fraktion 0 hat soll auch kommen " Du bist kein Mitglied des SAAPD. "

MFG

Kaliber · 21. Januar 2013

Wie sieht die Funktion IsAPoliceCar(newcar) aus ?

PS: Es ist nicht unlogisch, schreibs mal einfach (bin zu faul diese if-Abfrage jetzt ausführlich zu erkären...)

mfg.^^

HunterBoY998 · 21. Januar 2013

forward IsAPoliceCar(carid);
public IsAPoliceCar(carid)
{
new modelid = GetVehicleModel(carid);
for(new i = 0; i < sizeof(PoliceCar); i++)
{
if(modelid == PoliceCar[i]) return 1;
}
return 0;
}

Kaliber · 21. Januar 2013

Okay und du bist sicher, dass du bei den Cars das so gemacht hast:

PoliceCar[0] = CreateVehicle(....);
PoliceCar[1] = CreateVehicle(....);
PoliceCar[2] = CreateVehicle(....);
PoliceCar[3] = CreateVehicle(....);
//Usw...

und anstatt der 0 hast du jetzt eine andere Zahl eingesetzt ?

Wenn ja, dann probiere das nochmal so aus

HunterBoY998 · 21. Januar 2013

new PoliceCar[33];

PoliceCar[0] = AddStaticVehicle(427,1528.9585,-1688.1089,6.0220,270.1985,0,1); // lspdcar 1
PoliceCar[1] = AddStaticVehicle(427,1528.9513,-1683.9355,6.0226,269.5105,0,1); // lspdcar 2
PoliceCar[2] = AddStaticVehicle(528,1538.7559,-1645.2733,5.9347,180.5644,0,0); // lspdcar 3
PoliceCar[3] = AddStaticVehicle(528,1534.6256,-1645.2428,5.9345,179.7258,0,0); // lspdcar 4
PoliceCar[4] = AddStaticVehicle(528,1530.4272,-1645.2274,5.9343,180.0705,0,0); // lspdcar 5
PoliceCar[5] = AddStaticVehicle(528,1526.5116,-1645.2708,5.9344,180.4315,0,0); // lspdcar 6
PoliceCar[6] = AddStaticVehicle(596,1558.8679,-1711.0403,5.6108,0.3289,0,1); // lspdcar 7
PoliceCar[7] = AddStaticVehicle(596,1562.5826,-1710.9552,5.6119,0.3211,0,1); // lspdcar 8
PoliceCar[8] = AddStaticVehicle(596,1566.0231,-1710.8680,5.6108,1.2654,0,1); // lspdcar 9
PoliceCar[9] = AddStaticVehicle(596,1570.4457,-1710.6808,5.6110,1.1276,0,1); // lspdcar 10
PoliceCar[10] = AddStaticVehicle(596,1574.5294,-1710.5968,5.6121,1.1661,0,1); // lspdcar 11
Bis 32 noch...

Kaliber · 21. Januar 2013

und funktioniert es

PS: Hast du die if-Abfrage angepasst...schick mal den Code davon

HunterBoY998 · 21. Januar 2013

Wie meinst du das?

Kaliber · 21. Januar 2013

Zitat von HunterBoY998

Wie meinst du das?

Also:

if(GetPVarInt(playerid,"Fraktion") != 0)
Das ist falsch, da 0 nicht der fraktionsid vom SAPD entspricht !

So könnte es richtig sein:

if(GetPVarInt(playerid,"Fraktion") != 5)
Sprich 5 = Fraktionsid vom SAPD

mfg.

HunterBoY998 · 21. Januar 2013

SAPD ist aber 1

Kaliber · 21. Januar 2013

Zitat von HunterBoY998

SAPD ist aber 1

Omg...bitte lese meine Texte richtig

So, da SAPD 1 ist schreibe als abfrage es so:

if(GetPVarInt(playerid,"Fraktion") != 1)
und jetzt müsste das funktionieren, wenn du die Fahrzeuge fahren kannst, bist du entweder im SAPD oder du bist in einem anderen Fahrzeug

mfg.

HunterBoY998 · 21. Januar 2013

Ja aber man wird trz. nicht rausgeschmissen!

Kaliber · 21. Januar 2013

Zitat von HunterBoY998

Ja aber man wird trz. nicht rausgeschmissen!

Omg, lese:

Zitat von Kaliber

wenn du die Fahrzeuge fahren kannst, bist du entweder im SAPD oder du bist in einem anderen Fahrzeug