[SAMMELTHREAD] Kleine Scripting Fragen

Maho · 16. Januar 2024

GitHub - maddinat0r/samp-discord-connector: SA:MP plugin which controls a Discord bot

SA:MP plugin which controls a Discord bot. Contribute to maddinat0r/samp-discord-connector development by creating an account on GitHub.

github.com

Durchgelesen? Ich weiß ob es ein zsmhang haben kann.

Mr.Pawn · 16. Januar 2024

Servus zusammen!

Dumme Frage: Einer eine Idee wie ich auf einem Mac mit Pawn arbeiten kann ? Hoffe, ich kann die Frage hier stellen. Finde irgendwie keine anständigen Ergebnisse.

Lieben Grüße

seegras · 16. Januar 2024

Prizipiell kannst du jede IDE verwenden und dann das ganze durch den Compiler jagen, Anleitung findest du bspw. hier

Gamemodes auf einem Li... | bithost Docs

Compiler herunterladen Wir finden den Compiler auf GitHub, einfach den aktuellsten Release herunter...

docs.bithost.co

Ob die Dependencies 1:1 auf macOS so zu finden sind kann ich allerdings nicht sagen - Versucht macht Klug!

**Andosius** · 16. Januar 2024

VSCode mit dem PAWN Plugin von Southclaws.

Kompilieren kannst du, indem du den Compiler selbst baust (https://github.com/pawn-lang/c…-Source#building-on-macos) oder eine ältere Version herunter lädst (https://github.com/pawn-lang/compiler/releases).

Musst wahrscheinlich bisschen was konfigurieren aber dann sollte es klappen.

DJSatrox · 18. Januar 2024

Guten Morgen,

Macht es viel aus wenn ich bei dieses hier es so mache ?

Code

else return SendClientMessage(playerid,-1,"Hi ^^");

oder doch lieber so ?

Code

else SendClientMessage(playerid,-1,"Hi ^^");

Kaliber · 18. Januar 2024

Zitat von DJSatrox

Macht es viel aus wenn ich bei dieses hier es so mache ?

Das kommt darauf an, was darunter geschieht

Falls darunter auch ein return 1; steht, dann macht es keinen Unterschied.

Generell ist es eher dafür gedacht bestimmte Clean Code Prinzipien einzuhalten.

Hier mal ein Beispiel, was ich oft sehe:

C++

if(IsPlayerInRangeOfPoint(playerid,x,y,z)) {
    new a=random(10);
    if(a > 7) {
        if(y > 20) {
            if(IsPlayerInAnyVehicle(playerid)) {
                if(IsPlayerDriver(playerid)) {
                    if(IsPlayerLeader(playerid)) {
                        for(new i; i<MAX_HOUSES; i++)
                        {
                            if(i == 5 && IsPlayerInRangeOfPoint(playerid,i,i,i))
                            {
                                if(HasItem(playerid)) {
                                    return SendClientMessage(playerid,-1,"ok");
                                }
                            }
                        }
                        SendClientMessage(playerid,-1,"Nicht im Haus");
                    }
                    else {
                        SendClientMessage(playerid,-1,"Kein Leader");
                    }
                }
                else {
                    SendClientMessage(playerid,-1,"Kein Fahrer");
                }
            }
            else {
                SendClientMessage(playerid,-1,"nicht im Auto");
            }
        } else {
            SendClientMessage(playerid,-1,"nicht ok");
        }
    } else {
        SendClientMessage(playerid,-1,"nicht geklappt");
    }
}
SendClientMessage(playerid,-1,"nicht am ort!");

Alles anzeigen

Das ist ein sogenannter if-Tree des Todes 😂 Meistens geht das ja noch viel weiter.

Und oft ist das auch logisch nicht in der Reihenfolge, so wird meistens zum Schluss geprüft, ob man wirklich Dinge darf, obwohl vorher schon Dinge getriggert wurden.

Durch das return und gezielte Negation ist es möglich das aufzuräumen und deutlich übersichtlicher zu gestalten, dass du keine Ebenen mehr hast.

C++

if(!IsPlayerLeader(playerid)) return SendClientMessage(playerid,-1,"Kein Leader");
if(!IsPlayerInRangeOfPoint(playerid,x,y,z)) return SendClientMessage(playerid,-1,"nicht am ort!");
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid,-1,"nicht im Auto");
if(!IsPlayerDriver(playerid)) return SendClientMessage(playerid,-1,"Kein Fahrer");
if(random(10) <= 7) return SendClientMessage(playerid,-1,"nicht geklappt");
if(y <= 20) return SendClientMessage(playerid,-1,"nicht ok");
if(!HasItem(playerid)) return SendClientMessage(playerid,-1,"kein item");
if(!InHouse(playerid)) return SendClientMessage(playerid,-1,"Nicht im Haus");

