[SAMMELTHREAD] Kleine Scripting Fragen

.snx · 15. Juni 2017

if(!IsPlayerInRangeOfPoint(playerid, 10.0, 1260.3502, -1698.7749, 13.2889))return SendClientMessage(playerid,dunkelrot,"Du bist zu weit entfernt!");
if(pInfo[playerid][Fraktion] != 1)return SendClientMessage(playerid,dunkelrot,"Du bist kein Polizist!");
if(lspdschranke == false)
{
lspdschranke = true;
MoveObject(LSPDSchranke,1260.3502+8, -1698.7749, 13.2889,4);
SendClientMessage(playerid,dunkelgrün,"Schranke geöffnet!");
}
else if(lspdschranke == true)
{
MoveObject(LSPDSchranke,1260.3502, -1698.7749, 13.2889,4);
SendClientMessage(playerid,dunkelrot,"Schranke geschlossen!");
lspdschranke = false;
}

Kann man die Bools noch weiter kürzen? Will so wenig Ressourcen wie möglich verbrauchen und zeilensparend arbeiten.

**XeroX** · 15. Juni 2017

Würde von den resourcen her kein Untschied machen und immer versuchen alles in einer Zeile zu klatschen kann ehr die Lesbarkeit eschweren als das es wirklich hilft.

Hakaishin · 15. Juni 2017

Zitat von FlawGraphics

Warum hast du unter deinem SetPlayerPos ein Return drinne, schreib es so, und probiere es.

public OnPlayerCommandText(playerid, cmdtext[]){if(strcmp("/teleport", cmdtext, true, 10) == 0){SetPlayerPos (playerid,-1600.3464,853.8167,466.9222);}return 1;}

Unbenannt.PNG

Jeffry · 15. Juni 2017

Zitat von FlawGraphics

Kann man die Bools noch weiter kürzen? Will so wenig Ressourcen wie möglich verbrauchen und zeilensparend arbeiten.

Du könntest es so schreiben:
if(!IsPlayerInRangeOfPoint(playerid, 10.0, 1260.3502, -1698.7749, 13.2889)) return SendClientMessage(playerid,dunkelrot,"Du bist zu weit entfernt!");
if(pInfo[playerid][Fraktion] != 1) return SendClientMessage(playerid,dunkelrot,"Du bist kein Polizist!");
lspdschranke = !lspdschranke;
MoveObject(LSPDSchranke,1260.3502+(8*_:lspdschranke), -1698.7749, 13.2889,4);
if(lspdschranke) SendClientMessage(playerid,dunkelgrün,"Schranke geöffnet!");
else SendClientMessage(playerid,dunkelrot,"Schranke geschlossen!");

Zeile 3 tauscht praktisch immer den Wert zum Gegenteil (wenn true, dann false und andersherum). MoveObject nimmt die +8 nur dann, wenn das bool true (1) ist. Das "_:" entfernt den "bool" Tag, damit true als 1 interpretiert wird, sonst gibt es einen Tag Mismatch.
Die Nachrichten sollten klar sein.

Performance-technisch ändert das aber nicht wirklich was, es wird eher schlechter, da bei MoveObject eine Rechenoperation dazu kommt.
Zeilen-mäßig hast du natürlich weniger.

Ganz übertrieben kannst du es auch so schreiben:
if(!IsPlayerInRangeOfPoint(playerid, 10.0, 1260.3502, -1698.7749, 13.2889)) return SendClientMessage(playerid,dunkelrot,"Du bist zu weit entfernt!");
if(pInfo[playerid][Fraktion] != 1) return SendClientMessage(playerid,dunkelrot,"Du bist kein Polizist!");
MoveObject(LSPDSchranke,1260.3502+(8*_:(lspdschranke = !lspdschranke)), -1698.7749, 13.2889,4);
SendClientMessage(playerid, (lspdschranke) ? (dunkelgrün) : (dunkelrot), (lspdschranke) ? ("Schranke geöffnet!") : ("Schranke geschlossen!"));
Das ist dann aber nicht mehr auf den ersten Blick lesbar.

Hakaishin · 15. Juni 2017

Zitat von Monkys

Unbenannt.PNG

Kann mir jemand helfen, diesen Komischen Fehler zu beseitigen - Ich weiß leider nicht wo ran es liegt.

Madd92 · 15. Juni 2017

Zitat von Madd92

SetPlayerPos(playerid, -1600.3464, 853.8167, 466.9222); Leerzeichen raus.

Jeffry · 15. Juni 2017

Das hat mit den Leerzeichen nichts zu tun, die spielen keine Rolle.

Das Callback OnPlayerCommandText wird in dem Gamemode wahrscheinlich gar nicht aufgerufen, Grund dafür könnte ein verwendeter Command Prozessor (z. B. zcmd) sein, der die Ausführung verhindert, oder ein Filterscript.

Hakaishin · 15. Juni 2017

Es ist ein Mapping Filterscript drinnen

.snx · 15. Juni 2017

Benutzt du ZCMD denn? oder OCMD etc? Wenn ja, wäre es eine Überlegung mal alle Befehle mit diesen auszukommentieren(?), das Include oben im Script ebenfalls. Dann mal gucken ob es klappt.
Versuche aber am besten erst einmal Das Filterscript aus der Config zu nehmen, und es dann zu testen.

