[SAMMELTHREAD] Kleine Scripting Fragen

Romnus · 24. Januar 2019

Zitat von Kaliber
Du erstellst einen Timer für jeden Spieler und Loopst bei jedem Spieler durch alle Spieler o:
Daaavon ist abzuraten

//Edit:
Ahh wait, du willst, dass für jeden Spieler der Timer individuell gestartet wird.

Mach es so:
C
//Unter OnPlayerConnect
SetPVarInt(playerid,"reward_timer", SetTimerEx("PlayReward", 900000, true, "i", playerid));


//Unter OnPlayerDisconnect
KillTimer(GetPVarInt(playerid, "reward_timer"));


forward PlayReward(i);
public PlayReward(i)
{
    if(!PlayerInfo[i][LoggedIn] || isPlayerAFK(i)) return 1;
    PlayerInfo[i][Tokens] += 5;
    SFM(i,COLOR_COP,"Reward: {FFFFFF}Du bekommst 5 Tokens!");
    SavePlayer(i); //Are you sure about this?
    return 1;
}
Alles anzeigen
Bist du sicher, dass du da den Spieler direkt speichern magst?

Also kannst du machen, aber reicht es nicht unter OnPlayerDisconnect oder dient das der Server-Crash Vorbeugung?
Alles anzeigen

Vielen dank, funktioniert jetzt. Ja das mit dem Spieler speichern ist unnötig, habs jetzt bei OnPlayerDisconnect.

Dr. Frauenarzt · 24. Januar 2019

Der Compeiler meckert überall wo gebraucht steht.

C:\Users\Patrick\Desktop\Reallife\gamemodes\Reallife.pwn(20036) : warning 202: number of arguments does not match definition

new gebraucht = floatround(Tanke[i][tMaxLiterBenzin] - Tanke[i][tLiterBenzin]);
SendClientMessage(playerid, COLOR_WHITE,"{21DD00}Es konnten nur %d Liter Benzin abgeliefert werden, die Tankstelle ist voll.", gebraucht);

Muss man da kein float davor machen oder so?

Kaliber · 24. Januar 2019

Zitat von Dr. Frauenarzt

Muss man da kein float davor machen oder so?

Entweder fällt dir lesen schwer, oder das kopieren.

Habe ich da irgendwo in meinem Code SendClientMessage stehen?

Und jetzt frage dich, wieso das du das?

Theoretisch muss man da kein Float vor machen, da du ja ganze Liter haben willst und das über das floatround gerundet wird.

Und..meine letzte Frage, in welcher Zeile der Fehler auftritt sagst du mir nicht und gibst uns keine Debug Informationen...

Sub Royal · 24. Januar 2019

Ist das angeben eines :default wertes bei switch sinvoll? Bzw. warum macht man das? Oder dient das einfach nur als "else" ?

Dr. Frauenarzt · 24. Januar 2019

Habe SCM ersetzt wie gewohnt wen ich

#define SCME(%0,%1,%2,%3) format(string,sizeof(string),%2,%3),SCM(%0,%1,string)

Oben im Script einbaue findet er SCM nicht. Den anderen fehler habe ich nun behoben habe was vergessen^^

Kaliber · 24. Januar 2019

Zitat von Sub Royal

Ist das angeben eines :default wertes bei switch sinvoll? Bzw. warum macht man das? Oder dient das einfach nur als "else" ?

Dient einfach nur als else.

Muss man nicht angeben, der Code darunter würde ja quasi einfach weiter laufen.

Zitat von Dr. Frauenarzt

Oben im Script einbaue findet er SCM nicht

Das muss natürlich unter das SCM oder schreibe es so:

#define SCME(%0,%1,%2,%3) format(string,sizeof(string),%2,%3),SendClientMessage(%0,%1,string)

Sub Royal · 24. Januar 2019

Alles klar, danke dir!

Habe mal eine Frage:
Ist es machbar NPC, bzw. Bots zu bauen, welche bspw. als annehmbare Gegner in einem Deathmatch dienen koennen?
Die sich also auf den Spieler zu bewegen, man diese vernichten kann, und auch den Gegner angreifen?

Kaliber · 24. Januar 2019

Zitat von Sub Royal

Ist es machbar NPC, bzw. Bots zu bauen, welche bspw. als annehmbare Gegner in einem Deathmatch dienen koennen?
Die sich also auf den Spieler zu bewegen, man diese vernichten kann, und auch den Gegner angreifen?

Jup, das geht mit FCNPC.

Zwar etwas fricklig, aber es wurde mal n gutes Include realisiert, was dir vielleicht weiter hilft: https://forum.sa-mp.com/showthread.php?t=663155

Sub Royal · 24. Januar 2019

