komische fehler

Meisternuke · 1. September 2010

hey @ all...

hab n scriptschnipsel gefunden, welcher die cars automatisch repaired wenn die health unter 250 fällt..

jetzt bekomm ich 1 error:

Code

C:\DOKUME~1\Erik\Desktop\GODFAT~1\GAMEMO~1\ex.pwn(871) : error 076: syntax error in the expression, or invalid function call

also oben bei onplayerconnect:

SetTimerEx("Autorepair",500,true,"d",playerid);

dann bei onplayerdisconnect:

KillTimer(Autorepair); // hier liegt der error

und dann der eigentliche code:

forward Autorepair(playerid);
public Autorepair(playerid)
{
if(IsPlayerInAnyVehicle(playerid))
{
new Float:health,vehicleid;
vehicleid = GetPlayerVehicleID(playerid);
GetVehicleHealth(vehicleid,health);
if(health <250)
{
SetVehicleHealth(vehicleid,1000);
}
}
}

|-|auke · 1. September 2010

Wäre es nicht besser, einen Timer zu nehmen, der alle Autos einmal "durchschleift"?
Das wäre wohl Recourcen sparender.

Meisternuke · 1. September 2010

kannste mir zeigen wie das geht?

Grex · 1. September 2010

Würd ich nicht machen, man will ja nur für einen bestimmen User den Timer ausführen.
Das Problem ist, dass du dir das nicht richtig angeschaut hast:
http://wiki.sa-mp.com/wiki/KillTimer

Also daher musst du es so machen:
new timer = SetTimerEx("Autorepair",500,true,"d",playerid);

Und dann
KillTimer(timer);

Der Grund liegt darin, dass du nicht dem Funktionsnamen killst, sondern ja den Timer, und da du die Nummer speichern willst, musst du diese auf eine Variable speichern.

|-|auke · 1. September 2010

Ok:
Das kannst du oben in´s Script machen:
forward Autohealer();
Das kommt unter OnGameModeInit:
SetTimer("Autohealer",2000,1);
Wenn du im Script schon einen ähnlichen Timer hast, solltest du das bei dem einfügen
Dann kommt das hier nach ganz unten in´s Script

public Autohealer()
{
for(new i = 0; i < MAX_VEHICLES; i++)
{
new Float:carhealth;
GetVehicleHealth(i,carhealth);
if(carhealth<250)
{
SetVehicleHealth(i,1000);
}

}
}

Das sollte so gehen

Gruß Hauke

Grex · 1. September 2010

Zitat von |-|auke

Ok:
Das kannst du oben in´s Script machen:
forward Autohealer();
Das kommt unter OnGameModeInit:
SetTimer("Autohealer",2000,1);
Wenn du im Script schon einen ähnlichen Timer hast, solltest du das bei dem einfügen
Dann kommt das hier nach ganz unten in´s Script

public Autohealer()
{
for(new i = 0; i < MAX_VEHICLES; i++)
{
new Float:carhealth;
GetVehicleHealth(i,carhealth);
if(carhealth<250)
{
SetVehicleHealth(i,1000);
}

}
}

Das sollte so gehen

Gruß Hauke

Alles anzeigen

Das ist doch Unfug, er will doch nur das Auto von einem Spieler reparieren, und das nur, wenn die Funktion aufgerufen wird. Wie er den Timer gemacht hat, passt das doch. Ansonst würde von jedem das Auto repariert.

|-|auke · 1. September 2010

Achsooo...
Ich dachte es geht darum alle zu reparieren

Edit: Aber wozu soll das denn unter OnPlayerConnect?
Dann ist es doch das selbe wie meins. Außer, wenn er keine Fraktionsautos (oder was auch immer) heilen will...

Meisternuke · 1. September 2010

@ grex

jetzt kommen die errors:

Code

C:\DOKUME~1\Erik\Desktop\GODFAT~1\GAMEMO~1\ex.pwn(856) : warning 204: symbol is assigned a value that is never used: "timer"
C:\DOKUME~1\Erik\Desktop\GODFAT~1\GAMEMO~1\ex.pwn(872) : error 017: undefined symbol "timer"

bei onplayerconnect:

new timer = SetTimerEx("Autorepair",500,true,"d",playerid);

bei onplayerdisconnect:

KillTimer(timer);

...

1. error zu connect...

2. error zu disconnect

|-|auke · 1. September 2010

Versuch´s mal, wenn du ganz oben in´s Script new timer; machst.
Und das new bei SetTimer weglassen.

Meisternuke · 1. September 2010

jo bin selber druf gekommen xD

danke für die hilfe.. wenn was ich meld ich michnochmal.. ich test jetzt ma

Grex · 1. September 2010

Aso du machst das bei OnPlayer Connect Oo, dann passt schon Hauke seine Methode, würde aber das ganze mit playerid machen anstatt mit der VehicleID, und 500 MS sind zu oft, wäre 1000 MS schon besser.

Meisternuke · 1. September 2010

soll ichs dann lieber machen wir hauke es gemacht hat?

ps er hats nich mehr drin =P

|-|auke · 1. September 2010

Wäre ganz gut, wenn du uns sagst, was du genau damit vor hast.
Dann können wir dir sagen, was besser dazu ist
Wenn du allerdings alle Autos heilen möchtest, kannst du meine Variante nehmen.

Meisternuke · 1. September 2010

ich möchte einfach wenn die car health unter 250 fällt... das car auf 1000 gesetzt wird... also das das auto automatisch repaired wird wenn es eigentlich anfängt mit brennen

Grex · 1. September 2010

Ich würds so machen, wie oben bei OnGameModeInit() den Timer einfügen, (der mit 2 Sekunden) und das hier als Funktion nehmen:

public Autohealer()
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
new Float:carhealth;
GetVehicleHealth(GetPlayerVehicleID(i),carhealth);
if(carhealth<250)
{
SetVehicleHealth(GetPlayerVehicleID(i),1000);
}

}
}

EDIT: Anstatt carhealth < 250 sollte man carhealth <= 250 machen.

Meisternuke · 1. September 2010

habs ma grad getestet funktioniert auch klasse^^

aber mir gefällt des irgendwie net das die karosserie kaputt geht...

also...

wie kann ich machen das am besten die car health immer 1000 bleibt und der zustand der karosserie sich nicht verändert...

wenn das mit der karosserie nicht geht...

kann ich ja die carhealth von 250 auf 999 wechseln, damit repaired wird wenn die health auf unter 100 sinkt

Grex · 1. September 2010

Dann machst du:
public Autohealer()
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
SetVehicleHealth(GetPlayerVehicleID(i),1000);
}
}

EDIT:
Noch besser wäre:

public Autohealer()
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
IsPlayerInAnyVehicle(i)
{
SetVehicleHealth(GetPlayerVehicleID(i),1000);
}
}
}

EDIT 2: Schöner formatiert und Funktionsnamen richtig geschrieben.

Meisternuke · 1. September 2010

muss ich die timer drin lassen?
bzw den forward auch?

Grex · 1. September 2010

Ja, ansonst wird die Funktion ja nie ausgeführt.

Meisternuke · 1. September 2010

oki hat funktioniert...
danke an euch beide