Hakaishin · 15. Juni 2017

OCMD

.snx · 15. Juni 2017

Dann probier mal das aus, was ich geschrieben habe im letzten post.

NoxiZ · 15. Juni 2017

kann mir hier jemand helfen?
Es geht nur darum für jede fraktion eine kasse erstellen zu lassen nur ich komm da nicht weiter

https://breadfish.de/index.php…d/238868-fraktionskassen/

Hakaishin · 16. Juni 2017

http://prntscr.com/fkhmrv

http://prntscr.com/fkhmz6

http://prntscr.com/fkhn3p

Kann mir da jmd helfen?

JustMe.77 · 16. Juni 2017

Befindet sich die Definition auch ganz oben im Script?

PS: Kopiere dein Code hier rein anstatt es mit Bildern hochzuladen, ansonsten bin ich nicht bereit dir zu helfen..

Hakaishin · 16. Juni 2017

Code

#define red 0xFF0000AA
#define green 0x33FF33AA
#define COLOR_INVIS 0xAFAFAF00
#define COLOR_SPEC 0xBFC0C200
#define NICESKY 0x99FFFFAA
#define AQUAGREEN 0x03D687FF
#define COLOR_ADD 0x63FF60AA
#define COLOR_ORANGE 0xFF9900AA
#define COLOR_GRAD1 0xB4B5B7FF
#define COLOR_GRAD2 0xBFC0C2FF
#define COLOR_GRAD3 0xCBCCCEFF
#define COLOR_GRAD4 0xD8D8D8FF
#define COLOR_GRAD5 0xE3E3E3FF
#define COLOR_REPORT 0x990000FF
#define COLOR_GRAD6 0xF0F0F0FF
#define COLOR_GREY 0xAFAFAFAA
#define COLOR_GREEN 0x33AA33AA
#define LIGHTGREEN 0x00FF00FF
#define COLOR_RED 0xAA3333AA
#define COLOR_BLUE 0x33CCFFAA
#define COLOR_RED2 0xAA3333AA
#define COLOR_LIGHTRED 0xFF6347AA
#define COLOR_LIGHTBLUE 0x33CCFFAA
#define COLOR_LIGHTGREEN 0x9ACD32AA
#define COLOR_LIGHTYELLOW 0xFFFF99FF
#define COLOR_YELLOW 0xFFFF00AA
#define COLOR_YELLOW2 0xF5DEB3AA
#define COLOR_WHITE 0xFFFFFFAA
#define COLOR_FADE1 0xE6E6E6E6
#define COLOR_FADE2 0xC8C8C8C8
#define COLOR_FADE3 0xAAAAAAAA
#define COLOR_FADE4 0x8C8C8C8C
#define COLOR_FADE5 0x6E6E6E6E
#define COLOR_PURPLE 0xC2A2DAAA
#define COLOR_DBLUE 0x2641FEAA
#define COLOR_ALLDEPT 0xFF8282AA
#define COLOR_NEWS 0xFFA500AA
#define COLOR_OOC 0xE0FFFFAA

Alles anzeigen

Code

public OnPlayerConnect(playerid)
{
	SendClientMessage(playerid,red,"Willkommen auf meinen Showroom Server!");
	return 1;
}

Code

Script.pwn(100) : error 017: undefined symbol "red"

NoxiZ · 17. Juni 2017

stock Fkassespeichern()
{
new fkasse[100];
format(fkasse,sizeof(fkasse),"/Fraktionskassen.txt");
if(dini_Exists(fkasse))
{
dini_IntSet(fkasse,"Grove",Kasse[Grove]);
}
return 1;
}

stock Fkasseladen()
{
new fkasse[100];
format(fkasse,sizeof(fkasse),"/Fraktionskassen.txt");
if(dini_Exists(fkasse))
{
Kasse[Grove] = dini_Int(fkasse,"Grove");
}
return 1;
}

Was muss ich bei nem stock, wo etwas für nichtspieler gespeichert werden soll, in die klammern packen und wie definier ich das im script das es funktioniert?

Includ · 17. Juni 2017

Zitat von NoxiZ

stock Fkassespeichern()
{
new fkasse[100];
format(fkasse,sizeof(fkasse),"/Fraktionskassen.txt");
if(dini_Exists(fkasse))
{
dini_IntSet(fkasse,"Grove",Kasse[Grove]);
}
return 1;
}

stock Fkasseladen()
{
new fkasse[100];
format(fkasse,sizeof(fkasse),"/Fraktionskassen.txt");
if(dini_Exists(fkasse))
{
Kasse[Grove] = dini_Int(fkasse,"Grove");
}
return 1;
}

Alles anzeigen

Soweit ich weiß kannst du die Klammern so lassen

NoxiZ · 17. Juni 2017

und wie lass ich die kasse dann bei gamemodeexit speichern ohne den error tag mismatch zu bekommen?

Includ · 17. Juni 2017

Code

public OnGameModeExit()
{
    savekasse();
	return 1;
}






stock savekasse()
{


}

Alles anzeigen

So sollte es gehen musst einfach das savekasse durch dein stock ersetzten

NoxiZ · 17. Juni 2017

trotzdem tag mismatch