Und du hast es kompiliert, die amx am Server getauscht und neu gestartet und es funktioniert weiterhin kein Befehl?!
Beiträge von Kaliber
-
-
Du benötigst den Callback OnPlayerCommandText gar nicht mehr. Lösche diesen am Besten.
Und nutze nur:
Cpublic OnPlayerCommandPerformed(playerid, cmd[], params[], result, flags) { if (result == -1) { SCM(playerid, 0xFFFFFFFF, "SERVER: Befehl nicht gefunden."); return 0; } return 1; }
Und du bist sicher, dass du oben im Skript #include <ocmd> quasi rausgeschmissen hast und nur #include <Pawn.CMD> verwendest?
-
Was sagt denn der server log? Wurde das Plugin geladen?
-
ich glaub Lösung 1 hat mir gerade das gesamte script "zerschossen" Login System fehlt etc. kein Befehl funktioniert
Dann check mal den Server Log...
Wurde das Plugin überhaupt geladen? Musst es in den Plugin Order packen und die Server.cfg anpassen...
-
Jedoch frag ich mich immer noch wieso sobald ich beim accept Befehl InGame /accept eingebe sofort die einladung angenommen wird!? weil wie gesagt gibt ja noch weitere dinge die man accepten kann bspw. /accept dice etc.
Ah, ja stimmt ich habe nicht bedacht, dass ocmd ja gar nicht \0 in den String packt sondern \1...
Lösung 1:
Ich empfehle dir nutze nicht ocmd, sondern: https://github.com/katursis/Pawn.CMD/releases
Das ist deutlich schneller und weniger Fehleranfällig
Kannst oben dann in Pawno einfach Ersetzen machen und ocmd: durch CMD: oder cmd: ersetzen
Oder Lösung 2:
-
Also wäre es zum Vorteil dies auch bei meinem accept Befehl umzuändern? von quasi so:
Das kommt darauf an...
In diesem Fall funktioniert es, da keine Weiteren Parameter außer dem Keyword eingegeben werden. Aber falls doch, solltest du das so machen.
//Und hier dann immer die nächsten keyword hinter invite einfügen?
Joaa, ansonsten weiß der Nutzer ja nicht, welche Optionen er alle hat
Und anstatt dem return 1 unten würde ich nochmal wie oben erklärt SCM(playerid, WEISS, info); nutzen, damit er direkt weiß, welche keywords es gibt.
Nur als Info noch:
Cif(!strcmp(item, "invite",false)) // Das sorgt dafür, dass nur "invite" geschrieben werden kann. Es wird quasi geprüft, ob die Groß- und Kleinschreibung richtig ist. Für den Nutzer ist das oft nervig, da es ja egal ist, ob er Invite oder invite, invitE schreibt, ist ja alles das Selbe und es soll funktionieren if(!strcmp(item, "invite")) //<< das sorgt dafür, dass die Groß-und Kleinschreibung ignoriert wird und alles gültig ist ;)
jedoch wird mein Fraktionsrang in der DB auf 70 gesetzt wie kann das zustande kommen?
Kann ich dir so nicht sagen, vllt geht etwas beim Speichern schief?
Dazu müsste ich dein Speichersystem begutachten.
SetPVarInt(playerid, "getfrak", 0);
Noch ein Hinweis dazu, lösche das ruhig einfach:
DeletePVar(playerid, "getfrak");
Dann sinkt der RAM Verbrauch vom Server
-
Genau dieses System wird ironischer Weise schon als Standard-Beispiel für den Callback selbst angegeben: https://team.sa-mp.com/wiki/OnPlayerDeath_DE.html
Hoffe das hilft dir weiter
-
Also zuerst zu der Info Variable:
Gibt der User nur /erstellen ein, soll er ja eine Nachricht erhalten mit den möglichen Parametern. Aber gibt er jetzt z.B. instinktiv /erstellen Car ein, dann würde ja nichts passieren, da das Keyword "Car" nicht gefunden werden würde. Dachte es wäre eine nette Geste einfach dann nochmal die Info auszugeben, was mögliche Keywords sind, einfach dass man weiß, dass man ein ungültiges Keyword eingegeben hat.
Damit du jetzt aber nicht immer oben und unten die Liste erweitern musst, wenn neue Keywords wie keine Ahnung /erstellen [Auto, Objekt] oder sonst was, gibt es quasi diese Konstante, die nur einmal verändert werden muss und dann wird die Info oben und unten angepasst
Als Programmier sollte man immer Redundanzen vermeiden. Sprich wenn du z.B. mehrfach den selben Text verwendest, leg eine Konstante an und verwende ihn quasi nur einmal.
Es hat hier sogar einen Vorteil eine Konstante, statt eines Makros (#define) zu verwenden. Beim define wird während des Kompiliervorgangs ja das Makro ersetzt. Sprich:
C//Vor dem Kompiliervorgang #define TEST_INFO "Das ist ein Test" print(TEST_INFO); print(TEST_INFO); //Danach: print("Das ist ein Test"); print("Das ist ein Test");
Jetzt muss trotzdem 2x Speicher für den Text reserviert werden. Hätte man hingegen folgendes getan:
Würde nur 1x Speicher reserviert werden und es würde immer auf diesen Speicherbereich zugegriffen werden
Aber das sind nur so mini Optimierungen
Primär geht es auch darum, dass du so weniger Rechtschreibfehler machst und alles gleich formuliert ist und somit schön konsistent und professionell wirkt.
Zudem, wenn eine Änderung gemacht werden muss, du nur einen Punkt anpassen musst.
Zu dem Code:
Dein Code hat nicht funktioniert, da sscanf quasi nur einen Parameter oben erwartet hat und nicht mehrere. Also ist es besser das hier manuell zu machen.
Du willst ja aus dem Befehl den ersten Parameter haben.
Gibt man /erstellen Auto 1 1 1 z.B. ein, dann steht in params[] "Auto 1 1 1".
Nun wollen wir ja nur das "Auto" haben.
strfind sucht nun nach dem Leerzeichen (und gibt die Position im String zurück, an welcher Stelle es gefunden wurde).
Nun können wir den String splitten, indem wir von Position 0 - zum Position des Leerzeichens den Inhalt kopieren. Das ist nämlich genau unser gewünschtes Keyword.
Existiert kein Leerzeichen, nehmen wir die gesamte Länge des Strings, das wird mit dem Ternary Operator ? : geprüft.
Dient im Endeffekt nur dazu das erste Wort zu bekommen
-
Probiers mal so:
C
Alles anzeigenocmd:erstellen(playerid, params[]) { static const info[] = "[ {ECFF00}Information{FFFFFF} ]: /erstellen [Auto]"; // Dann musst es nur an einer Stelle ändern ;) if(!params[0]) return SCM(playerid, WEISS, info); new item[64], idx = strfind(params, " "); strmid(item, params, 0, (idx==-1)?strlen(params):idx); if(!strcmp(item, "auto")) { new pID, vehid, farbe, farbe2; if(sscanf(params,"s[64]iiii", item,pID, vehid, farbe, farbe2))return SCM(playerid,WEISS,"[{ECFF00} AdminInfo {FFFFFF}]: /erstellen [Auto] [playerid] [VehID] [Model] [Farbe] [Farbe 2]"); new Float:xc,Float:yc,Float:zc,Float:rc; GetPlayerPos(pID,xc,yc,zc); GetPlayerFacingAngle(pID,rc); createPlayerCar(pID,vehid, xc, yc, zc, rc); } return SCM(playerid, WEISS, info); }
-
Wenn du es im Map-Editor markierst und dabei alles markierst (also die gesamte Base), dann geht das nicht, da das Tor zu dem Model gehört.
Da müsste man das ummappen, dass es so ähnlich aussieht, aber da dann kein Tor ist.
-
Wieso Logs als Screenshot, wenn du die paar Zeilen hier easy mit dem Spoiler & Code BBCode super einpflegen kannst?
Ich kann am Handy da nix lesen und es lädt super langsam...
-
Hast du auch eine main Funktion im Skript?
-
Es ging nur um das Thema. Am Handy sieht man nicht, dass es sich um FiveM handelt.
-
Kann mir wer verraten wie man eine Farbe abspeichert mit Dini?
Eine Farbe ist eine Zahl, also ein Integer, also https://team.sa-mp.com/wiki/Us…unctions.html#dini_IntSet
-
Ja sizeof(tInfo[]) ist Quatsch, du willst sizeof(BagItems) und lösche das =1...ich wiederhole: Arrays starten bei 0.
#Handy
-
Bei diesem Code fallen mir so viele komische Dinge auf, dass ich mich frage, wieso du nicht erst einmal die Basics mehr lernst...
- Der Hauptfehler besteht daran, dass vermutlich BagItems und tInfo nicht die selbe Größe haben...musst natürlich schauen, dass du die Schleife über die wesentliche Variable laufen lässt.
- Wieso startest du bei der Schleife bei 1?!?!?!?!?! Arrays starten bei 0
- Zeile 30 ist nonsense
- Zeile 16..wieso eine Klammer? Das ist Quatsch
- Wenn du schon DEBUGs machst, zeige uns doch diese auch
Aber wie gesagt, der Fehler liegt in der Schleife. Du greifst auf einen Index zu, der nicht existiert.
-
Moin wieder mal ne frage ich habe das problem das ich dem spieler eine kettensäge gegeben habe aber wie kann ich nun nur die kettensäge entfernen ohne ihm alle waffen zu nehmen?
C
Alles anzeigenstock RemovePlayerWeapon(playerid, weaponid) { new plyWeapons[13],plyAmmo[13]; for(new slot,wep,ammo; slot < 13; slot++) { GetPlayerWeaponData(playerid, slot, wep, ammo); if(wep != weaponid) { plyWeapons[slot] = wep, plyAmmo[slot] = ammo; } } ResetPlayerWeapons(playerid); for(new slot; slot < 13; slot++) { GivePlayerWeapon(playerid, plyWeapons[slot], plyAmmo[slot]); } } //Zum Entfernen der Kettensäge dann einfach: RemovePlayerWeapon(playerid, WEAPON_CHAINSAW);
-
Wie erstellst du denn das Objekt?
Wieso gibst du dir nicht mehr Parameter aus? Dann wurdest du merken, dass BULLET_HIT_TYPE_PLAYER_OBJECT aufgerufen wird, da es sich bei "dynamic Objekten" bzw Streamer Objekten immer um Player Objekte handelt.
Cprintf("%d|%d|%d|%d|%.02f|%.02f|%.02f",playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ);
Alternativ kannst du auch OnPlayerShootDynamicObject nehmen.
-
Kannst den Code vereinfachen und so Redundanzen vermeiden:
C
Alles anzeigenif(IsPlayerAtAnyJob(playerid,jid)) { static const tDiagID[] = {DIALOG_FISCHERJOB, DIALOG_CITYJOB, DIALOG_LOGISTICJOB, DIALOG_OELJOB}; new jobstring[64]; format(jobstring,sizeof(jobstring),"Hier arbeiten!\nVerdienst:\t%d$",sInfo[playerid][paycheck][jid]); ShowPlayerDialog(playerid,tDiagID[jid-1],DIALOG_STYLE_TABLIST,jobnames[jid],jobstring,"ENTER","X"); SetPVarInt(playerid,"StateChange",1); SetTimerEx("StateChangeTimer",2000,false,"i",playerid); return 1; }
Die Frage ist, wie und wo du sInfo[playerid][paycheck] setzt und dem einen Wert gibst
-
Empfehle dir auf YouTube Finanzfluss (für generelle Investments & Krypto)
Und für mehr theoretische (auch oft subjektive) Ansichten den Blocktrainer auf YouTube