Oha danke! Ich weiss noch, vor einigen Jahren war sowas Utopie.
Dann nehme ich mir das in den naechsten Wochen als Projekt vor

Kaliber · 24. Januar 2019

Zitat von Sub Royal

vor einigen Jahren war sowas Utopie.

Awas, gab schon vor 5 Jahren, wenn nicht noch länger her Zombie Server, die einen angegriffen haben oder so.

Halt alles um einiges schlechter realisiert, aber gegeben hat es das schon.

Gerade in der englischen bzw russischen Szene xD

Aber ich erinnere mich auch mal an einen Deutschen Server von Beavis und Blackace oder so xD

Sub Royal · 24. Januar 2019

Hehe alles klar.
Ich habe mit NPC nur einmal zutun gehabt, undzwar etwa 2010, da habe ich etwa 25 NPCs auf meinen Server gehabt, das Ding hatte 100% CPU Auslastung, und ist immer wieder abgeschmiert Seitdem habe ich es nie wieder probiert.

pgsmith · 25. Januar 2019

Hallo, jemand da der mir evtl weiterhelfen könnte? Habe vor einiger Zeit mit einem eigenen Gamemode angefangen, lerne aber noch ziemlich viel. Viel bekomme ich mit dem SAMP Wiki und Beiträgen hier raus selbst gelöst, aber bei folgendem komme ich nicht weiter. Für das Admsystem hatte/habe ich mir das Selfmade für Anfänger, bzw zum weiter Scripten angesehen/teilweise übernommen, nun kommt aber im /a(dmchat) nur 1/4 meiner Nachricht im Chat an.

Mein /a Code:
ocmd:a(playerid, params[])
{
new atext[128], astring[128];
if(PlayerInfo[playerid][pAdmin] < 1) return SendClientMessage(playerid, COLOR_RED, ""err""COLOR_HEX_WHITE""a404"");
if(sscanf(params, "s", atext)) return SendClientMessage(playerid, COLOR_WHITE, "[Nutzung] /a(dminchat) [Text]");
format(astring, sizeof(astring), "%s "COLOR_HEX_YELLOW"%s"COLOR_HEX_WHITE": %s", admName(playerid), PlayerName(playerid), atext);
SendToAdms(COLOR_WHITE, astring);
return 1;
}

SendToAdms:
stock SendToAdms(COLOR, msg[])
{
for(new i = 0;i < MAX_PLAYERS; i++)
if(PlayerInfo[i][pAdmin] >= 1) {
SendClientMessage(i, COLOR, msg); }
return 0;
}

Habe schon viel rumprobiert, bspw auch im Befehl die astring Größe erhöht (gleiches mit atext); Ohne Erfolg.

Kaliber · 25. Januar 2019

Zitat von pgsmith

jemand da der mir evtl weiterhelfen könnte

Schauen wir uns das doch mal an

sscanf scheint glaube bei dir veraltet zu sein (du scheinst die Funktion zu nutzen, aber das Plugin ist um einiges besser), in diesem Fall, wo man auch nur einen Parameter nutzt, sollte man gänzlich darauf verzichten (erst bei einem String).

Schreib das alles mal so:

C

ocmd:a(playerid, params[])
{
    if(PlayerInfo[playerid][pAdmin] < 1) return SendClientMessage(playerid, COLOR_RED, ""err""COLOR_HEX_WHITE""a404"");
    if(!params[0] || params[0] == '\1') return SendClientMessage(playerid, COLOR_WHITE, "[Nutzung] /a(dminchat) [Text]");
    new string[144];
    format(string, sizeof(string), "%s "COLOR_HEX_YELLOW"%s"COLOR_HEX_WHITE": %s", admName(playerid), PlayerName(playerid), params);
    SendToAdms(COLOR_WHITE, string);
    return 1;
}


stock SendToAdms(COLOR, const msg[])
{
    for(new i=GetPlayerPoolSize(); i!=-1; i--)
    {
        if(IsPlayerConnected(i) && PlayerInfo[i][pAdmin] >= 1) SendClientMessage(i, COLOR, msg);
    }
    return 1;
}

Alles anzeigen

pgsmith · 25. Januar 2019

Achja... Den Umstieg aufs Plugin zwischendurch hatte ich total vergessen. Hätte man natürlich anpassen sollen
Bin mittlerweile aufm Sprung, teste das später & schaue mir auch noch mal die genauere Funktionsweise des sscanf Plugins an.

Danke dir soweit + melde mich später

Dr. Frauenarzt · 25. Januar 2019

@Kaliber

C