SendClientMessage(playerid,-1,"ok");
return 1;

Alles anzeigen

Alle negativen / Fehler - Fälle fängst du quasi vorne ab und übrig bleibt der Happy-Path.

Dann ist das auch sehr übersichtlich und eindeutig.

Aber Performance-Technisch ist da kein Unterschied. Nur deutlich übersichtlicher, da ja auch hier auf else etc verzichtet werden kann.

DJSatrox · 18. Januar 2024

Zitat von Kaliber

Das kommt darauf an, was darunter geschieht

Falls darunter auch ein return 1; steht, dann macht es keinen Unterschied.

Generell ist es eher dafür gedacht bestimmte Clean Code Prinzipien einzuhalten.

Hier mal ein Beispiel, was ich oft sehe:

C++

if(IsPlayerInRangeOfPoint(playerid,x,y,z)) {
    new a=random(10);
    if(a > 7) {
        if(y > 20) {
            if(IsPlayerInAnyVehicle(playerid)) {
                if(IsPlayerDriver(playerid)) {
                    if(IsPlayerLeader(playerid)) {
                        for(new i; i<MAX_HOUSES; i++)
                        {
                            if(i == 5 && IsPlayerInRangeOfPoint(playerid,i,i,i))
                            {
                                if(HasItem(playerid)) {
                                    return SendClientMessage(playerid,-1,"ok");
                                }
                            }
                        }
                        SendClientMessage(playerid,-1,"Nicht im Haus");
                    }
                    else {
                        SendClientMessage(playerid,-1,"Kein Leader");
                    }
                }
                else {
                    SendClientMessage(playerid,-1,"Kein Fahrer");
                }
            }
            else {
                SendClientMessage(playerid,-1,"nicht im Auto");
            }
        } else {
            SendClientMessage(playerid,-1,"nicht ok");
        }
    } else {
        SendClientMessage(playerid,-1,"nicht geklappt");
    }
}
SendClientMessage(playerid,-1,"nicht am ort!");

Alles anzeigen

Das ist ein sogenannter if-Tree des Todes 😂 Meistens geht das ja noch viel weiter.

Und oft ist das auch logisch nicht in der Reihenfolge, so wird meistens zum Schluss geprüft, ob man wirklich Dinge darf, obwohl vorher schon Dinge getriggert wurden.

Durch das return und gezielte Negation ist es möglich das aufzuräumen und deutlich übersichtlicher zu gestalten, dass du keine Ebenen mehr hast.

C++

if(!IsPlayerLeader(playerid)) return SendClientMessage(playerid,-1,"Kein Leader");
if(!IsPlayerInRangeOfPoint(playerid,x,y,z)) return SendClientMessage(playerid,-1,"nicht am ort!");
if(!IsPlayerInAnyVehicle(playerid)) return SendClientMessage(playerid,-1,"nicht im Auto");
if(!IsPlayerDriver(playerid)) return SendClientMessage(playerid,-1,"Kein Fahrer");
if(random(10) <= 7) return SendClientMessage(playerid,-1,"nicht geklappt");
if(y <= 20) return SendClientMessage(playerid,-1,"nicht ok");
if(!HasItem(playerid)) return SendClientMessage(playerid,-1,"kein item");
if(!InHouse(playerid)) return SendClientMessage(playerid,-1,"Nicht im Haus");

