Angepinnt [SAMMELTHREAD] Kleine Scripting Fragen

Kaliber · 27. Mai 2018, 19:45

3 Dinge zu dem Code:

Das liegt an dem return 1; in der Funktion getPlayerSpeed. Das bedeutet in deinem TextDraw müsstest du immer eine 1 sehen, wenn du im Auto bist Also einfach das löschen.
Du könntest die Funktion noch optimierten, indem du anstatt floatsqroot einfach [wiki]VectorSize[/wiki] nutzt, diese wurde dafür extra entwickelt
In einer Schleife solltest du keine Variablen deklarieren, da das immer viel Zeit braucht.

Im Spoiler ist die optimierte Version

Spoiler anzeigen

reaL. · 28. Mai 2018, 08:47

Warum werden bei diesen Code meine Login Boxen mit Angezeigt ?

Quellcode

new Text:Register[19];
new Text:Login[2];
new PlayerText:Registername[MAX_PLAYERS],
PlayerText:Registerpw[MAX_PLAYERS],
PlayerText:Registertag[MAX_PLAYERS],
PlayerText:Registermonat[MAX_PLAYERS],
PlayerText:Registerjahr[MAX_PLAYERS],
PlayerText:Registerpfeill[MAX_PLAYERS],
PlayerText:Registerpfeilr[MAX_PLAYERS],
PlayerText:Registerpfeill2[MAX_PLAYERS],
PlayerText:Registerpfeilr2[MAX_PLAYERS],
PlayerText:Registergeschlecht[MAX_PLAYERS],
PlayerText:Registerb[MAX_PLAYERS],
PlayerText:Registerskin[MAX_PLAYERS],
PlayerText:Registeremail[MAX_PLAYERS],
PlayerText:Loginname[MAX_PLAYERS],
PlayerText:Loginskin[MAX_PLAYERS],
PlayerText:Loginb[MAX_PLAYERS];

Alles anzeigen

Quellcode

if(rows == 0)
{
for(new i = 0; i < sizeof(Register); i++)
{
TextDrawShowForPlayer(playerid, Register[i]);
}
PlayerTextDrawSetString(playerid, Registername[playerid], pInfo[playerid][pName]);
PlayerTextDrawShow(playerid, Registername[playerid]);
PlayerTextDrawSetSelectable(playerid, Registerpw[playerid], 1);
PlayerTextDrawShow(playerid, Registerpw[playerid]);
PlayerTextDrawSetSelectable(playerid, Registertag[playerid], 1);
PlayerTextDrawShow(playerid, Registertag[playerid]);
PlayerTextDrawSetSelectable(playerid, Registermonat[playerid], 1);
PlayerTextDrawShow(playerid, Registermonat[playerid]);
PlayerTextDrawSetSelectable(playerid, Registerjahr[playerid], 1);
PlayerTextDrawShow(playerid, Registerjahr[playerid]);
PlayerTextDrawShow(playerid, Registerpfeill[playerid]);
PlayerTextDrawShow(playerid, Registerpfeilr[playerid]);
PlayerTextDrawSetSelectable(playerid, Registergeschlecht[playerid], 1);
PlayerTextDrawShow(playerid, Registergeschlecht[playerid]);
//PlayerTextDrawSetSelectable(playerid, Registeremail[playerid], 1);
//PlayerTextDrawShow(playerid, Registeremail[playerid]);
PlayerTextDrawSetSelectable(playerid, Registerb[playerid], 1);
PlayerTextDrawShow(playerid, Registerb[playerid]);
PlayerTextDrawShow(playerid, Registerskin[playerid]);
SelectTextDraw(playerid, 0xFF0000FF);
Regstep[playerid] = 1;
SendClientMessage(playerid, -1, "Herzlich Willkommen, Gebe hier zunächst deine Daten ein.");
SendClientMessage(playerid, -1, "Soblad du damit fertig bist kannst du dein Skin auswählen.");
return 1;
}

