[SAMMELTHREAD] Kleine Scripting Fragen

Xazor · 28. Mai 2018

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:

Code

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

Code

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

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

Aktualisiere diese gegebenenfalls.

John_Dree · 29. Mai 2018

@Xazor

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

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!");
}
}

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

Zitat von iZAG2k18

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

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

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

Code

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

Zitat von Dr. Frauenarzt

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

Code

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

Zitat von Jeffry

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

Aktualisiere diese gegebenenfalls.

Vielen Dank hat die Lösung eingebracht.

Zitat von Kaliber

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

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

Hallo,

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

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:

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

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

Code Optimierung

Kaliber · 5. Juni 2018

Zitat von Keslo

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

#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:

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

Gehen würde auch sowas wie:

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

Zitat von seegras

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

Code Optimierung

Alles klar, vielen Dank euch beiden!

--

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

Zitat von Keslo

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

Zitat von Kaliber

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
Habs nochmal versucht, zwar connected er aber dann steht da nur noch: Connected to MeinServer...

Kaliber · 7. Juni 2018

Tjaa...was sollen wir jetzt tun?

Unsere Wunderkugel befragen?

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

Keslo · 7. Juni 2018

Zitat von Kaliber

Tjaa...was sollen wir jetzt tun?

Unsere Wunderkugel befragen?

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

haha die gute alte Wunderkugel
Natürlich könnt ihr nicht alles wissen, aber ich dachte, vielleicht gab es ja schon vorfälle

trotzdem danke

//edit:
hab nochmal rumprobiert...
hab herausgefunden, dass wenn ich nicht connecten kann, dann aber wieder connecte, dann gehts einwandfrei, versuch
ich danach wieder zu connecten, gehts wieder nicht, dann wieder schon... was ist hier los
auf anderen servern, mit anderem script funkioniert alles einwandfrei...

Kaliber · 7. Juni 2018

Zitat von Keslo

was ist hier los

Ja wie schon erwähnt ist dies ein Skript Fehler.

Nun bräuchten wir Code...so von OnPlayerConnect, OnPlayerRequestClass und eben da wo du den Spieler Einloggen willst usw.

Oder du kannst auch auf eigene Faust ermitteln und mal überall prints verteilen und schauen, wie weit du kommst bzw wo der Code hängen bleibt

Wenn du das gezielt machst, kannst du genau feststellen, an welcher Zeile es hängen bleibt

[SAMMELTHREAD] Kleine Scripting Fragen

Beitrag von John_Dree (30. Mai 2018)

Beitrag von John_Dree (31. Mai 2018)

Beitrag von Dr. Frauenarzt (4. Juni 2018)

Beitrag von John_Dree (8. Juni 2018)