if(Vehicle[vehicleid][vehGeladen][2] == 3)
				{
				    new gebraucht = floatround(Tanke[i][tMaxLiterBenzin] - Tanke[i][tLiterBenzin]);
				    new kosten = Tanke[i][tBenzinAnkauf]*gebraucht;
				    if(Tanke[i][tKasse] < kosten) return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Die Tankstelle hat nicht genügend Geld für den Benzin.");
				    if(!gebraucht) return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Dieses Tankstelle benötigt keinen Benzin mehr.");
				    if(Vehicle[vehicleid][vehGeladen][2] == 0) return SendClientMessage(playerid, COLOR_WHITE, "{AFAFAF}Dein Tank Anhänger hat kein Benzin mehr geladen.");
					if(Vehicle[trailer][vehProds] > gebraucht)
					{
					    Vehicle[trailer][vehProds] -= gebraucht, Tanke[i][tMaxLiterBenzin] += floatadd(gebraucht, Tanke[i][tLiterBenzin]);
					    format(string, sizeof(string), "{21DD00}Es konnten nur %d Liter Benzin abgeliefert werden, die Tankstelle ist voll.", gebraucht);
					    SendClientMessage(playerid, COLOR_WHITE, string);
					}
					else Tanke[i][tMaxLiterBenzin] += Vehicle[trailer][vehProds], Vehicle[vehicleid][vehGeladen][0] = 0, Vehicle[trailer][vehProds] = 0;


					format(string, sizeof(string), "{21DD00}Gehalt: %d$", kosten);
					SendClientMessage(playerid, COLOR_WHITE, string);
					format(string, sizeof(string), "{21DD00}Benzin: %d/2500", Vehicle[trailer][vehProds]);
					SendClientMessage(playerid, COLOR_WHITE, string);
					Tanke[i][tKasse] -= kosten, GiveMoney(playerid, kosten);
					UpdateTankLabel(i, Tanke[i][tOwned]);
					printf("Tankstelle: Diese Tankstelle hat %d Benzin erhalten.", floatadd(gebraucht, Tanke[i][tLiterBenzin]));
				}

Alles anzeigen

Tankstelle: Diese Tankstelle hat 1176256512 Benzin erhalten.

Werde den Code noch genauer anpassen keine sorge und danke für deine hilfe mit den abkürzungen fällt mir leichter

Kaliber · 25. Januar 2019

Zitat von Dr. Frauenarzt

printf("Tankstelle: Diese Tankstelle hat %d Benzin erhalten.", floatadd(gebraucht, Tanke[i][tLiterBenzin]))

floatadd gibt immer einen Float Wert zurück.

Also musst du:

printf("Tankstelle: Diese Tankstelle hat %d Benzin erhalten.", floatround(floatadd(gebraucht, Tanke[i][tLiterBenzin])));

machen.

Dr. Frauenarzt · 25. Januar 2019

Tankstelle: Diese Tankstelle hat 10000 Benzin erhalten.

Habe 2500 Liter abgeladen

Kaliber · 25. Januar 2019

Zitat von Dr. Frauenarzt

Habe 2500 Liter abgeladen

Ja und jetzt?

Wenn, dann printe dir ALLE Variablen vorher und nacher..und sag genau, was nicht stimmt...

Jeffry · 25. Januar 2019

Vehicle[trailer][vehProds] -= gebraucht, Tanke[i][tLiterBenzin] += floatadd(gebraucht, Tanke[i][tLiterBenzin]);
zu:
Vehicle[trailer][vehProds] -= gebraucht, Tanke[i][tLiterBenzin] = Tanke[i][tMaxLiterBenzin];

else Tanke[i][tMaxLiterBenzin] += Vehicle[trailer][vehProds], Vehicle[vehicleid][vehGeladen][0] = 0, Vehicle[trailer][vehProds] = 0;
zu:
else Tanke[i][tLiterBenzin] += Vehicle[trailer][vehProds], Vehicle[vehicleid][vehGeladen][0] = 0, Vehicle[trailer][vehProds] = 0;

printf("Tankstelle: Diese Tankstelle hat %d Benzin erhalten.", floatadd(gebraucht, Tanke[i][tLiterBenzin]));
zu:
printf("Tankstelle: Diese Tankstelle hat %d Benzin erhalten.", floatround(floatsub(gebraucht, Tanke[i][tLiterBenzin])));

Dr. Frauenarzt · 25. Januar 2019

Tankstelle: Diese Tankstelle hat 2000 Liter Benzin gebraucht.
Tankstelle: Diese Tankstelle hat 5000 Liter Benzin erhalten.

Das Meine ich gebraucht hat die Tankstelle nur 2000 Liter Benzin aber es wurden 5000 anscheind abgeladen aber es wurden ja nur 2000 abgegeben.
Habe 2500 geladen. habe abgeladen also wurden 2000 abgezogen somit waren am schluss 500 Liter noch vorhanden.

Wie kann das dann zu stande komm das 5000 Liter Benzin abgeladen wurden obwohl es nur 2000 waren.

Spoiler anzeigen