Probleme mit Motorschaden Script...

einfach den Spieler freezen und dann eine Nachricht senden "Benutze /raus um das Fahrzeug zu verlassen" oder so

Es ist glaube ich nicht möglich, nur das Fahrzeug zu freezen.
Der Name der Funktion verrät es ja schon ein wenig "TogglePlayerControllable".

Dementsprechend muss der Code so aussehen (jedoch wird der Spieler gefreezt und nicht das Fahrzeug

new ms[MAX_VEHICLES]; // oben im Script einfügen

public OnPlayerUpdate(playerid)
{
new vid = GetPlayerVehicleID(playerid);
//----Motorschaden----
{
new Float:hxx;
GetVehicleHealth(vid, hxx);
if(!IsPlayerInAnyVehicle(playerid))return 1;
if(GetVehicleHealth(vid, hxx) <=300.0 && ms[vid] == 0)
{
TogglePlayerControllable(playerid, true);
ms[vid] = 1;
SendClientMessage(playerid,ROT,"Du hast einen Motorschaden!");
}
}
return 1;
}
Die Variable "ms" muss dann beim reparieren wieder auf 0 gesetzt werden.

- Ungetestet -

ist ja auch kein wunder -.-

du benutzt das public OnPlayerUpdate. ich denke mal das wird jede sekunde oder alle paar sekunden aufgerufen und immer dann passiert das.
außerdem haste da 2 unnötige abfrage sowie etwas was ich nicht so ganz verstehe. du fragst 2mal den status des wagens ab und dann fragste noch ob er in einem auto sitzt und wenn er NICHT in einem auto sitzt, return 1. das kapiere ich nicht. warum wird wenn er nicht in einem auto sitzt der vorgang fprtgesetzt?

das wird häftg laggen wenn du das so unter dem public auch noch hast.

hier die richtige variante:

public OnPlayerStateChange(playerid, newstate, oldstate)
{
if(newstate == PLAYER_STATE_DRIVER)
{
new vehicleid = GetPlayerVehicleID(playerid), Float:vhp;
if(GetVehicleHealth(vehicleid, vhp) <= 300)
{
TogglePlayerControllable(playerid,0);
SendClientMessage(playerid,0xFF0000FF,"Das Fahrzeug hat einen Motorschaden!");
}
}
return 1;
}

//edit: habe gerade gelesen das du dein auto freezen willst, das geht wenn überhaupt nur mit VeloCity glaube ich, damit kenne ich mich aber nicht aus.
man kann den spieler aus dem auto werfen.

mit velocity wäre es dann so wenn ich mich nicht irre:

new Float:vx, Float:vy, Float:vz;
SetVehicleVelocity(vehicleid, vx*0, vy*0, vz*0);

das machste einfach an stelle von TogglePlayerControllable(playerid,0);

das mit velocity klappt nicht damit wird nur dein fahrzeug gestoppt, du kannst aber danach weiter fahren.
sobald du in dein auto steigst, wird der schaden des wagens abgefragt. wenn der unter 300 ist, wirste gefreezed und es kommt die meldung.

du müsstes nun nen anderen command machen um auszusteigen, oder du machst es mit velocity. dann musst du bei OnPlayerKeyStateChange abfragen ob er im die tasten "w", "s", "d", "a" drückt, ob er im auto sitzt und ob der schaden unter 300 ist. wenn das alles zutrifft wieder die selbe funktion wie schon oben genannt mit velocity machen. so wird jedesmal wenn er losfahren will und der schaden unter 300 ist das auto gestoppt.

//edit: hier ich habe es mal für dich gemacht:

public OnPlayerKeyStateChange(playerid, newkeys, oldkeys)
{
if(newkeys == KEY_UP || newkeys == KEY_DOWN || newkeys == KEY_LEFT || newkeys == KEY_RIGHT)
{
if(IsPlayerInAnyVehicle(playerid))
{
new vehicleid = GetPlayerVehicleID(playerid), Float:vhp, Float:vx, Float:vy, Float:vz;
if(GetVehiclehealth(vehicleid, vhp) <= 300)
{
SetVehicleVelocity(vehicleid,vx*0,vy*0,vz*0);
}
}
}
return 1;
}

habs dir gemacht, siehe oben.

was genau geht nicht? ich denke mal das liegt daran das er überprüft ob man w,s,d,a drückt, wenn mans gedrückt hällt passiert nichts glaube ich.

Wenn man zusätzlich ein andere Taste drückt würde nach dem obigen Code nichts passieren - deswegen bitweisen Vergleich (&) verwenden anstelle von gesamten Vergleich (==).

if(newkeys & KEY_UP || newkeys & KEY_DOWN || newkeys & KEY_LEFT || newkeys & KEY_RIGHT || HOLDING( KEY_UP | KEY_DOWN | KEY_RIGHT | KEY_LEFT))

ich glaube so muss es, wobei ich mir nicht sicher bin ob man das HOLDING braucht und wenn mans braucht obs so richtig ist, aufjedenfall stehts so bei wiki.

#define HOLDING(%0) \ ((newkeys & (%0)) == (%0))

und dann müsste es auch so aussehen:

if(newkeys & KEY_UP || newkeys & KEY_DOWN || newkeys & KEY_LEFT || newkeys & KEY_RIGHT || HOLDING( KEY_UP) || HOLDING( KEY_DOWN) || HOLDING( KEY_RIGHT) ||HOLDING( KEY_LEFT))

glaube ich, ich habe es vorher noch nie so gemacht.

probiers mal so:

#define HOLDING(%0) \ ((newkeys & (%0)) == (%0))

if(newkeys & KEY_UP || newkeys & KEY_DOWN || newkeys & KEY_LEFT || newkeys & KEY_RIGHT || HOLDING(KEY_UP) || HOLDING(KEY_DOWN) || HOLDING(KEY_RIGHT) || HOLDING(KEY_LEFT))