SendClientMessage(playerid,-1,"ok");
return 1;

Alles anzeigen

Alle negativen / Fehler - Fälle fängst du quasi vorne ab und übrig bleibt der Happy-Path.

Dann ist das auch sehr übersichtlich und eindeutig.

Aber Performance-Technisch ist da kein Unterschied. Nur deutlich übersichtlicher, da ja auch hier auf else etc verzichtet werden kann.

Alles anzeigen

Sehr verständlich, danke dir

//e

Wie währe es denn in diesen fall hier richtig da es nur mit else geht wo ich aber auch drauf verzichten will

Code

if(RechteInfo[playerid][Admin] >= 1)return true;

derzeit geht es nur so

Code

cmd:blub(playerid,params[])
{
    if(RechteInfo[playerid][Admin] >= 1)
    {
        jo ist admin kann machen 
    }
    else KeineRechte  // ist so definiert
    return 1;
}

Muss ich da ein stock anwenden um es zu umgehen wie zb

Code

if(AdminLevel(playerid,1))

Kaliber · 18. Januar 2024

Nein, du musst die Logik nur drehen.

Also einfach negieren, das Gegenteil anwenden.

Anstatt den Fall, dass das Admin Level größer Gleich 1 sein soll, sagst du, wenn sein Admin Level kleiner 1 ist, (bzw in diesem Fall einfach 0), dann darf er es nicht tun.

C++

cmd:blub(playerid,params[])
{
    if(RechteInfo[playerid][Admin] < 1) return KeineRechte; // oder !RechteInfo[playerid][Admin]
    
    // Jo kann machen
    return 1;
}

DJSatrox · 18. Januar 2024

Zitat von Kaliber
Nein, du musst die Logik nur drehen.

Also einfach negieren, das Gegenteil anwenden.

Anstatt den Fall, dass das Admin Level größer Gleich 1 sein soll, sagst du, wenn sein Admin Level kleiner 1 ist, (bzw in diesem Fall einfach 0), dann darf er es nicht tun.
C++
cmd:blub(playerid,params[])
{
    if(RechteInfo[playerid][Admin] < 1) return KeineRechte; // oder !RechteInfo[playerid][Admin]
    
    // Jo kann machen
    return 1;
}
Alles anzeigen

als stadt = nur < 1 was dann auch für höher als 1 steht dann ?

/e

habe es getestet jedoch leider mit beiden verianten kein erfolg

die mit ! davor ergibt das

Code

warning 213: tag mismatch: expected tag "bool", but found none ("_")

// Habs gelöst, habe nicht richtig aufgepasst, hatte die "<" noch falschrum, geht nun ohne "!" davor, danke dir

Kaliber · 19. Januar 2024

Du musst die Umkehroperation einfach anwenden.

Bei + ist es -

Bei / ist es *

Bei > ist es <=

Bei < ist es >=

Willst du quasi ausdrücken:

C++

if(Admin > 5)
{
    //Ok
}
else KeineRechte();

//Ist die Umkehroperation von >, <= ergo
if(Admin <= 5) return KeineRechte();
// Ok

DJSatrox · 19. Januar 2024

Zitat von Kaliber
Du musst die Umkehroperation einfach anwenden.

Bei + ist es -
Bei / ist es *
Bei > ist es <=
Bei < ist es >=

Willst du quasi ausdrücken:
C++
if(Admin > 5)
{
    //Ok
}
else KeineRechte();

//Ist die Umkehroperation von >, <= ergo
if(Admin <= 5) return KeineRechte();
// Ok
Alles anzeigen

