ocmd: Medic Heal-Befehl fail !

Zitat von NicoAiko

alles falsch

ocmd:heal(playerid,params[])
{
for(new i = 0; i < sizeof(MedicCars); i++) //Statt MedicCars die Variable nehmen die du für die Krankenhaus Fahrzeuge genommen hast
{
if(!IsAMedicCar(i) && i == sizeof(MedicCars)-1) //Ungetestet!
return SendClientMessage(playerid,rot,"Du bist in keinem Krankenhaus-Fahrzeug!");
new pID, string[128], money, namepID[MAX_PLAYER_NAME], nameplayerid[MAX_PLAYER_NAME];
if(!isPlayerInFrakt(playerid,2))return 0;
if(sscanf(params,"ui", pID, money))return SendClientMessage(playerid, Hellblau, "Benutze: /heal [Spieler-ID] [Preis]");
if(GetPlayerMoney(pID) < money)return SendClientMessage(playerid, Gruen, "Der Spieler hat nicht genügend Geld");
SetPlayerHealth(pID, 100);
GivePlayerMoney(playerid, money);
GivePlayerMoney(pID, -money);
GetPlayerName(pID, namepID, MAX_PLAYER_NAME);
GetPlayerName(playerid, nameplayerid, MAX_PLAYER_NAME);
format(string, sizeof(string), "Du hast %s für $%d geheilt.", namepID, money);
SendClientMessage(playerid, Orange, string);
format(string, sizeof(string), "Du wurdest von Medic %s für %d$ geheilt.", nameplayerid, money);
SendClientMessage(pID, Orange, string);
return 1;
}
return 1;
}

Alles anzeigen

Dies Frisst aber mehr Resourcen da du ja die variable durch gehst.
Bei mir nimmt er die die er grade Benutz.

Schick mal dein new medicar da.

Steht irgendwo im Script new medicar[ANZAHL]; und auch unter Ongamemodeinit mediccar[0] = Add.... ?

Dann muss das so

new medicauto[2];

public IsAMedicCar(carid)
{
if(carid >= medicauto[0] && carid <= medicauto[2])
{
return 1;
}
return 0;
}

oder so bin mir nicht Sicher.

new medicauto[3];

public IsAMedicCar(carid)
{
if(carid >= medicauto[0] && carid <= medicauto[2])
{
return 1;
}
return 0;
}

Zitat von DJ D!_Deagle

ich hab insgesamt 15 Mediccars.
So muss er new medicauto[14];

definen !

Hast nur 3 Angezeigt also hab ich es nicht gedacht

new medicauto[14];

public IsAMedicCar(carid)
{
if(carid >= medicauto[0] && carid <= medicauto[13])
{
return 1;
}
return 0;
}
oder

new medicauto[15];

public IsAMedicCar(carid)
{
if(carid >= medicauto[0] && carid <= medicauto[14])
{
return 1;
}
return 0;
}

Zitat von DJ D!_Deagle

ah, danke dir...

Alter Mann(naja bin 13) am Verzweifeln

hast du es hinbekommen ?