Alles anzeigen

Xazor · 28. Mai 2018, 21:13

Hallo, ich verstehe da jetzt nicht was daran falsch ist.
Mir wird folgende Fehlermeldung ausgespuckt:

s.p(324) : error 010: invalid function or declaration
s.p(326) : error 010: invalid function or declaration
s.p(329) : error 010: invalid function or declaration
s.p(331) : error 021: symbol already defined: "SendMessageToAdmins"
s.p(333) : error 010: invalid function or declaration
s.p(388) : warning 203: symbol is never used: "achat"
s.p(388) : warning 203: symbol is never used: "msg"

Code:

Quellcode

ocmd@3:a,achat,adminchat(playerid,params[])
{
if(PlayerInfo[playerid][pAdmin]>=1)
{
new achat[128],msg[128];
if(sscanf(params,"s",achat))return SendClientMessage(playerid,COLOR_RED,"[INFO] Nutze /a[chat] [Text]");
format(msg,sizeof(msg),"%s %s: %s",Adminrang(playerid),PlayerInfo[playerid][pName],achat);
SendMessageToAdmins(COLOR_ADMIN,msg);
}
else{
SendClientMessage(playerid,COLOR_RED,"Du bist kein Admin!");
}
}

Alles anzeigen

Quellcode

stock SendMessageToAdmins(color, const string[])
{
for(new i = 0; i < MAX_PLAYERS; i++)
{
if(PlayerInfo[i][pAdmin] != 0)
{
SendClientMessage(i,color, string);
}
}
}

Jeffry · 29. Mai 2018, 17:32

Nutzt du die aktuelle ocmd Include?
ocmd 2.1.1 [ Update 3.11.2013 ]

Aktualisiere diese gegebenenfalls.

John_Dree · 29. Mai 2018, 18:42

@Xazor

Falls du die neuste ocmd Include hast, und du das sscanf Plugin nutzt, versuch es mal bitte so:

PAWN-Quellcode

ocmd@3:a,achat,adminchat(playerid,params[])
{
if(PlayerInfo[playerid][pAdmin]>=1)
{
new achat[128],msg[128];
if(sscanf(params,"s[128]",achat))return SendClientMessage(playerid,COLOR_RED,"[INFO] Nutze /a[chat] [Text]");
format(msg,sizeof(msg),"%s %s: %s",Adminrang(playerid),PlayerInfo[playerid][pName],achat);
SendMessageToAdmins(COLOR_ADMIN,msg);
}
else{
SendClientMessage(playerid,COLOR_RED,"Du bist kein Admin!");
}
}

Alles anzeigen

Dazu schaut es aus, als wenn du schon die Funktion "symbol already defined: "SendMessageToAdmins"" irgendwo hast, und, ist der stock über den Befehl oder unter den Befehl ?
Hatte tatsächlich schon einmal das Problem, als der Stock unter dem Befehl war, das mir der Compiler auch Fehler ausgespuckt hat.

Kaliber · 29. Mai 2018, 18:55

iZAG2k18 schrieb:

und du das sscanf Plugin nutzt, versuch es mal bitte so:

Nur um das mal zu erwähnen, sscanf in so einem Fall zu nutzen ist sehr schlecht, da das komplett überflüssig ist.

Man sollte bei so einem Fall direkt params nutzen:

C-Quellcode

ocmd@3:a,achat,adminchat(playerid,params[])
{
if(!PlayerInfo[playerid][pAdmin]) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Admin!");
if(params[0] == EOS || params[0] == '\1') return SendClientMessage(playerid,COLOR_RED,"[INFO] Nutze /a[chat] [Text]");
new msg[128];
format(msg,sizeof(msg),"%s %s: %s",Adminrang(playerid),PlayerInfo[playerid][pName],params);
SendMessageToAdmins(COLOR_ADMIN,msg);
return 1;
}

Dr. Frauenarzt · 30. Mai 2018, 00:40

