im an idiot

  • Welcome to skUnity!

    Welcome to skUnity! This is a forum where members of the Skript community can communicate and interact. Skript Resource Creators can post their Resources for all to see and use.

    If you haven't done so already, feel free to join our official Discord server to expand your level of interaction with the comminuty!

    Now, what are you waiting for? Join the community now!

  • LOOKING FOR A VERSION OF SKRIPT?

    You can always check out skUnity Downloads for downloads and any other information about Skript!

Status
Not open for further replies.
What is this supposed to do exactly? Your code is very repetitive
 
This is supposed to be a lottery script. You can choose 3 slots and an end sum is then added from the 3 slots. That means you click on a slot and that is, for example, 8 $, then you click on the 2 slot and that is then 3 $ and then you can only click on a slot with e.g. $ 6. Then a new GUI opens where all 3 slot amounts are added up and added to the player's account. My problem is that the amounts are not added correctly but that it is more than it should be.
 
Also ich hab noch kein Skript gesehen das so ineffizient ist wie das hier.

1. Wieso nutzt du SkQuery für GUIS
2. Wieso nutzt du normale Variablen keine List Variablen
3. Das hier
Code:
        set slot 0 of current inventory of player to emerald named "&7Klick"
        set slot 1 of current inventory of player to emerald named "&7Klick"
        set slot 2 of current inventory of player to emerald named "&7Klick"
        set slot 3 of current inventory of player to emerald named "&7Klick"
        set slot 4 of current inventory of player to emerald named "&7Klick"
        set slot 5 of current inventory of player to emerald named "&7Klick"
        set slot 6 of current inventory of player to emerald named "&7Klick"
        set slot 7 of current inventory of player to emerald named "&7Klick"
        set slot 8 of current inventory of player to emerald named "&7Klick"
kann man damit abkürzen
Code:
        set slot (intergers from 0 to 8) of current inventory of player to emerald named "&7Klick"
4. Wieso kopierst du denselben Code immer wieder unter dasselbe Event "clicked slot is 0:" und änders nur die Zahl anstatt abzufragen ob es einer von vielen Werten ist also
Code:
clicked slot is 0 or 1 or 2...:
 
Also als erstes, danke für die Antwort.
Zweitens tut es mir leid, dass ich schlecht im Skripten bin. Ich habe auch noch nie so einen unfreundlichen Supporter hier gesehen.
Das hier ist eine Seite, um Usern zu helfen! Nicht um deren Skript zu kritisieren.

Deine Antwort die du mir da gegeben hast, hat mich jetzt überhaupt nicht weiter gebracht. Ich weiß jetzt immer noch nicht, warum die Variable höher als das eigentliche Ergebnis ist.
 
Last edited:
1. Entschuldigen musst du dich nicht
2. Ich bin kein Support, Supporter = Unterstützer, offizielle Teammitglieder tragen den Moderator oder Administratoren Rang
3. Helfen = Kritisieren, wenn du auf der Arbeit jemand seine Arbeit schlecht machen siehst, als erfahrener Mitarbeiter gehst du auch hin und sagst was und lässt ihn nicht weiter so machen. Das ergibt kein Sinn, wie sollst du dich sonst verbessern? Denkst du ich hab meine Skript-Erfahrung durch Zuckerschlecken?
4. Da du anscheinend nun keine Lust mehr hast und den Code gelöscht hast, anstelle ihn einfach dort zu lassen oder eine verbesserte Variante reinzusenden kann ich dir nun auch nicht mehr helfen
 
Status
Not open for further replies.