Siehe mein Post den edit hatte es zu spät gemerkt xD

Danke dir aber nochmal

DJSatrox · 27. Januar 2024

Guten Morgen

Gibt es eine möglichkeit, oder ein Tool, was mir die Arbeit, mit hoch nummerieren erleichtert?

zb

Code

    lspd1int[1] = CreateDynamicObject(2310, -274.975494, 899.826477, -37.414501, 0.000000, 0.000000, 180.000000);
    SetDynamicObjectMaterial(lspd1int[1], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
    SetDynamicObjectMaterial(lspd1int[1], 1, 19480, "signsurf", "sign", 0x00000000);
    lspd1int[2] = CreateDynamicObject(2310, -274.975494, 900.646484, -37.414501, 0.000000, 0.000000, 180.000000);
    SetDynamicObjectMaterial(lspd1int[2], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
    SetDynamicObjectMaterial(lspd1int[2], 1, 19480, "signsurf", "sign", 0x00000000);
    lspd1int[3] = CreateDynamicObject(2310, -274.975494, 901.446472, -37.414501, 0.000000, 0.000000, 180.000000);
    SetDynamicObjectMaterial(lspd1int[3], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
    SetDynamicObjectMaterial(lspd1int[3], 1, 19480, "signsurf", "sign", 0x00000000);
    lspd1int[4] = CreateDynamicObject(2310, -274.975494, 902.266479, -37.414501, 0.000000, 0.000000, 180.000000);
    SetDynamicObjectMaterial(lspd1int[4], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
    SetDynamicObjectMaterial(lspd1int[4], 1, 19480, "signsurf", "sign", 0x00000000);
    lspd1int[usw...] = CreateDynamicObject(2310, -274.995513, 899.826477, -37.434501, 0.000000, 0.000000, 180.000000);
    SetDynamicObjectMaterial(lspd1int[usw..], 0, 18646, "matcolours", "grey-93-percent", 0x00000000);
    SetDynamicObjectMaterial(lspd1int[usw..], 1, 19480, "signsurf", "sign", 0x00000000);

Alles anzeigen

Bei über 400 Objecte ist es echt müheselig, alles zu nummerieren.

Kaliber · 27. Januar 2024

Zitat von DJSatrox

Gibt es eine möglichkeit, oder ein Tool, was mir die Arbeit, mit hoch nummerieren erleichtert?

Das kommt darauf an, was genau du tun magst.

Die ID bei Objekten, als auch bei Fahrzeugen, werden sequenziell erstellt. Also einfach immer +1.

Diese Eigenschaft lässt sich super ausnutzen.

Willst du zum Beispiel immer die selben 2 Materialen setzen, bietet sich extrem ein 2 Pointer Ansatz an.

Das heißt, wir nehmen das Erste und Letzte Objekt und kennen nun alle Objekte die dazwischen liegen.

Sprich, sowas:

C++

new firstLSPD = CreateDynamicObject(...);
CreateDynamicObject(...);
CreateDynamicObject(...);
CreateDynamicObject(...);
new lastLSPD = CreateDynamicObject(...);

for(new obj=firstLSPD; obj<=lastLSPD; obj++) //Geht alle Objekte durch
{
    SetDynamicObjectMaterial(obj, 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
    SetDynamicObjectMaterial(obj, 1, 19480, "signsurf", "sign", 0x00000000);
}

Alles anzeigen

Falls dies nicht anwendbar sein sollte, weil die Texturen zu komplex für bestimmte Objekte sind, empfehlt sich ein einfacher auto-Counter:

C++

new i=-1; //Wichtig bei -1 zu starten
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 899.826477, -37.414501, 0.000000, 0.000000, 180.000000);
SetDynamicObjectMaterial(lspd1int[i], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
SetDynamicObjectMaterial(lspd1int[i], 1, 19480, "signsurf", "sign", 0x00000000);
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 900.646484, -37.414501, 0.000000, 0.000000, 180.000000);
SetDynamicObjectMaterial(lspd1int[i], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
SetDynamicObjectMaterial(lspd1int[i], 1, 19480, "signsurf", "sign", 0x00000000);
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 901.446472, -37.414501, 0.000000, 0.000000, 180.000000);

So musst du nicht manuell zählen, sondern er macht das automatisch.

Bei Fahrzeugen zum Beispiel, ist der erste Ansatz aber super effizient, da wenn man prüfen mag, ob der Spieler z.B. in einem LSPD Fahrzeug ist, keine Schleifen mehr benötigt, sondern einfach lspdCar[0] <= vehicleid <= lspdCar[1] nutzen kann und somit sofort weiß, ob er drinnen ist oder nicht.

DJSatrox · 27. Januar 2024

Zitat von Kaliber
Falls dies nicht anwendbar sein sollte, weil die Texturen zu komplex für bestimmte Objekte sind, empfehlt sich ein einfacher auto-Counter:
C++
new i=-1; //Wichtig bei -1 zu starten
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 899.826477, -37.414501, 0.000000, 0.000000, 180.000000);
SetDynamicObjectMaterial(lspd1int[i], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
SetDynamicObjectMaterial(lspd1int[i], 1, 19480, "signsurf", "sign", 0x00000000);
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 900.646484, -37.414501, 0.000000, 0.000000, 180.000000);
SetDynamicObjectMaterial(lspd1int[i], 0, 14415, "carter_block_2", "mp_shop_floor2", 0x00000000);
SetDynamicObjectMaterial(lspd1int[i], 1, 19480, "signsurf", "sign", 0x00000000);
lspd1int[++i] = CreateDynamicObject(2310, -274.975494, 901.446472, -37.414501, 0.000000, 0.000000, 180.000000);
So musst du nicht manuell zählen, sondern er macht das automatisch.

Bei Fahrzeugen zum Beispiel, ist der erste Ansatz aber super effizient, da wenn man prüfen mag, ob der Spieler z.B. in einem LSPD Fahrzeug ist, keine Schleifen mehr benötigt, sondern einfach lspdCar[0] <= vehicleid <= lspdCar[1] nutzen kann und somit sofort weiß, ob er drinnen ist oder nicht.

Hey ich habe es jetzt so gemacht aber nun spawnt nix mehr an objecte ?

Code

    new hplus=-1,hotel[301];
    hotel[++hplus] = CreateDynamicObjectEx(4033, 1721.875000, -1643.054687, 12.734397, 0.000000, 0.000000, 0.000000,500.00,500.00);
    SetDynamicObjectMaterial(hotel[++hplus], 1, 4593, "buildblk55", "snpdsqdoor", 0x00000000);
    SetDynamicObjectMaterial(hotel[++hplus], 2, 7555, "bballcpark1", "ws_carparknew2", 0x00000000);
    SetDynamicObjectMaterial(hotel[++hplus], 3, 14847, "mp_policesf", "mp_cop_tile", 0xFFFFFFFF);
    SetDynamicObjectMaterial(hotel[++hplus], 5, 6873, "vgnshambild1", "fitzwallvgn6_256", 0xFFFFFFFF);
    SetDynamicObjectMaterial(hotel[++hplus], 6, 14847, "mp_policesf", "mp_cop_vinyl", 0xFFFFFFFF);

Kaliber · 27. Januar 2024

@DJSatrox musst genau schauen, nur bei CreateDynamicObject natürlich den counter erhöhen

Bei SetDynamicObjectMaterial verwendest du i direkt.

#Handy

Kaliber · 28. Januar 2024

Genau

[SAMMELTHREAD] Kleine Scripting Fragen

Beitrag von kleinersampzocker (23. Januar 2024)

Beitrag von kleinersampzocker (24. Januar 2024)

Beitrag von DJSatrox (27. Januar 2024)

Beitrag von DJSatrox (27. Januar 2024)