Skocz do zawartości
DemoN

DemoN-News - Autorski plugin na NEWSY na serwerze!

Rekomendowane odpowiedzi

Użytkownik

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.

DownloadZarejestruj się lub zaloguj, aby zobaczyć ukrytą treść!

SkanZarejestruj się lub zaloguj, aby zobaczyć ukrytą treść!

GithubZarejestruj się lub zaloguj, aby zobaczyć ukrytą treść!

Zarejestruj się lub zaloguj, aby zobaczyć ukrytą treść!

 

53575543178253378894.png

Edytowane przez DemoN
Dodanie githuba

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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ę 

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik

Ja zamiast metod statycznych zrobiłbym to takim sposobem:

Niby to samo, ale jednak zawsze metod statycznych mniej

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

Edytowane przez Ever

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Senior administrator

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ą.

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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ę

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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 =/

Edytowane przez yooniks

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik

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: Zarejestruj się lub zaloguj, aby zobaczyć ukrytą treść! 

 

Ocena 6/10

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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: Zarejestruj się lub zaloguj, aby zobaczyć ukrytą treść! 

 

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

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
    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 ? :thinking:

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik
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 ? :thinking:

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 :P

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach
Użytkownik

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ę.

Udostępnij tę odpowiedź


Odnośnik do odpowiedzi
Udostępnij na innych stronach

Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto

Jedynie zarejestrowani użytkownicy mogą komentować zawartość tej strony.

Zarejestruj nowe konto

Załóż nowe konto. To bardzo proste!

Zarejestruj się

Zaloguj się

Posiadasz już konto? Zaloguj się poniżej.

Zaloguj się

  • Przeglądający   0 użytkowników

    Brak zarejestrowanych użytkowników przeglądających tę stronę.

×