Skocz do zawartości

  • 👋 Witaj na MPCForum!

    Przeglądasz forum jako gość, co oznacza, że wiele świetnych funkcji jest jeszcze przed Tobą! 😎

    • ✅ Pełny dostęp do działów i ukrytych treści
    • ✅ Możliwość pisania i odpowiadania w tematach
    • ✅ System prywatnych wiadomości
    • ✅ Zbieranie reputacji i rozwijanie swojego profilu
    • ✅ Członkostwo w jednej z największych społeczności graczy

    👉 Dołączenie zajmie Ci mniej niż minutę – a zyskasz znacznie więcej!

    Zarejestruj się teraz

[Plugin] Maintenance mode

jucy

Przez jucy
w Minecraft - Projekty i download

Rekomendowane odpowiedzi

Zimmex129 Uczeń

Zimmex129

Opublikowano
Uczeń

Zimmex129

Opublikowano

Serio tak ciężko jest wrzucenie src protocollib do pluginu i zostawienie tylko potrzebnych class do działania?

4 godziny temu, jucy napisał:

Wymagany jest ProtocolLib.

Po co protocollib skoro wrzuciłeś src protocol'a do pluginu?

Plugin nawet spoczko :D

Mine1998a Bywalec

Mine1998a

Opublikowano
Bywalec

Mine1998a

Opublikowano

Ja mam podobny ale na bungeecorda,a plugin nawet fajny

CPU i5 6600k 4,1GHZ | RAM HyperX SAVAGE 16GB RAM | GPU ASUS GTX 1070 ROG STRIX 8GOC |

SSD1 Samsung EVO 850 120GB| SSD2 ADATA SP600 120GB |HDD Toshiba 1TB

Reklama serwera minigames w starym stylu Kliknij tutaj

 

__AHA__ Uczeń

__AHA__

Opublikowano
Uczeń

__AHA__

Opublikowano
3 godziny temu, Zimmex129 napisał:

Serio tak ciężko jest wrzucenie src protocollib do pluginu i zostawienie tylko potrzebnych class do działania?

Po co protocollib skoro wrzuciłeś src protocol'a do pluginu?

Plugin nawet spoczko :D

PACKET WRAPPER TO NIE JEST PROTOCOLIB GOSCIU

 

 

Co to ma byc?? 

 Bukkit.getServer().broadcastMessage(ChatColor.GRAY.toString() + "§7Maintenance has been " + ChatColor.GREEN + "enabled" + ChatColor.GRAY + "!");

 

Najpierw dajesz ChatColor.GRAY po czym jeszcze toString() ? Czemu ma to sluzyc? a potem znowu uzywasz paragrafow xD..

 

Zrob sobie taka metode 

    public static String fixColors(String message) {
        return ChatColor.translateAlternateColorCodes('&', message);
    }

I uzywaj jej chociazby tak 

Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!");

 

A najlepiej to jeszcze sobie zapisz stringi w finalnych zmiennych i uzywaj metody tylko raz przy starcie.

 

Polecam poczytac: http://wiki.c2.com/?ArrowAntiPattern

Tak to chyba wyglada lepiej c'nie?

  

    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String l, String[] args) {
        if (cmd.getName().equalsIgnoreCase("maintenance")) {
            return true;
        }
        if (sender.hasPermission("jelizy.maintenance")) {
            sender.sendMessage(fixColors("&cNo permission!"));
            return true;
        }
        if (args.length == 1) {
            sender.sendMessage(fixColors("&cWrong!"))
            return true;
        }
        if (args[0].equalsIgnoreCase("on")) {
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().setWhitelist(true);

            maintenance = true;

            for (Player player : Bukkit.getOnlinePlayers()) {
                if (!(player.hasPermission("jelizy.maintenance"))) {
                    player.kickPlayer(fixColors("Maintenance mode has been enabled!"));
                }
            }
            return true;
        }
        if (args[0].equalsIgnoreCase("off")) {
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().setWhitelist(false);

            maintenance = false;
        }
        return true;
    }

 

jucy Nowicjusz

jucy

Opublikowano
Nowicjusz

jucy

  • Autor
Opublikowano
17 hours ago, __AHA__ said:

PACKET WRAPPER TO NIE JEST PROTOCOLIB GOSCIU

 

 

Co to ma byc?? 


 Bukkit.getServer().broadcastMessage(ChatColor.GRAY.toString() + "§7Maintenance has been " + ChatColor.GREEN + "enabled" + ChatColor.GRAY + "!");

 

Najpierw dajesz ChatColor.GRAY po czym jeszcze toString() ? Czemu ma to sluzyc? a potem znowu uzywasz paragrafow xD..

 

Zrob sobie taka metode 


    public static String fixColors(String message) {
        return ChatColor.translateAlternateColorCodes('&', message);
    }

I uzywaj jej chociazby tak 


Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!");

 

A najlepiej to jeszcze sobie zapisz stringi w finalnych zmiennych i uzywaj metody tylko raz przy starcie.

 

Polecam poczytac: http://wiki.c2.com/?ArrowAntiPattern

