DemoN-News - Autorski plugin na NEWSY na serwerze!

DemoN-News

Wersja: 1.0 

Opis wersji 1.0:

Jest to plugin polegający na tzw. "newsach" czyli wiadomości pokazanej na środku ekranu po wejściu na serwer przez gracza. Może być tam cokolwiek tylko chcemy tam dać. Wiadomości do ustawienia w configu. Wiadomość pokazywana jest przez metodę SendTitle.

Download: http://hostuje.net/file.php?id=61cf0f0f1d89c023da5616eb81c69450

Skan: https://www.virustotal.com/#/file/4c2f716b084f69379dc5351e8d199186bcdd6a44ddfed213cf78ab85c56eaffa/detection

Github: https://github.com/DemoNdevJava/DemoN-Newss

Screen

Mógłbyś dodać jakieś zdjęcia, kod na githuba bo na telefonie nic nie można zobaczyć ;(

10 godzin temu, PietregTM napisał:

Mógłbyś dodać jakieś zdjęcia, kod na githuba bo na telefonie nic nie można zobaczyć ;(

Jasne, dodam screena jak wrócę 

kiedy github

13 godzin temu, Queito napisał:

kiedy github

Jak wrócę do domu z uczelni

W dniu 27.11.2018 o 10:02, DemoN napisał:

Jak wrócę do domu z uczelni

still waiting for github

5 godzin temu, yooniks napisał:

still waiting for github

Gotowe

Ja zamiast metod statycznych zrobiłbym to takim sposobem:

A co do kodu, działa jak ma działać, 6/10

No takie średnie

Pełno syfnego kodu do sprawdzania autora i wysyłania jakiegoś niepotrzebnego tekstu do konsoli, nie używasz refleksji, masz w kodzie jakieś puste metody, puste klamry. No i możesz ustawić tylko jedną wiadomość do wysyłania. Słabe

@Ever akurat tutaj użycie statycznych metod jest o wiele lepszą opcją.

13 minut temu, Ixidi napisał:

No takie średnie

Pełno syfnego kodu do sprawdzania autora i wysyłania jakiegoś niepotrzebnego tekstu do konsoli, nie używasz refleksji, masz w kodzie jakieś puste metody, puste klamry. No i możesz ustawić tylko jedną wiadomość do wysyłania. Słabe

@Ever akurat tutaj użycie statycznych metod jest o wiele lepszą opcją.

Wiem, dałem pomysł jakbym ja to zrobił, tylko popełniłem jeden błąd. W sumie to ma tak mało linijek, że niepotrzebnie robić moim sposobem

Przepraszam za wpadkę

W dniu 29.11.2018 o 16:13, Ever napisał:

Wiem, dałem pomysł jakbym ja to zrobił, tylko popełniłem jeden błąd. W sumie to ma tak mało linijek, że niepotrzebnie robić moim sposobem

Przepraszam za wpadkę

Utily powinny byc statyczne i slabo poprawiles

a tak ogolnie to plugin jak dla mnie slabo napisany, zbyt duzo wiadomosci o autorze i tak jak wyzej ktos napisal - puste klamry, metody i po tym sprawdzaniu autora moglbys od razu walnac return zeby kod nie szedl dalej i wywalal bledy

czemu replaceAll w zamienianiu kolorkow? Uzyj ChatColor.translaetAlternateColorCodes, i te wysylanie title i subtitle.. przeciez w 1.8+ jest wbudowane chyba Player#sendTitle i mozesz od razu wyslac title i subtitle, a tutaj zadnego wsparcia dla innych wersji =/

3 minuty temu, yooniks napisał:

Utily powinny byc statyczne i slabo poprawiles

a tak ogolnie to plugin jak dla mnie slabo napisany, zbyt duzo wiadomosci o autorze i tak jak wyzej ktos napisal - puste klamry, metody i po tym sprawdzaniu autora moglbys od razu walnac return zeby kod nie szedl dalej i wywalal bledy

czemu replaceAll w zamienianiu kolorkow? Uzyj ChatColor.translaetAlternateColorCodes, i te wysylanie title i subtitle.. przeciez w 1.8+ jest wbudowane chyba Player#sendTitle i mozesz od razu wyslac title i subtitle, a tutaj zadnego wsparcia dla innych wersji =/

heh, wiem. Poprawiłem na górze, że fail

Plugin nie jest jakiś trudny do napisania ale możesz dodać komendę która można dodać newsy i podczas dołączanie na serwer wyświetlać listę stringów np mamy 2 stringi to przez 5 sekund widać string 1 i później string 2 też przez 5 sek. Generalnie nie powinno się nazywać klasy Util albo Utils tutaj masz link do artykułu: https://lostechies.com/chrismissal/2009/06/01/anti-patterns-and-worst-practices-utils-class/ 

Ocena 6/10

10 godzin temu, PietregTM napisał:

Plugin nie jest jakiś trudny do napisania ale możesz dodać komendę która można dodać newsy i podczas dołączanie na serwer wyświetlać listę stringów np mamy 2 stringi to przez 5 sekund widać string 1 i później string 2 też przez 5 sek. Generalnie nie powinno się nazywać klasy Util albo Utils tutaj masz link do artykułu: https://lostechies.com/chrismissal/2009/06/01/anti-patterns-and-worst-practices-utils-class/ 

 

Ocena 6/10

znaczy utili nikt nie zabrania robić (ale jakiś przedrostek fajnie jakby był, bo tak to tu jest bez sensu)  no i tutaj akurat lepiej byłoby jakiś interfejs sobie zrobić z actionbarem itd zamiast utila i o to w tym artykule chodzi, a nie żeby wgl nie używać utilow/nie robić nich

    public void JoinMessage(final PlayerJoinEvent event) {
        final Player p = event.getPlayer();
        {
        }
        Util.sendTitle(p, Util.fixChatColor(Config.news1));
        Util.sendSubTitle(p, Util.fixChatColor(Config.news2));
    }

final PlayerJoinEvent event
final Player p = event.getPlayer()
Hmmm czyżby skopiowane z jakiegoś pluginu ?

3 godziny temu, JanekMaSkrypta_ napisał:

    public void JoinMessage(final PlayerJoinEvent event) {
        final Player p = event.getPlayer();
        {
        }
        Util.sendTitle(p, Util.fixChatColor(Config.news1));
        Util.sendSubTitle(p, Util.fixChatColor(Config.news2));
    }

final PlayerJoinEvent event
final Player p = event.getPlayer()
Hmmm czyżby skopiowane z jakiegoś pluginu ?

bo ma final? ja juz bardziej bym sie przyczepil do tego czemu metoda jest z duzej litery rozpoczeta, czemu jest 'event' a potem juz 'p' - skoro pelnymi nazwami to pelnymi..

i te klamerki tez niepotrzebne

juz lepiej zobacz co ty kopiujesz z innych pluginow 

Koledzy tak jak mówiłem jestem dość "POCZĄTKUJĄCY" i często zaczerpuje porad od bardziej doswiadczonych kolegow badz stron internetowych. Dziekuje za opinie na temat pluginu, stale będę poprawiał moje umiejętności. Pozdrawiam i wesołych świąt życzę.