ocmd: Medic Heal-Befehl fail !

DJ Deagle · 8. Dezember 2011

Hey Leute,

beim Compilen von dem hier, bekomme ich immer 1 Error:

// Medic Heal
ocmd:heal(playerid,params[])
{
if(IsPlayerInVehicle(playerid,medicauto))//Zeile 435
{
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, Grün, "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;
}

Der Error:
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(435) : error 035: argument type mismatch (argument 2)
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Was ist Falsch ?

Buschmensch · 8. Dezember 2011

Du musst "medicauto" definieren.

//Edit : Ich glaube mal new medicauto[MAX_PLAYER_VEHICLES];
Aber das mit den MAX_PLAYER_VEHICLES ist wahrscheinlich falsch, musst du herausfinden.
Mir fällt die Definition grad nicht ein...

DJ Deagle · 8. Dezember 2011

hab ich auch...

new medicauto[14]; (14 Autos sinds insgesamt)

Buschmensch · 8. Dezember 2011

Neeein
Du musst bei deinem Command wo deine ganzen new's sind auch
new pID, string[128], money, namepID[MAX_PLAYER_NAME], nameplayerid[MAX_PLAYER_NAME], medicauto[MAX_VEHICLES];
machen.

Aber mit dem [MAX_VEHICLES]; bin ich mir nicht sicher, musst raus finden.

// Edit : Wenn du 14 Autos hast musst du bei new medicauto[14]
anstatt 14
new medicauto[15]
machen.

Weil man immer von 0 anfängt zu zählen.

DJ Deagle · 8. Dezember 2011

davon funkt garnichts!

Ausserdem hab ich von 0 angefangen !

0
1
2
usw

Buschmensch · 8. Dezember 2011

Schick mir mal die Errormeldung wenn du diesen new medicauto[MAX_VEHICLES];
machst.

DJ Deagle · 8. Dezember 2011

C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(437) : warning 219: local variable "medicauto" shadows a variable at a preceding level
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(437) : warning 204: symbol is assigned a value that is never used: "medicauto"
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Buschmensch · 8. Dezember 2011

ocmd:heal(playerid,params[])
{
new medicauto[MAX_VEHICLES];
if(IsPlayerInVehicle(playerid,medicauto))
{
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;
}

Also so müsste es gehen, bei dir stimmt irgendwas in der if(IsPlayerInVehicle(playerid,medicauto)) nicht!

DJ Deagle · 8. Dezember 2011

C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(434) : warning 219: local variable "medicauto" shadows a variable at a preceding level
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Warning.

Diese Zeile:
new medicauto[MAX_VEHICLES];

SrBiN15 · 8. Dezember 2011

Du kannst eine Variabel nicht zwei mal definieren -.-
du hast sie ja schon global definiert das reicht auch, das ist völliger Quatsch, das du die Variable im Command nochmal definierst.

MfG, SrBiN15

DJ Deagle · 8. Dezember 2011

Ja, aber wenn ich nur das hier hab:

if(IsPlayerInVehicle(playerid,medicauto[0]

dann geht es ja NUR in einem Medicauto...

SrBiN15 · 8. Dezember 2011

Wie wäre es wenn du eine Schleife drum herum machst?! Dass wäre die einzige Lösung für diese Situation. Man kannst natürlich anders und viel schöner scripten.

MfG, SrBiN15

DJ Deagle · 8. Dezember 2011

Kann man das ned irgendwie anders machen ?
Das ich z.B. sage if(IsPlayerInVehicle(playerid,medicauto[0][1][2]usw...

?

SrBiN15 · 8. Dezember 2011

lol

Du kannst natürlich ein extra public machen das abfragt ob das Auto ein Mediccar ist. Aber streng mal dein Köpfchen an dann schaffst du es auch.

DJ Deagle · 8. Dezember 2011

-.- ein Hau(b)ptschüler soll sein Käpfchen anstregen...
2. Sitzen geblieben...

Ich bin zu blöd dafür - Wirklich !
das einzigste was ich kann si english (ohne Google Übersetzer ^^)

GamerXIOS · 8. Dezember 2011

mach es so:

new medicauto[ANZAHL];
forward IsAMedicCar(carid);

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

ocmd:heal(playerid,params[])
{
new carid= GetPlayerVehicleID(playerid);
if(medicauto(carid))
{
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;
}

DJ Deagle · 8. Dezember 2011

C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(378) : error 032: array index out of bounds (variable "medicauto")
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(469) : error 012: invalid function call, not a valid address
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(469) : warning 215: expression has no effect
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(469) : error 001: expected token: ";", but found ")"
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(469) : error 029: invalid expression, assumed zero
C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(469) : fatal error 107: too many error messages on one line

5 Errors.

/edit

Zeile 469
if(medicauto(carid))

GamerXIOS · 8. Dezember 2011

ups sry so

ocmd:heal(playerid,params[])
{
new carid= GetPlayerVehicleID(playerid);
if(IsAMedicCar(carid))
{
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;
}

DJ Deagle · 8. Dezember 2011

C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(378) : error 032: array index out of bounds (variable "medicauto")
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Hier die Zeile:
if(carid >= medicauto[0] && carid <= medicauto[14])

GamerXIOS · 8. Dezember 2011

Zitat von DJ D!_Deagle

C:\Users\Joshua.haus-pc\Desktop\SAMP Server\gamemodes\LV-RP.pwn(378) : error 032: array index out of bounds (variable "medicauto")
Pawn compiler 3.2.3664 Copyright (c) 1997-2006, ITB CompuPhase

1 Error.

Hier die Zeile:
if(carid >= medicauto[0] && carid <= medicauto[14])

Alles anzeigen

if(carid >= medicauto[0] && carid <= medicauto[15]) vllt so