Tak to chyba wyglada lepiej c'nie?

  


    @Override
    public boolean onCommand(CommandSender sender, Command cmd, String l, String[] args) {
        if (cmd.getName().equalsIgnoreCase("maintenance")) {
            return true;
        }
        if (sender.hasPermission("jelizy.maintenance")) {
            sender.sendMessage(fixColors("&cNo permission!"));
            return true;
        }
        if (args.length == 1) {
            sender.sendMessage(fixColors("&cWrong!"))
            return true;
        }
        if (args[0].equalsIgnoreCase("on")) {
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().setWhitelist(true);

            maintenance = true;

            for (Player player : Bukkit.getOnlinePlayers()) {
                if (!(player.hasPermission("jelizy.maintenance"))) {
                    player.kickPlayer(fixColors("Maintenance mode has been enabled!"));
                }
            }
            return true;
        }
        if (args[0].equalsIgnoreCase("off")) {
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().setWhitelist(false);

            maintenance = false;
        }
        return true;
    }

 

te paragrafy to zapomniałem usunąć xD

us3rn0tf0und Bywalec

us3rn0tf0und

Opublikowano
Bywalec

us3rn0tf0und

Opublikowano

1. Dlaczego nie korzystasz z Mavena? 

2. Dlaczego korzystasz z paragrafów? Nie lepiej stworzyć sobie jakąś metodę od kolorowania w Utilsach? :)

3. 

Cytat

Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));

To ma być reklama? Z tego co się orientuję, bardziej poprawna forma to "xxx was enabled" ;)

4. Na jakiego ciula sprawdzasz  

     if (cmd.getName().equalsIgnoreCase("maintenance")) {
            return true;
        }

skoro używasz tylko jednej komendy?

5. Dlaczego w klasie MaintenanceCommand używasz metody "fixColors" której tam nie ma? Chyba zapomniałeś jej dodać.

6. Dlaczego  

        if (args[0].equalsIgnoreCase("on")) {
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().setWhitelist(true);

            maintenance = true;

            for (Player player : Bukkit.getOnlinePlayers()) {
                if (!(player.hasPermission("jelizy.maintenance"))) {
                    player.kickPlayer(fixColors("Maintenance mode has been enabled!"));
                }
            }
            return true;
        }
        if (args[0].equalsIgnoreCase("off")) {
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().setWhitelist(false);

            maintenance = false;
        }

w argumencie on returnujesz, a w argumencie off zapominasz o returnowaniu? 

7. Trochę namieszałeś z pakietami podczas wrzucania na git'a. Troszkę niepoprawne nazewnictwo pakietów.

8. Dlaczego jeśli ktoś ma permisję, to nie zezwalasz mu do korzystania z komendy? Powinno być odwrotnie, zapomniałeś wykrzyknika ;)

9. Dlaczego podczas kickowania gracza stosujesz kolorowanie kodu, gdzie nie występuje znak '&'?

10. Po co cały czas getServer()?

Generalnie dość fajny plugin, masz tu kodzik poprawiony. Wprowadź zmiany na githubie :)

https://hastebin.com/vagehaqefo.java

 

__AHA__ Uczeń

__AHA__

Opublikowano
Uczeń

__AHA__

Opublikowano
14 godzin temu, ScoreBoard napisał:

1. Dlaczego nie korzystasz z Mavena? 

2. Dlaczego korzystasz z paragrafów? Nie lepiej stworzyć sobie jakąś metodę od kolorowania w Utilsach? :)

3. 

To ma być reklama? Z tego co się orientuję, bardziej poprawna forma to "xxx was enabled" ;)

4. Na jakiego ciula sprawdzasz  


     if (cmd.getName().equalsIgnoreCase("maintenance")) {
            return true;
        }

skoro używasz tylko jednej komendy?

5. Dlaczego w klasie MaintenanceCommand używasz metody "fixColors" której tam nie ma? Chyba zapomniałeś jej dodać.

6. Dlaczego  


        if (args[0].equalsIgnoreCase("on")) {
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().setWhitelist(true);

            maintenance = true;

            for (Player player : Bukkit.getOnlinePlayers()) {
                if (!(player.hasPermission("jelizy.maintenance"))) {
                    player.kickPlayer(fixColors("Maintenance mode has been enabled!"));
                }
            }
            return true;
        }
        if (args[0].equalsIgnoreCase("off")) {
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().setWhitelist(false);

            maintenance = false;
        }

w argumencie on returnujesz, a w argumencie off zapominasz o returnowaniu? 

7. Trochę namieszałeś z pakietami podczas wrzucania na git'a. Troszkę niepoprawne nazewnictwo pakietów.

8. Dlaczego jeśli ktoś ma permisję, to nie zezwalasz mu do korzystania z komendy? Powinno być odwrotnie, zapomniałeś wykrzyknika ;)

9. Dlaczego podczas kickowania gracza stosujesz kolorowanie kodu, gdzie nie występuje znak '&'?

10. Po co cały czas getServer()?