Habe kurz eine frage das mit unload funktioniert nicht wen er hinfährt und einfach /unload macht,

Quellcode

ocmd:unload(playerid, params[])
{
new vehicleid = GetPlayerVehicleID(playerid);
if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich in keinem Tanker.");
for(new it=1;it<MAX_TANKEN;it++)
{
if(Tanke[it][tCreated] == 1)
{
if(IsPlayerInRangeOfPoint(playerid, 5.0, Tanke[it][tPickupX], Tanke[it][tPickupY], Tanke[it][tPickupZ]))
{
new string[128], int;
Tanke[it][tMaxLiter] -= Tanke[it][tLiter] = int;
new cash = Tanke[it][tKasse];
new kosten = (Tanke[it][tBenzinAnkauf]*int);
if(cash < kosten)return SendClientMessage(playerid, COLOR_GREY, "** Die Kasse hat nicht genügend Geld für das Benzin.");
Vehicle[GetVehicleTrailer(vehicleid)][vehProds] -= int;
new Float:intfloat = float(int);
new Float:add = floatadd(intfloat, Tanke[it][tLiter]);
Tanke[it][tLiter] = add;
if(Tanke[it][tLiter] == Tanke[it][tMaxLiter])
{
format(string, sizeof(string), "{21DD00}Es konnten nur %d Liter abgeliefert werden, die Tankstelle ist voll.", int);
SendClientMessage(playerid, COLOR_WHITE, string);
format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);
SendClientMessage(playerid, COLOR_WHITE, string);
format(string, sizeof(string), "{21DD00}Liter: %d/2500", Vehicle[vehicleid][vehProds]);
SendClientMessage(playerid, COLOR_WHITE, string);
}
format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);
SendClientMessage(playerid, COLOR_WHITE, string);
format(string, sizeof(string), "{21DD00}Liter: %d/2500", Vehicle[vehicleid][vehProds]);
SendClientMessage(playerid, COLOR_WHITE, string);
Tanke[it][tKasse] -= kosten;
GivePlayerMoneys(playerid, kosten);
UpdateTankLabel(it, Tanke[it][tOwned]);
}
}
}
return 1;
}

Alles anzeigen

Janes · 30. Mai 2018, 01:40

Dr. Frauenarzt schrieb:

Habe kurz eine frage das mit unload funktioniert nicht wen er hinfährt und einfach /unload macht,

Quellcode

ocmd:unload(playerid, params[])

{

new vehicleid = GetPlayerVehicleID(playerid);

if(!IsPlayerInAnyVehicle(playerid))return SendClientMessage(playerid, COLOR_GREY, "Du befindest dich in keinem Tanker.");

for(new it=1;it<MAX_TANKEN;it++)

{

if(Tanke[it][tCreated] == 1)

{

if(IsPlayerInRangeOfPoint(playerid, 5.0, Tanke[it][tPickupX], Tanke[it][tPickupY], Tanke[it][tPickupZ]))

{

new string[128], int;

Tanke[it][tMaxLiter] -= Tanke[it][tLiter] = int;

new cash = Tanke[it][tKasse];

new kosten = (Tanke[it][tBenzinAnkauf]*int);

if(cash < kosten)return SendClientMessage(playerid, COLOR_GREY, "** Die Kasse hat nicht genügend Geld für das Benzin.");

Vehicle[GetVehicleTrailer(vehicleid)][vehProds] -= int;

new Float:intfloat = float(int);

new Float:add = floatadd(intfloat, Tanke[it][tLiter]);

Tanke[it][tLiter] = add;

if(Tanke[it][tLiter] == Tanke[it][tMaxLiter])

{

format(string, sizeof(string), "{21DD00}Es konnten nur %d Liter abgeliefert werden, die Tankstelle ist voll.", int);

SendClientMessage(playerid, COLOR_WHITE, string);

format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);

SendClientMessage(playerid, COLOR_WHITE, string);

format(string, sizeof(string), "{21DD00}Liter: %d/2500", Vehicle[vehicleid][vehProds]);

SendClientMessage(playerid, COLOR_WHITE, string);

}

format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);

SendClientMessage(playerid, COLOR_WHITE, string);

format(string, sizeof(string), "{21DD00}Liter: %d/2500", Vehicle[vehicleid][vehProds]);

SendClientMessage(playerid, COLOR_WHITE, string);

Tanke[it][tKasse] -= kosten;

GivePlayerMoneys(playerid, kosten);

UpdateTankLabel(it, Tanke[it][tOwned]);

}

}

}

