Ranking

Zawsze lepiej dodac wait 1 tick command /easy: trigger: wait 1 tick set {_gui} to a new chest inventory with 1 row with name "&a&lMAPY" set slot 0 of {_gui} to lime wool named "&a&lEASY" with lore "&1BlueJump" set slot 1 of {_gui} to lime wool named "&a&lEASY" with lore "&1Bisek" set slot 2 of {_gui} to lime wool named "&a&lEASY" with lore "&1Fly Wool" set slot 3 of {_gui} to lime wool named "&a&lEASY" with lore "&1Sprint" set slot 4 of {_gui} to lime wool named "&a&lEASY" with lore "&1BlueWool" set slot 5 of {_gui} to lime wool named "&a&lEASY" with lore "&12-ki" set slot 6 of {_gui} to lime wool named "&a&lEASY" with lore "&1MossyJump" set slot 7 of {_gui} to lime wool named "&a&lEASY" with lore "&1OrangeJuice" set slot 8 of {_gui} to lime wool named "&a&lEASY" with lore "&1Caves" wait 1 tick open {_gui} to player

Krytyka jest bardzo potrzebna w takich projektach. Tak naprawdę to dzięki niej ktoś może się wiele nauczyć. if (sender != null) { to nie będzie nullem i tak czy siak, co najwyżej sender będzie albo graczem albo konsolą for (int i = 0; i < 101; ++i) { for (Player p2 : Bukkit.getOnlinePlayers()) { p2.sendMessage(" "); } } 100x pobierasz każdego gracza po kolei i wysyłasz mu wiadomośc, zamiast stworzyć sobie w konstruktorze klasy jakiś StringBuilder i tylko go wysyłać #przykład private final StringBuilder emptyMessage = new StringBuilder(); public JakasKlasa() { for (int i = 0; i < 101; i++) { emptyMessage.append(" \n"); } #gdzieś przy czyszczeniu for (Player player : Bukkit.getOnlinePlayers()) { player.sendMessage(emptyMessage.toString()); } } WTF public Helpop(PluginController pluginController){} Tu znowu public MeCommand(PluginController plugin) { Nazwy metod zaczynają się od małej litery:c private int SecondsDelay(){ W ten sposób porównuje się enumy a nie klasy:< if(sender!=invited) { XDD package me.opkarol.opchat.utils; public class BukkitUtils { } Dlaczego integer jest w formie Obiektu? public static String argBuilder(String[] args, Integer removeArgumentsNumber){ Po co te event priority o.0? @EventHandler(priority = EventPriority.LOW) MemoryLeak halo public static HashMap<Player, Player> lastPlayer = new HashMap<Player, Player>(); Jest wiele więcej błędów ale nie chce mi się już wymieniać. 1. Za dużo metod statycznych, brak programowania obiektowego 2. Dziwne nazwy klas/dziwne metody 3. Nazewnictwo jednej klasy z małej litery 4. Pobieranie żywcem z configu, tragedia, tak się nie robi. Zrób sobie jakaś klase Config, ConfigLoader, Settings co ci tam pasuje i przy starcie serwera zapisuj wszystkie stringi a potem je tylko wczytuj. Zobacz chociażby na kod paweu: https://github.com/paweUdev/paweU-Safes/blob/master/src/main/java/pl/paweu/safes/config/Settings.java

To już byłoby zdecydowanie prędzej w ramach drugiej części dla nieco bardziej zaawansowanych, a już nie wspominam o tym ostatnim. Chciałbyś zapisać metadatę w graczu, tak? Warto byłoby również wspomnieć o 'hotbar button' przydatne po sprawdzeniu czy 'click type' to 'number key'.

GUI w Skript Czym jest GUI? Jest to skrzynia z wcześniej ustalonymi przedmiotami, z której nie możesz wyciągać przedmiotów, lecz możesz np. przenieść się na inny serwer klikając w odpowiedni przedmiot. Przykładowe GUI: Dlaczego nie powinieneś korzystać z SkQuery do tworzenia GUI? SkQuery posiada sporo błędów, jednym z nich jest ukryta możliwość wyciągania przedmiotów z GUI robionego za pomocą tego addonu. Jakie addony są potrzebne do zrobienia GUI? Nie potrzebujesz żadnych addonów, wystarczy sam Skript. Jak zrobić GUI w Skripcie? Jak w każdym skrypcie, na początku musi być jakiś event. W przykładzie użyję komendę: (Zapamiętaj, że przy dodawaniu komendy zawsze musi być 'trigger' w następnej linijce.) command /gui [<text>]: trigger: Gdy już masz komendę, musisz stworzyć swoje GUI. Robisz to w ten sposób: (3 to ilość linijek, "Nazwa" to nazwa twojego gui.) set {_gui} to chest inventory with 3 rows named "Nazwa" Teraz musisz ustawić slot. Pamiętaj, że są one liczone od 0. Żeby ustawić pierwszy slot w swoim GUI, zrób to: set slot 0 of {_gui} to diamond Twoje GUI jest już gotowe! Najwyższy czas je otworzyć dla gracza, który wykonał komendę: open {_gui} to player Cały kod powinien wyglądać tak: command /gui [<text>]: trigger: set {_gui} to chest inventory with 3 rows named "Nazwa" set slot 0 of {_gui} to diamond open {_gui} to player Aktualnie z twojego GUI można wyciągać przedmioty i nic się nie dzieje, jak to zmienić? Przyda ci się event 'on inventory click': on inventory click: Sprawdzaj czy naciśnięty slot nie jest ekwipunkiem gracza: event-inventory is not player's inventory Teraz musisz sprawdzić nazwę GUI: if name of event-inventory is "Nazwa": Jeśli chcesz zablokować wyciąganie przedmiotów, dodaj teraz 'cancel event': cancel event Sprawdź kliknięty slot: if clicked slot is 0: I wykonaj co chcesz wykonać: add diamond to player's inventory Cały kod powinien wyglądać tak: on inventory click: event-inventory is not player's inventory if name of event-inventory is "Nazwa": cancel event if clicked slot is 0: add diamond to player's inventory Teraz połącz te dwa kody i przeładuj skrypt. Gratulacje! Zrobiłeś swoje pierwsze GUI w Skripcie!

Oto i mój przepiękny kod na drop. Wymagane pluginy: - Skript - Skellett I chyba więcej żadnych, ale ja mam prawie wszystkie dodadki do skryptów, a więc jak będziecie mieli jakiś problem to wgrajcie: - WildSkript - SkQuery - SkRayFall

Siemka napisałby ktoś skript żeby nie dało się rzucaę rzucaka na spawnie?

rzucać rzucaka sorry za błąd @SelniKs Że jak jest się na regionie spawn1 to nie można rzucić rzucaka

@Norbi353 jezeli chcesz, mogę to zrobić za ciebie Jak tak, pv zapraszam.

nikt nie odpowie?