Leider nicht aber wie oben schon beschrieben - schau selber erstmal durch das auskommentieren des halben Codes in dem Callback, wo der Fehler genau liegt.
So siehst du den Fehler am schnellsten und falls du ihn am Ende nicht direkt siehst, kann du den Codeabschnitt immernoch hier reinposten.
Der Fehler wurde heute gelöst.
unter onplayerdissconect bei resetplayervriabels hat ein argument gefehlt.
Guten Abend,
ich versuch mich mal wieder in SAMP und bin auf folgendes Problem gestoßen.
ocmd:kick(playerid,params[])
{
if(AccountInfo[playerid][account_Admin] > 0)
{
new pID,string[128],string1[128],string2[128],grund[64];
if(sscanf(params, "us[128]", pID,grund)) return SendBenutzung(playerid, "/kick [ID/Name] [Grund]");
if(!CharacterInfo[pID][character_Eingeloggt])return SendFehler(playerid,"Dieser Spieler ist nicht eingeloggt");
format(string, sizeof(string),"%s wurde von %s %s gekickt. Grund: %s",GetAdminRang(AccountInfo[playerid][account_Admin],false),GetFarbName(playerid),grund);
SendClientMessageToAll(ORANGE,string);
format(string1, sizeof(string1),"Du wurdest von %s gekickt. Grund: %s",GetFarbName(pID),grund);
SendInfo(pID,string1);
printf("Kicknachricht: %s", string1);
SetTimerEx("DelayedKick", 10000, false, "i", pID);
format(string2, sizeof(string2),"Du hast %s gekickt. Grund: %s",GetFarbName(pID),grund);
SendInfo(playerid,string2);
Kick(pID);
}
return 1;
}
FUNCTION DelayedKick(pID)
{
Kick(pID);
return 1;
}Der Nachricht "Du wurdest von ... gekickt" kommt beim Spieler nicht an, woran kann das liegen?
Vielen Dank im Voraus.
Mit freundlichen Grüßen
Allrounder18
woran kann das liegen?
Das wird daran liegen, dass du trotz deines DelayedKick Timers die Kick-Funktion in Zeile 16 aufrufst. Einfach die entfernen.
Das wird daran liegen, dass du trotz deines DelayedKick Timers die Kick-Funktion in Zeile 16 aufrufst. Einfach die entfernen.
Danke hat geklappt, jetzt kommt nen anderer Fehler.
ocmd:kick(playerid,params[])
{
if(AccountInfo[playerid][account_Admin] > 0)
{
new pID,string[256],string1[256],string2[256],grund[64];
if(sscanf(params, "us[256]", pID,grund)) return SendBenutzung(playerid, "/kick [ID/Name] [Grund]");
if(!CharacterInfo[pID][character_Eingeloggt])return SendFehler(playerid,"Dieser Spieler ist nicht eingeloggt");
format(string, sizeof(string),"%s wurde von %s %s gekickt. Grund: %s",GetFarbName(playerid), GetAdminRang(AccountInfo[playerid][account_Admin],true),GetFarbName(playerid),grund);
SendInfoToAll(string);
format(string1, sizeof(string1),"Du wurdest von %s %s gekickt. Grund: %s",GetAdminRang(AccountInfo[pID][account_Admin],true),GetFarbName(pID),grund);
SendInfo(pID,string1);
printf("Kicknachricht: %s", string);
SetTimerEx("DelayedKick", 1000, false, "i", pID);
format(string2, sizeof(string2),"Du hast %s gekickt. Grund: %s",GetFarbName(pID),grund);
SendInfo(playerid,string2);
}
return 1;
}
FUNCTION DelayedKick(pID)
{
Kick(pID);
return 1;
}
stock GetAdminRang(adminlevel,bool:colored = false)
{
new rang[255];
switch(adminlevel)
{
case 0: {//Spieler
if(!colored) format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang0]);
else format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang0]);
}
case 1: {//Guide
if(!colored) format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang1]);
else format(rang, sizeof(rang), "{00AB1A}%s{FFFFFF}",AdminInfo[0][admin_Rang1]);
}
case 2: {//Administrator
if(!colored) format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang2]);
else format(rang, sizeof(rang), "{2E9AFE}%s{FFFFFF}",AdminInfo[0][admin_Rang2]);
}
case 3: {//Entwickler
if(!colored) format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang3]);
else format(rang, sizeof(rang), "{d225ba}%s{FFFFFF}",AdminInfo[0][admin_Rang3]);
}
case 4: {//Projektleiter
if(!colored) format(rang, sizeof(rang), "%s",AdminInfo[0][admin_Rang4]);
else format(rang, sizeof(rang), "{FF0000}%s{FFFFFF}",AdminInfo[0][admin_Rang4]);
}
}
return rang;
}Wenn ich in der Zeile "... wurde von ... vom Server gekickt" true anstatt false eingebe erscheint die Message nicht, woran kann das liegen?
Vielen Dank im Voraus.
Mit freundlichen Grüßen
Allrounder18
In Zeile 10 muss das playerid und nicht pID sein.
Zum Anderen musst evtl mal print-Debuggen, welchen Wert AccountInfo[playerid][account_Admin] hat und welchen Wert der dazugehörige AdminInfo-AdminRang hat.
In Zeile 10 muss das playerid und nicht pID sein.
Zum Anderen musst evtl mal print-Debuggen, welchen Wert AccountInfo[playerid][account_Admin] hat und welchen Wert der dazugehörige AdminInfo-AdminRang hat.
Danke, ich hab das Gefühl es lieg an folgendem Code.
stock SendAdminInfo(pID,farbe, string[])
{
new message[500];
if(AccountInfo[pID][account_Admin] > 0)
{
format(message, sizeof(message), "{40FF00}ADMININFO:{FFFFFF} %s.", string);
}
SendClientMessage(pID, farbe, message);
return 1;
}
SendAdminInfo(pID,WEISS,string);Edit: Der Print sieht so aus: Kicknachricht: {FF0000}Projektleiter{FFFFFF}{FF0000} Allrounder18(0){FFFFFF} hat {FF0000}Allrounder18(0){FFFFFF} gekickt. Grund: Test
Edit: Fehler gefunden, das Script wollte keine zwei Codes mit Fabre hintereinander.
Danke für die Hile.
Moin, hat jemand noch die funktion IsEMailValid? das SAMP Forum is ja leider dwon 
Moin, hat jemand noch die funktion IsEMailValid? das SAMP Forum is ja leider dwon
stock IsValidEmail(const email[])
{
new len=strlen(email);
new cstate=0;
for(new i=0;i<len;i++)
{
if ((cstate==0 || cstate==1) && (email[i]>='A' && email[i]<='Z') || (email[i]>='a' && email[i]<='z') || (email[i]>='0' && email[i]<='9') || (email[i]=='.') || (email[i]=='-') || (email[i]=='_'))
{
}
else
{
if ((cstate==0) &&(email[i]=='@'))
{
cstate=1;
}
else
{
return false;
}
}
}
if (cstate<1)
{
return false;
}
if (len<6)
{
return false;
}
if ((email[len-3]=='.') || (email[len-4]=='.') || (email[len-5]=='.'))
{
return true;
}
return false;
}Moin weiß jemand warum der Fehler kommt?
ocmd:auninvite(playerid, params[])
{
new p_ID, msg[512];
if(!pInfo[playerid][pLoggedIn])return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
if(pInfo[playerid][pTeam] == 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
if(pInfo[playerid][pTeam] < 4)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
if(sscanf(params, "u", p_ID))return SendClientMessage(playerid,COLOR_GELB, "* Benutze"HTML_WEISS": /auninvite [Spieler/ID]");
if(pInfo[p_ID][pFraktion] == 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Der angegebene Spieler ist in keiner Fraktion!");
format(msg, sizeof(msg), "* Info"HTML_WEISS": Du hast %s aus der Fraktion %s administrativ uninvited.", pInfo[p_ID][pName], GetFraktionName(pInfo[p_ID][pFraktion]));
SendClientMessage(playerid, COLOR_GELB, msg);
format(msg, sizeof(msg), "* Info"HTML_WEISS": Du wurdest von %s auf der Fraktion %s administrativ uninvited.", pInfo[p_ID][pName], GetFraktionName(pInfo[p_ID][pFraktion]));
SendClientMessage(p_ID, COLOR_GELB, msg);
pInfo[p_ID][pFraktion] = 0;
pInfo[p_ID][pFraktionRang] = 0;
return 1;
}
public OnPlayerCommandText(playerid, cmdtext[])
{
new msg[256];
format(msg, sizeof(msg), "* Fehler"HTML_WEISS": Der Befehl ("HTML_GELB" %s "HTML_WEISS") wurde nicht gefunden. Benutze /help!",cmdtext);
return SendClientMessage(playerid, COLOR_ROT, msg);
}Console
[debug] Run time error 4: "Array index out of bounds"
[debug] Attempted to read/write array element at index 65535 in array of size 1000
[debug] AMX backtrace:
[debug] #0 0000e2ec in public ocmd_auninvite (0, 2579148)
[debug] #1 native CallLocalFunction () in samp-server.exe
[debug] #2 00006200 in public OnPlayerCommandText (0, 2579096)Nabend,
ich hab folgendes Problem. Und zwar wenn ich die Schleife durchlaufe wird mir nicht die MAX_FRAK_VEHICLE hochgezält sondern eine viel zu hohe zahl.
#define MAX_FRAK_VEHICLE 200
enum FraktionVehicleData {
fvehID,
fvehModel,
fvehColor1,
fvehColor2,
fvehFraktion,
Float:fvehX,
Float:fvehY,
Float:fvehZ,
Float:fvehR,
fvehSirene
};
new fVehInfo[MAX_FRAK_VEHICLE][FraktionVehicleData];
for(new i = 0; i < MAX_FRAK_VEHICLE; i++)
{
printf("%d", fVehInfo[i][fvehID]);
}In der Console wird folgedes geloggt:
Hoffe mir kann jmd helfen ich verzweilfe langsam 
Du gibst ja auch printf("%d", fVehInfo[i][fvehID]); in deiner Schleife an. Ich gehe mal davon aus, dass in der fvehID die Datensatz-ID des Fahrzeuges gespeichert ist.
Damit der Schleifendurchgang geprinted wird, brauchst du nur die lokale Variable i in deinem print angeben.
Du gibst ja auch printf("%d", fVehInfo[i][fvehID]); in deiner Schleife an. Ich gehe mal davon aus, dass in der fvehID die Datensatz-ID des Fahrzeuges gespeichert ist.
Damit der Schleifendurchgang geprinted wird, brauchst du nur die lokale Variable i in deinem print angeben.
Vielen dank mein Hirn war gestern matsch so ein dummer Fehler.
Allerdings bekomme ich jetzt in der Console folgender Fehler:
4
[debug] Run time error 4: "Array index out of bounds"
[debug] Attempted to read/write array element at index 16 in array of size 16Code:
for(new i = 0; i < MAX_FRAK_VEHICLE; i++)
{
if(IsValidVehicle(fVehInfo[i][fvehID]) == 1)continue;
printf("%d", i);
format(msg, sizeof(msg), "* AdminInfo"HTML_WEISS": Du hast ein Fraktionsvehicle für die Fraktion %s erstellt.", fInfo[frak][fName]);
return SendClientMessage(playerid, COLOR_GELB, msg);
}Allerdings bekomme ich jetzt in der Console folgender Fehler:
Ich denke, dass es sich wieder um denselben Fehler handelt.
Ich gehe mal davon aus, dass in der fvehID die Datensatz-ID des Fahrzeuges gespeichert ist.
Falls das so ist, wieder nur i verwenden. (Zeile 3)
vehicleID ist nicht der eindeutige Key in der Datenbank.
Das Ding ist das ist der aktuelle Code ohne erstellen des Vehicle oder speichern. Ich gehe nur die Schleife durch und printe i
es gibt aktuell 4 fahrzeuge in der Datenbank die werden auch korrekt geladen
so lade ich sie:
forward OnFraktionVehicleLoad();
public OnFraktionVehicleLoad()
{
new rows, fvehid;
cache_get_row_count(rows);
for(new i = 0; i < rows; i++)
{
cache_get_value_name_int(i, "VehID", fvehid);
cache_get_value_name_int(i, "VehModel", fVehInfo[fvehid][fvehModel]);
cache_get_value_name_int(i, "VehFraktion", fVehInfo[fvehid][fvehFraktion]);
cache_get_value_name_float(i, "VehX", fVehInfo[fvehid][fvehX]);
cache_get_value_name_float(i, "VehY", fVehInfo[fvehid][fvehY]);
cache_get_value_name_float(i, "VehZ", fVehInfo[fvehid][fvehZ]);
cache_get_value_name_float(i, "VehR", fVehInfo[fvehid][fvehR]);
cache_get_value_name_int(i, "VehTowed", fVehInfo[fvehid][fvehTowed]);
cache_get_value_name_float(i, "VehTowedX", fVehInfo[fvehid][fvehTowedX]);
cache_get_value_name_float(i, "VehTowedY", fVehInfo[fvehid][fvehTowedY]);
cache_get_value_name_float(i, "VehTowedZ", fVehInfo[fvehid][fvehTowedZ]);
cache_get_value_name_float(i, "VehTowedR", fVehInfo[fvehid][fvehTowedR]);
cache_get_value_name_int(i, "VehSirene", fVehInfo[fvehid][fvehSirene]);
}
printf("[Server] Es wurden %d Fraktionsvehicle geladen!", rows);
return 1;
}//edit fehler gefunden es lag an meiner SendClientMessage hahah
Ich gehe mal davon aus, dass der Fehler von fInfo[frak][fName] erzeugt wird. frak scheint größer zu sein, als der mögliche Index von fInfo zulässt.
Dabei noch eine Frage: Fragst du innerhalb der Schleife ab, ob das Fahrzeug valide ist und wenn nicht, dann soll er das Fahrzeug erstellen?
Das sieht alles nämlich etwas zu unkonventionell aus. Magst du mal sonst dein Befehl hier reinschreiben, dann kann man mal schauen, was man da verbessern kann.
Klar Edgar
ocmd:fcreateveh(playerid, params[])
{
new frak, model, Float:pos[4], color[2], sirene, msg[128], query[1024];
if(!pInfo[playerid][pLoggedIn])return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
if(pInfo[playerid][pTeam] == 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
if(pInfo[playerid][pTeam] < 5)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
if(sscanf(params, "iiiii", frak, model, color[0], color[1], sirene))return SendClientMessage(playerid,COLOR_GELB, "* Benutze"HTML_WEISS": /fcreateveh [FraktionID] [ModelID] [Color1] [Color2] [Sirene(0/1)]");
if(frak < 1 || frak > 16)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Die Fraktion existiert nicht. Benutze /fraktionen");
if(model < 400 || model > 611)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Diese ModelID existiert nicht!");
if(GetPlayerVirtualWorld(playerid) != 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du musst in der Spielwelt sein!");
if(GetPlayerInterior(playerid) != 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du darfst in keinem Interior sein!");
GetPlayerPos(playerid, pos[0], pos[1], pos[2]);
GetPlayerFacingAngle(playerid, pos[3]);
for(new i = 0; i < MAX_FRAK_VEHICLE; i++)
{
if(!IsValidVehicle(fVehInfo[i][fvehID]))
{
fVehInfo[i][fvehModel] = model;
fVehInfo[i][fvehColor1] = color[0];
fVehInfo[i][fvehColor2] = color[1];
fVehInfo[i][fvehFraktion] = frak;
fVehInfo[i][fvehX] = pos[0];
fVehInfo[i][fvehY] = pos[1];
fVehInfo[i][fvehZ] = pos[2];
fVehInfo[i][fvehR] = pos[3];
fVehInfo[i][fvehSirene] = sirene;
mysql_format(Handle, query, sizeof(query), "INSERT INTO `fraktion_vehicle` (`VehID`, `VehModel`, `VehColor1`, `VehColor2`, `VehFraktion`, `VehX`, `VehY`, `VehZ`, `VehR`, VehSirene`) VALUES ('%d', '%d', '%d', '%d', '%d', '%f', '%f', '%f', '%f', '%d')",
i, fVehInfo[i][fvehModel], fVehInfo[i][fvehColor1], fVehInfo[i][fvehColor2], fVehInfo[i][fvehFraktion], fVehInfo[i][fvehX], fVehInfo[i][fvehY], fVehInfo[i][fvehZ], fVehInfo[i][fvehR], fVehInfo[i][fvehSirene]);
mysql_tquery(Handle, query);
fVehInfo[i][fvehID] = CreateVehicle(fVehInfo[i][fvehModel], fVehInfo[i][fvehX], fVehInfo[i][fvehY], fVehInfo[i][fvehZ], fVehInfo[i][fvehR], fVehInfo[i][fvehColor1], fVehInfo[i][fvehColor2], fVehInfo[i][fvehSirene]);
PutPlayerInVehicle(playerid, fVehInfo[i][fvehID], 0);
format(msg, sizeof(msg), "* AdminInfo"HTML_WEISS": Du hast ein Fraktionsvehicle für die Fraktion %s erstellt.", fInfo[frak-1][fName]);
SendClientMessage(playerid, COLOR_GELB, msg);
break;
}
}
return 1;
}Ich würde das folgendermaßen aufbauen:
Füge deinem Vehicle-Enum noch einen Wert hinzu, bspw: fvehInUse (als Boolean-Wert)
Damit kannst du prüfen, ob der Index/Slot bereits in Verwendung ist.
Diesen Wert musst du immer auf true setzen, wenn du ein Fahrzeug erstellst und somit diesen Slot verwendest und auf false, wenn du diesen wieder freigibst.
Mit dieser Funktion kannst du den nächsten freien slot erhalten.
stock getNextFreeVehicleSlot() {
for(new i; i < MAX_FRAK_VEHICLE; i++) {
if (!fVehInfo[i][fvehInUse]) return i;
}
return -1;
}Dann kannst du das folgendermaßen in deinen Befehl einbauen:
ocmd:fcreateveh(playerid, params[])
{
new frak, model, Float:pos[4], color[2], sirene, msg[128], query[1024];
if(!pInfo[playerid][pLoggedIn]) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
if(pInfo[playerid][pTeam] == 0) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
if(pInfo[playerid][pTeam] < 5) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
if(sscanf(params, "iiiii", frak, model, color[0], color[1], sirene))return SendClientMessage(playerid,COLOR_GELB, "* Benutze"HTML_WEISS": /fcreateveh [FraktionID] [ModelID] [Color1] [Color2] [Sirene(0/1)]");
if(frak < 1 || frak > 16) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Die Fraktion existiert nicht. Benutze /fraktionen");
if(model < 400 || model > 611) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Diese ModelID existiert nicht!");
if(GetPlayerVirtualWorld(playerid) != 0) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du musst in der Spielwelt sein!");
if(GetPlayerInterior(playerid) != 0) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du darfst in keinem Interior sein!");
GetPlayerPos(playerid, pos[0], pos[1], pos[2]);
GetPlayerFacingAngle(playerid, pos[3]);
new freeSlot = getNextFreeVehicleSlot();
if (freeSlot == -1) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Es exisitieren keine weiteren freien Slots mehr.");
fVehInfo[freeSlot][fvehInUse] = true;
fVehInfo[freeSlot][fvehModel] = model;
fVehInfo[freeSlot][fvehColor1] = color[0];
fVehInfo[freeSlot][fvehColor2] = color[1];
fVehInfo[freeSlot][fvehFraktion] = frak;
fVehInfo[freeSlot][fvehX] = pos[0];
fVehInfo[freeSlot][fvehY] = pos[1];
fVehInfo[freeSlot][fvehZ] = pos[2];
fVehInfo[freeSlot][fvehR] = pos[3];
fVehInfo[freeSlot][fvehSirene] = sirene;
mysql_format(Handle, query, sizeof(query), "INSERT INTO `fraktion_vehicle` (`VehID`, `VehModel`, `VehColor1`, `VehColor2`, `VehFraktion`, `VehX`, `VehY`, `VehZ`, `VehR`, VehSirene`) VALUES ('%d', '%d', '%d', '%d', '%d', '%f', '%f', '%f', '%f', '%d')",
i, fVehInfo[freeSlot][fvehModel], fVehInfo[freeSlot][fvehColor1], fVehInfo[freeSlot][fvehColor2], fVehInfo[freeSlot][fvehFraktion], fVehInfo[freeSlot][fvehX], fVehInfo[freeSlot][fvehY], fVehInfo[freeSlot][fvehZ], fVehInfo[freeSlot][fvehR], fVehInfo[freeSlot][fvehSirene]);
mysql_tquery(Handle, query);
fVehInfo[freeSlot][fvehID] = CreateVehicle(fVehInfo[freeSlot][fvehModel], fVehInfo[freeSlot][fvehX], fVehInfo[freeSlot][fvehY], fVehInfo[freeSlot][fvehZ], fVehInfo[freeSlot][fvehR], fVehInfo[freeSlot][fvehColor1], fVehInfo[freeSlot][fvehColor2], fVehInfo[freeSlot][fvehSirene]);
PutPlayerInVehicle(playerid, fVehInfo[freeSlot][fvehID], 0);
format(msg, sizeof(msg), "* AdminInfo"HTML_WEISS": Du hast ein Fraktionsvehicle für die Fraktion %s erstellt.", fInfo[frak-1][fName]);
SendClientMessage(playerid, COLOR_GELB, msg);
return 1;
}Vergiss nicht, an den Stellen in deinem Script, an der du Fahrzeuge erstellst auch fVehInfo[freeSlot][fvehInUse] = true; zu verwenden und
da, wo du welche wieder löscht dann auf false.
Und dein Enum um fvehInUse erweitern.
Anmerkung zur deiner Methode mit IsValidVehicle:
Sollte das Fahrzeug irgendwann mal irgendwie verschwinden (Explodiert, nicht geladen, Troller und so weiter) dann wird das Fahrzeug durch dein Befehl überschrieben, da es nunmal nicht valide ist.
Mit meiner Empfehlung auf fvehInUse zu prüfen, hast du die alleinige Kontrolle über den "Datensatz", ob dieser verwenden wird oder nicht. Mit IsValidVehicle hingegen kann dir auch mal das Spielgeschähen dazwischen funken, das ist aber in diesem Fall nie gewollt.
Habe das schnell frei aus dem Kopf erledigt, könnte sein, dass da auch Denkfehler mitbei sind, einfach mal testen.
Warum zum teufel wird das destroyen aufgerufen und gleichzeitig auch die Fehlermessage dass ich in keinem FrakVeh bin? obwohl beim printen bei beide das gleiche raus kommt
ocmd:fdelveh(playerid, params[])
{
new query[128], msg[248];
if(!pInfo[playerid][pLoggedIn])return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
if(pInfo[playerid][pTeam] == 0)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
if(pInfo[playerid][pTeam] < 5)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER)return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht in einem Vehicle als Fahrer.");
for(new i = 0; i < MAX_FRAK_VEHICLE; i++)
{
if(GetPlayerVehicleID(playerid) == fVehInfo[i][fvehID])
{
//Veh löschen
DestroyVehicle(fVehInfo[i][fvehID]);
fVehInfo[i][fvehInUse] = false;
mysql_format(Handle, query, sizeof(query), "DELETE FROM `fraktion_vehicle` WHERE `VehID` = '%d'", i);
mysql_pquery(Handle, query);
format(msg, sizeof(msg), "* Fehler"HTML_WEISS": Du hast ein Fraktionsvehicle der Fraktion %s gelöscht.", fInfo[fVehInfo[i][fvehFraktion]-1][fName]);
SendClientMessage(playerid, COLOR_GELB, msg);
}else return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist in keinem Fraktionsvehicle.");
}
return 1;
}9 9Warum zum teufel wird das destroyen aufgerufen und gleichzeitig auch die Fehlermessage dass ich in keinem FrakVeh bin?
Weil du da ein else in der Schleife hast.
Kannst so Funktionen auch wunderbar auslagern:
ocmd:fdelveh(playerid, params[])
{
if(!pInfo[playerid][pLoggedIn]) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht eingeloggt.");
if(pInfo[playerid][pTeam] == 0) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist kein Teammitglied.");
if(pInfo[playerid][pTeam] < 5) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Dein Teammitglied Rang ist zu niedrig.");
if(GetPlayerState(playerid) != PLAYER_STATE_DRIVER) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist nicht in einem Vehicle als Fahrer.");
new i = GetFrakCarIndex(playerid);
if(i == -1) return SendClientMessage(playerid, COLOR_ROT, "* Fehler"HTML_WEISS": Du bist in keinem Fraktionsvehicle.");
new msg[248];
DestroyVehicle(fVehInfo[i][fvehID]);
fVehInfo[i][fvehInUse] = false;
mysql_format(Handle, msg, sizeof(msg), "DELETE FROM `fraktion_vehicle` WHERE `VehID` = '%d'", i);
mysql_pquery(Handle, msg);
format(msg, sizeof(msg), "* Fehler"HTML_WEISS": Du hast ein Fraktionsvehicle der Fraktion %s gelöscht.", fInfo[fVehInfo[i][fvehFraktion]-1][fName]);
SendClientMessage(playerid, COLOR_GELB, msg);
return 1;
}
stock GetFrakCarIndex(playerid)
{
for(new i,v=GetPlayerVehicleID(playerid); i<MAX_FRAK_VEHICLE; i++)
{
if(v == fVehInfo[i][fvehID]) return i;
}
return -1;
}