return 1;

}

Alles anzeigen

C&P lassen und es mal neu schreibe, vielleicht klappt es dann?

Xazor · 30. Mai 2018, 21:28

Jeffry schrieb:

Nutzt du die aktuelle ocmd Include?
ocmd 2.1.1 [ Update 3.11.2013 ]

Aktualisiere diese gegebenenfalls.

Vielen Dank hat die Lösung eingebracht.

Kaliber schrieb:

Nur um das mal zu erwähnen, sscanf in so einem Fall zu nutzen ist sehr schlecht, da das komplett überflüssig ist.
Man sollte bei so einem Fall direkt params nutzen:

C-Quellcode

ocmd@3:a,achat,adminchat(playerid,params[])

{

if(!PlayerInfo[playerid][pAdmin]) return SendClientMessage(playerid,COLOR_RED,"Du bist kein Admin!");

if(params[0] == EOS || params[0] == '\1') return SendClientMessage(playerid,COLOR_RED,"[INFO] Nutze /a[chat] [Text]");

new msg[128];

format(msg,sizeof(msg),"%s %s: %s",Adminrang(playerid),PlayerInfo[playerid][pName],params);

SendMessageToAdmins(COLOR_ADMIN,msg);

return 1;

}

Vielen Dank auch hier, ich wollte nur so ne Art return wie bei dir beschrieben ausgeben und du hast mir da einen kürzeren und besseren Lösungsansatz mitgegeben

Keslo · 5. Juni 2018, 00:17

Hallo,

wie kann man seine for-Schleifen effizient machen? (oder eine andere methode nutzen? foreach?)
Da die meisten es so machen:

PAWN-Quellcode

for(new i=0;i<MAX_PLAYERS;i++)

Durchgeht er ja jedes mal das Server Limit von 500 Usern. Wie geht es perfomanter?

2. Frage:
Wie kann man die globalen Variabeln, beispiel:

PAWN-Quellcode

new Hallo[MAX_PLAYERS];

ebenfalls perfomant machen? Da ich gelesen habe, dass man "GetMaxPlayers()" nutzen kann, aber die ID 0 nicht mitgezählt wird.

seegras · 5. Juni 2018, 00:38

Schau dir mal folgendes Thema bzw. explizit den verlinkten Beitrag an

Code Optimierung

Kaliber · 5. Juni 2018, 00:52

Keslo schrieb:

Durchgeht er ja jedes mal das Server Limit von 500 Usern. Wie geht es perfomanter?

Am besten wäre es einfach, sowas zu machen, das würde beide deiner Probleme auf einen Schlag lösen:

C-Quellcode

#undef MAX_PLAYERS
#define MAX_PLAYERS 25

Dann würdest du eben nur 25 Einträge durchgehen hahaha

Aber schon richtig foreach wäre am besten:

forum.sa-mp.com/showthread.php?t=570868

Gehen würde auch sowas wie:

PAWN-Quellcode

for(new i=GetPlayerPoolSize(); i!=-1; i--)

Das beispielsweise beginnt ab der höchsten spielerid auf dem Server und nicht z.B. ab 500

Also zur Verdeutlichung:

