/afixveh Befehl?

Welche Errors kommen denn? Mit dem Code allein können wir nix anfangen.

if(strcmp(cmd, "/afixveh", true) == 0)
{
if(IsPlayerConnected(playerid))
{
if(PlayerInfo[playerid][pAdmin] < 4)
{
return SendClientMessage(playerid, COLOR_GRAD1, " Du darfst den Befehl nicht benutzen!");
}
if(IsPlayerInAnyVehicle(playerid))
{
SetVehicleHealth(GetPlayerVehicleID(playerid), 1000.0);
SendClientMessage(playerid, COLOR_GREY, " Auto repairt!");
}
else SendClientMessage(playerid, COLOR_GREY, " Du bist in keinem Auto!");
}
return 1;
}

1. Was kommen für Errors?
2. Zum Reparieren ist RepairVehicle besser geeignet. (Sehbarer Schaden wir dadurch auch repariert)
3. Der Befehl heißt bei dir /fixveh nicht /afixveh (Ist der Error, dass du Ingame /afixveh eingeben willst und nix passiert?)
4. Warum am Anfang abfragen, ob der Spieler Connected ist? Wenn er den Befehl eingibt, ist er auch Connected...

Mfg,
UncleSub

Mach mal :

if(PlayerInfo[playerid][pAdmin] >= 4)

PS : Markier die Zeile wo dir Errors sind

if(PlayerInfo[playerid][pAdmin] < 4)
Ist schon richtig..

Du hast anscheinend irgendwo eine Funktion, die ebenfalls SetVehicleHealth heißt (??).

Nutze stattdessen RepairVehicle, ist eh für deinen Befehl besser.

Mfg,
UncleSub

@[TVR]SkillerFX:

Hast du JunkBuster drin?

Das Ding verursacht gerne so Probleme. Hatte das auch mal, aber das ist schon zu lang her und jetzt nutze ich das System auch nicht mehr.

Wie bei den vorherigen Posts schon beschrieben, nimm lieber RepairVehicle . Sieht schöner aus

public OnPlayerCommand(playerid, cmdtext[])
{
if (!strcmp("/arepair", cmdtext))
{
// Ich glaube hier ist der Fehler. :)
if(PlayerInfo[playerid][pAdmin] < 4) // Aussage: Spieler unter Level 4
if(PlayerInfo[playerid][pAdmin] >= 4) // Aussage: Spieler gleich oder über Level 4
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, 0xFFFFFFFF, "Du bist in keinem Auto");
RepairVehicle(GetPlayerVehicleID(playerid));
SendClientMessage(playerid, 0xFFFFFFFF, "Auto repairt!");
return 1;
}
}
}

Entweder du machst:

if(PlayerInfo[playerid][pAdmin] < 4) return SendClientMessage(.....);

Oder:

if(PlayerInfo[playerid][pAdmin] >= 4)

Liegt bei dir.

Mfg,
UncleSub

//Edit: Housefrau war mal wieder schneller. Das Problem ist grade halt bei dir, dass der Befehl nur Ausgeführt wird, wenn du Adminlevel kleiner 4 bist..

ocmd:afixveh(playerid,params[])
{
if(!isPlayerAnAdmin(playerid,2))return SendClientMessage(playerid,COLOR_GRAU,"Du bist kein Admin!");//anpassen
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_GRAU, "Du musst in einem Fahrzeug sitzen!");
new vehicleID = GetPlayerVehicleID(playerid);
RepairVehicle(vehicleID);
SendClientMessage(playerid, COLOR_GRAU, "Du hast das Fahrzeug erfolgreich repariert!");
}
return 1;
}

Hier auch Einmal ohne ocmd.. hab mich verschaut.. dachte du arbeitest mit ocmd..

if(strcmp(cmdtext,"/afixveh",true)==0)
{
if(PlayerInfo[playerid][admin] >= 4)
{
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid, COLOR_GRAU, "Du musst in einem Fahrzeug sitzen!"); //Überprüfen ob er im Fahrzeug sitzt und wenn nicht Abbrechen und Fehlermeldung zeigen
new vehicleID = GetPlayerVehicleID(playerid); //VehicleID herausfinden
RepairVehicle(vehicleID); //Fahrzeug mit der VehicleID reparieren
SendClientMessage(playerid, COLOR_GRAU, "Du hast das Fahrzeug erfolgreich repariert!");
}
else return SendClientMessage(playerid,COLOR_GRAU,"Du brauchst Mindestend Adminlevel 4");
return 1;
}

Sowas liebe ich auch zu sehen ..

Es verwirrt ihn doch nur noch mehr wenn du plötzlich OCMD, statt das "normale" System nutzt!

isPlayerAnAdmin(...) gibts nicht. Anfänger können somit damit nichts anfangen.
Außerdem kann man auch ab und zu Sachen erklären. Nicht jeder wurde von Gott mit einer Hyperscriptinggabe erschaffen.

Mfg