Generalnie dość fajny plugin, masz tu kodzik poprawiony. Wprowadź zmiany na githubie :)


https://hastebin.com/vagehaqefo.java

 

Jak juz uzywasz fixColors to wszedzie

 

Ty w ogole wiesz co dodajesz na tego githuba?

 

BTW. ten plugin nie ma prawa zadzialac z nowym protocolibem i to sporo wersji wstecz.

PacketWrapper tez jest przestarzaly, tutaj znajdziesz nowszy: http://repo.dmulloy2.net/content/groups/public/   ..  com.comphenix.packetwrapper

lap: https://hastebin.com/tosivutovu.java

ogolnie to nie widze sensu tego pluginu skoro nawet nie ma configu do ustawienia motd czy czegokolwiek.

toWcaleNieOn_;/ Rekrut

toWcaleNieOn_;/

Opublikowano
Rekrut

toWcaleNieOn_;/

Opublikowano
Dnia 3.05.2018 o 01:28, ScoreBoard napisał:

1. Dlaczego nie korzystasz z Mavena? 

2. Dlaczego korzystasz z paragrafów? Nie lepiej stworzyć sobie jakąś metodę od kolorowania w Utilsach? :)

3. 

To ma być reklama? Z tego co się orientuję, bardziej poprawna forma to "xxx was enabled" ;)

4. Na jakiego ciula sprawdzasz  


     if (cmd.getName().equalsIgnoreCase("maintenance")) {
            return true;
        }

skoro używasz tylko jednej komendy?

5. Dlaczego w klasie MaintenanceCommand używasz metody "fixColors" której tam nie ma? Chyba zapomniałeś jej dodać.

6. Dlaczego  


        if (args[0].equalsIgnoreCase("on")) {
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("&7Maintenance has been &3enabled &7!"));
            Bukkit.getServer().setWhitelist(true);

            maintenance = true;

            for (Player player : Bukkit.getOnlinePlayers()) {
                if (!(player.hasPermission("jelizy.maintenance"))) {
                    player.kickPlayer(fixColors("Maintenance mode has been enabled!"));
                }
            }
            return true;
        }
        if (args[0].equalsIgnoreCase("off")) {
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().broadcastMessage(fixColors("§7Maintenance has been &cdisabled &7!"));
            Bukkit.getServer().setWhitelist(false);

            maintenance = false;
        }

w argumencie on returnujesz, a w argumencie off zapominasz o returnowaniu? 

7. Trochę namieszałeś z pakietami podczas wrzucania na git'a. Troszkę niepoprawne nazewnictwo pakietów.

8. Dlaczego jeśli ktoś ma permisję, to nie zezwalasz mu do korzystania z komendy? Powinno być odwrotnie, zapomniałeś wykrzyknika ;)

9. Dlaczego podczas kickowania gracza stosujesz kolorowanie kodu, gdzie nie występuje znak '&'?

10. Po co cały czas getServer()?

Generalnie dość fajny plugin, masz tu kodzik poprawiony. Wprowadź zmiany na githubie :)


https://hastebin.com/vagehaqefo.java

 

Cytat 

https://hastebin.com/vagehaqefo.java

po co do javaplugin implementujesz commandexecutor?

po co nie uzywasz streamow skoro uzywasz javy 8? 

        if (args.length == 1) {
            sender.sendMessage(fixColors("&cWrong!"));
            return true;
        }

        if (args[0].equalsIgnoreCase("on")) {

ktos tu chyba nie zna sie na indexowaniu ;/

Cytat

To ma być reklama? Z tego co się orientuję, bardziej poprawna forma to "xxx was enabled"

has been enabled, present perfect, zostal wlaczony - wiec mozesz go uzywac

Cytat

1. Present Perfect używamy do opisania przeszłej czynności (nie podając, kiedybyła ona wykonana) która ma wpływ na teraźniejszość - i najważniejszy właśnie jest ten związek z teraźniejszością. Czyli ważna jest przyczyna i jej skutek - natomiast nie jest istotne, kiedy dana czynność została (lub nie) wykonana

Cytat

10. Po co cały czas getServer()?

dostępność

Cytat

w argumencie on returnujesz, a w argumencie off zapominasz o returnowaniu? 

a po co ma dodawac tam return skoro jest to ostatni if i nic po nim sie nie wykona ;/

chociaz ja i tak bym wywali ostatniego ifa i od razu walnal ten kodzik co jest po ifie

 

i tak wgl to jaki jest sens tego pluginu, bo nie rozumiem tych komend i glupiego kodu

Cytat

Prosty plugin, napisany w 4 minuty, dziala przy uzyciu wbudowanej whitelisty.

jakosc, a nie szybkosc zrobienia pluginu, zbucki napisal drop czy tam gildie w 6h (bardzo ladnie sie tym chwalil) ale kod juz nie zadowala ;/

Zarchiwizowany

Ten temat przebywa obecnie w archiwum. Dodawanie nowych odpowiedzi zostało zablokowane.

Tematy
×
×
  • Dodaj nową pozycję...