Nachricht wird gespammt?

Guten Abend,

hab soweit mein Motorschaden System fertig. Doch er spammt die Nachricht jede Sekunde.

public Motorschaden(){
for(new i = 0; i < MAX_PLAYERS; i++){
if(!IsPlayerNPC(i)){
if(IsPlayerConnected(i)){
if(GetPlayerState(i) == PLAYER_STATE_DRIVER){
new veh,Float:vhp;
veh = GetPlayerVehicleID(i);
GetVehicleHealth(veh,vhp);
if(vhp < 350.0){
SendClientMessage(i,Rot,"Du hast einen Motorschaden.");
SendClientMessage(i,Rot,"Ruf den Abschleppdienst, so dass ein Mechaniker dein Auto reparieren kann.");
GetVehicleParamsEx(veh,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(veh,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
Engine[veh] = false;
break;
}
}
}
}
}
return 1;
}

SetTimer("Motorschaden",1000,1);
Jede Sekunde

Grad getestet..

Dann funktioniert der Timer nur 1x. Wenn man nochmal ein Auto kaputt macht funktioniert der Timer nicht mehr.

Bringt mir nix?

Ich möchte das er nicht die Nachricht spammt. Das muss doch einfacher gehen, oder nicht?
Oder muss ich SetTimerEx verwenden?

Zitat

Einfach den check NUR für die SendClientMessage machen?

Wie meinst du das?

Okay, und wie mach ich das?

Zitat von GreenG

PAWN Quelltext




1
2
3
4
5
6
7
8
9
10
11

new bool:messagesend[MAX_PLAYERS];

if(vhp < 350.0 && messagesend[i] == false)
{
SendClientMessage(i,Rot,"Du hast einen Motorschaden.");
SendClientMessage(i,Rot,"Ruf den Abschleppdienst, so dass ein Mechaniker dein Auto reparieren kann.");
GetVehicleParamsEx(veh,engine,lights,alarm,doors,bonnet,boot,objective);
SetVehicleParamsEx(veh,VEHICLE_PARAMS_OFF,lights,alarm,doors,bonnet,boot,objective);
messagesend[i] = true;
Engine[veh] = false;
}

So sollte es gehen, keine Ahnung wie man es anders lösen sollte.

Alles anzeigen

Weil damit geht das nur bei einem Auto..

Richtig, wenn ich es so mache wie bisher gilt der Timer nur für ein Auto..