MAX_PLAYERS = Ein fester (konstanter) Wert, den du für Arrays nutzen kannst. (Da dort nur feste Werte verwendet werden können, also Dinge die sich nicht ändern. Die müssen von vorne herein fest definiert sein.) Habe oben ja gezeigt, wie du das anpassen kannst.

GetMaxPlayers = Diese Funktion liest dir aus der server.cfg den Wert aus. Ist schlecht und kann nicht für Arrays verwendet werden, da es eine Funktion ist und keine konstante! Also am Besten einfach MAX_PLAYERS den Wert von der server.cfg mittels dem Makro trick oben anpassen.

GetPlayerPoolSize = Gibt die höchste Spielerid auf dem Server wieder. Halt gut geeignet für Loops, aber foreach ist besser, da bei foreach keine Lücken zwischen den IDs entstehen. Quasi wenn ID 4 die höchste Spielerid ist, aber nur ID 1 noch online. Dann beginnt die Loop trotzdem von 4 runter zu zählen. Und bei foreach hättest du nur 2 anstatt 5 (4-0) Einträge.

Zu deinem 2. Punkt kann man nicht viel sagen.
Es gibt Memory Editing Plugins, so dass man Variablen dynamisch allozieren kann.

Einfacher wäre der spezielle Gebrauch von PVars, da du diesen Speicher wieder freigeben kannst.

Oder eben gezielte Anpassung des Makros von MAX_PLAYERS

Keslo · 7. Juni 2018, 22:39

seegras schrieb:

Schau dir mal folgendes Thema bzw. explizit den verlinkten Beitrag an

Code Optimierung

Alles klar, vielen Dank euch beiden! :rolleyes:

--

Hey,

ich hab da ein Problem, leider hab ich absolut keine Ahnung woran es liegen könnte.

Wenn ich mein Server Starte, läuft alles super, keine Warns/Errors von Crashdetect/MySQL etc...
Dann joine ich wie gewohnt auf mein Server, logge mich ein und logge mich wieder aus.
Wenn ich dann versuche zu joinen, tut er es nicht, sondern da steht dann einfach nur "Connecting to Server...."

Anschließend muss ich Server und Client neu starten, damit ich wieder connecten kann.

Hätte jemand eine Idee, was es evtl. sein könnte?

Kaliber · 7. Juni 2018, 22:41

Keslo schrieb:

Hätte jemand eine Idee, was es evtl. sein könnte?

Ja, dein Server schmiert ab, lade mal crashdetect und schaue dann in den server_log bzw poste den hier

Keslo · 7. Juni 2018, 22:47

Kaliber schrieb:

Ja, dein Server schmiert ab, lade mal crashdetect und schaue dann in den server_log bzw poste den hier

Das ist der Punkt, da steht nichts :huh:

Habs nochmal versucht, zwar connected er aber dann steht da nur noch: Connected to MeinServer...

Kaliber · 7. Juni 2018, 23:03

Tjaa...was sollen wir jetzt tun?

Unsere Wunderkugel befragen?

Offensichtlich ist dann etwas an deinem Code verkehrt, wenn wirklich nichts im Log steht

Angepinnt [SAMMELTHREAD] Kleine Scripting Fragen

C-Quellcode

Quellcode

Quellcode

Beitrag von Dr. Frauenarzt (28. Mai 2018, 19:02)

Quellcode

Quellcode

PAWN-Quellcode

iZAG2k18 schrieb:

C-Quellcode

Quellcode

Dr. Frauenarzt schrieb:

Quellcode

Beitrag von John_Dree (30. Mai 2018, 16:15)

Jeffry schrieb:

Kaliber schrieb:

C-Quellcode

Beitrag von John_Dree (31. Mai 2018, 10:12)

Beitrag von Dr. Frauenarzt (4. Juni 2018, 14:46)

PAWN-Quellcode

PAWN-Quellcode

Keslo schrieb:

C-Quellcode

PAWN-Quellcode

seegras schrieb:

Keslo schrieb:

Kaliber schrieb: